ああああああああああああああああああああああああああああああああああああ

Question

ああああああああああああああああああああああああああああああああああああ

Comments

[K_3578]

> ソースが少し冗長になっていると指摘をされた
なら指摘した本人に聞くべきでは？

[jimbe]

break の使い方の見直しが足りないものと思います。

[1T2R3M4]

fall throughさせるとか。
というかⅹは常に0ならswitch自体いらないのでは。

[dodox86]

私も1T2R3M4さんのご指摘と同じくfall throughを考えて、そうすればゴッソリ削れるはずですが、レビューしたら今度は「フォールスルーはバグのもとだから使っちゃダメ」とか言われそうですね。その辺り、レビュアーに確認したらどうでしょう。何か、試されているようなコードに思います。※尚、私自身は意図したフォールスルーなら迷わず使います。

[BeatStar]

パッと見、別にswitch...caseを使わなくても実装できそうなんですけど……

必ずswitch...casseを使わないといけませんか?

[YT0014]

switchを使うのは必須ですか？if文で書き直すのは許されるのでしょうか？

[wassan_nikoniko]

皆さん、ご指摘ありがとうございます。
おそらくフォールスルーという技法が一番削れるということでしたがそこで確認なのですが、
フォールスルーというのは条件式に当てはまったところ以降も処理が続くという認識なのですが、
それをどのようにうまく使っていけば削れるものなのでしょうか...?
初学者なため、まったくアイディアが浮かばないです...
また、何か参考になるサイト等ございましたらご教授いただければ幸いです。

[wassan_nikoniko]

必ずswitchでやらないといけないみたいでした

[jimbe]

コードに kadai と入っているくらいですしね。

Answer 1

こんな風にも書けます。

		switch (x) {
		default: System.out.println("ランクなし"); break;
		case 1: case 2: case 3: case 4: case 5:
			while (x <= 5)
				System.out.println("ランク" + ".ABCDE".charAt(x++) + "突破");
        }

Comments

[BeatStar]

トリッキーな書き方ですが、確かにアリっちゃアリですね。

[wassan_nikoniko]

おお...さきにdefaultを定義してしまうということですね...
case:1 2 3 4 5を定義したのちに出力で.charatで.ABCDEの順番をデクリメントで指定するということですね...

すごく興味深いコードで参考になりました！（使いこなせるかは別ですが...）
ありがとうございました！

Answer 2

今回の場合、switch...caseを使わなくても実装できそうです。
でも追記では「必ずswitch...caseを使え」という定義のようなので、
他の方も仰っているように、「フォールスルー」を行う必要がありますね。

(この質問に出会うまで、知らなかったけど)

「フォールスルーとは」で検索すると、ここがヒットしました。

上記によると、単純にbreakを置かないだけ。らしいです。

私ならまず最初に仕様を確認します。

で、今回の冗長と言われたコード(質問にあるコード)を見ると、

1のとき:
    Aランク
    Bランク
    ...
    Eランク

2のとき:
    Bランク
    ...
    Eランク

3のとき:
    Cランク
    ...

4のとき:
    Dランク
    ...

...

と言う風になっていますね。

では、これらの規則性を見てみましょう。

1 -> 2 -> 3 ... と数字が上がるごとに、A,B,C, ... のアルファベット表記のランクが下がっています。

逆を言えば、「数字が小さくなれば小さくなるほど、高ランクも通る」的なものですね。

別の視点で見てみましょうか。

Aランクを突破するのはどういう条件でしょうか。
そう、x＝1 を満たすときだけです。
{ 1 } だけ。

後はどうやっても突破できない。

では、Bランクは? そう、x≧2 を満たすときだけ。
つまり、{ 1, 2 } のどちらかじゃないとできない。

では Cランクは? そう、x > 3 を満たすときだけ。
つまり、{ 1, 2, 3 } のときだけ。

……と考えていくと、

「単純に、数字を1,2,3,4, ... と並べていき、フォールスルー方式でやればいい」ですね。

あえて疑似コードで書くと、

SWITCH( 対象 ):
   1の場合:
      // Aランク到達
   2の場合:
      // Bランク到達
   3の場合:
      // Cランク到達
   ...

って感じでしょうか。

ただ、1~5以外のもの(default)だと、「ランク無し」となりますから、区別を付けないと、「ランク無し」まで行ってしまいます。

ということで、1~4はフォールスルー方式でbreak無しでやり、5のときだけbreakで抜ける。

的な処理になるかと。

そうすれば重複している部分( 2のときは 1の時の「Aランク突破」が重複しているし ) を省くことができますよ。

---

[追記1]

※ ご指摘を受けての追記です。

上記でやっているフォールスルーをやる場合は、規則性を見付ける必要があります。

1~4は、「下位ランクも含まれている」ため、あえてbreakを省きます。

まずは、上記で述べたように1の場合,2の場合,... のそれぞれの「到達するランク」を列挙してみてください。

おそらく、以下のようになるはずです。

1: A, B, C, D, E
2:    B, C, D, E
3:       C, D, E
4:          D, E
5:             E
その他: <ランク無し>

1~4は 下位ランクも含んでいますね。

つまり、1は 2のやつを含んでいて、2のやつは 3のやつを含んでいて、...という感じです。

ちょっと考えを変えてみると、3の場合は、 <5> + <4> + <3> となっていますね。

よって、疑似コードで書くと、

SWITCH( 対象 ):
    1の場合:
        「Aランク」
    2の場合:
        「Bランク」
    3の場合:
        「Cランク」
    4の場合:
        「Dランク」
    5の場合:
        「Eランク」
        BREAK
    その他:
        「該当なし」系

となるはずです。

上記疑似コードを使ってシミュレーションしてみましょう。

たとえば、X = 1 の場合。

1. まず SWICH(...) で調べる
2. 「1の場合」を見る
3. 「Aランク」が該当する
4. breakが無いので「2の場合」も見る
5. 「Bランク」も該当する
6. breakが無いので「3の場合」も見る
7. 「Cランク」も該当する
...
10. breakが無いので「5の場合」を見る
11. 「Eランク」も該当する
12. breakがあるのでSWITCHから抜ける

となり、

Aランク到達
Bランク到達
Cランク到達
Dランク到達
Eランク到達

となります。

X = 2 の場合も、

1. まずSWITCH(...)で調べる
2. 「1の場合」を見る
3. (→ 条件を満たさないため スルー)
4. 「2の場合」を見る
5. 「Bランク」に該当
6. breakが無いため 「3の場合」を見る
7. 「Cランク」も該当
...
12. breakがあるのでSWITCHから抜ける

となり、

Bランク該当
Cランク該当
Dランク該当
Eランク該当

となる。

X = 5 のときは、

1. まずSWITCH(...)で調べる
2. 「1の場合」を見る
3. (→ 条件を満たさないため スルー)
4. 「2の場合」を見る
5. (→ 条件を満たさないため スルー)
6. breakが無いため 「3の場合」を見る
7. (→ 条件を満たさないため スルー)
...
10. breakが無いので「5の場合」を見る
11. 「Eランク」も該当する
12. breakがあるのでSWITCHから抜ける

となり、

Eランク該当

のみとなる。

で、X が 15以外の数字の場合は、上記の(1)(12)は完全スルーして、

1. まずSWITCH(...)で調べる
2. 「1の場合」を見る
3. (→ 条件を満たさないため スルー)
4. 「2の場合」を見る
5. (→ 条件を満たさないため スルー)
6. breakが無いため 「3の場合」を見る
7. (→ 条件を満たさないため スルー)
...
13. 上記の条件すべてに満たさないため defaultに
14. よって「該当無し」とする

となり、

該当なし

的なものになるはずです。

switch...caseは上からチェックしていき、該当するものがあれば、breakが出るか、最後までと到達するまで処理するようなので、今回の X が 1~4 までは break無しで処理をして、
5 で 処理をした直後にbreakを置く。

となるはずです。

Comments

[wassan_nikoniko]

ご返信いただきありがとうございます。
フォールスルーの技法私自身も初めて知りました...!
そうですね、ランクを一つずつ追加していくという書き方をしてあげればよかったんですが
どうしても浮かびませんでした！とても参考になりました。ありがとうございます！

[YT0014]

フォールスルーの場合、意図してbreakしていないことを、コメントなどで明記することをお勧めします。

[BeatStar]

@ YT0014さん
確かにそうですね。追記しておきます。

Answer 3

たぶんフォールスルーを勉強するための課題っぽいですが
どうしても switch 使うなら

package kadai.pda.k.oonuki2;

public class KadaiSwitch2 {

    public static void main(String[] args) {
        int x=0;
        switch(x) {
        case 1: p("ABCDE");
            break;
        case 2: p("BCDE");
            break;
        case 3: p("CDE");
            break;
        case 4: p("DE");
            break;
        case 5: p("E");
            break;
         default :
            System.out.println("ランクなし");
        }

    }

    public static void p(String s) {
        String[] a = s.split("");
        for (int i=0; i<a.length; i++) {
            System.out.println("ランク" + a[i] + "突破");
        }
    }

}

自分ならこんなかんじでにげるかな。