2019年7月29日2019年8月1日投稿者: takunoji

Java Mid Basic 〜Lv2_3_Javaの基本(リファクタリング実践)ゲームループ付き〜

イントロダクション

前回は、リファクタリングについて触れました。
しかしコードまではいかなかったので、今回はコードの話をします。

リファクタ実践

まずは修正するコードを眺めます。
そーすると、だんだん同じような処理をしているところが見えてきます。

同じ処理をしているのはどこでしょう？

Switch文のところです、switch(kigo) {とコードを書いている部分が複数見当たります。しかも結構な行数を使っている。。。

そんなわけで、対象の部分をまとめてしまおうと考えるわけなのですが、キーワードは「同じような処理(コード)」です、同じような処理なので、値を変えてやれば、ちょっと工夫をすれば、なんとかなりそうです。

自分の出した答えは

「メソッドを作成する」です。そして、作成するメソッドは入力された計算式から「足し算なのか、引き算なのか。。。」を判定するメソッドを作成しました。実際にコードを記載します。

/*
 * judgeKigo()のテスト
 */
@Test
public void testJudgeKigo() {
    Lv2_3_Calculate test = new Lv2_3_Calculate();
    try {
        int kigo = test.jadgeKigo("1+1");
        assertEquals(0, kigo);
        int kigo1 = test.jadgeKigo("1-1");
        assertEquals(1, kigo1);
        int kigo2 = test.jadgeKigo("1*1");
        assertEquals(2, kigo2);
        int kigo3 = test.jadgeKigo("1/1");
        assertEquals(3, kigo3);
    } catch(Exception e) {
        e.printStackTrace();
        fail();
    }
}

処理の内容としては以下の通りです。