Question

我正在尝试在C中制作战舰模拟游戏。我在这里要做的是获得潜艇上的起点和终点。为此我使用了一个函数。现在问题是程序在获得2分后崩溃。

int getsub(char* filename, Submarine* list)
{
    int subcount=0;
    int i=0;
    char* token;
    char line[100];
    FILE *fptr=fopen(filename,"r");
    if(fptr==NULL)
    {
        printf("no file found");
        exit(-1);
    }
    while(fgets(line,100,fptr))
    {
        if(isalpha(line[0])|| isalnum(line[0]))
        {
            ++subcount;
        }
    }

    list=malloc(sizeof(Submarine)*subcount);
    rewind(fptr);
    while(fgets(line,100,fptr)){
    if(isalpha(line[0]))
    {
        list[i]->start=malloc(sizeof(Point));
        token=strtok(line,"-");
        list[i]->start=TranslateCoordinate(token);
        list[i]->end=malloc(sizeof(Point));
        token=strtok(NULL,"-");
        list[i]->end=TranslateCoordinate(token);
        i++;
    }
    }
    return subcount;
}

这是我的源文件。我使用的另一个函数是获取一个坐标（如A9）并将其转换为0到9之间的数字

Point TranslateCoordinate(char* c)
{
    Point newpoint;
    newpoint=malloc(sizeof(Point));
    int row=c[0];
    int column;
    newpoint->x=row-'A';
    if(c[1]== 1 && c[2]==0)
    {
        column=9;
        newpoint->y=column;
        return newpoint;
    }
    column=c[1];
    newpoint->y=column-'1';;
    return newpoint;
}

这些是我的结构：

struct Point_s
{
    int x;
    int y;
};

struct Submarine_s
{
    Point start;
    Point end;
    int Life;
};

struct SubmarinePart_s
{
    Submarine* Sub;
    int IsHit; // 0 = not hit, <1 = exploded
};

这是我的主要计划：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "header.h"
#define BoardSize 10

int main(int argc, char *argv[])
{
    if (argc < 3){
        printf("Error not enough files");
        exit(-1);
    }
    char* P1setup=argv[2];
    char* P2setup=argv[3];
    Submarine* P1list;
    Submarine* P2list;
    getsub(P1setup,P1list);

    char Board1[BoardSize][BoardSize];
    char Board2[BoardSize][BoardSize];
    int x,y;
    for(x=0;x<BoardSize;x++){
        for(y=0;y<BoardSize;y++){
            Board1[x][y]='0';
        }
    }
    for(x=0;x<BoardSize;x++){
        for(y=0;y<BoardSize;y++){
            Board2[x][y]='0';
        }
    }
    return 1;
}

这是header.h：

#ifndef HEADER_H_INCLUDED
#define HEADER_H_INCLUDED
#include <stdio.h>
#include <string.h>
typedef struct Point_s* Point;
typedef struct Submarine_s* Submarine;
typedef struct SubmarinePart_s* SubmarinePart;
Point TranslateCoordinate(char* c);
int getsub(char* filename,Submarine* list);
void printpoint(Point p);
void printsub(Submarine s);

#endif // HEADER_H_INCLUDED

Answer 1

您没有分配list的子元素。请注意Submarine是typedef struct Submarine_s* Submarine;的类型转换。您已分配主列表，但在您在此处取消引用list[i]->start

之前未分配该成员

int getsub(char* filename, Submarine* list)
...

list=malloc(sizeof(Submarine)*subcount);
rewind(fptr);
while(fgets(line,100,fptr)){
if(isalpha(line[0]))
{
    list[i]->start=malloc(sizeof(Point));

在取消引用它之前，你应该首先将list [i]分配为sizeof（submarine_s）。

脚注：无论如何，我不同意使用隐藏指针的typedef而不指示新类型是指针。如果你把它改为SubmarinePtr，我宁愿这样做。

C程序模拟潜艇游戏

1 个答案: