-1

多次元配列を再割り当てするときのコードは次のとおりです。関数 add_line を複数回使用すると、コードが機能しません。一日中それを理解しようとしていました。誰か助けてくれませんか？

void add_line(char ** wlist, char * word, int * size) // Extending wordlist or cross
{
    (*size)++;
    char ** new_wlist = (char**)realloc(wlist,(*size)*sizeof(char*));
    if(new_wlist == NULL)
        show_error("Reallocation error",1);

    wlist = new_wlist;
    wlist[(*size)-1] = (char*)malloc(ROW_SIZE*sizeof(char));
     if(strlen(word)>ROW_SIZE)
        show_error("Word too long", 1);
    strcpy(wlist[(*size)-1],word);
}
int main()
{
    int * w_size = (int*)malloc(sizeof(int));
    int * c_size = (int*)malloc(sizeof(int));
    *w_size = 0;
     *c_size = 0;
    char ** wordlist = (char**)malloc(sizeof(char*));
    char ** cross = (char **)malloc(sizeof(char*)); 

    add_line(cross,"test1",c_size);
    add_line(cross,"test2",c_size);
    return 0;
}
4

1 に答える 1

1

問題は、変更されたものを返していないことですwlist-これはコードの修正された（ただしテストされていない）バージョンです。

void add_line(char *** wlist, const char * word, int * size) // Extending wordlist or cross
              //   ^^^ note extra level of indirection here
{
    int new_size = *size + 1;
    char ** new_wlist = realloc(*wlist, new_size*sizeof(char*));
    if (new_wlist == NULL)
        show_error("Reallocation error",1);

    new_wlist[new_size-1] = malloc(ROW_SIZE);
    if (strlen(word)>ROW_SIZE)
        show_error("Word too long", 1);
    strcpy(new_wlist[new_size-1],word);
    *wlist = new_wlist;
    *size = new_size;
}

int main()
{
    int c_size = 0; // NB: no need for dynamic allocation here

    char ** cross = NULL; // NB: initial size is zero - realloc will do the right thing

    add_line(&cross, "test1", &c_size);
          // ^ pass pointer to cross here
    add_line(&cross, "test2", &c_size);
          // ^ pass pointer to cross here

    return 0;
}

他にもいくつかの小さな問題を修正しました。 の初期サイズcrossは 0 になりました (1 でした) c_size。sizeof(char)また、C では潜在的に危険な不要なキャストと、 (定義により 1 に等しい)の冗長な使用を削除しました。

于 2013-08-12T15:38:18.280 に答える