Wie die Rückkehr der matrix (2D-array) in der Funktion? (C)

Erstellte ich eine Funktion, die erzeugt ein bingo board und ich wollen die Rückkehr der bingo-board.

da ich nicht erwartet habe , es funktioniert nicht.

hier ist die Funktion:

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;
}

Ich habe eine compilcation Fehler (die rote subline) bei "return-board" - Linie.

ist es Weg, um wieder ein 2D-array ohne mit structs \ dynamische Zuordnungen?

Ich bin mit Microsoft Visual C++ Express 2010.

InformationsquelleAutor der Frage Billie | 2012-12-30

  1. 13

    Jemand besitzen muss, um die Erinnerung an dieses board irgendwound noch wichtiger ist, dass das Eigentum erweitern müssen zurück an den Aufrufer dieser Funktion. Ohne die dynamische Zuweisung, nur andere echte alternative ist, es zu senden, in der Funktion als in - /out-parameter.

    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;
                }
            }
        }
    }
}

    aufrufen und wie diese von Ihrem Anrufer:

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

    generateBoard(N,M,board);

    ...
}

    Ich würde auch überlegen, relocatting die srand() Aufruf der startup-code in main(). Es sollte im Idealfall nie in die einige potenziell wiederholen-callable-Funktion, und es sollte sichergestellt sein, nur ausgeführt werden, einmal pro Prozessausführung. (Hinweis: ich kann ehrlich nicht erinnern, wenn es einmal pro thread Ausführung, aber an diesem Punkt in der Codierung Lernkurve ich vermute, multi-threading ist nicht auf dem radar noch).

    Schließlich wird Ihre random-fill-Schleife ist unnötig wiederholten. Es gibt bessere alternativen zu generieren, was man offenbar versucht, zu tun: erstellen Sie eine zufällige permutation von einem vorhandenen Satz von zahlen. Wie geschrieben, Sie könnte spin für einige Zeit versuchen, zu füllen, die letzten paar slots, je nachdem, wie viel größer MAX_RANGE ist im Vergleich zu (N*M).

    InformationsquelleAutor der Antwort WhozCraig

  2. 17

    Ihnen angegebenen board als lokale variable - seine Erinnerung ist dealoccated die Funktion den Gültigkeitsbereich verlässt.

    Können Sie erklären, den Vorstand global, oder Sie können es dynamisch wie so:

    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;
}

    Müssen Sie dynamisch das gratis-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);
 }

    InformationsquelleAutor der Antwort Israel Unterman

Schreibe einen Kommentar