Question

这是我第一次做这样的事情，所以我有点不确定我需要做什么。我有一个非常简单的类，它包含一些简单的值和一些getter：

class Nucleotide{
    private:
        char Base;
        int Position;
        int Polymorphic;
    public:
        Nucleotide(char ch, int pos);
        int getPos();
        char getBase();
        int getPoly();
};

此类存在于另一个包含它们向量的类中：

class NucleotideSequence{
    private:
        std::string Name;
        std::vector<Nucleotide> Sequence;
    public:
        NucleotideSequence(std::string name, std::vector<Nucleotide> seq);
        std::string getName();
        Nucleotide getBase(int pos1);
};

我希望名为getBase的第二个类的方法能够取一个整数 - 比如1，并返回向量中的第一个Nucleotide对象。我写的内容如下：

Nucleotide NucleotideSequence::getBase(int pos1)
{
    for(std::vector<Nucleotide>::iterator i = Sequence.begin(); i != Sequence.end(); i++)
    {
        if(pos1 == (*i).getPos())
        {
            return i; // Return a pointer to the correct base.
        }
    }
}

我有Nucleotide作为返回类型，但我真的想知道我应该如何改变它 - 因为如果我因为传递值而返回核苷酸，那么它不会仅仅返回该向量中该对象的副本吗？所以我宁愿返回一个指针/引用。我在循环中使用了一个迭代器，所以我应该只返回一个带迭代器值的指针吗？我该怎么做呢？在函数中我返回i但是我应该返回i＆amp ;?我不确定具体情况 - 大概是如果我要返回一个指针，我的返回类型需要是核苷酸*或者可能是核苷酸＆amp;从＆amp;是指地址？我已经考虑过了并阅读了Cpp tuts，但我仍然不确定正确的答案。

谢谢，本。

Answer 1

您必须通过引用返回Nucleotide ：

Nucleotide & NucleotideSequence::getBase(int pos1) { for(std::vector<Nucleotide>::iterator i = Sequence.begin(); i != Sequence.end(); i++) { if(pos1 == (*i).getPos()) { return *i; // Notice the *i instead of i } } }

引用与指针的工作方式非常相似（允许您传递实际对象，而不是其副本），但不能为null且不能指向不存在的对象，因此它比指针。

但请注意，如果找不到所需的Nucleotide，则不返回任何内容，通常不是一个好主意。在这种情况下，使用指针实际上可能是一个更好的主意：

Nucleotide * NucleotideSequence::getBase(int pos1) { for(std::vector<Nucleotide>::iterator i = Sequence.begin(); i != Sequence.end(); i++) { if(pos1 == (*i).getPos()) { return &(*i); } } return nullptr; }

Answer 2

您没有返回指针，您尝试返回迭代器。并且声明函数返回实例而不是指针。此外，如果您没有找到Nucleotide，如果您尝试使用“返回”值，则根本不会返回任何导致未定义行为的内容。

你可以更改函数以返回一个指针，一个引用，或者只是一个by值（复制就像声明它一样。

您也可以更改，以便函数将Nucleotide作为参数，然后返回一个布尔指示符（如果找到它）。

bool NucleotideSequence::getBase(int pos1, Nucleotide& n)
{
    for (...)
    {
        if (...)
        {
            n = *i;
            return true;
        }
    }
    return false;  // Not found
}

Answer 3

就您的问题而言，按照其他人的建议返回参考（＆amp;）是解决方案。

为了改进您的代码，我还建议进行更改：

要么使用运算符[]，要么使用std :: vector中的at（）。

因此，您可以直接说：

返回序列[pos1]; 或返回Sequence.at（pos1）;

Answer 4

为了提高效率，您的代码将受益于某些参考资料的使用。 getBase方法签名应如下所示：

const Nucleotide& NucleotideSequence::getBase(int pos1)

NucleotideSequence构造函数签名应如下所示：

NucleotideSequence(const std::string& name, const std::vector<Nucleotide>& seq);

这样的getName方法：

const std::string& getName();

（虽然return value optimisation可能会降低这一点。）

对于getBase的内容，可能有助于理解将代码分解为：

const Nucleotide* NucleotideSequence::getBase(int pos1)
{
    for(std::vector<Nucleotide>::iterator i = Sequence.begin(); i != Sequence.end(); ++i)
    {
        Nucleotide& ref = *i; //Get a reference to the object this iterator points to
        if(pos1 == ref.getPos()) //compare its base to the argument
        {
            return &ref; // Return a pointer to the correct object.
        }
    }
    return NULL; //or null if we didn't find the object we wanted
}

返回指向类中类的指针

4 个答案: