"a"を使っているのに上書きされてしまう

Question

前提・実現したいこと

プログラミング初心者です。現在C言語を学習しています。
正の整数を入力して、ファイルにその数値が存在しない場合、新しい行に書き加え、負の整数が入力されるまでその処理を繰り返し、２回目以降のプログラム実行でもファイル数値は保存されているようにするというプログラムを作成したいと持っています。助けていただけると幸いです。

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

ファイルに何も記載されていない状態だとうまく動いているようなのですが、２回目以降の実行時に前回入力した数値が上書きされてしまいます。

該当のソースコード

#include <stdio.h>

int main(){
    printf("ファイルの数値を確認し、新しい数値のみ記録する。\n");
    FILE *FP;
    FP=fopen("inputLog.txt", "w");
    fclose(FP);
    int num, Log, judge, count, loop, i;

    
    if((FP=fopen("inputLog.txt", "r"))==NULL){
            return -1;
        }else{
            do{ 
                count++;
            }while(fscanf(FP, "%d", &Log)!=EOF);  
        }
        fclose(FP);

    
    do{  
    printf("記録したい正の整数を入力してください。また、正でない整数が入力されるとプログラムを終了します。: ");
    scanf("%d", &num);
    FP=fopen("inputLog.txt", "r");
        for(loop=0;loop<count;loop++){  
            fscanf(FP, "%d", &Log);
            if(num==Log){
            printf("この数値はすでに記載されています。\n");
                fclose(FP); i=-1; break; 
            }else if(num<0){
            printf("プログラムを終了します。\n");
                judge=-1; fclose(FP); i=-1; break; 
            }
        }if(i!=-1){  
            fclose(FP);  
            FP=fopen("inputLog.txt", "a");
            fprintf(FP, "%d\n", num);
            fclose(FP);
            count++;  
        }
    }while(judge!=-1);
    return 0;
}

試したこと

fopen()で"a"を使いましたが上書きされてしまいます。

補足情報（FW/ツールのバージョンなど）

Xcodeでプログラミングしています。

Answer 1

全然読んでないけどこれじゃないですか？

FP=fopen("inputLog.txt", "w");
fclose(FP);

Comments

[papukin]

そこの部分はファイルがない場合に新しくファイルを作成するために入れました。

[Zuishin]

入れた理由はわかりましたが、ファイルがある状態でこれを取り除いてからテストしてみてください。これが原因なら追記されるはずです。

[Y.H.]

>pupkin
ファイルが無い場合だけでなく、ファイルが有る場合にも新しく空のファイルが作られますよ。

[Zuishin]

また、"w" をやめて "a" にしても、ファイルの無い状態でなら新しいファイルが作られます。"w" だと Y.H. さんのおっしゃる通り、既存のファイルは初期化されます。

[papukin]

"w"を使うと毎回ファイルの中身が更新されしまうので、"a"を使う方が良いのですね。"w"を"a"に変えてみましたが、やはり１回目は上手く動作しているようですが、２回目以降に入力した数値がファイルに記録されていないようです。

[Zuishin]

他にもあるということですね。時間があったら読んでみますが、その前にどうなるのか、またどうなったら正解なのかを詳しく書いてください。

[Zuishin]

見てみました。直接の原因は count が 0 で初期化されていないことです。また、不具合が起こる確率は低いといえば低いのですが、fana さんのおっしゃる通り i の初期化がなされていないのもよくありません。不具合が起こる時には起こります。

他にもうなるようなところはありますが、とりあえずそこを直せば動くと思います。

[fana]

iを-1にした後，それを（-1以外に)戻す処理が無いので，以降は何を入力してもファイルに追記されない状態ですね．

[Zuishin]

そこは見落としていました。確かに記録済みの数を一度入力するともう追記できませんね。do ループの先頭で初期化しなくてはいけません。

[papukin]

お二人の助言通りiの値を初期化できるようにi=0;をdo,while処理の先頭に追加したところ、上手く処理ができるようになりました！ありがとうございます！

Answer 2

後半の do～while ループ内における
変数 iやcountの役割を考えて，常に適切な値になるようにしてください．
（まず，初期値すら与えられていない…）