ファイル内のデータを格納

Question

前提

ここに質問の内容を詳しく書いてください。
（例）
ファイル内(words.txt)にある二千数百個の単語（1行に１単語）を読み込んで配列に格納します。それを１列１０個ずつ出力します。

ここに実現したいことを箇条書きで書いてください。

•読み込んだ単語を配列に格納したいです。

発生している問題・エラーメッセージ

Format specifies type 'char *' but the argument has type 'int'
Incompatible pointer types passing 'char [1]' to parameter of type 'FILE *' (aka 'struct __sFILE *')

該当のソースコード

#include <stdio.h>
 
int main()
{

    char siny[] = {0};
    FILE *fp;
    char words[10000] = {0};
 
    fp = fopen("words.txt","r");
    if(fp == NULL) {
        printf("ファイルを開くことが出来ませんでした\n");
        return -1;
    }
 
    rewind(fp);//先頭にファイルポインタを戻す     Return file pointer to beginning
    
    int i = 0;
    while(fgets(siny,3000,fp) != NULL) {
        printf("%s",siny);
        fscanf(siny,"%s",words[i]);
        i++;
    }
 
    fclose(fp);
    
    return 0;
    
    
}

試したこと

1行ごとに読み込んでいるのでそれごとにi++して１単語ずつ入れているつもりです。
3000なのは二千数百個の単語なのでそれよりも多い適当な数字です
fscanfで格納を試みましたができませんでした。

*１列１０個ずつ出力します。と書いていますが、このコードは途中なので、とりあえず単語を全て格納する所を解決したいです。

Answer 1

コードみるに文字列がメモリ上でどのように扱われているのかをあまり理解できていないようなので、その辺をもう一度調べてみてはいかかでしょうか？

単語(文字列)自体が配列なので配列の一つの要素に単語を読み込むのは無理です。
単語一つで配列の要素を複数使用する事になります。
1次元配列に入れた数値のように複数の文字列を扱うには2次元配列にする必要があります。
↓サンプル

#include <stdio.h>
#define WMax 5000 // 単語の最大数
#define WCMax 64 // 単語の最大文字数(バイト数)

int main()
{
    FILE* fp;
    char words[WMax][WCMax] = { 0 };
    int wcnt;

    fp = fopen("words.txt", "r");
    if (fp == NULL) {
        printf("ファイルを開くことが出来ませんでした\n");
        return -1;
    }

    //2次元配列に単語(文字列)を読み込む
    for (wcnt = 0; wcnt < WMax; wcnt++) {
        if (fgets(words[wcnt], WCMax, fp) == NULL)break; //文字列を読み込む
        //改行コードを探して消す(↑でfscanf使えば不要だがfscanfは改行以外も分割される)
        for (int i = 0; i < WCMax; i++) {
            if (words[wcnt][i] == '\n') {
                words[wcnt][i] = NULL;
                break;
                }
        }
    }

    fclose(fp);

    //表示
    for (int i = 0; i < wcnt; i++) {
        printf("%s   ", words[i]);//単語の後ろにタブをつけて表示
        if (!((i + 1) % 10))printf("\n");//10単語ごとに改行
    }

    return 0;
}

あと全部を1次元配列に読み込んで単語をNULL文字で区切りポインタ配列に単語の先頭のポインタを代入すると、2次元配列に入れたのと同じようにアクセスできます。
↓サンプル

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

int main()
{
    FILE* fp;
    char* data;
    char** words;
    fpos_t fsize = 0;
    int cnt = 1;

    fp = fopen("words.txt", "r");
    if (fp == NULL) {
        printf("ファイルを開くことが出来ませんでした\n");
        return -1;
    }

    //ファイルサイズ取得
    fseek(fp,0,SEEK_END);
    fgetpos(fp, &fsize);
    fseek(fp, 0, SEEK_SET);

    //ファイルサイズ+1分メモリ確保
    data = (char*)calloc(fsize+1, sizeof(char));
    if (data == NULL) {
        printf("メモリの確保に失敗しました。\n");
        return -1;
    }

    //すべて読み込む
    fread(data,fsize,1,fp);
    //ファイルを閉じる
    fclose(fp);

    //単語数（改行）をカウント
    for(int i = 0; i < fsize; i++) {
        if (data[i] == '\n')cnt++;
    }

    //ポインタ配列を単語数分メモリ確保
    words = (char**)calloc(cnt, sizeof(char*));
    if (words == NULL) {
        printf("メモリの確保に失敗しました。\n");
        return -1;
    }

    //ポインタ配列に単語の先頭のポインタを代入
    words[0] = &data[0];
    for (int i = 0, j = 1; j < cnt; i++) {
        if (data[i] == '\n') {//もし改行コードなら
            data[i] = NULL;    //NULL文字に変換(単語で区切る)
            words[j] = &data[i] + 1;//次の単語の頭のポインタを代入
            j++;
        }
    }
    
    //表示(ポインタ配列も2次元配列と同じようにアクセスできる)
    for (int i = 0; i < cnt; i++) {
       printf("%s   ", words[i]);
       if(!((i+1)%10))printf("\n");
    }

    //メモリ開放
    free(words);
    free(data);

    return 0;
}

Answer 2

そのエラーメッセージは fscanf(siny, "%s", words[i]); に、型の不一致が２つあることを指摘しています。

まず fscanf() の引数はどうあるべきか確認します。
C言語 fscanf 使い方
書式　 int fscanf(FILE *fp, 書式指定文字列, 格納可変個引数);　となってます。

(1) Format specifies type 'char *' but the argument has type 'int'

は「 char * 型（charへのポインタ型）を渡すべきなのに、 int 型の値を渡しているよ」ということらしい。第２引数が "%s" なので char へのポインタでなければならない、ということではないかと。

fscanf(siny, "%s", words[i]); の第三引数が words[i] では型が合わないのです。
words[i] だと words というの配列の、i 番目の値です。文字配列の中の、ある一要素の値です。その値はポインタではありませんから。

char words[10000] ; と変数定義されているので、その一要素の型は char です。その char 型の値は、自動的に int 型に拡張されて、関数（この場合は fscanf()）に渡ります。なのでエラーメッセージに 'int' という型名が現れたのです。

(2) Incompatible pointer types passing 'char [1]' to parameter of type 'FILE *' (aka 'struct __sFILE *')

これは fscanf(siny, "%s", words[i]); の第一引数 siny の型が違うと言ってます。
「fscanf() の第一引数は FILE * 型 なのに siny は char 型の配列だ。それでは互換性のないポインタを渡してしまう」といった意味です。

ちなみに, 'char [1]' とは、「char型の要素が１個だけある（要素数が１の）配列」という意味です。
char siny[] = {0}; と変数定義したので、そうなります。つまり、この変数定義に誤りがあります。

---

fscanf(fp, "%s", &words[i]); とでもすればエラーメッセージは出なくなりますが、これだけでは済まないでしょう。

私は、 fscanf() ではなく sscanf() を使って

    while(fgets(siny, 3000, fp) != NULL) {
        printf("%s", siny);
        sscanf(siny, "%s", &words[i]);

とするか

    while(fscanf(fp, "%s", &words[i]) != EOF) {
        printf("%s", &words[i]);
        i++;

とすることを考えましたが、これでもまともに動くとは思えません。一見動いたように見えても、質問者が期待したようには動いていないでしょう。
siny[], words[] 等の データ構造を見直し、その使い方も見直す必要があると思います。目的に合ったデータ構造を用意しないとまともな結果を出せません。

---

追加。
rewind() する必要はありません。fopen()直後はファイル先頭から読み出せます。

1行ごとに読み込んでいるのでそれごとにi++して１単語ずつ入れているつもり

一行に一単語、それを words[i] に格納する「つもり」が、実際はどうなのか。
エラーなくコンパイルできる次のコードが

    i = 0;
    while (fscanf(fp, "%s", &words[i]) != EOF) {
        printf("%s\n", &words[i]);  // 読み込んだ単語を表示する
        i++;
    }
    printf(" => %s\n", words);     // words[] 配列の結果を表示する

"This" "is" "a" "pen" という４単語（４行）を char words[10000]; 配列に読み込む様子です。
結果、words[] には "Tiapen" という文字列が残ります。こうしたいわけじゃないでしょ。

Answer 3

char siny[] = {0};

こいつの配列の要素数は、１です

while(fgets(siny,3000,fp) != NULL) {

さっそくアクセス違反ですね