Tbpgr Blog

Employee Experience Engineer tbpgr(てぃーびー) のブログ

不要な分岐

概要

レビュー指摘「不要な分岐」について
Javaの話です

内容

私が実際に指摘した「不要な分岐」について。
コードを見ていただくのが手っ取り早いかと思います。

こういうコードを書くJava開発経歴10年のプログラマーとかが
居たりするのが日本の開発現場の怖いところ。

サンプルコード

package review.ex2.incorrect_if;

public class MusicPlayer {
  public void uncorrectPlay(Music music, String musicName) {
    // 以下、全く意味不明な分岐。分岐させる意味が無い
    if (musicName.equals("JPOPの何か")) {
      music.name = "JPOPの何か";
    } else if (musicName.equals("演歌の何か")) {
      music.name = "演歌の何か";
    } else {
      music.name = musicName;
    }
    System.out.println(music.name + "を再生中");
  }

  public void play(Music music, String musicName) {
    music.name = musicName;
    System.out.println(music.name + "を再生中");
  }

  public static class Music {
    String name = "";

    public Music(String name) {
      this.name = name;
    }
  }
}