C 的 printf 不打印字符串（字符数组）

Question

我的问题基本上是使用 printf 打印一个字符数组。

在某些情况下，它会打印出结果：

int main(int argc, char** argv) {
  char* orig = "@reveals#?the treasure chest#$President Barack H. Obama#";
  printf("The input: %s\n", orig);
  printf("The output: %s\n", reArrange(orig));

  return (EXIT_SUCCESS);
}

有时不会：

int main(int argc, char** argv) {
  char* orig = "@reveals#?the treasure chest#$President Barack H. Obama#";
  printf("%s\n", reArrange(orig));

  return (EXIT_SUCCESS);
}

下面是完整代码（包含main函数）：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define SUBJECT '$'
#define ACTION '@'
#define OBJECT '?'
#define END '#'

char* reArrange(char* orig) {
  int origSize = strlen(orig);

  char subject[origSize], action[origSize], object[origSize];

//int i;
//for(i = 0; i < origSize; i++) {
//  subject[i] = ' ';
//  action[i] = ' ';
//  object[i] = ' ';
//}
  int subjectIndex = 0,   actionIndex = 0,  objectIndex = 0;

  int timesEndCharShowUp = 0;

  char state;
  int i;
  for(i = 0; i < origSize; i++) {
    char ch = orig[i];

    if(ch == SUBJECT) {
      state = SUBJECT;
    }
    else if(ch == ACTION) {
      state = ACTION;
    }
    else if(ch == OBJECT) {
      state = OBJECT;
    }
    else if(ch == END) {
      if(timesEndCharShowUp == 3) {
        break;
      }
      else {
        timesEndCharShowUp++;
      }
    }
    else {
      if(state == SUBJECT) {
        subject[subjectIndex++] = ch;
      }
      else if(state == ACTION) {
        action[actionIndex++] = ch;
      }
      else if(state == OBJECT) {
        object[objectIndex++] = ch;
      }
    }
  }

  subject[subjectIndex] = '\0';
  action[actionIndex] = '\0';
  object[objectIndex] = '\0';

  char rearranged[origSize];
  sprintf(rearranged, "%s %s %s.\0", subject, action, object);
  //printf("%s\n", rearranged);

  orig = rearranged;
  return orig;
}

int main(int argc, char** argv) {
  char* orig = "@reveals#?the treasure chest#$President Barack H. Obama#";
//  printf("The input: %s\n", orig);
//  printf("The output: %s\n", reArrange(orig));
  printf("result: ");
  printf("%s\n", reArrange(orig) );
//fflush(stdout);

  return (EXIT_SUCCESS);
}

Answer 1

您正在返回一个指向驻留在堆栈上的内存的指针。 rearranged 在封闭函数 (reArrange) 返回后不可用，并且可能包含垃圾。

您可能想要malloc rearranged 或全局声明它。

Answer 2

不是只返回一个char *，而是让reArrange() 接受一个可以写入结果的缓冲区。现在调用者必须为您的函数提供合适的缓冲区，并且您不再有内存管理问题。您只需将 strncpy() 安排到缓冲区。为了确保缓冲区足够大，用户还应该在第三个参数中提供缓冲区的大小：

char *rArrange(char *orig, char *result, int resultSize)
{
    if (!result || resultSize == 0)
        return NULL;

    /* your code here */

    strncpy(result, arranged, resultSize);
    return result;
}

存储结果的替代malloc() 对用户来说不是很友好（用户必须 freemem() 缓冲区，但可能不知道这一点）。使用静态/全局缓冲区不是线程安全的。

Answer 3

你的函数需要两个参数：

char* reArrange(char* orig)

应该是：

char* reArrange(char *rearragned, char* orig) {

// make your magic here!

}

调用顺序：

char input[SIZE];
char rearrange [SIZE];

// initialize everything! Don't forget to rearrange[0] ='\0\;!!!

rearrange(rearranged, input);

// do you printing....

您还应该学习如何正确使用指针并查找“开关”。

Answer 4

问题是您的函数reArrange 返回一个指向它不再控制的内存的指针。返回rearranged 数组的地址。在返回发生后，这个数组实际上不再存在——内存可以并且将会被重用。

修复错误的快速技巧是将rearranged 声明为static。长期的解决方法是学习 C 的工作原理并使用 malloc() 或等效的代码编写一些东西。

Answer 5

使用char rearranged[origSize];，您创建了一个新的字符数组，一旦reArrange 终止，它就会超出范围。所以在reArrange 的生命周期中，rearranged 是一个指向有意义的东西的指针；因此，orig = rearranged 是有意义的。

但一旦超出范围，reArrange(orig) 将返回指向 rearranged 的指针，这是一个悬空指针，现在 rearranged 不再存在。

Answer 6

您确定以下代码部分有效吗？

char* reArrange(char* orig) {
  int origSize = strlen(orig);

  char subject[origSize], action[origSize], object[origSize];

我的意思是 origSize 必须是 const 它不能是动态值。您应该使用malloc 为subject , action and object 分配空间。 此外，您可以考虑一些准则：

1 代替：

for(i = 0; i < origSize; i++) {
    char ch = orig[i];    
    if(ch == SUBJECT) {
      state = SUBJECT;
}

你可以拥有：

char ch;//declare char ch outside for loop
for(i = 0; i < origSize; i++) {
    ch = orig[i];

    if(ch == SUBJECT) {
      state = SUBJECT;
    }

2 您可能希望使用switch 语句而不是if，这将使您的代码看起来很棒。