C++シンプルなポインタの受け渡し

Question

私はC++には新しく、すべてのポインタのものに慣れていく上でいくつか問題があります。基本的には、関数へのポインタを渡して、そのポインタで配列を作成しています。メインの関数に戻り、私はこの配列にアクセスできません。ここで

は私のコードです：

#include <iostream> 

using namespace std; 

void createArray(char** dict, int* arraysize) 
{ 
    *arraysize = 26*26*26*26; 

    delete dict; 
    dict = 0; 

    //Initialisiere character array of character 
    //char **wortliste = 0; 
    dict = new char*[*arraysize]; 
    for(int i = 0; i < *arraysize; i++) 
     dict[i] = new char[5]; 

    int ctr = 0; 

    //Erstelle Einträge (sortiert) 
    for (char i = 'A'; i <= 'Z'; i++) 
    { 
     for (char j = 'A'; j <= 'Z'; j++) 
     { 
      for (char k = 'A'; k <= 'Z'; k++) 
      { 
       for (char l = 'A'; l <= 'Z'; l++) 
       { 
        dict[ctr][0] = i; 
        dict[ctr][1] = j; 
        dict[ctr][2] = k; 
        dict[ctr][3] = l; 
        dict[ctr][4] = '\0'; 
        ctr++; 
       } 
      } 
     } 
    } 
} 

int main(void) 
{ 
    char** dict = 0; 
    int arraysize; 

    createArray(dict, &arraysize); 

    cout << dict[0] << endl << dict[arraysize-1] << endl; 

    return 0; 
} 

私は私の誤りを把握することはできません事前にどうもありがとうございました。

Answer 1

私はあなたがそのようなデータで何をするのか想像することはできませんが、少なくとも現代的な技術を使うことはできます。

#include <iostream> 
#include <vector> 
#include <string> 

using namespace std; 

vector<string> create() { 
    vector<string> result; 
    for (char i = 'A'; i <= 'Z'; ++i) { 
     for (char j = 'A'; j <= 'Z'; ++j) { 
      for (char k = 'A'; k <= 'Z'; ++k) { 
       for (char l = 'A'; l <= 'Z'; ++l) { 
        result.push_back(string() + i + j + k + l); 
       } 
      } 
     } 
    } 
    return result; 
} 

int main() { 
    vector<string> data = create(); 
    cout << data.front() << endl << data.back() << endl; 
} 

Answer 2

いくつかの間違いがあります。

1. 配列を削除するには：あなたがnot- newを反復処理しないようにするために、

   char **array = new char *[array_size]; 
   for (i = 0; i < array_size; ++i) 
       array[i] = new char[array_size_2]; 
   

2. delete ING：新しい配列へ

   char **array = /* new with whatever */; 
   /* do your work */ 
   for (i = 0; i < array_size; ++i) 
       delete[] array[i]; 
   delete[] array; 
   

3. をそれをチェックしてNULL：

   for (i = 0; i < array_size; ++i) 
   { 
       if (array[i] != NULL) /* Or simply if (array[i]) */ 
        delete[] array[i]; 
       array[i] = NULL; 
   } 
   if (array != NULL) 
       delete[] array; 
   array = NULL; 
   

   deleteはとにかくNULLのチェックを行うので、その代わりに、あなたはこれを簡素化することができます。

   if (array != NULL) 
       for (i = 0; i < array_size; ++i) 
        delete[] array[i]; /* no need to set to NULL after if going to delete the array */ 
   delete[] array; 
   array = NULL; 
   

注：deletedelete[]は、アレイを削除しながら、単一のオブジェクトを削除します。 C++のパラメータで

Answer 3

（明示的に参照パラメータとしてマークされていない限り）値渡しされているので、あなたはdictを渡し、createArrayへ（charへのポインタ）のポインタ、あなたの関数内dictが異なるオブジェクトである、とはいえ同一の初期値で、がmainにあります。 dictへの変更をmainに表示する場合は、参照渡しするか、またはそのアドレスをchar ***という関数に渡す必要があります。

など。

void createArray(char**& dict, int* arraysize) 

または

void createArray(char*** pdict, int* arraysize) 
{ // use (*pdict) instead of dict ... 

と

// ... 
createArray(&dict, &arraysize); 

あなたが望むものを達成するために、より "C++" の方法は、持っているだろう：

void createArray(std::vector<std::string>& dict); 

と単に持っていますcreateArrayのサイズを変更するを必要なサイズに設定します。 vectorやstringのような標準的なコンテナを使用すると、あなたのコードから現在割り当てられていないメモリを割り振り解除する義務がなくなります。