メソッドのリターン値

Question

このメソッドのリターン値を条件によって変えたいと思っているのですが、if, else文の中にリターン値を書いているにも関わらず、リターン値が無いという警告が出る理由が分かりません。
拙い文章ですがどなたかご教授頂ければ幸いです。

protected char lookAtTile(int y, int x) {
		ClientThread thread = new ClientThread();
		int otherPlayerPos[];
		int threadid=1;
		int theNumberOfThreadid = thread.getClientList().size();
		while(threadid<theNumberOfThreadid){
			otherPlayerPos=(int[]) thread.getClientList().get(threadid);
			if (otherPlayerPos[0]==y && otherPlayerPos[1]==x){
				threadid++;
				return 'P';
			}
			else if (y < 0 || x < 0 || y >= map.length || x >= map[0].length){
				threadid++;
                return '#';
			}
			else{
				char output = map[y][x];
                threadid++;
				return output;
			}
		}
	}

Answer 1

「threadid<theNumberOfThreadid == false」の場合、whileを抜けてしまいます。
while{}ブロック後のreturnが必要になります。

Comments

[mnmtksk]

納得しました。
素早いご回答ありがとうございました。

Answer 2

こんにちは。

解決されているようなのですが、ちょっと気になったのでコメさせていただきます。

ループの中でreturnすると確かにループ処理は終わるのですが、
業務などでメンテすることを考えると、メソッドの途中でのreturnは見逃されがちです。

メソッドが短いものだったら見つけやすいのですが、
中には長いメソッドも存在します。
アチコチ別のメソッドを呼び出しているメソッドもあります。

なので、処理を追いやすいようにreturnはメソッドの一番最後に書くようにして、
ループはbreakやcontinueで抜けるようにした方がいいです。
(一番いいのは、ループの途中で抜けるような条件にしないことですが ^^;)
breakやcontinueにはラベルも付けられるので、活用してみてください。

Comments

[mnmtksk]

なるほど、そうなのですね。
次回は条件の設定にもう少し気を付けてみます。
今回のように複数の条件によって返す値が違う場合、どのように最後にまとめればよいのか、考えてみる事にします。
参考になりました。ありがとうございます。

[kaputaros]

そうですねー、たとえば、↑のコードですと、
「char resultChar」みたいにreturn用の変数を用意し、
現在returnしている部分をreturn用の変数に設定してやり、
そこでbreak;してループを抜け、
最後にreutnr resultChar; すればいいと思いますよ。

コードにしてみると↓こんな感じです。
protected char lookAtTile(int y, int x) {
　char resultChar = '';
　～
　while(threadid < theNumberOfThreadid){
　　if (otherPlayerPos[0]==y && otherPlayerPos[1]==x) {
　　　～
　　　resultChar = 'P';
　　　break;
　　} else if(y < 0 || x < 0 || y >= map.length || x >= map[0].length) {
　　　～
　　　resultChar = '#';
　　　break;
　　} else {
　　　～
　　　resultChar = map[y][x];
　　　break;
　　}
　　threadid++;
　}
　return resultChar;
}

がんばってください^^

Answer 3

これwhile いりますか？
whileに入った後1回でreturnしてしまうのにループの存在意義がないのでは？

Comments

[mnmtksk]

仰る通りですね。ご指摘ありがとうございます。
そこで質問なのですが、下のバージョンで、whileループのif文の条件がtrueならPをリターンしてこのメソッドは終了するのでしょうか。それとも後述のif文の条件に当てはまれば#やoutputをリターンしてしまうのでしょうか。
"protected char lookAtTile(int y, int x) {
ClientThread thread = new ClientThread();
int otherPlayerPos[];
int threadid=1;
int theNumberOfThreadid = thread.getSizeOfClientList();
System.out.println("theNumberOfThreadid: "+ theNumberOfThreadid);
otherPlayerPos=thread.getSinglePosition(threadid);
while(threadid<theNumberOfThreadid+1){
otherPlayerPos=thread.getSinglePosition(threadid);
if (otherPlayerPos[0]==y && otherPlayerPos[1]==x){
threadid++;
return 'P';
}
}
if (y < 0 || x < 0 || y >= map.length || x >= map[0].length){
return '#';
}
else{
char output = map[y][x];
return output;
}
}"

[unau]

Java に限らず一般に return 文を実行したらそのメソッド（or 関数）から抜け出ます。

[mnmtksk]

ご回答ありがとうございます。
今のところ、プログラム全体にエラーは無いものの、自分の求めているoutputが出ないので、ご指摘いただいたところも含め、もう一度考えてみます。

Answer 4

threadid<theNumberOfThreadid の条件が偽になって while 抜けた後は return なしでメソッドを抜けますよね？
あと、ちゃんと解読していないのであれですが、threadid++; は if ブロックや else ブロックの中から括りだして if の前に置いてははいけないのですか。

Comments

[mnmtksk]

incrementsを条件の前に書くという発想はありませんでした。
ありがとうございます。