「missing return statement」の原因

Question

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.util.*;

public class Main {
  public static void main(String[] args) throws Exception {
    Main m = new Main();
    m.input();
    m.Aggregate();
    }
    
    StringBuilder sb = new StringBuilder();
    Scanner sc = new Scanner(System.in);
    int roop = sc.nextInt();
    String com = sc.next();

  public void input(){
    for(int i = 0 ; i < roop ; i++){
      sb.append(com);
    }
  }

    int sum = 0;
    int count = 0;

  public void Aggregate(){
    for(int i = 0 ; i < sb.length() ; i++){
      Character c = sb.charAt(i);
      System.out.print(sum + " ");

      if(c.equals("A")){
          sum += (1+count);
          count++;
          System.out.print("A");
      }else{
          count = 0;
          System.out.print("B");
      }
    }
    System.out.println(sum);
  }
  
    public boolean equals(Object obj) {
      if (obj == obj)
          return true;
     
      if (obj == null)
          return false;
          
    }
}

■入力内容
3
ABBAA

■質問
Aggregateメソッドの中で、sb変数の中身を1文字ずつ取り出して
if文に使いたいと考えています。

charのままではcharAtメソッドが使えないので、characterにしたのですが、
そうするとequalsメソッドが文字と変数の中身を等値と判断してくれません。
equalsをオーバーロードしようとしてみたのですが、
「missing return statement」が出てきて出来ません。

オーバーロードしてるequalsを消せばエラーは消えます。
このエラーが発生する原因を教えていただけますでしょうか。

Answer 1

エラーとなる以下のコードですが、もし2つのif文の条件式が両方ともfalseだったとしたら、このメソッドは何を返しますか？ 考えてみればエラー・メッセージの意味がわかるでしょう。

public boolean equals(Object obj) {
    if (obj == obj)
        return true;

    if (obj == null)
        return false;
}

ちなみに「obj == obj」の比較式にどのような意味がありますか？
さらに言えば、そもそもequalsメソッドをオーバーライド（オーバーロードではなく）する必要はないですよね？
Characterクラスのequalsメソッドに渡す引数の型が正しいかどうか考えてみましょう。

Comments

[KSwordOfHaste]

obj == objはthis == objの書き間違えなのでしょうが、obj == nullの行に意味がない点を指摘したいところですね。thisはnullではありえないので、this == objのテストの後にobjがnullかどうかチェックする必要はないので。

[swordone]

意味はありますよ。
obj == nullを通るのはobj == obj改めthis == objが偽の時なので、nullである可能性もあります。
もっとも、そのあとキャストしてメンバ比較をしないと意味がないことに変わりはないのですが…

[KSwordOfHaste]

これはしたり。この後の処理を想定せずに意味があるとかないとかいうこと自体に意味がなかったですねｗ；
自分はつい
if (this == obj) return true
return false
を想定して物をいっていました。ご指摘ありがとうございました。

[swordone]

それだとObjectのequalsと全く同じなのでオーバーライドする意味すらないですがね…

[KSwordOfHaste]

意味がないのでamadablamさんの「オーバーライドの意味がない」に続く・・・という意図だったのですが、先に述べたようにif (obj == null)の後ろの処理を想定せずにコメントしたこと自体に意味がなかったという顛末です。

[sakuman0]

>amadablamさん

返答が遅くなり申し訳ありません。
回答いただき、ありがとうございます。

「obj==obj」は確かにおかしいですね。。。
両方ともfalseだったらというのは、equalsメソッドがtrueを返す可能性が
無い状態になっているのが問題ということでしょうか。
そういうことであれば、理解できました！

また、characterクラスのequalsメソッドの型はobject型なので
何でも入れられるはずだと、理解に時間がかかっていたのですが、
文字列と文字を比較していて為、等値ではないと判断されたと理解しました。

回答有難うございました。

[swordone]

「trueを返す可能性がない」のではなく、
「trueもfalseも返せない可能性がある」のが問題だったのです。

[amadablam]

sakuman0さん、プログラムの問題は答えがすべてではなく、自分で考えて答えを導き出せる過程が大切です。そういう意味では、今回のことで新たに身に付いたことがあり、よかったですね。
sordoneさんが書いていることが正しい答えですが、そこに至る仕組みを理解するように頑張りましょう。
ちなみに、今回のプログラムの意図が、入力された文字列に含まれるAの数をカウントするだけであれば、以下コードだけでも十分ですよ。
Scanner sc = new Scanner(System.in);
long count = Arrays.stream(sc.next().split(""))
.filter(s -> s.equals("A"))
.count();
勉強頑張ってください。

[swordone]

long count = sc.next().chars().filter(c -> c == 'A').count();
これだとStringインスタンスを多数作らずに済む

[sakuman0]

> amadablamさん、swordoneさん

回答ありがとうございます。
エラーの原因について理解できました。
プログラムも修正して実行してみたところうまくいきました！

サンプルコードも参考にさせていただきます。
JAVA8の機能を勉強しなくては…。

Answer 2

ここで書いているequalsメソッドはMainクラスに書かれているため、Mainクラスのインスタンスと他のオブジェクトを比較する時にしか機能しません。ここにequalsをオーバーライドしたところでCharacterのequalsには何ら関係ありません。
Characterはchar値をラップしてオブジェクトとして扱えるようにしたものにすぎず、そのequalsメソッドは同じchar値を持つCharacterオブジェクトに対してのみtrueとなります。

      if(c.equals("A"))

ここで使用しているequalsはCharacterに対してString(ダブルクォーテーションで囲ったリテラルはString)を比較しているため、ifが成立しないのです。
またオーバーライドしようにも、Characterクラスはfinalクラスなので、継承自体が不可能です。

正しくはCharacterと比較するため、文字をシングルクォーテーションで囲ってcharリテラルとする必要があります。

      if(c.equals('A'))

もしくはそもそもCharacterなんぞ使わず、

char c = sb.charAt(i);
...
if (c == 'A')

これでいいのでは？

Comments

[sakuman0]

回答ありがとうございます。
char型はシングルクオーテーションで囲まないといけないのを
忘れていて気付きませんでした。

あと、characterを使ったのは、charAtではなくequalsを使う為でした。
すみません。

[sakuman0]

オーバーライドの説明は、Mainクラスのequalsをオーバーライドしたところで、
characterクラスのequalsは元のままだということですね。
説明を受けながら見ることで、ようやく理解できてきました。