C++ 从C中的函数返回静态字符串是不是一种糟糕的做法

这种方法的问题是，您有一个隐藏的static状态，这使得您的函数既不是线程安全的，也不是同一表达式中的多个可调用的，包括间接的.

以下面的代码为例:

int main(int argc, char *argv[]) {
    for (int i = 0; i < 256; i++) {
        printf("%sHello, World!%s\n", getAnsiColorCode(i), getAnsiColorCode(0));
    }
}

输出将是不确定的，因为调用getAnsiColorCode的顺序是未指定的，但在所有情况下，printf输出的空间序列对于两个%s替换都是相同的.

可以返回静态字符串，只要它的值是恒定的，即:不会在调用之间改变.

有两种方法可以获得您的转义序列代码:

• 在构造序列的位置使用足够长度的参数数组

  char *getAnsiColorCode(char colorCode[static 15], int colorId) {
      snprintf(colorCode, 15), "\e[38;5;%dm", colorId);
      return colorCode;
  }
  
  int main(int argc, char *argv[]) {
      for (int i = 0; i < 256; i++) {
          char colorCode[15];
          printf("%sHello, World!\n", getAnsiColorCode(colorCode, i));
      }
  }
  

• 使用静态答案数组(不是完全线程安全的，但可以不受限制地多次调用):

  const char *getAnsiColorCode(int colorId) {
      static char colorCode[256][12];
      colorId %= 255;
      if (!*colorCode[colorId]) {
          snprintf(colorCode[colorId], sizeof colorCode[colorId],
                   "\e[38;5;%dm", colorId);
      }
      return colorCode[colorId];
  }
  
  int main(int argc, char *argv[]) {
      for (int i = 0; i < 256; i++) {
          printf("%sHello, World!\n", getAnsiColorCode(i));
      }
  }