C++向量内存访问问题

Question

我有一个带有命令列表的向量，如下所示：

//COMMAND INITIALISATION
 std::vector<std::string> objectInitialisationAction;
 objectInitialisationAction.push_back("CREATE");              //0
 objectInitialisationAction.push_back("END_CREATE");          //1       
 objectInitialisationAction.push_back("START_TIMELINE");      //2

我只使用如下所示的函数访问这个向量：

int SearchFor(std::string search, std::vector<std::string> from)
{
int result=-1;
for(int i=0; i<from.size(); i++)
if(from[i]==search)
{
     result=i;
     break;
}
if(result == -1)
{
   std::ofstream error("searching.txt");
   error<<"search failed, original value = \""<<search<<"\""<<std::endl;
   error<<"values in the table:"<<std::endl;
   for(int i=0; i<from.size();i++)
   error<<"value "<<i<<": "<<from[i]<<std::endl;
   error.close();
}

return result;
}

只有一个函数调用：

commandNum=SearchFor(command[0], objectInitialisationAction);

这是我访问向量的唯一地方，但是当我第 n 次调用该函数（它总是在代码中的同一点制动）时，它访问错误并输出乱码。我在下面列出的一些代码：

    search failed, original value = "CREATE"
    values in the table:
    value 0: CREATE      Øç¼   Œ                      Ôç¼   Œ                      Ðç¼              Exit               ¼ç¼          ¸ç¼   Œ      p«üxðù   ;    ´ç¼   Œ      pëù@òø  €<    °ç¼   ŒBerlin Sans FB Demi e   ¬ç¼   ˆ°¿^nmra     œç¼   ŒBerlin Sans FB Demi e   ˜ç¼             help        ”ç¼   ˆ          object_dump ç¼             test        Œç¼   Ž          spawn       ˆç¼   ‹          load_map    „ç¼   Ž
//and so on...   

关于为什么向量可能会这样损坏的任何建议？

Answer 1

这一切似乎都是正确的。编译并执行这个。如果一切正确，则问题可能出在代码的另一部分。

#include <iostream>
#include <vector>
#include <fstream>

using namespace std;


int SearchFor(std::string search, std::vector<std::string> from)
{
    int result=-1;
    for(unsigned int i=0; i<from.size(); i++)
        if(from[i]==search)
        {
            result=i;
            break;
        }
    if(result == -1)
    {
        std::ofstream error("searching.txt");
        error<<"search failed, original value = \""<<search<<"\""<<std::endl;
        error<<"values in the table:"<<std::endl;
        for(unsigned int i=0; i<from.size(); i++)
            error<<"value "<<i<<": "<<from[i]<<std::endl;
        error.close();
    }

    return result;
}

int main()
{

    std::vector<std::string> objectInitialisationAction;
    objectInitialisationAction.push_back("CREATE");              //0
    objectInitialisationAction.push_back("END_CREATE");          //1
    objectInitialisationAction.push_back("START_TIMELINE");      //2

    for(unsigned int i=0; i<objectInitialisationAction.size(); i++)
    {
        cout<< objectInitialisationAction[i] << endl;
    }

    cout << "FOUND " << SearchFor("CREATE", objectInitialisationAction);

    return 0;
}

我建议您在文件的开头添加using namespace std;，这样您就不必在每个声明中添加std::blablabla...您的代码将更具可读性:) 请...缩进它： )

Answer 2

您的代码在我看来是正确的。在这种情况下，应用程序的另一部分应该会损坏内存。例如，某处可能存在超出范围的数组访问、悬空指针或删除后使用。 Valgrind 之类的工具可能会在这方面为您提供帮助。

Answer 3

代码看起来没问题。对我来说，最可能的情况是字符串的长度字段被无意覆盖，因为命令（即实际数据）仍然存在。只是字符串认为它比那更长。 （覆盖 characters 不会导致您报告的输出：命令将被覆盖，但字符串长度仍然很短。）覆盖内存通常通过数组索引或指针发生的界限。它指向的数据必须与字符串具有相同的链接（在您的示例中为本地或全局/静态）。

错误搜索的一种策略是偶尔打印 objectInitialisationAction 的元素字符串的长度；如果它们太长，您就知道出了点问题。

使用一种二进制搜索策略注释掉代码可能会有所帮助（注释掉一半——模拟它的功能以保持程序运行——并查看错误是否仍然存在，然后再次划分有故障的部分等。 )。

请注意，您将向量按值传递给 SearchFor()，这可能是无意的。损坏可能发生在调用方或被调用方，这应该很容易测试。--

希望对您有所帮助。

Answer 4

尝试通过 const refs 传递所有参数：

int SearchFor(const std::string& search, const std::vector<std::string>& from)
{
 ...
}