某些垃圾值正在输出中打印。是什么原因？

Question

#include <stdio.h> 

char *getString() 
{ 
    char str[]="textprint"; 
} 

int main() 
{ 
    printf("%c",getString());  
    return 0; 
} 

在输出中接收垃圾值的原因是什么？

Answer 1

您忘记添加返回语句。 return str;

Answer 2

你可能想看看这个：

#include <stdio.h> 

char * getString(void) 
{ 
    const * str = "textprint test"; 
    return str; 
} 

int main() 
{ 
    printf("%s\n", getString());  
    return 0; 
} 

观看演示here

我跑这个片段在精而不codepad.org const关键字，但根据本article它可能有必要避免编译器的警告。

该代码创建一个字符串字面值，它存储在内存中的只读位置，指针保存其地址。当getString（）执行时，它的返回值是一个指向该字符串文字的指针。

根据上述article：

“C和C++标准说字符串文字具有静态存储”

这表明OP的代码是可赎回，修改如下：

#include <stdio.h> 

char * getString(void) 
{ 
    static char str[] = "textprint"; 
    return str; 
} 

int main() 
{ 
    printf("\n%s",getString());  
    return 0; 
} 

观看演示here

在这个例子中，创建的是一个数组，每个元素包含一个包含字符串的字符。该数组只会存在于getString（）的范围内，除了添加关键字“static”，这大大延长了该数组的寿命。因此，当getString（）在main（）中执行时，返回值是一个指向数组在内存中保持不变的地址。

请注意，getString（）必须包含一个返回语句才能使函数作为printf（）的参数进行计算。此外，没有指定关键字静态，地址将指向损坏的存储器，从而产生垃圾数据。

Answer 3

• 编译你与gcc -Wall代码可以让你几个警告
• 固定他们（制作“版本1”）让你这会告诉你你的代码
• 固定，最后警告的核心问题（警告使第2版）让你的工作和安全的代码
• 我增加了一个选择，打印什么，你可能想打印
• 我说什么你的函数的目的是 猜测（它没有说服我不猜测）
• 我添加一种安全预防措施，以防止一些更易于使

代码被静态定义的炭的阵列，串错误：

#include <stdio.h> 

// version 2 
// this is a change done after all other warnings were resolved, 
// if you try it yourself, keep the version 1 (see below) to see the 
// enlightening warning 
char str[]="textprint"; 

// example for "localisation" strings 
char strFr[] = "imprimertexte"; 


const char *getString(void) // better signature 
// (with void as parameter list) for function, for good practice; 
// the keyword "const" prevents changing the global string, 
// which is a safety precaution 
{ 
    // version 1 char str[]="textprint"; 
    // this version 1, after all the other warnings are resolved, 
    // gets the enlightening warning 
    // "function returns address of local variable", 
    // so this got moved somewhere else for version 2 

    // Returning a pointer to a global variable/text from a function 
    // is not obviously useful - at first glance. 
    // One reason could be the design to be able to return different 
    // pointers to different strings, depending on some condition. 
    // I can imagine for example localisation of the texts. 
    // That could look like 
    // if(localisation == English) {return str;} 
    // else {return strFr;} 

    return str; // to avoid warnings 
    // a) "unused variable" 
    // b) "control reaches end of non-void function" 
} 

int main(void) // better signature for main, good practice 
{ 
    printf("%c",*getString()); // change what to print, in order to avoid warning 
    // "format '%c' expects type 'int', but argument 2 has type 'char *' " , 
    // by changing from "pointer to char" to 
    // "what a pointer to char points to", i.e. "char" 

    printf("\n"); // print a newline for better readable output 

    // Alternative, for printing the complete text: 
    printf("%s\n", getString()); 

    // this attempts to change the global string 
    // with the "const" keyword at the function head this does not work 
    // it gets: "error: assignment of read-only location '*getString()'" 
    // *getString()='T'; 

    return 0; 
}