我的工作中有重复的代码，我想摆脱它，所以我知道在 C++ 中使用宏不是一个好主意，但我必须使用内联函数，将此函数用作内联是个好主意：
list<Data>::iterator foo(int data){
if(dataExists(data)){
list<Data>::iterator i;
for(i = dataClass.begin(); i != dataClass.end(); ++i){
if(i->getData() == data){
break;
}
return i; //here I have one more problem, what can I return if data doesn't exist?
}
我认为这个功能非常不安全。如何改进我的代码？
您在这里所做的事情已经由该std::find()函数完成，因此最好使用它（尽管在练习中尝试自己实现这些东西当然是可以的）。
std::find()还演示了一种指示“未找到”条件的好方法——如果未找到该项目，则它在结束时返回迭代器。这样，调用者可以通过将返回的迭代器与 进行比较来确定是否找到了匹配项Data.end()。
您发布的代码毫无意义。有时您将Data其用作一种类型，而另一种则用作对象。假设它是一个对象，则表明未找到某物的方法是返回一个指向其末尾的迭代器。
return Data.end();
但是您的代码太混乱了，无法在没有重大更改的情况下工作。
您不会通过在 SO 上发布问题来学习 C++ - 它根本无法完成。相反，您需要阅读一本好的 C++ 教科书——我推荐Accelerated C++，它详细介绍了迭代器等内容。如果你继续问这样的问题，你只是在浪费你的时间，更重要的是，浪费我们的时间。
我真的不知道你在这里问什么，但你的帖子中有几个误解。
第一个内联函数是一种特定的优化，你还不需要关心。先了解正常功能，正确理解。
正如另一个答案已经说过的那样，std::find()做你似乎想做的事情。它不必是列表的成员才能工作，事实上现代 C++ 样式指南通常更喜欢非成员函数。
现在到你的代码。我很确定您发布的代码不是 C++ 代码，这使得您很难理解您正在尝试做什么。您使用的列表类型也不是 a std::list<>（迭代器的工作方式不同），您Data既可以用作变量（未在代码中定义）也用作类型。
正如有人已经建议的那样，也许你应该开始更容易一些，或者更好的是，为 C++ 初学者买一本好书（我推荐“Accelerated C++”和“Programming P&P using C++”）。