1

对不起，制作一个完全可重现的错误版本是非常困难的——所以请用我的原理图代码。

该程序从网页中检索信息，对其进行处理，并将输出保存到 ASCII 文件中。我还有一个“日志”文件（FILE *theLog---包含在一个Manager对象中）用于报告错误等。

一些后台方法：

// Prints string to log file
void Manager::logEntry(const string lstr) {
    if( theLog != NULL ) { fprintf(theLog, "%s", lstr.c_str()); }
}

// Checks if file with given name already exists
bool fileExists(const string fname) {
    FILE *temp;
    if( temp = fopen(fname.c_str(), "r") ) { 
        fclose(temp);
        return true;
    } else { return false; }
}

// Initialize file for writing (some components omitted)...
bool initFile(FILE *&oFile, const string fname) {
    if(oFile = fopen(fname.c_str(), "w") ) { return true; }
    else { return false; }
}

造成麻烦的东西：

// Gets data from URL, saves to file 'dataFileName', input control flag 'foreCon'
//                     stu is some object that has string which i want
bool saveData(Manager *man, Stuff *stu, string dataFileName, const int foreCon) {
    char logStr[CHARLIMIT_LARGE];          // CHARLIMIT_LARGE = 2048
    sprintf(logStr, "Saving Data...\n");
    man->logEntry( string(logStr) );       // This appears fine in 'theLog' correctly

    string data = stu->getDataPrefixStr() + getDataFromURL() + "\n";        // fills 'data' with stuff
    data += stu->getDataSuffixStr();

    if( fileExists(dataFileName) ) {
        sprintf(logStr, "save file '%s' already exists.", dataFileName.c_str() );
        man->logEntry( string(logStr) );
        if( foreCon == -1 ) {
            sprintf(logStr, "foreCon = %d, ... exiting.", foreCon);        // LINE 'A' : THIS LINE ENDS UP IN OUTPUT FILE
            tCase->logEntry( string(logStr) );
            return false;
        } else {
            sprintf(logStr, "foreCon = %d, overwriting file.", foreCon);   // LINE 'B' : THIS LINE ENDS UP IN LOG FILE
            tCase->logEntry( string(logStr) );
        }                                                                                                 
    }

    // Initialize output file
    FILE *outFile;
    if( !initFile(outFile, dataFileName) ) {
        sprintf(logStr, "couldn't initFile '%s'", dataFileName.c_str());
        tCase->logEntry( string(logStr) );
        return false;
    }

    fprintf(outFile, "%s", data.c_str());                 // print data to output file

    if( fclose(outFile) != EOF) {
        sprintf(logStr, "saved to '%s'", dataFileName.c_str());
        tCase->logEntry( string(logStr) );
        return true;
    }

    return false;
}

如果文件已经存在，并且“ int foreCon= -1”，那么代码应该将“A”行打印到日志文件中。如果文件存在并且foreCon!= -1，则旧文件将被覆盖data。如果文件不存在，则创建它，并将数据写入其中。

然而，结果是数据文件中出现了行'A'的分解版本，并且日志文件中打印了行'B'！！！！

数据文件的样子：

.. exiting.20130127 161456
20130127 000000,55,17,11,0.00
20130127 010000,54,17,11,0.00
... ...

第二行及以后看起来是正确的，但有一个额外的行包含行“A”的一部分。

现在，真正奇怪的部分。如果我注释掉if( foreCon == -1) { ... }块中的所有内容，则数据文件如下所示：

%d, ... exiting.20130127 161456
20130127 000000,55,17,11,0.00
20130127 010000,54,17,11,0.00
... ...

还有一个额外的行，但它是复制到数据文件中的 LITERAL CODE。

我认为我的代码中有一个 poltergeist。我不明白这怎么可能发生。

编辑：我尝试打印来控制台data字符串，它给出了相同的混乱值：即%d, ... exiting.20130127 161456- 所以它必须是关于string而不是FILE *

4

3 回答 3

2

根据您的最新评论回答：

getDataPrefixStr() 最终返回一个字符串，该字符串以字符串 retStr = COMCHAR + " file created on ..."; 开头 这样 const char COMCHAR = '#';。COMCHAR 可能是问题吗？

您不能像这样添加字符和字符串文字（它们是 s 的数组char，而不是strings 的数组）。

您将 35（“#”的 ASCII）添加到“file created on ...”的地址，即 getDataPrefixStr() 是从该字符串开头的 35 个字符开始的任何内容。由于所有文字字符串都存储在同一数据区域中，因此您将在输出中从程序中获取字符串。

相反，你冷做

   const string COMCHAR = "*"; 
   string retStr = COMCHAR + " file created on ...";
于 2013-01-27T22:00:38.493 回答
1

可能logStr是太短了，导致数据在其他缓冲区中被覆盖（您是否仔细检查过CHARLIMIT_LARGE？）。您可以通过评论所有写入logStr( sprintf) 来诊断此问题，并查看是否data仍然损坏。一般来说，如果用户可以设置dataFileName（很长的字符串），您的代码很容易受到攻击；使用snprintforostringstream代替。

否则，我会猜测要么 要么stu->getDataPrefixStr()正在getDataFromURL()返回损坏的结果或返回类型char*而不是string. 尝试将这些值直接打印到控制台以查看它们是否已损坏。如果他们返回 a char*，那么data = stu->getDataPrefixStr() + getDataFromURL()将有未定义的行为。

于 2013-01-27T21:53:51.240 回答
-1 
if( temp = fopen(fname.c_str(), 'r') ) {

应该

if( temp = fopen(fname.c_str(), "r") ) {
于 2013-01-27T21:07:43.510 回答