main関数内でmallocで文字列をコピーする

Question

メイン関数内でmallocを用いて別のやり方を検討しました。今回は配列を用いて、関数なしで、新たに生成された配列をコピーするというプログラムです。
前回の質問通りにmain関数で実行したところ、while文が実行されなくて条件がfalseとみなされて、ﾍﾍﾍﾍﾍﾍﾍﾍﾍﾍといった文字がコマンドプロンプトに出力されてコピーできない状態となっております。while文内でブレークポイントを入れても素通りしてしまいます。つまり、mallocでaのバイト数を確保したにもかかわらずfalseを満たすのはどうしてでしょうか。

プログラムコードは前回同様にポインタ変数resultを用いて、先頭のアドレスを確保し、while文内で、0が出でくるまで、aの文字列の値をcに代入させています。

そこで、 printf("%s\n", result);でcのアドレスの先頭部分であるポインタ変数resultで文字列のコピーを出力します。そこでいらなくなったメモリ領域を解放しています。
といったやり方でmallocでの確保した領域を文字列aをコピーします。
どうかご協力を願います。

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


    

char  main(void ) {

    int i=0;
     char* c;
   char a[10] =  "aaaaa" ;
    
   
    for (i = 0; a[i] != '\0'; ++i);


    c = (char*)malloc(sizeof(a)); 
    char* result = c;


    while (a[i] != '\0') {
        c[i] = a[i];
        i++;
      
    }
   

    printf("%s\n", result);
    free(result); 
    return 0;
}

Comments

[kazuma-s]

char main を int main にしてください。
c はポインタだけど、値を変更しないから、result にコピーしてそれを free しなくても
free(c); で済みます。

Answer 1

while (a[i] != '\0')ですが、最初のforループでiが進んでいるため、a[i]はちょうど'\0'になる位置を指しています。つまり、このループは一度も実行されません。

Comments

[退会済みユーザー]

先頭のアドレスを確保するために、新たなchar型の変数を用意する必要がありますか

[maisumakun]

ありません。最初の無駄なfor文を削除するだけで動くようになるかと思います。

[e-watt]

while の直前に
printf("i=%d\n", i);
と追加してみればやるべきことが解かると思います。

（そこが解決したうえで、）ついでにwhileから抜けるときにも問題があるので、c[0]からc[どこまで見るべきかも要確認>]の各要素にどんな値を入れなければならないかも確認する必要がりますよ。

（そして老婆心ながら、）そこも解決したうえで malloc(i); なんて書き換えると（気づかないかもしれないけれど）別の（似たような）誤りを犯すことになってしまいますのでご注意召され。