1

我是 C、linux 等的新手，我的代码可以编译并运行，但是一旦我输入我的第一个用户输入，我就会遇到分段错误。如果有人能指出我的代码有什么问题，这将非常有帮助，我认为它要么在“calculate()”或“main()”中，因为我试图在两者中都使用“malloc()”分配内存那些地方。

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

#define LIFE_YES 'X'
#define LIFE_NO 'O'

int HEIGHT, WIDTH;
typedef int **TableType;

void printTable(TableType table) {
    int height, width;

        for (height = 0; height < HEIGHT; height++) {
            for (width = 0; width < WIDTH; width++) {
                    if (table[height][width] == LIFE_YES) {
                        printf("X");
                    } 
            else {
                    printf("-");
                    }
            }
            printf("\n");
        }
        printf("\n");
}

void clearTable(TableType table) {
    int height, width;

        for (height = 0; height < HEIGHT; height++) {
            for (width = 0; width < WIDTH; width++) {
                    table[height][width] = LIFE_NO;
            }
        }
}

void askUser(TableType tableA) {
    int i;
        int n;
        int height, width;

        printf("Enter the amount of initial organisms: ");
        scanf("%d", &n);

        for (i = 0; i < n; i++) {
            printf("Enter dimensions (x y) where organism %d will live: ", i + 1);
            scanf("%d %d", &height, &width);

            tableA[height][width] = LIFE_YES;
        }

        printTable(tableA);
        printf("Generation 0");
}

int getNeighborValue(TableType table, int row, int col) {
        if (row < 0 || row >= HEIGHT || col < 0 || col >= WIDTH || table[row][col] != LIFE_YES ) {
            return 0;
        } 
    else {
            return 1;
        }
}

int getNeighborCount(TableType table, int row, int col) {
        int neighbor = 0;

        neighbor += getNeighborValue(table, row - 1, col - 1);
        neighbor += getNeighborValue(table, row - 1, col);
        neighbor += getNeighborValue(table, row - 1, col + 1);
        neighbor += getNeighborValue(table, row, col - 1);
        neighbor += getNeighborValue(table, row, col + 1);
        neighbor += getNeighborValue(table, row + 1, col - 1);
        neighbor += getNeighborValue(table, row + 1, col);
        neighbor += getNeighborValue(table, row + 1, col + 1);

        return neighbor;
}

void calculate(TableType tableA) {
        TableType tableB;
        int neighbor, height, width, i;
    tableB= malloc(HEIGHT * sizeof(int*));

    for (i = 0; i < HEIGHT; i++) {
            tableB[i] = malloc(WIDTH * sizeof(int));
    }

        for (height = 0; height < HEIGHT; height++) {
            for (width = 0; width < WIDTH; width++) {
                    neighbor = getNeighborCount(tableA, height, width);
                    if (neighbor==3) {
                        tableB[height][width] = LIFE_YES;
                    } 
            else if (neighbor == 2 && tableA[height][width] == LIFE_YES) {
                        tableB[height][width] = LIFE_YES;
                    } 
            else {
                        tableB[height][width] = LIFE_NO;
                    }
            }
        }

        for (height = 0; height < HEIGHT; height++) {
            for (width = 0; width < WIDTH; width++) {
                    tableA[height][width] = tableB[height][width];
            }
    }
    free(tableB);
}

/* test data
void loadTestData(TableType table) {
        table[3][4] = LIFE_YES;
        table[3][5] = LIFE_YES;
        table[3][6] = LIFE_YES;

        table[10][4] = LIFE_YES;
        table[10][5] = LIFE_YES;
        table[10][6] = LIFE_YES;
        table[11][6] = LIFE_YES;
        table[12][5] = LIFE_YES;
}
*/

int main(void) {
    int i;
        char end;
        int generation = 0;

    printf("Enter the amount of rows and columns you want in the grid: ");
    scanf("%i %i\n", &HEIGHT, &WIDTH);

        TableType table = malloc(HEIGHT * sizeof(int*));

    for (i = 0; i < HEIGHT; i++) {
            table[i] = malloc(WIDTH * sizeof(int));
    }

        clearTable(table);
        askUser(table);
        /*loadTestData(table);*/
        printTable(table);

        while (end != 'q') {
            calculate(table);
            printTable(table);
            printf("Generation %d\n", ++generation);
            printf("Press q to quit or 1 to continue: ");
            scanf(" %c", &end);
        }

        return 0;
}
4

2 回答 2

0

在主要 

scanf("%i %i\n", &HEIGHT, &WIDTH);  ==>    scanf("%i %i", &HEIGHT, &WIDTH);
             ^

因为那个\n你需要再输入一个输入。避免它

和分段错误askUser 

tableA[height][width] = LIFE_YES;

Program received signal SIGSEGV, Segmentation fault.
0x00000000004007c1 in askUser (tableA=0x603010) at seg3.c:49
49                  tableA[height][width] = LIFE_YES;
(gdb) bt
#0  0x00000000004007c1 in askUser (tableA=0x603010) at seg3.c:49
#1  0x0000000000400bf3 in main () at seg3.c:142
(gdb)

在这里你没有分配内存，但你正在访问。首先分配内存

于 2013-09-14T12:06:11.453 回答
0

这是段错误，因为：

for (i = 0; i < n; i++) 
{
   printf("Enter dimensions (x y) where organism %d will live: ", i + 1);
   scanf("%d %d", &height, &width);
    tableA[height][width] = LIFE_YES;  <--- Here 
}

如果用户指定的维度超出了您最初动态分配的“高度”和“重量”维度，该怎么办main？

当用户输入时，您需要检查坐标是否为非负且小于上述尺寸，如果不在范围内，则抛出错误消息并退出。

第二件事：
你end在 main 中的变量也没有初始化，也许你end = getchar()在if条件下错过了这个。
如果这未初始化也可能导致段错误。


\n第三件事：scanf真的需要吗？如果存在，它将把第三个输入作为下一个输入，scanf并且不会提示您输入该 scanf 的输入。
即，目前第三个输入将进入第。有机体没有被提示。

于 2013-09-14T12:09:08.777 回答