c言語 入門レベル問題 Control reaches the end of a non-void function

Question

下記コード実行したら、Control reaches the end of a non-void function　というエラー出ました。解決方法教えていただけますか。

#include <stdio.h>

// プロトタイプ宣言
int hantei(int l,int s);

int main()
{
    int i = 0 ,s = 0 ,l = 0 ,a = 0;
    printf("数値を入力してください==>");
    scanf("%d",&s);
    printf("数当てゲーム！チャンスは10回！\n");
    
    //10回繰り返す
    for(i=0;i<10;i++){
        printf("数値を入力!==>");
        scanf("%d",&l);
        
        //自作関数hantei の呼び出し
        a = hantei(s,l);
        
        //もしa==0ならばbreakで抜ける
        if(a == 0){
            break;
        }
    }
    
    //a != 0 ならば以下の処理を行う
    if(a != 0){
        printf("残念！正解の数値は%dです\n",s);
    }
 
    return 0;
}

//判定部分
int hantei(int s,int l)
{
    //l よりsが大きければ
    if(l < s){
        printf("正解の数値は大きいです\n");
        //戻り値として1を返す
        return 1;
    }
    
    //lよりsが小さければ
    if(l > s){
        printf("正解の数値は小さいです\n");
        //戻り値として2を返す
        return 2;
    }
    
    //lとsが同じならば
    if(l==s){
        printf(" 大正解です！\n");
        //戻り値として0を返す
        return 0;
    }
}

Answer 1

Control reaches the end of a non-void function　というエラー出ました

warning: control reaches end of non-void function [-Wreturn-type]ですね？
これはエラーではなく警告です。コンパイル自体は出来ています。

原因と対処

関数hantei内のif文の条件は全て排他的ですが、コンパイラはそれに気付いていないようです。
次のどちらかで対処できます。

if-elseを使う

int hantei(int s,int l)
{
    //l よりsが大きければ
    if(l < s){
        printf("正解の数値は大きいです\n");
        //戻り値として1を返す
        return 1;
    }

    //lよりsが小さければ
    else if(l > s){
        printf("正解の数値は小さいです\n");
        //戻り値として2を返す
        return 2;
    }

    //lとsが同じならば
    else {
        printf(" 大正解です！\n");
        //戻り値として0を返す
        return 0;
    }
}

最後に条件判定しないようにする

int hantei(int s,int l)
{
    //l よりsが大きければ
    if(l < s){
        printf("正解の数値は大きいです\n");
        //戻り値として1を返す
        return 1;
    }

    //lよりsが小さければ
    if(l > s){
        printf("正解の数値は小さいです\n");
        //戻り値として2を返す
        return 2;
    }

    //lとsが同じならば
    printf(" 大正解です！\n");
    //戻り値として0を返す
    return 0;
}

後者の方が個人的には綺麗でわかりやすく思えます。

Answer 2

    //lとsが同じならば
    if(l==s){
        printf(" 大正解です！\n");
        //戻り値として0を返す
        return 0;
    }
    return 0;

最後にreturn 0を入れる必要がありますね。

Answer 3

http://iphone-app-program.com/memo/error/【解決方法】control-reaches-the-end-of-a-non-void-function/

エラーメッセージでググったらすぐ出ます。

Comments

[GokaTokyo]

質問前に調べて、同じなもの出てきたが、頭悪くて、また、解決できなかったため、質問した。

[Zuishin]

頭の良し悪しは知りませんが、了解しました。

しかし何も書いていないのでは何もしていないように見えます。
解決法をみつけたならそのページを紹介してどこがわからないのか尋ねてください。

質問履歴を見ると C の基礎はわかったと書いてあったので、調べる方法を知らないだけだと思ってメッセージでググることを勧めました。

[GokaTokyo]

色々壁ありそうですので、引き続き応援お願いしますね