cpp 文件中的部分行最终出现在输出文件中 - 闹鬼的代码

Partial line from cpp file ending up in output file - haunted code?

本文关键字:文件 输出 代码 闹鬼 cpp      更新时间:2023-10-16

很抱歉,要制作一个完全可重现的错误版本是非常困难的---所以请裸露我的原理图代码。

该程序从网页检索信息,对其进行处理,并将输出保存到 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
... ...

仍然有一行额外的行,但它是复制到数据文件中的文字代码。

我认为我的代码中有一个恶作剧者。 我不明白这些怎么会发生。


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

根据您的最新评论回答:

getDataPrefixStr() 最终返回一个以 类似于字符串 retStr = COMCHAR + " 在...上创建的文件";这样 const char COMCHAR = '#';.COMCHAR会是问题吗?

你不能像那样添加字符和字符串文字(它们是 char 的数组,而不是 string s)。

您要将 35("#"的 ASCII)添加到" 在 ...",即getDataPrefixStr()是从该字符串的开头开始的35个字符。由于所有文本字符串都一起存储在同一个数据区域中,因此您将在输出中从程序中获取字符串。

相反,你冷淡地做

   const string COMCHAR = "*"; 
   string retStr = COMCHAR + " file created on ...";
可能是

logStr太短,导致数据在其他缓冲区中被覆盖(您是否仔细检查CHARLIMIT_LARGE?您可以通过将所有写入注释到logStrsprintf)来诊断此问题,并查看data是否仍然损坏。通常,如果用户可以将 dataFileName(设置为很长的字符串),则您的代码容易受到此影响;请改用snprintfostringstream

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

if( temp = fopen(fname.c_str(), 'r') ) { 

应该是

if( temp = fopen(fname.c_str(), "r") ) {