5

私はファイルを読んでいて、各行を配列の文字列に入れたいと思っています。ファイルの長さは任意で、各行の長さも任意です (ただし、100 文字未満であると仮定します)。

これが私が持っているもので、コンパイルされていません。基本的に、これは文字の配列に対する配列ですよね？では、そうあるべきではありませんchar** words = (**char)malloc(sizeof(*char));か？

#include <stdio.h>
#include <stdlib.h>

int main(){


 int BUFSIZE = 32767;//max number of lines to read
 char** words = (**char)malloc(sizeof(*char));//gives error: expected expression before 'char'
 FILE *fp = fopen("coll.txt", "r");
 if (fp == 0){
        fprintf(stderr, "Error opening file");
        exit(1);
 }

int i = 0;
words[i] = malloc(BUFSIZE);
while(fscanf(fp, "%100s", words[i]) == 1)//no line will be longer than 100
{
        i++;
        words[i] = realloc(words, sizeof(char*)*i);
 }

 int j;
 for(j = 0; j < i; j++)
    printf("%s\n", words);

 return 0;
}

注：「ファイルから読み取り、配列に保存する」を読みましたが、私の質問には答えていません。

4

2 に答える 2

14

プログラムにはいくつかの問題があります。realloc() ステートメントが正しく使用されていません。また、行を取得するために fgets() を好みます。これが私の解決策です。また、これは realloc() を使用してバッファ行の割り当てを増やすため、事前に行数を知る必要がなく、ファイルを 2 つのパスで読み取る必要もありません (その方が高速です)。これは、事前に割り当てる必要があるメモリの量がわからない場合に使用する一般的な手法です。

#include <stdio.h>
#include <stdlib.h>

int main(void)

    {
    int lines_allocated = 128;
    int max_line_len = 100;

    /* Allocate lines of text */
    char **words = (char **)malloc(sizeof(char*)*lines_allocated);
    if (words==NULL)
        {
        fprintf(stderr,"Out of memory (1).\n");
        exit(1);
        }

    FILE *fp = fopen("coll.txt", "r");
    if (fp == NULL)
        {
        fprintf(stderr,"Error opening file.\n");
        exit(2);
        }

    int i;
    for (i=0;1;i++)
        {
        int j;

        /* Have we gone over our line allocation? */
        if (i >= lines_allocated)
            {
            int new_size;

            /* Double our allocation and re-allocate */
            new_size = lines_allocated*2;
            words = (char **)realloc(words,sizeof(char*)*new_size);
            if (words==NULL)
                {
                fprintf(stderr,"Out of memory.\n");
                exit(3);
                }
            lines_allocated = new_size;
            }
        /* Allocate space for the next line */
        words[i] = malloc(max_line_len);
        if (words[i]==NULL)
            {
            fprintf(stderr,"Out of memory (3).\n");
            exit(4);
            }
        if (fgets(words[i],max_line_len-1,fp)==NULL)
            break;

        /* Get rid of CR or LF at end of line */
        for (j=strlen(words[i])-1;j>=0 && (words[i][j]=='\n' || words[i][j]=='\r');j--)
            ;
        words[i][j+1]='\0';
        }
    /* Close file */
    fclose(fp);

    int j;
    for(j = 0; j < i; j++)
        printf("%s\n", words[j]);

    /* Good practice to free memory */
    for (;i>=0;i--)
        free(words[i]);
    free(words);
    return 0;
    }
于 2013-10-04T06:06:50.997 に答える
1

次の行を変更する必要があります。

char** words = (**char)malloc(sizeof(*char));

これに：

char** words=(char **)malloc(sizeof(char *)*Max_Lines);
于 2013-10-04T06:25:45.143 に答える