なぜか初歩的なc言語のファイル読み込みができない

実現したいこと

一行ずつ数字が書かれたドキュメントを読み込み、それを一行分の数字をすべて表示させる
できれば読み込んだ文字としての数字をint型などに直したい

発生している問題・分からないこと

まったく読み込めない、そもそも文字としても読み込めない。たとえ読み込めたとしても数文字分しか表示されない。

該当のソースコード

c言語
1#include <stdio.h>
2#include <stdlib.h>
3int main(){
4FILE *file;
5char fname[] = "integer_data.txt";
6int num;
7file = fopen_s(file,fname, "r"); 
8if (file == NULL) {
9	printf("%s file not open!\n", fname);
10	return -1;
11}
12
13while (fscanf_s(file, "%d", &num) == 1) {  
14	printf("%d\n", num); 
15}
16fclose(file);
17return 0;
18}

試したこと・調べたこと

teratailやGoogle等で検索した
ソースコードを自分なりに変更した
知人に聞いた
その他

上記の詳細・結果

とにかくネットやchatGPTにあるものを試したがかすりもしない

補足

txtファイルには3200個の数字が書かれている

fana

2024/12/23 01:25

まず > fopen_s の使い方（引数や戻り値の型とか意味）を調べるところから始めれば良いと思う．

jimbe

2024/12/23 02:53

>まったく読み込めない、はご提示のコードであり得るとしても >そもそも文字としても読み込めない。たとえ読み込めたとしても数文字分しか表示されない。はご提示のコードでのことでしょうか。数値として読むコードで「文字として」はあり得ないように思うのですが。

dodox86

2024/12/23 03:57

> とにかくネットやchatGPTにあるものを試したがかすりもしない拙速に答えを求めるばかりでなく、コードの1行1行の処理と実行結果を確かめ、理解して進めましょう。そうでないと、いただいた回答を消費するだけです。

KAKAKASASASA

2024/12/27 15:18

>そもそも文字としても読み込めない。たとえ読み込めたとしても数文字分しか表示されない。の部分ですが、正確に表すと「� � 6」しか表示されません

行動規範の内容に同意します

回答6件

自己解決

回答が大変遅れてしまい、申し訳ございません。
皆さまがご提示くださったプログラムのおかげで、一歩前進し、文字を読み込むことができました。しかし、未初期化の変数が表示され続けるという問題が発生してしまいました。
それでも、この問題は最終的に自力で解決することができました。

以下に解決したプログラムを貼り付けます。（環境はwindows10 Visual Studio2022 ）

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


#define max 10
int main() {
    FILE* file;
    wchar_t line[128];  
    int ss[max] = { 0 };


    if (_wfopen_s(&file, L"float_data.txt", L"r, ccs=UTF-8") != 0) {
        perror("Error opening file");
        return 1;
    }
    while (fgetws(line, sizeof(line) / sizeof(wchar_t), file) != NULL) {
        float num = wcstol(line, NULL, 10);  

         for (int i = 0; i < max; i++) {
                if (num >= i * 100.00 && num <= i * 100.00 + 99.99) {
                    ss[i]++;
                }
            }


        printf("読み込んだ整数: %.2f\n", num); 
    }
for (int i = 0; i < max; i++) {
    printf("%.2f~%.2f ::", i * 100.00, i * 100.00 + 99.99);
            printf(" %d個 \n", ss[i]);
        }


    fclose(file);  

    return 0;
}

正しく文字を読み込めない原因が判明しました。どうやら、fopen_sはデフォルトでANSI形式でファイルを読み込んでいるため、UTF-8形式のテキストファイルを正しく処理できないようです。そこで、_wfopen_sを使用してUTF-8エンコーディングを指定しました。

（ファイルの文字コードはUTF-16でしたが、なぜかccs=UTF-16だと動作せず、UTF-8を指定すると正常に動作しました。）

また、UTF-16形式のため、char型ではなく広範囲の文字を扱えるwchar_t型を使用しました。
そのため_wfopen_sやfgetws、wcstolを使用しています。
以上の変更により、このプログラムは正常に動作するようになりました。

投稿2024/12/27 15:07

編集2024/12/27 15:26

KAKAKASASASA

総合スコア4

hiroki-o

2024/12/27 16:00

おめでとうございます！ 3点、補足説明があります。 - _wfopen_sとfopen_sの違いは、引数がワイド文字かどうかだけで、ファイルの文字コードは関係ありません。 - ccsを指定するとBOMを見て判定します。UTF-8を指定しても、BOMがUTF-16ならUTF-16で読み込みます。 - ccsに指定できるのは、UNICODE,UTF-8,UTF-16LEです。

行動規範の内容に同意します

このサイトで見かけたよくあるパターンとしては、「Cの学習テキストのプログラムをMS の VisualCでコンパイルしたら、「fopenじゃなくてfopen_sを使え」等の警告が出たので、単純にfopenをfopen_s、fscanf を fscanf_s 等と関数名だけ書き換えて、異なる関数であるにも関わらず、関数名を書き換えるだけでそのまま動くと思い込んでしまっている」
ですが、おそらく同じでしょうね。

案1：警告の意味を理解した上で、警告を無視する（ _s 付きの関数は使わない）
案2：警告の意味を理解した上で、警告に従って使う関数を変える。違う関数なので、その関数の使い方を調べてそれに応じてプログラムを書き換える
のどちらかです。C入門中の人の場合は、案1がお勧めです。
（「脆弱性の無いプログラムを書くというのはどういうことか」は基礎から体系的に学ぶべき事で、入門者がつまみ食い的に学ぶものではない）

なお、
案1改：警告の意味を理解せず、警告を無視する
は良くないです。

投稿2024/12/23 14:19

otn

総合スコア85989

まず、使用しているコンパイラは何でしょうか。
fopen_sやfscanf_sは、C言語の規格上は実装しなくてもよいので、gcc や clang では使えません。実質VisualStudio(VisualStudioCodeではないので注意)専用となります。

次に、fopen_sとfopenとでは、引数や戻り値の仕様、エラー処理の方法が異なります。fopen_sの情報が少ないので、誤ってfopenの情報を見て誤用するケースが後を絶ちません。

fopen_sを使える環境であれば、7行目から11行目を以下のように書き換えることで正常動作するようになるはずです。

c
1	if (fopen_s(&file, fname, "r") != 0) {
2		printf("%s file not open!\n", fname);
3		return -1;
4	}

投稿2024/12/22 21:34

actorbug

総合スコア2460

Windows 11、Visual Studio 2022で確認しました。
Visual StudioのC言語でも、先頭に1行おまじないを追加するだけで、fopenやfscanfが使えます。
otnさんの回答の「案1」ですね。私も、こちらをお勧めします。

// 先頭に追加
#define _CRT_SECURE_NO_WARNINGS

// 1行変更
file = fopen(fname, "r");

// 1行変更
while (fscanf(file, "%d", &num) == 1) {

他は変更無しです。惜しいところまで、できているので、がんばってください。

投稿2024/12/27 13:54

hiroki-o

総合スコア1145

Windows で Visual Studio を使っている前提ですが，fopen_s() の使い方はリンクを参照ください。

一行ずつ数字が書かれたドキュメントを読み込み，それを一行分の数字をすべて表示させる，できれば読み込んだ文字としての数字をint型などに直したい

については，fgets() と sscanf_s() を使って「１行分を文字列として読み込む」と「文字列が表す数値をint型の変数に格納する」を分けて記述すると分かり易いかもしれません。下記に記述例を示します。

C
1#include <stdio.h>
2#include <stdlib.h>
3
4#define N 100
5
6int main(void)
7{
8    FILE *file;
9    char fname[] = "integer_data.txt";
10    char line[N];
11    int num;
12
13    if (fopen_s(&file, fname, "r") != 0) {
14        printf("%s cannot be open!\n", fname);
15        exit(1);
16    }
17
18    while (fgets(line, N, file) != NULL) {
19        sscanf_s(line, "%d", &num);
20        printf("%d\n", num);
21    }
22
23    exit(0);
24}
25

（追記）

「自己解決」で提示されたコードとコメントを参考に更新してみました。 UTF-16（リトルエンディアン）のファイルから（BOM の有無にかかわらず）読み込めているようです。

なお，読み込むファイル名を引数として与えられるように変更しています。

C
1#include <stdio.h>
2#include <stdlib.h>
3#include <wchar.h>
4
5#define N 100
6
7int main(int argc, char *argv[])
8{
9    FILE *file;
10    char *fname;
11    wchar_t line[N];
12    int num;
13
14    if (argc != 2 || (fname = argv[1]) == NULL) {
15        fprintf(stderr, "usage: %s file_name\n", argv[0]);
16        exit(1);
17    }
18
19    if (fopen_s(&file, fname, "r, ccs=UTF-16LE") != 0) {
20        printf("%s cannot be open!\n", fname);
21        exit(2);
22    }
23
24    while (fgetws(line, N, file) != NULL) {
25        swscanf_s(line, L"%d", &num);
26        printf("%d\n", num);
27    }
28
29    exit(0);
30}