Question

我有以下功能：

ItemSet* Library::itemsForKeyword(const string& keyword)
{
    ItemSet temp;

    for(it=bookList.begin();it!=bookList.end();it++){
        if(it->getPtr()->findKeyWord(it->getPtr()->keywordsList, keyword))
            temp.insert(*it);
    }

    ItemSet* temp2 = &temp;
    return temp2;
}

ItemSet是一个包含ItemPtr实例的集合，它包含指向派生类实例的指针。 'it'是在Library中定义的迭代器，findKeyWord是一个函数，它将keywordsList与关键字进行比较，以确定该关键字是否在该列表中。如果它返回true，则我取消引用迭代器，并将当前迭代的实例添加到temp。我的问题是，当它完成循环时，我返回temp2，但它返回空，并在我的打印功能打印时导致错误。为什么它变回空？感谢。

Answer 1

您将返回一个指向本地对象的指针，该对象在从函数返回时会被销毁。这肯定是未定义的行为。

另外，我不明白为什么你需要返回指针。你可以返回对象本身：

ItemSet Library::itemsForKeyword(const string& keyword)
{
   ItemSet temp;
   //your code
   return temp;
}

如果你需要返回指针，那么我建议你返回智能指针，而不是原始指针。

std::unique_ptr<ItemSet> Library::itemsForKeyword(const string& keyword)
{
   std::unique_ptr<ItemSet> temp(new ItemSet());
   //your code, use this syntax =>  temp->insert(*it);
   return std::move(temp);
}

Answer 2

您正在返回本地变量的地址，这不起作用。

将一个项目作为参数提供给您在函数中修改或分配项目的函数并返回其地址。

Answer 3

用教师和主要代码评论后......

ItemSet* Library::itemsForKeyword(const string& keyword)
{
    ItemSet* p_temp = new ItemSet;

    for(it=bookList.begin();it!=bookList.end();it++){
        if(it->getPtr()->findKeyWord(it->getPtr()->keywordsList, keyword))
            p_temp->insert(*it);
    }

    return p_temp;
}

编辑：你应该向你的导师@Nawaz示例：）

Answer 4

您不得返回堆栈上分配的对象的地址。函数返回后，堆栈上的对象被销毁，指针指向一些垃圾。

函数返回空集

4 个答案: