16

ビンゴボードを生成する関数を作成しましたが、ビンゴボードを返却したいと思います。

期待していなかったので、うまくいきません。

関数は次のとおりです。

int** generateBoard() {
    int board[N][M], i, j , fillNum;
    Boolean exists = True;
    // initilize seed
    srand(time(NULL));
    // fill up..
    for(i = 0; i < N; ++i) {
        for(j = 0; j < M; ++j) {
            exists = True;
            while(exists) {
                fillNum = rand()%MAX_RANGE + 1; // limit up to MAX_RANGE
                if(beenAdded(board, fillNum) == Exist) {
                    continue;
                } else {
                    board[i][j] = fillNum;
                    exists = False;
                }
            }
        }
    }
    return board;
}

「リターンボード」行にコンパイルエラー（赤いサブライン）があります。

構造体\動的割り当てを使用せずに2D配列を返す方法はありますか？

Microsoft Visual C ++Express2010を使用しています。

4

3 に答える 3

20

ローカル変数として定義boardしました-関数がスコープ外になると、そのメモリの割り当てが解除されます。

ボードをグローバルに宣言することも、次のように動的に作成することもできます。

int **allocate_board(int Rows, int Cols)
{    
    // allocate Rows rows, each row is a pointer to int
    int **board = (int **)malloc(Rows * sizeof(int *)); 
    int row;

    // for each row allocate Cols ints
    for (row = 0; row < Rows; row++) {
        board[row] = (int *)malloc(Cols * sizeof(int));
    }

    return board;
}

ボードを動的に解放する必要があります。

// you must supply the number of rows
void free_board(int **board, int Rows) 
{
    int row;

    // first free each row
    for (row = 0; row < Rows; row++) {
         free(board[row]);
    }

    // Eventually free the memory of the pointers to the rows
    free(board);
 }
于 2012-12-30T06:53:08.513 に答える
17

誰かがそのボードのメモリをどこかで所有している必要があり、さらに重要なことに、その所有権はこの関数の呼び出し元にまで遡る必要があります。動的割り当てがない場合、他の唯一の実際の代替手段は、in/outパラメーターのように関数に送信することです。

void generateBoard(size_t N, size_t M, int board[N][M])
{
    int i, j , fillNum;
    Boolean exists = True;
    // initilize seed
    srand(time(NULL));
    // fill up..
    for(i = 0; i < N; ++i) {
        for(j = 0; j < M; ++j) {
            exists = True;
            while(exists) {
                fillNum = rand()%MAX_RANGE + 1; // limit up to MAX_RANGE
                if(beenAdded(board, fillNum) == Exist) {
                    continue;
                } else {
                    board[i][j] = fillNum;
                    exists = False;
                }
            }
        }
    }
}

発信者から次のように呼び出します。

int main()
{
    const size_t N = 10;
    const size_t M = 10;
    int board[N][M];

    generateBoard(N,M,board);

    ...
}

srand()また、のスタートアップコードへの呼び出しを再配置することも検討しmain()ます。理想的には、繰り返し呼び出し可能な関数に含まれることはなく、プロセスの実行ごとに1回だけ実行されることが保証されている必要があります。（注：スレッドの実行ごとに1回かどうかは正直覚えていませんが、コーディング学習曲線のこの時点では、マルチスレッドはまだ注目されていないと思います）。

最後に、ランダムフィルループは不必要に繰り返されます。あなたが明らかにしようとしていることを生成するより良い代替案があります：既存の数のセットのランダム順列を作成します。MAX_RANGE書かれているように、と比較してどれだけ大きいかに応じて、最後のいくつかのスロットを埋めようとしてしばらくの間スピンすることができます(N*M)。

于 2012-12-30T06:37:06.397 に答える