0

我写了这个函数。在调试时，我看到在函数的最后，它进入string cmonth[]声明，然后回到函数的最后，然后返回string cmonth[]声明，大约 10 次。然后它回到函数的第一行，然后回到函数的最后一行大约 100 次或更多。

int CheckLastDate(string file)
{
string line, dline[200];
int i = 0;
regex rxdate("[[:digit:]].:[[:digit:]].:[[:digit:]].");
ifstream infile;
infile.open(file.c_str());

if(! infile.is_open()) return -1;

while (infile.good())
{
    getline(infile, line);
    if(regex_search(line, rxdate))
    {
        dline[i] = line;
        i++;
    }
}
i--; //needed b/c dline starts at 0;
infile.close();

int imonth, day, hour, min, sec, year;
string month, ampm;
string cmonth[] = {"Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep", "Oct", "Nov", "Dec" };

month = dline[i].substr(5,3);
//convert month to number
for(int j = 0; j<12; j++)
{
    if(month == cmonth[j]) imonth=j+1;
}

day = atoi(dline[i].substr(9, 2).c_str());
hour = atoi(dline[i].substr(12, 2).c_str());
min = atoi(dline[i].substr(15, 2).c_str());
sec = atoi(dline[i].substr(18, 2).c_str());
ampm = dline[i].substr(21, 2);
year = atoi(dline[i].substr(24, 4).c_str());

if(ampm == "PM" && hour != 12) { hour += 12; } //turn into 24 hours
else if(ampm == "AM" && hour == 12) { hour = 0; }

time_t now, dif; //dif = date in file
double diff;
time(&now);
struct tm * timeinfo;
timeinfo = localtime(&now);
timeinfo->tm_mon = imonth - 1;
timeinfo->tm_mday = day;
timeinfo->tm_hour = hour;
timeinfo->tm_min = min;
timeinfo->tm_sec = sec;
timeinfo->tm_year = year - 1900;
timeinfo->tm_isdst = -1; //-1 = no info
dif = mktime(timeinfo);
diff = difftime(now, dif);

if(diff >= 86400) return 1; //more then 24 hours
else return 0;
}

有什么问题吗，还是 C++ 是如何工作的？感谢您的帮助。

4

2 回答 2

2

看起来不错，但有一些一般性的指示：

即使找到月份，您的 for 循环检查也会遍历所有月份，这似乎没有必要。

您应该养成始终初始化所有变量的习惯。在调试模式下，变量可能会被初始化，但在发布模式下，它们通常不会。

您应该添加一些检查以确保正则表达式返回的字符串具有您期望的格式。例如检查长度。strtok_s()如果标记用空格分隔，您也可以使用将字符串分开。它可能比期望令牌始终具有特定长度或位于特定索引处更安全。

当您从文件中读入时，如果文件超过 200 行，则不会进行检查以防止出现错误。你应该考虑这一点。例如while (infile.good() && i < 200)

于 2012-08-02T06:48:00.383 回答
0

它是调试版本还是发布版本？另外，您是在 C++ 代码级别还是在汇编程序级别进行调试的？

在发布版本中，编译器可能会生成初始函数中不存在的各种代码跳转。

于 2012-08-02T06:22:34.493 回答