偽の値を返すポインタ関数

Question

パラメータxがそれでもうまくいきません。たとえば、私がprintf("%s",maxCharac(2))をmainに入力すると、aaとその隣に余分な文字が表示されます。奇数では正しく動作します。

char *maxCharac(int x) 
{ 
    char *str=(char*)malloc(sizeof(char)*x); 
    for(int i=0;i<x;i++) 
    {    
     str[i]='a'; 
    } 
    return str; 
} 

Answer 1

C文字列はそう

char *maxCharac(int x) 
{ 
    char *str = malloc(x + 1); 

    if (str != NULL) 
    { 
     for (int i = 0; i < x; i++) 
     { 
      str[i] = 'a'; 
     } 
     str[i] = '\0'; 
    } 

    return str; 
} 

、終了NULです：

は

1. あなたはnull終端'\0'malloc(x + 1);
2. sizeof(char)のための余地を残しておく必要があります常に1標準
あたりです
3. あなたはチェックmalloc&co戻り値!= NULLを使用してください。

あるいは、割り当てられたメモリ

char *maxCharac(int x) 
{ 
    char *str = calloc(x + 1, 1); 

    if (str != NULL) 
    { 
     for (int i = 0; i < x; i++) 
     { 
      str[i] = 'a'; 
     } 
    } 

    return str; 
} 

最後のものをゼロにあなたがcallocを使用することができ、最後の命令を回避するために、機能ごとに、呼び出し側は、可能なを使用しないように保証するために、関数の戻り値をチェックする必要がありますNULLポインタが返さ：

int main(void) 
{ 
    char *str = maxCharac(2); 

    if (str != NULL) 
    { 
     printf("Test: %s\n", str); 
    } 
} 

Answer 2

あなたがそれをNULLで終了するために余分な文字を必要としてあなたは、文字列のための十分なメモリを割り当てていないので、この のように1つの余分を割り当てても、最後の文字はNULであることを確認します。最後に余分なNUL文字なし

char *str=(char*)malloc(sizeof(char)*(x+1)); 
for(int i=0;i<x;i++){ 
    str[i]='a'; 
} 

str[x]='\0'; 

、あなたは未定義の動作を経験している - それはNUL文字に遭遇するまで、あなたのコードは、文字列の終わりを超えて読み続けます。 xがあなたのストリングを終わらせるために正しいポイントで奇妙なときあなたが1つを見ていることは純粋な運です。あなたが見ることができるように