2次元のchar配列C++を返す方法は？

Question

私は関数内で2次元配列を作成しまし

、私はその配列を返すようにしたい、と他の関数にどこかにそれを渡す。..

char *createBoard(){ 
    char board[16][10]; 
    int j =0;int i = 0; 
    for(i=0; i<16;i++){ 
     for(j=0;j<10;j++){ 
       board[i][j]=(char)201; 
     } 
    } 
    return board; 
} 

が、これは私にエラー

Answer 1

うん、あなたがスタック上に作成されたオブジェクト（boardという配列）へのポインタが返却されて何をしているかを参照してください。スコープから外れると、配列は破棄されるため、ポインタはもはや有効なオブジェクト（ダングリングポインタ）を指していません。

あなたは、配列がnewを使用して、代わりにヒープに割り当てられていることを確認する必要があります。現代のC++で動的に割り当てられた配列を作成するための聖化された方法は、std::vectorクラスのようなものを使用することですが、2D配列を作成しようとしているのでここではもっと複雑です。

char **createBoard() 
{ 
    char **board=new char*[16]; 
    for (int i=0; i<16; i++) 
    { 
     board[i] = new char[10]; 
     for (int j=0; j<10; j++) 
     board[i][j]=(char)201; 
    } 

    return board; 
} 

void freeBoard(char **board) 
{ 
    for (int i=0; i<16; i++) 
     delete [] board[i]; 
    delete [] board; 
} 

Answer 2

を与え続けてあなたが代わりにchar**を返す必要がありますchar*

Answer 3

この方法は機能しません。ローカル変数へのポインタを返すと、未定義のビヘイビアが実行されます。代わりにというヒープに配列を割り当て、手動でインデックスを作成してデータをコピーします。

Answer 4

あなたの質問に対する簡単な答えはchar **です。

言われているように、しないでください！ "board"変数はcreateBoard（）の外側にはありません。

使用boost::multi_arrayと（あなたがそれを行う場合は、それがコピーされます）createBoardへの参照（）として渡したり、直接それを返します。

Answer 5

私は本当にSTLベクトル<を使用することをお勧めします>や/このためmulti_arrayコンテナを高めるでしょう。

アレイを使用する必要がある場合は、typedefを使用して配列を定義することをお勧めします。

typedef char[16][10] TBoard; 

また

char** 

を返すことができます...しかし、あなたは正しく、インデックス、それをするために、適切なサイズにそれを型キャストする必要があります。 C++では、動的な複数次元配列はサポートされていません。

また他の人はあなたが（すなわち、ローカル変数）をスタック上のオブジェクトを返すことができません示唆しているよう

Answer 6

このスペースは、すぐに関数として上書きされますので、あなたはローカル変数関数へのポインタを返すしてはなりません戻る。

ボードに関連付けられたストレージは、関数のスタック上にあります。

Answer 7

最善のアプローチは、ボードのクラスを作成し、ctreateBoardは、そのコンストラクタに機能しますです、このようなクラスを使用する方法については

class Board { 
    private: 
    char mSquares[16][10]; 

    public: 
    Board() { 
     for(int i=0; i<16;i++){ 
     for(int j=0;j<10;j++){ 
       mSquares[i][j]=201; 
     }  
    } 

    // suitable member functions here 
}; 

を、良い本を読みに勝るものはありません。 Andrew KoenigとBarbra MooのAccelerated C++を強くお勧めします。

Answer 8

他に述べたように、ローカル変数へのポインタを返さないでください。あなたが達成したいことを私が強制されたら、まずはstd :: vectorに行きます。あなたはstd :: vectorを学習していないので、別の方法があります：

void createBoard(char board[16][10]) 
{ 
    int j =0;int i = 0; 
    for(i=0; i<16;i++){ 
     for(j=0;j<10;j++){ 
       board[i][j]=(char)201; 
     }  
    } 
}