0

私がやろうとしているのは、文字列の2次元配列を作成することです。次のセグメンテーション違反は即座に発生しますが、何が問題になっていますか？

void add2(char***b, char *i)
{
    if (!i) {
       b[0][0] = (char*) malloc(120);
       sprintf(b[0][0], "%s", "hithere");
       b[0][1] = (char*) malloc(120);
       sprintf(b[0][1], "%s", "bithere");
    } else {
       strcat(b[0][0], "\\\\");
       strcat(b[0][0], i);
       strcat(b[0][1], "\\\\");
       strcat(b[0][1], i);
    }

}
void add1(char ***b)
{
 add2(b,NULL);
 add2(b,"one");
 add2(b,"two");
 add2(b,"three");
}

int main()
{
 char **keys[2] = {0};
 int i,j;

 add1(keys);

 for (i = 0; keys[i]; i++)
     for (j = 0; keys[j]; j++)
     {
         fprintf(stderr, "%s\n", keys[i][j]);
         free(keys[i][j]);
     }

}
4

2 に答える 2

2

配列キーを宣言するときは、charへのポインターへの2つのポインターの配列を処理し、charへのポインターからNULLへのポインターを初期化するようにコンパイラーに指示します。すべて良い。

次に、を呼び出しますadd1()。すべて良い。

次に、を呼び出して、からの戻り値add2()の最初の要素に入れようとします。ただし、の値はNULLです。NULLが関数に入れられました。要素はありません！b[0]malloc()b[0]main()b[0]

ポインタの形で（配列の...）配列の配列がある 場合は、すべてのレベルを個別に（および）する必要があります。malloc()free()

編集

#include <stdlib.h>

int main()
{
    char **keys[2] = {0};
    keys[0] = malloc(20 * sizeof *keys[0]); /* 20 strings */
    keys[1] = malloc(20 * sizeof *keys[1]); /* 20 strings */

    for (int k=0; k<20; k++) {
        keys[0][k] = malloc(120); /* string with 119 chars + '\0' */
        keys[1][k] = malloc(120); /* string with 119 chars + '\0' */
    }

    /* use k[0][0] through k[0][19] */
    /* use k[1][0] through k[1][19] */

    for (int k=0; k<20; k++) {
        free(keys[0][k]);
        free(keys[1][k]);
    }
    free(keys[0]);
    free(keys[1]);

    return 0;
}

私はそれをすべて関数に入れましたが、 sとsがそれぞれの関数にmain()ある場合も同じ考えです。malloc()free()

于 2009-10-06T20:16:40.937 に答える
1

の2D配列を動的に割り当てるための一般的な手順char *は、次のようなものです（他のタイプTchar *の場合は、目的のタイプに置き換えます）。

char ***new2DArr(size_t rows, size_t cols)
{
  char ***newArr = NULL;
  size_t i;
  newArr = malloc(sizeof *newArr * rows);
  if (newArr)
  {
    for (i = 0; i < rows; i++)
    {
      newArr[i] = malloc(sizeof *newArr[i] * cols);
      if (newArr[i])
      {
        /* initialize or assign newArr[i][0] - newArr[i][cols-1] here */
      }
    }
  }
  return newArr;
}

これは、必要な行と列の数が事前にわかっていることを前提としています。ポインタのテーブルを割り当てていることに注意してください。次のように、文字列エントリごとにメモリを割り当てる必要があります。

char **myArr = new2DArr(10, 10);
myArr[0][0] = malloc(strlen("Hello, World") + 1);
if (myArr[0][0])
{
    strcpy(myArr[0][0], "Hello, World");
}

既存の行に新しい行または新しいエントリを追加する場合は、追加の簿記を行う必要があります。

単一の行を拡張する1つの例を次に示します。

char **extendRow(char **row, char *newEntry, size_t *numEntries)
{
  char **tmp = realloc(row, *numEntries + 1);
  if (tmp)
  {
    row = tmp;
    row[*numEntries] = malloc(strlen(newEntry) + 1);
    if (row[*numEntries])
    {
      strcpy(row[*numEntries], newEntry);
      (*numEntries)++;
    }
  }
  return row;
}

そして、あなたはそれを次のように呼ぶでしょう

table[i] = extendRow(table[i], "This is a test", &entryCount);

table[i]ポインタ値がによって変更された場合、結果はに割り当てられrealloc()ます。ポインターの体操を最小限に抑えるために、table [i]にポインターを渡す代わりに、この方法で実行しています。

于 2009-10-08T12:42:47.540 に答える