分段错误,结构,ncurses,绘图

时间:2014-03-14 19:14:40

标签: c segmentation-fault ncurses

我的代码中出现了分段错误。我的代码是:

#define STARTING_POINT 5
int main(int argc, char *argv[])
{
input(argc, argv);
parser(argc, argv);
drawingRooms();
return 0;
}


void drawingRooms(room roomInfo[6]) 
{
int rooms;
int i;
int positionX;
int positionY;
rooms = 0;
initscr();
noecho();
for(rooms = 0; rooms < 6; rooms++)
{
    switch(rooms)
    {
        case 0:
        for(i = 0; i <= roomInfo[rooms].length; i++);
        {
            move(STARTING_POINT + i, STARTING_POINT);
            printw("|");
        }
        break;
    }
}   
getch();
endwin();
}


void parser(int argc, char **argv)
{
FILE * rooms;
char * theString;
char * theToken;
int width;
int length;
int k;
int x;
int placeInt;
int posX;
int posY;
char a[ROOM_STRING_LENGTH];
item gold;
item monster;
item potion;
item hero;
item stairs;
room roomInfo[6];
rooms = fopen(argv[1], "r");

if(rooms == NULL)
{
    printf("error opening file\n");
}
x = 0;
while(fgets(a, ROOM_STRING_LENGTH, rooms) != NULL)
{ 

    theString = malloc((sizeof(char)*(strlen(a)+1)));
    strcpy(theString, a);

    for(theToken = strtok(theString, " "); theToken; theToken = strtok(NULL, " "))
    {
        sscanf(theToken, "%dX%d", &length, &width);
        roomInfo[x].roomNumber = x;
        roomInfo[x].width = width;
        roomInfo[x].length = length;

        if(theToken[0] == 'd')
        {
            switch(theToken[1])
            {
                case 'e':
                {
                    placeInt = theToken[2] - '0';
                    roomInfo[x].eastDoor = placeInt;
                    break;
                }
                case 'w':
                {
                    placeInt = theToken[2] - '0';
                    roomInfo[x].westDoor = placeInt;
                    break;
                }
                case 's':
                {
                    placeInt = theToken[2] - '0';
                    roomInfo[x].southDoor = placeInt;
                    break;
                }
                case 'n':
                {
                    placeInt = theToken[2] - '0';
                    roomInfo[x].northDoor = placeInt;
                    break;
                }
                default:
                {
                    break;
                } 
            }       
        }

        else if(theToken[0] == 'g' || theToken[0] == 'm' || theToken[0] == 'p' || theToken[0] == 'h')
        {
             k = 0;
             while(k <= (strlen(theToken)))
             {

                 switch(theToken[k])
                 {
                     case 'g':
                     posY = theToken[1] - '0';
                     posX = theToken[3] - '0';

                     gold.Xposition = posX;
                     gold.Yposition = posY;
                     break;

                     case 'm':
                     posY = theToken[1] - '0';
                     posX = theToken[3] - '0';

                     monster.Xposition = posX;
                     monster.Yposition = posY;
                     break;

                     case 'p':
                     posY = theToken[1] - '0';
                     posX = theToken[3] - '0';

                     potion.Xposition = posX;
                     potion.Yposition = posY;
                     break;

                     case 'h':
                     posY = theToken[1] - '0';
                     posX = theToken[3] - '0';

                     hero.Xposition = posX;
                     hero.Yposition = posY;                  
                     break;

                     case 's':
                     posY = theToken[1] - '0';
                     posX = theToken[3] - '0';

                     stairs.Xposition = posX;
                     stairs.Yposition = posY;   
                     break;
                 }
                 k++;
             }
        }
        else if(theToken == NULL)
        {
            printf("end of file");
        }
    }

    if(theToken == NULL)
    {
        printf("\n");
    }
    x++;
}
free(theString);
fclose(rooms);
}

我的结构是:

typedef struct 
{
int roomNumber;
int length;
int width;
int eastDoor;
int westDoor;
int southDoor;
int northDoor;
}room;

我想要做的是访问我的结构数组中的第一个结构,然后使用我获得的信息做一些事情。每当我运行以下代码时,什么都没有打印,我得到一个seg错误

1 个答案:

答案 0 :(得分:0)

您创建的变量保持活动状态,直到它前面的左侧开始花括号}的相应右括号{

每对开合花括号和{ ... }内的内容都称为代码块,根据这个定义,我们可以说变量在代码块中保持活着状态它被宣布。

您在函数roomsInfo[6]中声明了一个parser数组,看起来您希望能够直接访问它而无需执行其他函数的任何操作:drawingRooms

事情不会那么成功。

另外一件事,C编译器不应该允许你调用一个请求参数的函数,而不需要参数。因此,您根本无法拨打电话drawingRooms( );,因为void drawingRooms( room roomInfo[6] ) { ... }定义了这样的电话。如果它允许,那么它可能将room roomInfo[6]视为未初始化的变量,如下所示:

    void drawingRooms(  )
    {
        room roomInfo[6];
        ...
    }

无论如何,你应该做的是为drawingRooms提供一个适当的参数,它是地址,指向6个room结构数组的第一个元素的指针;如果你已经宣布了它的功能。

要解决此特定问题,您有两个选择,我将从简单的选项开始:

1)使roomInfo[6]成为文件范围变量

通过使roomInfo[6]成为一个全局(文件范围)变量,该变量将贯穿程序的整个生命周期,因此可以从代码中的任何位置访问。要做到这一点:

  • 删除room roomInfo[6];函数
  • 中的行parser
  • 从函数room roomInfo[6]
  • 中删除参数drawingRooms
  • room roomInfo[6];函数
  • 上方声明main高于其他所有内容

像这样:

    #define STARTING_POINT 5

    //somewhere above the roomInfo declaration, somewhere here
    //there has to be the typedef for the room structure,
    //either through the inclusion of a header file
    //or the typedef struct {...} room; itself

    room roomInfo[6];

    int main( ... )
    {
        ...
    ...

2)将指针推送到drawingRooms

要做到这一点,你必须首先自己为room结构数组分配内存,这样当parser函数结束时它不会自动被杀死;然后使用parser函数将指针返回到第一个元素,在调用drawingRooms函数时使用返回的地址:

  • 将行room roomInfo[6];替换为room * roomInfo;函数
  • 中的parser
  • 为您的房间分配6 room结构值内存

像这样:

    roomInfo = calloc( 6, sizeof * roomInfo );
  • 确保您已<stdlib.h><malloc.h>包含calloc来电
  • parser函数末尾添加此行:return roomInfo;
  • parser函数的返回值从void更改为room *
  • 在调用main函数
  • 时,在drawingRooms函数内使用此返回值

像这样:

    //either like this:
    ...
        input( argc, argv );
        drawingRooms( parser( argc, argv ) );
    ...

    //or like this (allows you to free it inside main):
    ...
        room * roomInfo;
        input( argc, argv );
        roomInfo = parser( argc, argv );
        drawingRooms( roomInfo );
    ...
  • 如果您的程序要进一步运行,请确保free分配的内存。