java条件にすべて当てはまらなかった場合の処理を教えてください！

Question

前提

keywordに含まれる文字列があった場合、その文字列を出力
すべて当てはまらなかった場合にnoneを出力したいです。

実現したいこと

このコードだとすべてに当てはまらなかった場合が実現できません。
その前に該当の文字列があっても最後が非該当であれば、noneが出力されてしまいます。

このコードの間違っている部分は、
}else {
if (i == number - 1)
System.out.println("None");
}
の部分だと考えています。
特にi == number - 1の部分は最後のループで非該当の場合にnoneを出力するといった意味ですが
私が実現したいのは
if (strs[i].contains(keyWord)) {
System.out.println(strs[i]);
}
に一度も該当しなかった場合にnoneを出力するです。

発生している問題・エラーメッセージ

該当のソースコード

import java.util.*;


public class Main {
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        int number = sc.nextInt();
        sc.nextLine();
        String keyWord = sc.nextLine(); 
        String[] strs = new String[number]; 
        for (int i = 0; i < number; i++) { 
            strs[i] = sc.nextLine(); 
        }
        
        for (int i = 0; i < number; i++){
            if (strs[i].contains(keyWord)) {
                System.out.println(strs[i]);
            }else {
               if (i == number - 1)
               System.out.println("None");
            }
        }
        
        
    }
}

試したこと

補足情報（FW/ツールのバージョンなど）

Comments

[YT0014]

仕様が不明瞭です。複数のキーワードに該当した場合は、どのように動作する仕様なのか、質問文への追記をお願いします。

[YT0014]

https://teratail.com/questions/428at9fbd8bt9q にて、java_benkyoucuさんが、正しいコードを書けなかった原因を、コードを組む際の思考を思い出して、確認することをお勧めします。
それが正しく行えれば、この質問の答えも出ます。

[java_benkyoucu]

ありがとうございます！
もう一度考えてみます。

[jimbe]

>～に一度も該当しなかった場合にnoneを出力する

ご自身がコンピュータだったら、どのように処理するでしょう。
コードのように、1つ々々について「 keyword があるかチェックして有ったら表示、無かったら最後かどうかで "none" を表示」するでしょうか?
恐らくは、1つ々々について「 keyword があるかチェックして有ったら表示」を行った後に、「表示したかどうかで "none" を表示」するでしょう。(仕様のまんまですが。)
つまり、 "none" の表示(と表示するかの判定)はループの外であり、「表示をしたかどうか」を覚えておく(変数が)必要であるということです。

[java_benkyoucu]

なるほどですね！！！
ありがとうございます！！

[Zuishin]

こういうのは関数に切り出すのが良いと思います。

Answer 1

仕様が確定している場合のコーディングは、
アルゴリズムを決める
必要な変数を洗い出す
コードを書く
という手順を経る作業だと考えています。

前回も今回も、この２つめ、「必要な変数を洗い出す」が正しく行えていないことが原因だと思われます。

この仕様を見た場合、多くの場合、ループをさせ一致したらそれを出力、ループ後に一致が無かったら none を出力、と考えるのではないでしょうか？
次に、質問者さんは、変数として、keyword、検索する文字列配列、ループカウンタを洗い出したのでしょう。
ですが、実際にコードを組んでみて、後者の判定ができず、現在のコードになったかと思います。

これは、当初のアルゴリズムの「ループ後に一致が無かったら none を出力」を「最後のループで一致でないなら nonn を出力」への変更であり、仕様を満たせていません。

なので、アルゴリズムをキープし、変数の洗い出しで、一致の有無を保持する変数、を追加するべきでしょう。

前回の質問で、変数の洗い出しでの失敗だった、とか、入力はなくても状態を保持するのに変数が必要になる場合がある、とかを学んでおられれば、この質問は行う必要がなかったのではないでしょうか。

質問の答えを得て、障害が解消しても、そこで終わらず、その答えに到達しなかった理由まで検討しないと、学びにならないと思いますので、それらを心がけることをお勧めします。

かなり説教臭い回答となりましたが、少しでもお役に立てれば幸いです。

Answer 2

「contains(keyWord) が何回成功したか」を表す変数を作って、ループ終了後にその変数が0かどうかをチェックするのが1つの方法です。