1

ファイルをchar**に保存できない理由を理解しようとしています。

メモリを正しく割り当てたと思いました。誰か助けてくれませんか？ありがとうございました！

また、私のやり方が問題に対処する最も効率的な方法ではないことはわかっていますが、効率を心配する前に、まず問題を解決したいと考えています。ありがとうございました！

void readFile(int argc, char** argv)
{
    FILE *myFile;
    char** list;
    char c;
    int wordLine = 0, counter = 0, i;
    int maxNumberOfChars = 0, numberOfLines = 0, numberOfChars = 0;

    myFile = fopen(argv[1], "r");

    if(!myFile)
    {
        printf("No such file or directory\n");
        exit(EXIT_FAILURE);  
    }

    while((c = fgetc(myFile)) !=EOF)   //goes through the file to get # of lines
    {                                  //and columns so I can allocate array
        numberOfChars++;
        if(c == '\n')
        {
            if(maxNumberOfChars < numberOfChars)
                maxNumberOfChars = numberOfChars + 1;

            numberOfLines++;
        }
    }

    fseek(myFile, 0, SEEK_SET);   //resets file pointer

    list = malloc(sizeof(char*)*numberOfLines);  //dynamically allocating

    for(i = 0; i < wordLine ; i++)
        list[i] = malloc(sizeof(char)*maxNumberOfChars);


    while((c = fgetc(myFile)) != EOF)       //stores in words
    {
        if(c == '\n' && counter > 0)
        {
            list[wordLine][counter] = '\0';
            wordLine++;
            counter = 0;
        }
        else if(c != '\n')
        {
            list[wordLine][counter] = c;   //seg fault happens at first character
            counter++;
        }
    } 
    fclose(myFile);
}
4

3 に答える 3

2

この時点で：

for(i = 0; i < wordLine ; i++)

wordLine= 0 なので、メモリは割り当てられません。私はそれがあるべきだと思います：

for(i = 0; i < numberOfLines; i++)

そして、Grijesh Chauhan が言ったことと同様に、= 0 に設定する必要がありnumberOfCharsます。そうしないと、メモリを割り当てすぎてしまいます。

于 2013-02-09T10:51:54.193 に答える
2

ワードラインの割り当て:

for(i = 0; i < wordLine ; i++)
    list[i] = malloc(sizeof(char)*maxNumberOfChars);

を使用しwordLineますが、最初にこれを 0 に初期化し、変更されることはありません。

したがって、この for ループの malloc は実行されません。

于 2013-02-09T10:52:06.393 に答える
1

あなたのforループ状態はめちゃくちゃです、

for(i = 0; i < wordLine ; i++)
    list[i] = malloc(sizeof(char)*maxNumberOfChars);

wordLineに初期化されますが0、期待どおりに実行されず、メモリが割り当てられますlist[i]

あなたはそれをに変更したいかもしれません

for(i = 0; i < numberOfLines ; i++)
    list[i] = malloc(sizeof(char)*maxNumberOfChars);
于 2013-02-09T10:53:54.220 に答える