C言語　動的にメモリを割り当てて、文字列のポインタの配列を文字列にするには

例えば、{"abcde","fghi",NULL}というような文字列ポインタのリストを受け取り、
その文字列を一つにつなげて、ポインタを返したいです。

演算を⾏った結果を格納する構造体を動的に割り当てる、ということなのですが、新たに構造体を定義して割り当てるということなのでしょうか？この点はコードに実装していません。

以下、考えたコードです。自分で詰まっているところや、している事にコメント入れています。

C
1#include <string.h>
2#include <stdio.h>
3#include <stdlib.h>
4
5
6
7char *function(char **argv){
8
9    // リスト内の文字列の文字数を足し合わせる
10    int i = 0,len = 0;
11    while(argv[i] != NULL){
12        len += strlen(argv[i]);
13        i++;
14    }
15
16
17    printf("%d\n",len);
18    char *buffer = malloc(len + 1);
19    // エラーハンドリング
20    if(buffer != NULL){
21        int i = 0;
22        // １文字ずつ文字列をつなげる
23        while(argv[i] != NULL){
24            while(*argv[i] != '\0'){
25                *buffer = *argv[i];
26                buffer++;
27                *argv[i]++;
28            }
29            i++;
30        }
31        // 開放したら値を返しても、メモリの割当場所が不定になる？
32        free(buffer);
33        return buffer;
34    }else{
35        return NULL;
36    }
37}
38
39int main(void){
40    char *argv[] = {"abcde","fghi",NULL};
41    char *buffer = function(argv);
42    printf("%s",buffer);// つなげた文字列を出力したい
43    return 0;
44}

行動規範の内容に同意します

回答2件

ベストアンサー

while(argv[i] != NULL){

このループの中で、 bufferのアドレスが更新されています。
このため、

    free(buffer);
    return buffer;

が不正になります。最初の bufferのアドレスを保持して、return するようにしましょう。
と思ったが、その前に free(buffer)すると確保されたメモリ領域が解放され、不正となりますね。
free()は、関数から、戻った後(main)にしましょう。

投稿2021/04/24 08:04

pepperleaf

総合スコア6385

prof

2021/04/24 08:53

回答頂きありがとうございます。 - 最初の bufferのアドレスを保持して、return するこの点、見落としていました。 free()する場所も間違えていました。

行動規範の内容に同意します

確保した領域の先頭を返していない
使い終わるまでfreeしちゃダメ

C
1#include <string.h>
2#include <stdio.h>
3#include <stdlib.h>
4
5char *function(char **argv) {
6    int i = 0,len = 0;
7    while(argv[i] != NULL){
8        len += strlen(argv[i]);
9        i++;
10    }
11
12    printf("%d\n",len);
13    char *buffer = malloc(len + 1);
14    char* result = buffer;
15
16    if(buffer != NULL){
17        int i = 0;
18        while(argv[i] != NULL){
19            while(*argv[i] != '\0'){
20                *buffer = *argv[i];
21                buffer++;
22                argv[i]++;
23            }
24            i++;
25        }
26        return result;
27    }else{
28        return NULL;
29    }
30}
31
32int main(void){
33    char *argv[] = {"abcde","fghi",NULL};
34    char *buffer = function(argv);
35    printf("%s",buffer);
36    free(buffer); // freeするのはココ
37    return 0;
38}