if文を通らない理由がわかりません

Question

import java.io.*;
import java.util.*;
public class Main{
    
    public static void main(String[] args) throws IOException {
      Aisatsu aisatsu = new Aisatsu();
      aisatsu.execute();
      
      while(true){
      MenuHyouji menu = new MenuHyouji();
      menu.execute();
      
      System.out.println("メニュー番号を選んでください");
      Scanner scan = new Scanner(System.in);
      String str = scan.next();

      if(str.equals("1")){
          KaisyaJyouhou kaisyaJyouhou = new KaisyaJyouhou();
          kaisyaJyouhou.execute();
          continue;
      }
      
      if(str.equals("2")){
          HajimeteUser muryouSample = new HajimeteUser();
          muryouSample.execute();
          continue;
      }
      

      }
    }
}

import java.io.*;
import java.util.*;

class HajimeteUser{
    public HajimeteUser(){
        
    }
    
    public void execute()throws IOException{
      System.out.println("yes/no");
      Scanner scan = new Scanner(System.in);
      String str = scan.next();
      
      while(true){
          
      if(str.equals("yes")){
        System.out.println("住所を入力してください");
        String adress = scan.next();
        if(adress.equals("1")){
          System.out.println("住所を登録しました");
          break;
        }else if(adress == null){
          System.out.println("住所は必須です");
          continue;
        }
      }
      if(str.equals("no")){
        break;
      }  
      }
    }
  }

上記実行すると、
}else if(adress == null){
System.out.println("住所は必須です");
continue;
}

入力しないでエンターを押すと上記のif文を通らないのですが、理由がわかりません。
教えていただきたいです。

Answer 1

色んな考慮がされてない実装なので、自由入力を許す場合、いきなり書くのではなくフローチャートなど用いてケースパターンを洗い出すことを強くすすめます。

一番最初の入力でも考えることは沢山あります。
yes、no以外入力されたらどうなるか？
YesやNoだとどうなるか？
yesが来たとして、 1以外だとどうなるか？

となると「while(true)に入るタイミングが適切か」というのも見えてくるはず。

既に指摘されているように「分岐を通らないのはなぜか」を調べるのは大事なことなのですが、「どこまで考慮された設計となっているか」のほうが大事です。

ユーザーは何を入力するかアプリケーションは知りません。
ユーザーの入力は信用してはいけません。
「モンキーテスト」という工程もあるくらいです。
正常系は通って当たり前で、異常系をどれだけ設計時に考慮できるかで実装の粒度も精度も変わってきます。そのままアプリケーションの質にもかかわります。

別に「設計」といってもそこまで大それたことはしなくていいと思います。
先に書いたフローチャートのようなものに補足を入れたものでも十分です。
手書きメモレベルでもいいです。

今回だと「どういう入力があるか」「どういう入力があった時にどうするか」という点を重点的に考えてください。

もう少し具体的に言うと、
if-elseifだけではなくelseも考えることです。
ホワイトリスト、ブラックリストで言うと、今回は「ホワイトリスト」を採用することになると思います。
つまり、「許可している入力以外はNG」とするわけです。

もう少し細かく考えていくと今回「なぜ想定の場所を通っていないか」も見えてくるはずです。

「実装の考慮が足りない」のが本質。
「値の確認の仕方（デバッグの仕方）を身に着けていない」というのが手持ちスキルの問題。これに「改行、空文字、nullの違いなどの理解」も含みますかね。

直面している問題を解決するだけならすぐですが、
それ以前に理解していないと結局同じところに戻ってくるのは間違いないので、
ここでしっかり理解されるよう、強くすすめます。

書いたとおりにしか動かないプログラミングにおいては
「惜しい」というのは存在しません。true（できた）かfalse（できてない）かだけです。
時には今書いてきたコードを一旦捨てて全部書き直すようなことも必要です。
「あとちょっとなのに」を繰り返すと穴だらけのコードを助長するだけという事態も起きます。
なので実装に入る前の設計が大事なのです。

Answer 2

入力しないでエンターを押すと

エンターを入力しているので、改行コードだけ入った文字列が取れます。

Comments

[kanekoK]

入力がなかった場合の条件式にするにはどうしたらいいでしょうか？

[y_waiwai]

実際に入力がなかったときにはどういう文字列が取ってこれるのか見てみればどうでしょうか

[kanekoK]

文字列は何も返ってきませんでした。
↓実行結果
住所を入力してください

[kanekoK]

空行ができるのみでした

[maisumakun]

最初に書いたように、それが「改行コードだけ入った文字列」です。

[kanekoK]

System.out.println("住所は必須です");を
表示するにはどう変更したらいいでしょうか？

[y_waiwai]

そのなにも帰ってこないように見える文字列に、どういう「文字」が含まれてるのか表示してみたらどうですか？

[kanekoK]

それはどのように確認できますか？

[y_waiwai]

https://www.javadrive.jp/start/string/index5.html
これをどうぞ

Answer 3

}else if(adress == null){

これはadressというオブジェクトが存在しないということを指しています。
この「オブジェクトが存在しない」ということと「文字が１文字もない」ということは別物です。
今回はScannerクラスからのメソッドの戻り値を受け取っているのでオブジェクトは存在しています。
ですので、１文字も入力されていなくてもオブジェクトは存在するので、このelse if文は通らないわけです。

なので、チェックすることはオブジェクトの有無ではなく、オブジェクト内部に格納されている文字列についてです。

Stringクラスに入っている文字の長さは以下のメソッドで確認できます。
int length()

また、文字が格納されているかどうかは以下のメソッドで確認できます。
boolean isEmpty()

これで修正の仕方はわかるかと思います。

Comments

[rinjinto]

あと、質問とは違いますが、Scanner.next()の使い方が正しくないです。
Scanner.hasNext()の戻り値をチェックしてtrueの時のみnext()で取り出すようにする必要があります。
万が一、取り出すオブジェクトが存在しない場合、NoSuchElementException例外が発生します。

個人的には上記処理をしつつ、next()してる箇所をtry catchで例外処理コードを実装しておいたほうがいいと思いますが。

[m.ts10806]

>adressというオブジェクトが存在しない

単にnull代入したときもなので一概に「オブジェクトが存在しない」と言えないようには思いますが、意図したところと違ったらすみません。

[rinjinto]

adressにnullを代入するという意味がわかっていますか？
それは「adressのオブジェクトを削除する」という意味ですよ。

"null"というのはJavaでは"オブジェクトがない"ということを意味します。

[m.ts10806]

「削除する」となるとまた御幣が出そうな気がします。
「削除する」と明示されている公式の資料があればまた別ですが。。

[rinjinto]

質問者の方だと思っていました。
初心者の方相手だと思っていたので、簡単な理解である「オブジェクトが存在しない」「削除する」という記載をしました。

ご理解いただいていると思いますが、オブジェクト参照にnullを代入するということは「参照するオブジェクトをなくす」、ポインタ的な考え方でいえば「オブジェクトの参照アドレスをnullアドレスに設定する(nullアドレスにオブジェクトは存在しない)」ということになります。
「参照するオブジェクトをなくす」ということと、元々代入されていたオブジェクトが非参照になってガベージコレクション対象となること(他に参照がないという条件)をまとめた意味として「オブジェクトを削除する」という記載をしました。
厳密にいえば意味が違うとは思いますが、初心者の方相手にはここまで厳密な理解は不要で概念的な記載で十分ではないでしょうか？

(注：プログラムである以上、どんな言語であってもアドレスとは無縁ではないのでjavaであってもポインタ的な思考は活用可能)