Java Scanner 入力 重複してしまう なぜ

Question

前提・実現したいこと

Javaを用いて簡単な登録システムを作成しています。ソースコードは「Registration.java」「RentalMain.java」「User.java」の三種類で構成されています。「Registration.java」の中のRegistrationクラスのprintSignin()メソッドの中にあるScannerインスタンスの出力が重複してしまいます。
以下の写真のように動作して欲しいです。

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

Enter 1 or 2 (1. New Registration 2. Login): １
You entered option 1

**********************
 Enter Details
**********************
User Name: Password: 1 // <= ここが重なって表示されてしまっている。 
isVIP(true/false)1
Registration Successful

Registration.java

/*
 * This file is the registration class for the Vehicle Rental System.
 * 
 * 
 */

package model;

import java.io.BufferedReader;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.Map;
import java.util.Scanner;
import utility.FileLoader;

/**
 * Represents User Sign-in Object.
 * 
 * Methods of this class throw NullPointerException if required parameters are null.
 * 
 * @author Ayumu 
 * @version 1.0
 */

public class Registration {

    /**
     * User Storage File.
     */
    public static final String USERFILE_NAME = "./resources/registeredusers.txt";

    /**
     * The registered user list for signin.
     */
    private final Map<String, User> myUserList;

    /**
     * Constructs a sigin/registration system.
     * 
     * 
     */
    public Registration() {
        myUserList = FileLoader.readItemsFromFile(USERFILE_NAME);
    }

    /**
     * getter for myUserList.
     * 
     * @return myUserList
     */
    public Map<String, User> getMyUserList() {
        return myUserList;
    }

    /**
     * display sign-in or registration options.
     */
    public void printSignin() {

        final Scanner input = new Scanner(System.in);
        System.out.print("Enter 1 or 2 (1. New Registration 2. Login): ");
        final int intVal1 = input.nextInt();
        if (intVal1 == 1) { // Registration
            System.out.println("You entered option 1");
            System.out.println();
            System.out.println("**********************");
            System.out.println(" Enter Details");
            System.out.println("**********************");

            
            System.out.print("User Name: ");
            String uName = input.nextLine();
            System.out.print("Password: ");
            String uPass = input.nextLine();
            
            if (login(uName, uPass)) { //check if the user is already exist or not.
                while (login(uName, uPass)) {
                    System.out.print("User already exist, enter different user name: ");
                    uName = input.nextLine();
                    System.out.print("Password: ");
                    uPass = input.nextLine();
                }
            }
            
            System.out.print("isVIP(true/false)");
            final String uVIP = input.nextLine();
            
            // create a new user from given information
            final User newUser = new User(uName, uPass, Boolean.valueOf(uVIP));
            // register the User
            if (register(newUser)) {
                System.out.println("Registration Successful");
            } else {
                System.out.println("Registration Failed");

            }

        } else if (intVal1 == 2) { // Login
            System.out.println("You entered option 2");
            System.out.println();
            System.out.println("**********************");
            System.out.println(" Enter Details");
            System.out.println("**********************");
            
            // Login
            System.out.print("User Name: ");
            String uName = input.nextLine();
            System.out.print("Password: ");
            String uPass = input.nextLine();
            
            if (login(uName, uPass)) {
                System.out.println("Login Successful");
            } else {
                System.out.println("Wrong Credentials");
                while (login(uName, uPass)) { //loop until a user type the correct credentials.
                    System.out.print("User Name: ");
                    uName = input.nextLine();
                    System.out.print("Password: ");
                    uPass = input.nextLine();
                }
            }
        } else {
            System.out.println("Please select 1 or 2!");
            printSignin();
        }
        input.close();  
    }

    /**
     * Verify Sign-in procedure.
     * 
     * @param theUsername username for sign-in
     * @param thePassword password for signin
     * @return sign-in success
     */
    public boolean login(final String theUsername, final String thePassword) {

        final Path file = Paths.get(USERFILE_NAME);
        try (BufferedReader br = Files.newBufferedReader(file)) {
            String line;
            while ((line = br.readLine()) != null) {
                final String[] tokens = line.split(FileLoader.SPLIT_TOKEN);
                final String uName = tokens[0];
                final String uPwd = tokens[1];
                final boolean isVIP = Boolean.parseBoolean(tokens[2]); //just in case I use it maybe
                // return when the name and pass are matched.
                return uName.equals(theUsername) && uPwd.equals(thePassword);
            }
        } catch (final IOException e) {
            System.out.println(e);
        } 
        // return false when the name and pass are not matched for all lines.
        return false;
    }

    /**
     * Adds a user to the registered user list.
     * 
     * @param theUser an order to add to this shopping cart
     * @return true/false returns if registration is successful
     */
    public boolean register(final User theUser) {
        try {
            FileLoader.writeUserToFile(USERFILE_NAME, theUser);
            return true;

        } catch (final NullPointerException e) {
            e.printStackTrace();
            return false;
        }
    }

    /**
     * Empties the user list.
     */
    public void clear() {
        myUserList.clear();
    }

    @Override
    /**
     * String representation of the object
     * 
     */
    public String toString() {
        final StringBuilder sb = new StringBuilder();
        sb.append("Registered UserList {");
        for (String key : myUserList.keySet()) {
            sb.append(key + "=" + myUserList.get(key) + ", ");
        }
        sb.append("}");
        final String msg = sb.toString();
        return msg;
    }

}

javaを触るのは久しぶりで、コードに不備等や改善点が多々見受けられることかと思いますが、そちらの方も是非教えていただけると幸いです。

Comments

[jimbe]

> コードに不備等や改善点が多々見受けられることかと思いますが、そちらの方も是非～

とのことですので、チラ見ですがびっくりした所です。

> } catch (final NullPointerException e) {

try 内で使われている FileLoader も( import のパッケージ名から)自作されたものと思います。そこから本来ならバグ以外出ないはずの NullPointerException が出るのを無視させるというのは強引ではないでしょうか。

> } else {
> System.out.println("Please select 1 or 2!");
> printSignin();
> }

printSignin が再帰してしまいます。

[RoundCat]

ご回答ありがとうございます！　実はこれ大学の課題なんですが、適切な例外を暗示的にそれぞれのメソッドで投げなさいと指示が出ておりまして、まだ例外処理に関しての知識が深くないもので「むむ、これでいけるか？」と深く考えずに記述してしまいました（ダメですよね。。。）
例外処理の知識はちゃんと深めないといけないなと僕自身も思っております。
詳しく教えて頂きどうもありがとうございました！！

[jimbe]

> 適切な例外を暗示的にそれぞれのメソッドで投げなさい
"暗示的" とはどういうことでしょう…。

writeUserToFile メソッドに throws が付いていないのであれば、無理矢理 例外が発生するかのように記述するのは”適切”ではありません。
例外が無いのであればまず書かないのが適切と思います。

[RoundCat]

”The parameters should be checked for null values and should throw NullPointerException for null
parameters implicitly. ” と指示が出されておりまして、writeUserToFileにthrowsがついておらず、僕もよくわからなかったのですがとりあえずつけてみた次第です。週明けに教授に聞いてみます。
教えて頂きありがとうございました！

[jimbe]

それは、メソッドのパラメータが(オブジェクトを必ず指定しなければならないのに) null だったら NullPointerException を発生させること…という意味でしょうか。
login メソッドや register メソッドのパラメータは必ず必要でしょうから、もし null だったら NullPointerException を発生させるパラメータチェックをしなければならないのではないでしょうか。

Answer 1

nextLine() → next()

nextLineだとスキャナを現在行の先に進めて、スキップした入力を返すため、と思われます。

Comments

[RoundCat]

初歩的なミスでした。ご回答どうもありがとうございます！

[m.ts10806]

半角数字以外だとnextIntのところでException投げるはずなので、そこらへんも配慮した実装にしてくださいね。