0

我用qhash的qhash写了一个函数,如下图:

void scanOccurenceOnAllSequence(QString motif, QString chkMotif, qint32 offset, QString cell, QHash <QString, QHash <QString, QHash<qint32, qint32> > > *motifByCell2seq, QList<QString> *peakSequence){
    qint32 peakSequenceNumber = peakSequence->size();
    for(qint32 si=0; si < peakSequenceNumber; si++){
    if( motifByCell2seq->value(motif).value(cell).contains(si) ||  motifByCell2seq->value(motif).value(cell).contains(si) ){
        continue;
    }

    bool flag = checkMotifOccurence(motif,chkMotif,peakSequence->at(si),offset);

    if(flag){
        motifByCell2seq->value(motif).value(cell).insert(si,1);
    }
    }
}

但是,这一行有一个错误:

motifByCell2seq->value(motif).value(cell).insert(si,1);

错误是:

错误:将 'const QHash' 作为 'QHash::iterator QHash::insert(const Key&, const T&) 的 'this' 参数传递 [with Key = int; T = int]' 丢弃限定符 [-fpermissive]motifByCell2seq->value(motif).value(cell).insert(si,1);

即使我将此行更改为以下内容,仍然存在错误

motifByCell2seq[motif][cell].insert(si,1);

你能帮我找出问题吗?

4

1 回答 1

0

简短版本:value()方法返回一个常量,因此尝试value().insert()修改该常量值。 operator[]可以返回一个常量,显然在这种情况下它选择这样做。如果分配motifByCell2seq[motif][cell]给临时变量,则可以强制使用 non-const operator[]

长版: 作为解决此问题的策略(也就是我为达到上述短版所做的工作),我会将问题线分成几部分,将每个部分分配给它自己的临时变量。

因此,首先值得简化。例如,需要 bool 标志行和 if 语句上的标志来重新创建错误,所以现在将它们注释掉。对于 for 循环中的第一个 if 语句也是如此。

因此,目前,我已将问题简化为:

void scanOccurenceOnAllSequence(QString motif,
                                QString chkMotif,
                                qint32 offset,
                                QString cell,
                                QHash <QString, QHash<QString, QHash<qint32, qint32> > > *motifByCell2seq,
                                QList<QString> *peakSequence)
{ 
    qint32 si=0;
    motifByCell2seq[motif][cell].insert(si,1);                                                                               
}

从那里,我将问题行更改为:

QHash<QString, QHash<qint32, qint32> > tmp1;
tmp1 = motifByCell2seq[motif];
tmp1[cell].insert(si,1);

现在问题仍然存在并且仍然在最后一行,减少了搜索范围。

另一个迭代将我带到:

    QHash<QString, QHash<qint32, qint32> > tmp1;
    tmp1 = motifByCell2seq[motif];
    QHash<qint32, qint32> tmp2;
    tmp2 = tmp1[cell];
    tmp2.insert(si,1);                                                                                    

那行得通。因此,cell在这种情况下将查找分配给临时变量会改变行为。从那里,我想我应该仔细检查QHash的文档,这是我注意到operator[]. 另外,我注意到value()您首先尝试的方法仅在一个const版本中提供。

既然找到了问题,我可能会回到:

QHash<qint32, qint32> cellHash = (*motifByCell2seq)[motif][cell];
cellHash.insert(si,1);

然后当然,把我删除的其他东西放回去,然后写一个测试来确认它按预期工作。

于 2015-01-09T15:47:58.293 回答