我的int **在一个结构中的垃圾输出

时间:2018-12-05 01:46:46

标签: c structure c99

我从文件中读取整数值,用空格字符解析它们,然后将它们插入我的结构中称为游戏的int **中。使用print语句,我已确认将正确的值放置在int **中的正确位置,但是一旦退出while循环或返回该结构,则打印的值是错误的。

我的结构:

typedef struct Map_s{

        char * defender;
        char * attacker;
        int missles;
        int ** layout;
        size_t capacity;
        size_t size;
}map;

该结构的初始化:

map * newGame(){

        map * game = malloc(sizeof(map));
        game->layout = (int**)malloc(_DEFAULT_MAP_SIZE * sizeof(int*));
        game->defender = NULL;
        game->attacker = NULL;
        game->missles = 0;
        game->capacity = _DEFAULT_MAP_SIZE;
        game->size = 0;

        return game;
}

文件解析:

map * game = newGame();

    char * token;
    char * dup;
    char * ptr;
    int ret;
    const char delim[2] = " ";
    char * buf = NULL;
    char * temp = NULL;
    size_t size = _MAX_SIZE;
    int lineNum = 1;

    while( getline(&temp, &size, fp) > 1 ){
            buf = strtok(temp, "\n");
            dup = strdup(buf);
            if( buf[0] != '#' ){
                    if( lineNum == 1){
                            if( game->defender == NULL ){
                                    game->defender = dup;
                            }
                    }
                    else if( lineNum == 2 ){
                            if( game->attacker == NULL )
                                    game->attacker = dup;
                    }
                    else if( lineNum == 3 ){
                            game->missles = atoi(dup);
                    }
                    else if( lineNum > 3 ){
                            token = strtok(dup, delim);
                            while( token != NULL ){
                                    if( game->size >= game->capacity ){
                                            game->layout = (int**)realloc(game->layout, \
                                                            game->capacity*2 * sizeof(int*));
                                            game->capacity = game->capacity * 2;
                                    }
                                    ret = (int)strtol(token, &ptr, 10);
                                    game->layout[game->size] = &ret;
                                    game->size = game->size + 1;
                                    token = strtok(NULL, delim);
                                    //printf("%s ", token);
                            }
                    }**

                    lineNum++;
            }
    }



    return game;

我几乎肯定我的代码在第二个while循环中混乱了。我正在尝试将字符串转换为整数,然后将其保存到int **中的正确位置。

我是如何从主打印的:

for( size_t i = 0; i < thisgame->size; i++ ){
            printf("%d ", *thisgame->layout[i]);
            if( i == 0)
                    continue;
            else if( (i+1) % 10 == 0 )
                    printf("\n");
    }

返回struct后主函数的输出:

0 0 0 0 0 0 0 0 0 0 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10 
10 10 10 10 10 10 10 10 10 10

输出应该是什么样的(文件中的内容):

2 2 2 2 2 2 2 2 2 2
2 2 6 6 7 7 7 5 5 2
2 2 7 7 7 2 2 17 17 17 2
2 2 2 2 2 2 2 2 2 2
8 8 8 6 6 6 9 9 9 2
2 2 2 2 2 2 2 2 2 2
10 10 10 4 4 9 9 9 2
2 2 2 2 2 2 2 2 2 2
3 3 3 6 6 6 9 9 9 2
3 3 3 6 6 6 9 9 9 2
2 2 2 2 2 2 2 2 2 2

2 个答案:

答案 0 :(得分:0)

我看到几个问题。

首先是**token = strtok(dup, delim);行。在此行的第一行中,token没有有效值,因此,将其取消引用一次,而是两次,也会引起问题。我相信这应该只是token = strtok(dup, delim);

这将导致while循环的第一次迭代,因为if块将返回错误值(所有这些0似乎至少都符合此猜测)。

第二个是D Go指出的,因为您要提供ret的地址(该地址不变),因此game->layout的每个元素都具有相同的地址,因此看起来在相同的int(适合所有​​的10s)。

我认为解决方法是将layout做成int *的数组(调整分配代码以使其适合),然后将ret直接分配给每个元素。

答案 1 :(得分:0)

问题是当您真正想要的是 int 数组时,您正在使用 int * 数组。每次存储ret时,您将存储相同的地址。当您在循环中检查值时,它是正确的,因为ret包含您想要的值...直到下一次迭代。