成功回数を０と１で入力したい

Question

１０回結果を入力して成功回数を表示する。

結果は１と０で入力します。
その上で１と０以外を入力したらもう一度入力させる

どう書けばいいのか分からないです。

■■な機能を実装中に以下のエラーメッセージが発生しました。

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

１もしくは０を入力してもと表示され、最終的には回数を表示されます。
エラーした場合に入力回数を消費しない方法としてこれは正しいのでしょうか？
それとelse if で場合分けをしたつもりですが上手く行きません????

該当のソースコード

#include<stdio.h>
int main(void)
{
        int wins;
        int result=0;
        int i;
        for(i=0;i<=10;i++)
        {
                scanf("%d",&result);
                if(result==1)
                {
                        wins+=1;
                        result=0;


                else if(result==0)
                        result=0;
                else
                        i-=1;
                        printf("エラー\n");
                }
        return 0;
        }
        printf("勝利数は%dです\n",wins);
return 0;

}

試したこと

カギ括弧の位置を何回か変えましたが欲分からないです汗

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

リナックスで開発しています

Comments

[jimbe]

まず、{} を省略しない形で書いてみては如何でしょうか。

[majiponi]

仕様をもっと具体的に日本語にしてみてください。例えば、10回全部aって入力したら、10回入力させ直しますか? 間違ったのはスルーしますか? 入力のやり直しは何回でもさせますか? 寿限無のように長い入力にも対処できるようにしますか?
これ、意外と考えることが多いです。

Answer 1

「成功回数を０と１で入力したい」というタイトルが間違っています。

入力したいのは、成功回数ではなく、何かやった結果の成否を、
成功を 1、失敗を 0 として複数入力したい。そして、成功回数を出力したい。
タイトルは「成功回数を表示するプログラム」でいいでしょう。
入力や不具合は本文に書きましょう。

１０回結果を入力して成功回数を表示する。

タイトルは間違っていますが、この記述である程度のことは分かります。
しかし、成功を 1、失敗を 0 として入力するとは書かれていません。

その上で１と０以外を入力したらもう一度入力させる

やはり、成功を 1、失敗を 0 とするという記述がありません。
まあ、これは常識的に考えて、成功が 1 とするのが妥当でしょう。
1 と 0 以外と言っていますが、数値とは言っていません。
a や ? と入力した場合も再入力させたいのでしょうか？

■■な機能を実装中に以下のエラーメッセージが発生しました。

「■■な機能を実装中に」が意味不明です。

１もしくは０を入力してもと表示され、最終的には回数を表示されます。

何と表示されるのですか？
というか、該当のソースコードは、コンパイルエラーになります。
なぜ、エラーメッセージを書かないのですか？

そもそも、コンパイルエラーになるから、プログラムを実行できません。
すなわち、1 もしくは 0 を入力できません。

コードを見てみましょう。

int wins;
wins が初期化されていないので、値は不定です。
後で、wins+=1; で 1増やしても、値は不定のままです。

int result=0;
result は後で、scanf で値が格納されるので、この初期値 0 は使用されません。

for(i=0;i<=10;i++)
これでは、i は 0～10 の 11通りの値を取り、ループを 11回まわってしまいます。

if(result==1) の次の { に対応している } は
printf("エラー\n"); の後です。
この '{' から '}' が result が 1 の時に実行される文の集まりです。
その分の集まりの 3番目に else で始まる記述がありますが、
else で始まる文はありません。
wins+1=; と result=0: だけを result が 1 の時に実行してほしいのだから、
else の前に } を置かないといけません。
そして、result が 1でも 0でもない時に i-=1; と printf("エラー\n"); を
実行したいのだから、i-=1; の前に { が必要です。

return 0; は for文の { と } の中です。
i=0 でループを回り始めたのに、ここで main関数の終了となります。

Answer 2

if(result==1)

            {

こいつの閉じカッコがありません。
まず、C言語の基本的な文法をしっかり学びましょう
まだコード組む段階になってないように見えます

Answer 3

#include <stdio.h>
#include <stdbool.h>

int main(void) {
    int wins = 0;
    int i;

    for ( i = 0; i < 10; i++ ) {
        bool valid;
        do {
            int result;
            scanf("%d",&result);
            switch ( result ) {
            case 1:
                wins += 1;
                valid = true;
                break;
            case 0:
                valid = true;
                break;
            default:
                printf("エラー: 0か1を入力してね\n");
                valid = false;
            }
        } while ( !valid ); // 正当な入力でない間繰り返し
    }
    printf("勝利数は%dです\n",wins);
    return 0;
}