関数の中でelse ifを使った時のエラー

###前提・実現したいこと
C#でvoid Startの中にある三つの変数の中に入った数字を、関数を利用してどれが一番大きい値か振り分けてUnityのDebug.Logを利用して計算結果を表示するスクリプトを作っていたのですが、
このソースコードの１２行目に問題が見つかり、１２行目で「else if」の形にするとエラーが起こり、「else」だけにするとエラーが起きません。
これはelse ifと続けていったら、最後はelseで終わらなければエラーが起きるということでよろしいんでしょうか？
そうであればelse ifで終われない理由がありましたら教えて欲しいです。

###発生している問題・エラーメッセージ
（else ifで終わった場合）
エラー CS0161 '練習63.Big(int, int)': 値を返さないコードパスがあります。 New Unity Project 3

###該当のソースコード

C#
1
2using System.Collections;
3using System.Collections.Generic;
4using UnityEngine;
5
6public class 練習63 : MonoBehaviour {
7
8    int Big(int a, int b) {
9        if (a > b) {
10            return a;
11        } else if (a < b) {
12            return b;
13        } else if (a == b) {
14            Debug.Log("関数Bigの引数が同じ値です");
15            return a;
16        }
17    }
18
19	// Use this for initialization
20	void Start () {
21        int x = 5;
22        int y = 6;
23        int z = 9;
24        int big1;
25        int big2;
26
27        big1 = Big(x, y);
28        big2 = Big(big1, z);
29
30        Debug.Log(big2);
31	}
32}
33
34【成功したソースコード】
35
36using System.Collections;
37using System.Collections.Generic;
38using UnityEngine;
39
40public class 練習63 : MonoBehaviour {
41
42    int Big(int a, int b) {
43        if (a > b) {
44            return a;
45        } else if (a < b) {
46            return b;
47        } else {
48            Debug.Log("関数Bigの引数が同じ値です");
49            return a;
50        }
51    }
52
53	// Use this for initialization
54	void Start () {
55        int x = 5;
56        int y = 6;
57        int z = 9;
58        int big1;
59        int big2;
60
61        big1 = Big(x, y);
62        big2 = Big(big1, z);
63
64        Debug.Log(big2);
65	}
66}

###試したこと
関数で三つ目の「else if(a==b)」を「else」にして終わるようにしたら成功した

###補足情報(言語/FW/ツール等のバージョンなど)
C#

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

回答3件

こんにちは。

BAでてますが、ちょっと気になったので補足です。

最後はelseで終わらなければいけない

は誤りです。
「値を返さないコードパスがあります。」というメッセージの通り、最後のif (a == b)が成立しなかった時、return文が実行されないというエラーです。
ですので、elseを書く必要があるというわけではありません。下記でもOkです。

C#
1int Big(int a, int b) {
2        if (a > b) {
3            return a;
4        } else if (a < b) {
5            return b;
6        }
7        Debug.Log("関数Bigの引数が同じ値です");
8        return a;
9    }

投稿2018/01/13 07:27

Chironian

総合スコア23274

退会済みユーザー

2018/01/13 07:35

なるほど、回答を２つも貰いましたが、その中でもelseを書かなければ"if文が完璧でないからエラーになる"と思っていて本質を勘違いしていました... 問題はif文の処理の仕方ではなく、"関数の返り値がない場合を作らないようにする"ことが本質として大事だったんですね。勘違いをとくことができました、わかりやすい回答ありがとうございました！

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

値を返さないコードパスがあります。

上のエラーメッセージに現れているように、最後までelse ifで連ねてしまうと、最後のifが条件を満たさなかった場合に、returnのない最後部に突っ込んでしまいます。

intの場合はa > b、a < b、a == bのどれかが成立するので意味はなさそうに見えるのですが、例えばdoubleにした場合、aとbの両方にNaNを入れてしまうと、a > b、a < b、a == bのどれも成立しないという事態が実際におとずれてしまいます。

ということで、コンパイラは「a > b、a < b、a == bのどれかが成立する」と仮定出来ず、返り値を返せない可能性があるとエラーになります。

投稿2018/01/13 07:23