0

我有一个程序,它将类和结构集存储在列表中。

它执行以下操作:

  1. 通过引用将输入 (an int)、迭代器、列表和指针传递给函数check()
  2. 迭代列表,直到找到迭代器的数据和输入之间的匹配
  3. 将指针设置为迭代器的位置
  4. 返回真或假,取决于是否找到匹配项。

我的问题是,当我display()从函数检查中调用函数时,无论是 fromit->display()还是Ptr->display(),它都可以正常工作。但是当它通过引用传回时,我尝试显示它。它打印垃圾。

//it is the iterator, l is the list, Ptr is the passed pointer
template<class T, class T2, class P>
bool Inspection::check(int input, T it, T2 l, P * &Ptr)
{
    for(it = l.begin(); it != l.end(); ++it){   //Iterates through list using iterator
        if (it->checkExists(input)){        //if input == iterator class's data
            Ptr = &*it;

            //Display data - ERROR CHECKING//
            it->display();          
            Ptr->display();

            return true;
        }
    }
    return false;
}

checkExists是一个函数,它与它正在迭代的类中的私有数据进行比较,例如

bool Property::checkExists(int input)
{
    if (input == ID)
        return true;
    return false;
}

display也很简单

void Property::display()
{
    //Prints out property info
    cout << ID << ";" << address << ";" << landTypes[type] << ";" << price << endl;
}

标准调用是(是我在程序前面调用p的类的列表)Property

int input;
Property * temp; //Pointer to a class temp
list<Property>::iterator pIT;

cin >> input;


while(!check(input, pIT, p, temp)){
    ...
}
    temp->display();

一个典型的输出是(前两个是函数内部的调用并且正确,第三个是temp->display();函数外部的调用。

1001;5/10 Northfields Ave, North Wollongong, NSW 2500;Townhouse;280000
1001;5/10 Northfields Ave, North Wollongong, NSW 2500;Townhouse;280000
13;�������\314���@�ve, North Wollongong, NSW 2500;Townhouse;280000

编辑:对不起,我链接了错误的显示函数()。编辑代码以更新

4

1 回答 1

2

尽管 WhozCraig 指出了设计问题,但在您提供的代码中打印出垃圾的问题如下:

 template<class T, class T2, class P>
 bool Inspection::check(int input, T it, T2 l, P * &Ptr)
                                         ^^^^

l通过值而不是通过引用传递,因此您将返回一个指向临时变量的指针,当您在方法之外取消引用它时该临时变量将不存在。如果您按以下方式修改代码,它应该开始解决这个特定问题,尽管它确实需要重新设计:

template<class T, class T2, class P>
bool Inspection::check(int input, T it, T2 &l, P * &Ptr)     
于 2013-04-01T02:13:34.360 回答