C言語 文字列のポインタの配列を文字列にするには

Question

タイトルの通りです。

自分で考えたコードは以下です。流れは、

NULLになるまでargvの配列を前から順にとる
そのargv[i]が終端文字になるまで処理

その後の処理がうまくいきません。
どのようにすれば良いでしょうか。

#include<stdio.h>
void to_string(char **argv, char *buffer){
    int i = 0;
    while(argv[i] != NULL){
        while(*argv[i]){
            buffer = argv[i];
            printf("%c\n",*buffer);
            buffer++;
            *argv[i]++;
        }
        i++;
    }
    printf("%s",buffer);
}

int main(void){
    char *argv[] = {"abcde","fghi",NULL};
    char *buffer ;

    to_string(argv,buffer);
    return 0;
}

Answer 1

#include<stdio.h>
void to_string(char **argv, char *buffer){
    int i = 0;
    while(argv[i] != NULL){
        while(*argv[i]){
-           buffer = argv[i];
+           *buffer = *argv[i];
            printf("%c\n",*buffer);
            buffer++;
            *argv[i]++;
        }
        i++;
    }
-   printf("%s",buffer);
}

int main(void){
    char *argv[] = {"abcde","fghi",NULL};
-   char *buffer ;
+   char buffer[256] ;

    to_string(argv,buffer);
+   printf("%s",buffer);
    return 0;
}

Comments

[prof]

回答していただきありがとうございます。

質問ですが、あらかじめ何らかの文字列が入るだろうというときには、バッファオーバーフローを起こさないために、大きい文字の配列をとっておく、ということであっていますでしょうか？

もともと

```c
- buffer = argv[i];
+ *buffer = *argv[i];
```

にしていましたが、bufferの置き方を間違えておりセグメンテーションエラーとなっていました。

[ozwk]

はい

Answer 2

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

void to_string(char **argv, char *buffer){
    *buffer = '\0';
    while ( *argv != NULL){
        strcat(buffer, *argv++);
    }
    printf("%s",buffer);
}

int main(void){
    char *argv[] = {"abcde","fghi",NULL};
    char buffer[256]; // 十分な領域を用意すべし

    to_string(argv,buffer);
    return 0;
}