java if文のデバッグの指導をお願いします。

Question

java歴１ヶ月未満で下手くそなコードだと思います。

下のコードを実行した際、
まず「new Taipingのコンストラクタが出力されない」こと、
「ゲームの難易度を「簡単」または「難しい」とコンソールで入力してもelse文のコードが出力されてしまう」こと、
「正しく入力してもelse文の不正解！が出力される上、同じ単語の無限ループになってしまう」ことの解決方法のご教授をお願いします。

上のうち一つだけでも教えて頂けると嬉しいです。

public class Main {
    public static void main( String[] args){
        double random = Math.random() * 5 + 1;
        int number = (int)random;
        //乱数を持つフィールドの定義
        Taiping taiping = new Taiping("英単語タイピングゲーム","【簡単】または【難しい】を入力してください");
        //難易度の選択の入力
        System.out.println("英単語タイピングゲーム");
        System.out.print("【簡単】または【難しい】を入力してください : ");
        taiping.start( number );
    }
}
  import java.util.Scanner;
public class Taiping {
Scanner scanner = new Scanner( System.in);

    String name;
    String level;
    Taiping( String name, String level ){
     this.name = name;
     this.level = level;
    }
    //コンストラクタ
    
    String cheese   = "チーズ :";
    String dog      = "犬 :";
    String cat      = "猫 :";
    String yesterday= "昨日 :";
    String moon     = "月 :";
    //タイピングの単語
    public void start( int number ){
        for( int s = 1; s <= 5; s++ ){
            String levels = scanner.next();
            if(  levels == "簡単" || levels == "難しい"   ){
               s = 25;
            } else {
                s = 1;
                System.out.print("【簡単】または【難しい】を入力してください : ");
            }
       }
        int i = 1;
        while( i <= 5){
            System.out.println( number );//５〜１の乱数の出力
        switch( number ){
            case 1:
            System.out.println( this.cheese );
            String cheese0 = scanner.next();
            if( cheese0 == "cheese"){
                System.out.println("正解！");
            } else {
                System.out.println("不正解！");
            }
            break;
            case 2:
            System.out.println( this.dog );
            String dog0 = scanner.next();
            if( dog0 == "dog"){
                System.out.println("正解！");
            } else {
                System.out.println("不正解！");
            }
            break;
            case 3:
            System.out.println( this.cat );
            String cat0 = scanner.next();
            if( cat0 == "cat"){
                System.out.println("正解！");
            } else {
                System.out.println("不正解！");
            }
            break;
            case 4:
            System.out.println( this.yesterday );
            String yesterday0 = scanner.next();
            if( yesterday0 == "yesterday"){
                System.out.println("正解！");
            } else {
                System.out.println("不正解！");
            }
            break;
            case 5:
            System.out.println( this.moon);
            String moon0 = scanner.next();
            if( moon0 == "moon"){
                System.out.println("正解！");
            } else {
                System.out.println("不正解！");
            }
            break;
            default:
            System.out.println("いやん");
            }
            i++;
        }
    }
}

Comments

[javahack]

コピーに失敗したのかもしれませんが、提示のコードではコンパイルが通りません。
質問は編集できますのでコンパイルが通るコードを提示してください。

[m.ts10806]

「デバッグの指導」とは何の意図があってつけたタイトルなのでしょうか。
質問本文にはデバッグの話は一切触れられていません。丸投げしてるだけです。

[m.ts10806]

コードブロックにはコードだけを記載してください。1クリックでブロック内がコピーされるので、コード以外があると邪魔になります。

[Rikuto_]

m.tsさん言い方はうざいですがありがとうございます。

[m.ts10806]

ウザいも何も、質問内容を見たら当然の反応かと思いますが。

[Rikuto_]

言い方がきついということです。
私がテラテイルやプログラミング歴が長いならわかりますが、初学者に対する接し方として適切ではありません。
それがわからないのは友人関係もおおよそ想像できます。

[m.ts10806]

>友人関係もおおよそ想像できます。
本件と関係あるのでしょうか。
単なる思い込みによる個人攻撃なら通報します。

>初学者に対する接し方として適切ではありません。

淡々としているだけで、勝手にそちらがそう受け取られてるだけかと思います。
何も事情を知らない他人が見たらどう思うかくらいは技術レベルや歴は関係ないですよ。一般的な配慮の範疇です。

[Rikuto_]

色々な方法を模索した上で質問しています。
それを丸投げしてるだけと言うのはかなり印象悪いです。
ただ、私は「デバッグの指導」をバグ修正という目的に向かって教え導いてもらうこと（一応wikiから引用してます）と思っていますが、
間違っているなら丸投げという発言も納得し謝罪します。

[javahack]

”初学者”であってもサイトのルールに反していたり間違いがあれば、指摘され修正を求められます。ただ、それだけです。
初学者だから優遇しろ、優しくしろと言うのは正直甘えだと思います。
ちやほやされたかったら、有料サイトを利用した方が良いかもしれません。
お金を払った分は優しく懇切丁寧に教えてくれるでしょう。

[Rikuto_]

初学者関係なく　エラーに苦しんだ上質問して「丸投げしただけ」と言われ良い気持ちになる人なんていないと思います。
「デバッグの指導」についての返答がないのはどういう意図でしょうか。
私の解釈が間違っていれば謝罪すると明言していますが。
これまでの会話で私がちやほやされたいと思っていると感じてしまうのも　あなたの感性が一般の人々と乖離していますね。

[m.ts10806]

文句があるなら運営にいってください。
https://teratail.com/help/avoid-asking
>コードをください・デバッグしてください等の丸投げの質問
>何かを作りたいのでコードを書いてほしい、学校の課題を解いてほしい等の質問は、具体的にプログラミングで困っている質問ではないと考え、推奨していません。
>問題や質問は実際に調査や作業に取り組み、具体的なところで生まれると考えるためです。
>まずは実際に作業に取り組み、つまづいたところで投稿をしてみてください。

他人には質問に書いてあることが全てです。
やりたいことは書いてあっても、調べたこと、試したこと　何も書いてません。
これで丸投げと思わない方が無理があります。

いずれにしても「デバッグの指導をしてほしい」とタイトルに書きながら自身でデバッグした形跡がなければ誰が見ても丸投げですよ。
「デバッグを指導してほしい」ではなく「デバッグしてほしい」という作業依頼になっています。解釈は大いに間違っています。
「デバッグの仕方を教えてほしい」なら分かります。でも、質問内容はそうなっていません。

第三者になったつもりで質問を読みなおしてみると良いでしょう。
・質問になっているか、読みやすいか・要件以外に余計な文言はないか
　（初学者云々は不要。「初心者アイコン」を質問につければ十分）
・コードは読みやすく提示されているか（コピペで再現可能か）
・起きている問題に対し自身が取り組んだことは明示されているか

確か、質問投稿時にチェックリストのようなものがあったかと思いますが、それらはきちんと見たのでしょうか。
「質問する前に確認していただきたいこと」という画面も出るはずです。
それらを全て無視して「俺が書きたいように書いた」だと「お客様扱いしてほしいのね」と理解されても仕方がないことです。

[Rikuto_]

運営に対する文句なんてありませんが。
エラーと表示されず思うように動作しないことに数時間かけて自分で調べながら対処しようとしました。
「具体的なところで生まれる」の「具体的」とはあなたの基準です。今の私の知識は質問したバグの原因さえ全く検討がつかないといった段階であり、私にとっては具体的なところで生まれたものです。
そのため質問に「解決方法のご教授」と記述しています。たんに答えを教えてほしいと言っているのではありません。
解決方法を教えてもらうことはデバッグの指導の範疇に収まらないのでしょうか？
質問の仕方の調べが足りず、質問内容に欠落した部分があるという非は認めます。次に質問を投稿する際は正しく記述できるよう努力します。
最後に。それを「お客様扱いしてほしいのね」と理解するのは飛躍し過ぎです。それは単なる思い込みです。

[m.ts10806]

運営のガイドラインに「問題や質問は実際に調査や作業に取り組み、具体的なところで生まれると考えるためです。」とあって、「実際に行った調査や作業に取り組み」が一切書かれていないことを指摘しているのに終始そこを無視されるので「文句があるなら私ではなく運営に」という意味で書きました。
あとはおおよそ理解されていないようですし繰り返すのは面倒なので、反論したいのでしたらきちんと読んで理解してから反論してください。

[Rikuto_]

ガイドラインの引用だったんですね。＞これが引用を指すのは知りませんでした。失礼しました。
「実際に行った調査や作業」については
・「数時間かけて自分で調べながら対処」
・「今の私の知識は質問したバグの原因さえ全く検討がつかない」
つまり質問する前はどうコードを変えたらいいか検討もつかず訳わからないという状態でした。
その上で「欠落した部分があるという非は認めます」と述べています。これは「実際に行った調査や作業」の記述をしていないところを言っています。
他の話は理解できないと思うのでもう終わりでいいです。

[m.ts10806]

なるほど。いつ来るかわからない次の機会を楽しみにしておきましょう。
ただ、せめて、「文章」と「コード」くらいは別にしてもらえませんか？
この質問は削除されない限り半永久的に残り続けるので、後から見る人が読みにくいのは非常によろしくないです。

[Rikuto_]

すみません先日指摘されてすぐ直したつもりでしたが変更内容に記述が抜けていて更新できていませんでした。
今直しました。
「いつ来るかわからない次に機会を楽しみにしておきましょう」
とありますが、煽っていると捉えていいのですよね。
私は当分質問する側なので、あなたは回答に来ないでください。

Answer 1

Stringの内容の比較は==ではできません。
equalsメソッドを使ってください。

foo.equals(bar)
で、fooとbar二つのStringの値を比較できます。

Comments

[deaf_tadashi]

foo.equals(bar)はnull時の考慮がどうのこうので論争になるので、Java7以降ならObjects.equals(foo, bar)の方が安心です。

[A-pZ]

+1

[Rikuto_]

ありがとうございます！！　無事きちんと動作しました????‍♂️