5

这是我第一次做这样的事情，所以我有点不确定我需要怎么做。我有一个非常简单的类，其中包含一些简单的值和一些 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& 吗？我不确定具体细节 - 大概如果我返回一个指针，我的返回类型需要是 Nucleotide* 或者 Nucleotide& 因为 & 表示地址？我已经考虑过了并阅读了 Cpp tuts，但我仍然有点不确定正确的答案。

谢谢，本。

4

4 回答 4

5

您必须通过引用返回核苷酸：

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
        }
    }
}

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

但请注意，如果您没有找到所需的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;
}
于 2013-11-13T12:53:09.243 回答
1

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

您可以更改函数以返回指针、引用或仅按值返回（复制就像它声明的不是。

您还可以进行更改，以便该函数将Nucleotide用作参数，然后返回一个布尔指示符（如果找到与否）。

bool NucleotideSequence::getBase(int pos1, Nucleotide& n)
{
    for (...)
    {
        if (...)
        {
            n = *i;
            return true;
        }
    }
    return false;  // Not found
}
于 2013-11-13T12:53:00.310 回答
0

就您的问题而言，按照其他人的建议返回参考 (&) 是解决方案。

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

要么使用 operator[]，要么使用 std::vector 中的 at()。

因此，您可以直接说：

返回序列[pos1]；或返回 Sequence.at(pos1);

于 2013-11-13T13:02:56.653 回答
0

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

const Nucleotide& NucleotideSequence::getBase(int pos1)

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

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

像这样的getName方法：

const std::string& getName();

（尽管返回值优化可能会使它变得不那么重要。）

至于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
}
于 2013-11-13T13:09:58.100 回答