条件分岐で継承されたクラスを使う

Question

Javaでクラスの継承について学んでいます。
ユーザーからの入力でplusが入力されたらPlusCalcクラスを用いて足し算へ、mulが入力されたらMulCalcクラスを用いて掛け算をして結果を出力したいと思っています。
現在入力を受け付けるだけでその後のif文の実行がされない状態です。
3つ入力するだけで何も起こりません。エラーなども出ていないです。
Calcクラスのメンバーの使い方がよくわからず詰まってしまいました。
どのようにすれば継承先のメソッドが使えるのかご教示お願いいたします。

import java.util.*;

public class Calculate{
    public static void main(String[] args) {
        Calc c = new Calc();
        Scanner scan1 = new Scanner(System.in);
        Scanner scan2 = new Scanner(System.in);
        Scanner scan3 = new Scanner(System.in);
        c.x = scan1.nextInt();
        c.input = scan2.next();
        c.y = scan3.nextInt();
        if (c.input == "plus") {
            PlusCalc pulsRes = new PlusCalc();
            pulsRes.plus(c.x, c.y);
        }else if(c.input == "mul"){
            MulCalc mulRes = new MulCalc();
            mulRes.mul(c.x, c.y);
        }
    }
}

class Calc {
    Integer x;
    String input;
    Integer y;
    Integer res = null;

    // public Calc(){}

    // public Calc(Integer x, Integer y){
    // this.x = x;
    // this.y = y;
    // }
}

class PlusCalc extends Calc{
    // public PlusCalc(Integer x, Integer y){
    //     super(x, y);
    // }
    public void plus(Integer x, Integer y){
        Integer res = x + y;
        System.out.println("結果: " +res);
    }
}

class MulCalc extends Calc{
    // public MulCalc(Integer x, Integer y){
    //     super(x, y);
    // }
    public void mul(Integer x, Integer y){
        Integer res = x * y;
        System.out.println("結果: " +res);
    }
}

以下理想の出力です
100
plus
200
結果：300

Comments

[maisumakun]

「その後のif文の実行がされない状態です」とありますが、何かエラーなどは出ていますか？何も表示せずに終了する感じでしょうか？

[gentaro]

まぁなんというか、そもそもPlusCalcとMinusCalcというクラスを派生させるのが設計的におかしいと思いますよ。plusもmulも普通に考えればCalcクラスの責務だと思うし。
百歩譲ってそのクラス設計で行くとして、mainメソッドの中でCalcクラスのオブジェクトを作っているのも意味がわかりません。

Answer 1

        if (c.input == "plus") {
            PlusCalc pulsRes = new PlusCalc();
            pulsRes.plus(c.x, c.y);
        }else if(c.input == "mul"){
            MulCalc mulRes = new MulCalc();
            mulRes.mul(c.x, c.y);
        }

入力された「文字」と比較するなら、==ではなくequals()です。
if文の条件に何も引っかからずにそのまま終了しています。

あと、以下も気になったのでコメントします。

• scan2、scan3はscan1と同じなので不要です。scan1だけでもその後の処理は期待通りになります。
• 最後にscan1.close();をしてストリームを閉じる癖をつけたほうがいいです。
• switch文で判定するのもありですね。

Comments

[kaputaros]

あ、回答被った(笑)

Answer 2

× c.input == "plus"
○ c.input.equals("plus")

---

継承の使い方として練習の途中のコードと思いますので、ひとまずこれだけで。

Comments

[cacao86]

ご提示通り書き換えたら無事動作いたしました！ありがとうございます！
そこが悪いとは気付きませんでした。

自分の中でこのコードすごく汚く見えるのですが、お手隙でしたら、どう直すべきか意見などいただければ幸いです！

[quickquip]

いや、練習になってないは言い過ぎでした。訂正します。
練習の途中だと思うので「継承の使い方に関してはここでは立ち入りません」という意図なだけでした。