Question

enter image description here 所以这里基本上我有两个循环基本相同的东西，除了它们fscanf到不同的目录。第二个应该fscanf到一个结构，一个导致程序崩溃。这是为什么？？？？？导致程序崩溃的代码是程序中的最后一个循环。

#include <stdio.h>
#include <conio.h>
#include <string.h>
#define MAXLEN 100

int main()
{
    char filename1[MAXLEN];
    char filename2[MAXLEN];
    char filename3[MAXLEN];
    char filename4[MAXLEN];


    char itemname[MAXLEN];
    printf("Enter the input file: ");
    scanf("%s", filename1);
    strcpy(filename3,filename1);
    strcat(filename3,"Output.txt");
    strcpy(filename4,filename1);
    strcat(filename4,"Log.txt");
    strcpy(filename2, filename1);
    strcat(filename2, "Customers.txt");
    strcat(filename1, ".txt");
    printf("%s will be used",filename1);

    FILE *inputfile1 = NULL;
    FILE *inputfile2 = NULL;
    FILE *outputfile = NULL;
    FILE *logfile = NULL;

    inputfile1 = fopen(filename1, "r");
    inputfile2 = fopen(filename2, "r");
    outputfile = fopen(filename3, "w");
    logfile = fopen(filename4, "w"); 

    int numberofitems =0;
    while (fscanf(inputfile1,"%s",itemname)==1){
        numberofitems++;
    }
    rewind(inputfile1);
    numberofitems /= 4;

    struct storestock{ 
    char itemnames[numberofitems][MAXLEN];
    int isdecimal[numberofitems];
    double stock[numberofitems];
    double price[numberofitems];
    };

    typedef struct storestock store;

    store inventory;
    int i;
    for (i=0; i < numberofitems; i++)
    {
    fscanf(inputfile1,"%s %d %lf %lf",inventory.itemnames[i],&(inventory.isdecimal[i]),
    &(inventory.stock[i]),&(inventory.price[i]));
    printf("\n %dst item %s %d %lf %lf", i+1,inventory.itemnames[i],inventory.isdecimal[i]
    ,inventory.stock[i],inventory.price[i] );
    }

    struct customers{
        char customername[MAXLEN];
        char wanteditems[10][MAXLEN];
        double amountwanted[10];
    };


    int j,k,l;
    int numberofcustomers = 0;
    int itemnumber=0;
    double itemamount;
    char string[MAXLEN];
    for (j=0;j<100;j++){
        if (fscanf(inputfile2,"%s %lf", string,&itemamount)==1){
            numberofcustomers++;
            printf("\n%s",string);
    }}
    printf("%d", numberofcustomers);

    struct customers mycustomers[numberofcustomers];
    rewind(inputfile2);

    **for (k=0;k<100;k++){
        if (fscanf(inputfile2,"%s %lf", mycustomers[k].customername,&itemamount)==1){
            printf("\n%s", mycustomers[k].customername);}
            }**
    getch();
    return 0;       
}

Answer 1

此代码非法：

struct storestock{ 
    char itemnames[numberofitems][MAXLEN];

结构中数组的维度必须是常量表达式（灵活数组成员除外，这不是）。

您需要重新设计代码才能执行此操作。很难看出你的编译器如何超越这一行。

更好的方法是struct storestock实际只有每个项目中的一个，然后你有一个这样的结构数组（可以的大小为numberofitems ）。与您对struct customers所做的类似。

代码的第一部分，在FILE *行之前，对没有大小检查的缓冲区执行大量写操作。这可能导致缓冲区溢出，从而导致不可预测的行为。最好用经过长度检查的scanf替换所有这个guff，然后使用snprintf代替strcpy和strcat。

for struct with struct会导致崩溃

1 个答案: