我的程序输出没有在 C 中更新

Question

我的 c 代码在下面，它工作正常一次，再次调用该函数后，它给出了相同的结果。缓冲区未更新。

#define _XOPEN_SOURCE
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>

char *EpochToMMDDYYYY(long );

int main() {
    long epoch = 1492151737;
    printf("date of %ld is %s\n",epoch,EpochToMMDDYYYY(epoch));
    long ep = 1492222737;
    printf("date of %ld is %s\n",epoch,EpochToMMDDYYYY(ep));
    long epc = 1491111737;
    printf("date of %ld is %s\n",epoch,EpochToMMDDYYYY(epc));
    return 0;

}

char *EpochToMMDDYYYY(long ep)
{
    struct tm tm;
    char b[25];
    memset(b,0,sizeof(b));
    //setenv("TZ", "PST8PDT", 1);
    /* set your own time zone PST8PDT for PDT timezone */
    //tzset();

    char epoch[20];
    sprintf(epoch,"%ld",ep);

    memset(&tm, 0, sizeof(struct tm));
    strptime(epoch, "%s", &tm);
    strftime(b, sizeof(b), "%m%d%Y", &tm);
    puts(b); /* -> 04 24 2017 */
    return b;
}

输出如下

04142017
date of 1492151737 is 04142017
04142017
date of 1492151737 is 04142017
04022017
date of 1492151737 is 04022017

谁能告诉我这背后的原因，以及解决方法？

Answer 1

您正在返回一个指向函数中本地保存的内存的指针：

char *EpochToMMDDYYYY(long ep) {
    char b[25];
    ...
    return b;
}

当函数返回时，b 超出范围，不再有效。访问此类内存会导致未定义的行为，其中一种可能的未定义行为可能会产生您的输出。

为了克服这个问题，我看到了两种方法：

首先，您可以按照 BLUEPIXY 的建议将局部变量 b 更改为静态，即 char b[25]; --> static char b[25]。但是请注意，随后的调用会覆盖先前调用的结果。如果同时没有复制或最终使用结果，这可能会出现问题：

printf("%s %s", EpochToMMDDYYYY(epoch), EpochToMMDDYYYY(ep))

这将打印两次相同的值，甚至不确定是哪一次。

所以，第二，我实际上建议将写入结果的内存作为函数参数传递。修改代码如下：

void EpochToMMDDYYYY(long, char*);

int main() {
    char result[25];

    long epoch = 1492151737;
    EpochToMMDDYYYY(epoch, result);
    printf("date of %ld is %s\n",epoch,result);

    long ep = 1492222737;
    EpochToMMDDYYYY(ep, result);
    printf("date of %ld is %s\n",epoch,result);

    ...
    return 0;

}

void *EpochToMMDDYYYY(long ep, char* b) {
   ...
}