即使使用易失性，没有线程，变量也不会更新

Question

我有以下printf语句，它打印函数的输出：

printf("\nStart-Time %s,End-Time %s",
       get_time(myfields[i].start_mn),
       get_time(myfields[i].end_mn));

在语句中， get_time函数使用不同的参数被调用两次。 但是，即使函数在两次调用中返回不同的字符串， printf只打印第二次调用的第一次调用的返回值。 我已经为函数返回变量和函数本身尝试了volatile关键字，但输出仍然相同。 但是，如果我将printf语句拆分为两个printf语句以分别打印值，它们将打印出预期的不同值。

那么任何人都可以指出我这里发生了什么以及这样做的正确方法是什么？

---

[来自评论的更新：]

get_time正在返回一个全局char数组。 char *get_time(int tval)将时间以分钟为单位转换为小时。

Answer 1

那么任何人都可以指出我这里发生了什么以及这样做的正确方法是什么？

每次调用get_time()都会将相同的地址返回到相同的缓冲区，然后将这些地址传递给printf() ，它会在这个缓冲区中找到最后放入的内容，“last”表示“最后的时间” .

要解决此问题，请创建临时缓冲区以传递给printf() ：

char * p1 = strdup(get_time(...));
char * p2 = strdup(get_time(...));

printf("\nStart-Time %s,End-Time %s", p1, p2);

/* Free the temporary buffers. */
free(p1);
free(p2);

如果您提前知道get_time()返回的最大大小，则可以使用更简单的方法：

#define GET_TIME_LEN_MAX 32

...

char b1[GET_TIME_LEN_MAX];
char b2[GET_TIME_LEN_MAX];

strcpy(b1, get_time(...));
strcpy(b2, get_time(...));

printf("\nStart-Time %s,End-Time %s", b1, b2);

后一个例子的安全版本：

#define GET_TIME_LEN_MAX 32

...

/* Define buffers and initialise them to ALL zeros. */
char b1[GET_TIME_LEN_MAX] = "";
char b2[GET_TIME_LEN_MAX] = "";

strncpy(b1, get_time(...), GET_TIME_LEN_MAX - 1);
strncpy(b2, get_time(...), GET_TIME_LEN_MAX - 1);

printf("\nStart-Time %s,End-Time %s", b1, b2);

---

由于strdup()不是标准 C，为了完整strdup() ，请在下面找到一个本土实现：

#include <stdlib.h> /* for malloc() */
#include <string.h> /* for strcpy() */


char * strdup(const char * s)
{
  char * p = NULL;

  if (NULL != s)
  {
    p = malloc(strlen(s) + 1);
    if (NULL != p)
    {
      strcpy(p, s);
    }
  }

  return p;
}