Получение переполнения кучи даже при освобождении памяти - PullRequest
0 голосов
/ 21 февраля 2019

Я пытаюсь решить проблему n-queen на leetcode.Но это дает ошибку переполнения кучи на Leetcode.

Но на моем компьютере, когда я даю один вход, он дает правильный ответ, но при многократном вводе он дает ошибку сегментации: ошибка 11.

И когда я не печатаю полную шахматную доску.Просто распечатайте количество различных возможных решений.Тогда тоже работает нормально.

#include <stdio.h>
#include <stdlib.h>

#define bool int
#define true 1
#define false 0

//checking positioned queens
bool checkPlacedQueens(char **board, int queeni, int queenj, int n) {
    int i=queeni;
    int j=queenj;

    //checking complete row
    for(int i=queeni; i>=0; i--) {
        if(board[i][j] == 'Q')
            return false;
    }

    i=queeni;
    j=queenj-1;
    //checking left diagonal
    while(i>=0 && j>=0) {
        if(board[i--][j--] == 'Q') 
            return false;
    }

    i=queeni;
    j=queenj+1;
    //checking right diagonal
    while(i>=0 && j<=n) {
        if(board[i--][j++] == 'Q') 
            return false;
    }

    return true;
}

char ***placeQueens(char **board, int queenI, int n, int *returnSize, char ****result) {

    //all queens are on their correct position
    if(queenI == n) {
        (*returnSize)++;

        /*
            reallocating the memory to save all the outputs in 3D 
            array
        */
        (*result) = (char ***) realloc(*result, sizeof(char **)*(*returnSize));
        (*result)[*returnSize-1] = (char **)malloc(sizeof(char *)*n);

        for(int i=0; i<n; i++) {
            (*result)[*returnSize-1][i] = (char *)malloc(sizeof(char)*n);
        }

        for(int i=0; i<n; i++) {
            for(int j=0; j<n; j++) {
                (*result)[*returnSize-1][i][j] = board[i][j];
            }
        }

        return *result;
    }//if

    for(int j=0; j<n; ++j) {
        char save = board[queenI][j];
        board[queenI][j] = 'Q';
        if(checkPlacedQueens(board, queenI-1, j, n)) {
            placeQueens(board, queenI+1, n, returnSize, result);
        }
        board[queenI][j] = save;
    }//for Loop

    return *result;
}//function

char *** solveNQueens(int n, int* returnSize) {
    char **board;
    char ***result = (char ***)malloc(sizeof(char **));

    board = (char **)malloc(sizeof(char *)*n);

    for(int i=0; i<n; i++) {
        board[i] = (char *)malloc(sizeof(char)*n);
    }

    for(int i=0; i<n; i++) {
        for(int j=0; j<n; j++) {
            board[i][j] = '.';
        }
    }

    placeQueens(board, 0, n, returnSize, &result);

    for(int i=0; i<n; i++)
        free(board[i]);
    free(board);

    return result;
}//char

int main(void) {
    int returnSize=0;
    int n=4;
    char ***arr;

    while(n<10) {
        arr = solveNQueens(n, &returnSize);

        for(int i=0; i<returnSize; ++i) {
            for(int j=0; j<n; j++) {
                for(int k=0; k<n; k++) {
                    printf("%c", arr[i][j][k]);
                }
                printf("\n");
            }
            printf("\n");
        }
        printf("\n\n\n");

        for(int i=0; i<returnSize; ++i) {
            for(int j=0; j<n; j++) {
               free(arr[i][j]);
            }
            free(arr[i]);
        }

        free(arr);
        arr=NULL;
        n++;
    }
}//main

Ответы [ 2 ]

0 голосов
/ 21 февраля 2019

Проблема в том, что вы не сбрасываете значение returnSize при каждом повторном вызове функции solveNQueens.

Ниже приведен рабочий код:

#include <stdio.h>
#include <stdlib.h>

#define bool int
#define true 1
#define false 0

//checking positioned queens
bool checkPlacedQueens(char **board, int queeni, int queenj, int n) {
    int i=queeni;
    int j=queenj;

    //checking complete row
    for(int i=queeni; i>=0; i--) {
        if(board[i][j] == 'Q')
            return false;
    }

    i=queeni;
    j=queenj-1;
    //checking left diagonal
    while(i>=0 && j>=0) {
        if(board[i--][j--] == 'Q') 
            return false;
    }

    i=queeni;
    j=queenj+1;
    //checking right diagonal
    while(i>=0 && j<=n) {
        if(board[i--][j++] == 'Q') 
            return false;
    }

    return true;
}

char ***placeQueens(char **board, int queenI, int n, int *returnSize, char ****result) {

    //all queens are on their correct position
    if(queenI == n) {
        (*returnSize)++;

        /*
            reallocating the memory to save all the outputs in 3D 
            array
        */
        (*result) = (char ***) realloc(*result, sizeof(char **)*(*returnSize));
        (*result)[*returnSize-1] = (char **)malloc(sizeof(char *)*n);

        for(int i=0; i<n; i++) {
            (*result)[*returnSize-1][i] = (char *)malloc(sizeof(char)*n);
        }

        for(int i=0; i<n; i++) {
            for(int j=0; j<n; j++) {
                (*result)[*returnSize-1][i][j] = board[i][j];
            }
        }

        return *result;
    }//if

    for(int j=0; j<n; ++j) {
        char save = board[queenI][j];
        board[queenI][j] = 'Q';
        if(checkPlacedQueens(board, queenI-1, j, n)) {
            placeQueens(board, queenI+1, n, returnSize, result);
        }
        board[queenI][j] = save;
    }//for Loop

    return *result;
}//function

char *** solveNQueens(int n, int* returnSize) {
    char **board;
    char ***result = (char ***)malloc(sizeof(char **));

    board = (char **)malloc(sizeof(char *)*n);

    for(int i=0; i<n; i++) {
        board[i] = (char *)malloc(sizeof(char)*n);
    }

    for(int i=0; i<n; i++) {
        for(int j=0; j<n; j++) {
            board[i][j] = '.';
        }
    }

    placeQueens(board, 0, n, returnSize, &result);

    for(int i=0; i<n; i++)
        free(board[i]);
    free(board);

    return result;
}//char

int main(void) {
    int n=4;
    char ***arr;

    while(n<10) {
        // Reset the value of returnSize
        int returnSize=0;

        arr = solveNQueens(n, &returnSize);

        for(int i=0; i<returnSize; ++i) {
            for(int j=0; j<n; j++) {
                for(int k=0; k<n; k++) {
                    printf("%c", arr[i][j][k]);
                }
                printf("\n");
            }
            printf("\n");
        }
        printf("\n\n\n");

        for(int i=0; i<returnSize; ++i) {
            for(int j=0; j<n; j++) {
               free(arr[i][j]);
            }
            free(arr[i]);
        }

        free(arr);
        arr=NULL;
        n++;
    }
}//main
0 голосов
/ 21 февраля 2019

Ваш код имеет недопустимый доступ к памяти.Прямой тест здесь находится здесь segfault Diagnoser ,

Ниже приведено диагностическое сообщение о прочтении за пределами поля.

====== Сообщение начала DTS Stensal == (56.133) == скопировать начало здесь ======

[чтение за пределами допустимого]Stensal DTS обнаружил .
Продолжение выполнения может привести к неопределенному поведению.======= скопировать конец здесь =========

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...