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); 
} 

下面是完整的代码（包含的主要功能）：

#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

你的功能需要两个参数：

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 2

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

您可能想要mallocrearranged或全球声明。

Answer 3

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

解决您的错误的快速入门程序是声明rearranged为static。长期的解决方法是学习C如何工作并使用malloc()或相当的代码进行编码。

Answer 4

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

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

Answer 5

而不是只返回一个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()存储结果不是很用户fiendly（用户必须freemem在（ ）缓冲区，但可能不知道这一点）。使用静态/全局缓冲区不是非常安全的。

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，即会让你的代码看起来很棒。