質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.38%

  • Java

    16504questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

適当な数字を入れた場合にエラー文に入らない

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 167

KAZUGOD

score 10

適当な数字を入れた場合にエラーではなく ”4”という謎の数字が出力されてしまう

import java.util.Scanner;
public class Kadai1203 {
 public static void main(String[] args) {
  Scanner sc = new Scanner(System.in);

  LoginManager logmana = new LoginManager();
  System.out.print("Select(LI:Login PU:PrintUser E:End)--> ");
  String select = sc.next();
  while("E".equals(select) != true) {
   if("LI".equals(select)) {
    System.out.print("UserId --> ");
    String userid = sc.next();
    System.out.print("Password --> ");
    String pass = sc.next();
    User user = new User(userid,pass);
    try {
     logmana.login(user);
     System.out.println("Login is Successful.");
    }
    catch(Exception e) {
     System.out.println(e.getMessage());
    }
   }
   else if("PU".equals(select)) {
    System.out.println("UserId : " + logmana.getLoginUser().getUserid());
    System.out.println("Password : " + logmana.getLoginUser().getPassword());
   }
   System.out.println();
   System.out.print("Select(LI:Login PU:PrintUser E:End)--> ");
   select = sc.next();
  }
 }
}
public class User {
 private String userid; //ユーザID
 private String password; //パスワード
 User(String userid,String password){
  this.userid = userid;
  this.password = password;
 }
 String getUserid() {
  return userid;
 }
 String getPassword() {
  return password;
 }
}
public class LoginManager {
 private User loginUser = null;//ログイン中のユーザ
 private User[] userList = { new User("user01","Denshi"), //登録済みユーザの配列
         new User("user05","Qazwsx"),
         new User("user12","PoiUytRewq"),
         new User("user21","ZxcVbnm")
 };
 User getLoginUser() {
  return loginUser;
 }
 void login(User user) throws Exception { //引数で受け取ったユーザのIDの組み合わせが登録済みならログイン成功とし、ログイン中のユーザをセットする。登録済みでないときは、ログイン中のユーザををnullにして、("UserId or Password Error")のエラーメッセージをセットしたException例外をスローする
  for(int i = 0;i <= userList.length;i++) {
   if(userList[i].getUserid().equals(user.getUserid())) {
    if(userList[i].getPassword().equals(user.getPassword())) {
     loginUser = user;
     break;
    }
   }
  }
  if(loginUser != user) {
   throw new Exception("UserId or Password Error");
  }
 }
}
Select(LI:Login PU:PrintUser E:End)--> LI
UserId --> sss
Password --> sss
4 //謎の数字
Select(LI:Login PU:PrintUser E:End)-->
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • y_waiwai

    2019/07/12 13:50

    提示のコードはなにをするものなんでしょう
    んで、エラー分とはどこのことでしょう
    あまりにも説明不足です

    キャンセル

  • mts10806

    2019/07/12 13:51

    コードブロックの冒頭はファイル名等ではなく言語名を入れてください。

    キャンセル

  • mts10806

    2019/07/12 13:58

    えっと
    [Java]
    だけで良いです。

    キャンセル

  • 退会済みユーザー

    2019/07/12 14:11

    複数のユーザーから「やってほしいことだけを記載した丸投げの質問」という意見がありました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 3

checkベストアンサー

+4

表層的な原因

// LoginManager
for(int i = 0;i <= userList.length;i++) {

userList[4]にアクセスしています。

潜在的な原因

try {
    logmana.login(user);
    System.out.println("Login is Successful.");
}
catch(Exception e) {
    System.out.println(e.getMessage());
}
if(loginUser != user) {
    throw new Exception("UserId or Password Error");
}

例外の型をExceptionにするのは広過ぎです。


なお、以前の質問でも同様の議論をしています。読み飛ばしたまま進めてはいませんか。

LouiS0616 2019/07/07 16:40
@katoy さん
Exceptionでキャッチする処理を例示するのはちょっとまずい気がします。
わざと改良の余地を残しているのでしたら、ごめんなさい。

katoy 2019/07/07 16:48
..Exceptionでキャッチする処理を例示するのはちょっとまずい気がします。 ...
はい、そのとうりです。
学校の課題の回答としても原点されるポイントです。
適切な Exception でcatch するように書き直したものを提出して欲しいとおもいますし、先生もそのことを説明して欲しいものです。

Java - 配列の添え字を変えずに再入力する方法がわかりません。|teratail

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/12 14:47

    例えば 例外の型はどのようなやり方がありますでしょうか?

    キャンセル

  • 2019/07/12 15:27

    独自に例外クラスを作ってやるのが良いかと思います。
    既存のものを使うのであれば、一番それっぽいのはIllegalArgumentExceptionあたりですかね。

    キャンセル

+2

Louis0616さんの回答で十分かと。

デバッグ実行の方法は、わかりますでしょうか?

少なくともログインのあとにLogin is Successful.が表示されないなら
例外が発生したことがわかる(想定)できます。

で、Exceptionのところ

System.out.println(e.getMessage());

が、実行されているだろうことがわかる(といいなぁ)
なので、ここでデバッガでExceptionの中身を確認すれば、意味不明の文字の正体がわかります。

そもそものcatchうんぬんもありますが、e.getMessageの中身は、今回みたように、ほぼ役に立たないメッセージです。
※自分で例外クラスを設計&生成するなら別ですが

なので、catchの中身を

e.printStackTrace();


にすると、原因の特定が早くなると思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

配列をスキャンするのに配列の index で for ループする代わりに 拡張 for をつかうとよいです。
for (User user : userList ) {

このことも含めて、全体をかきなおしてみました。
とくの password の処理を変更しています。
getPassword()で パスワードをアクセスできてしまうのはセキュリティ上の問題になる可能性があります。
パスワードに関する操作は、設定とチェックだけのとし, 取得はできないようにしました。

package teratail;

import java.util.Scanner;

public class Kadai1203 {
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);

        LoginManager logmanager = new LoginManager();
        while (true) {
            try {
                System.out.print("Select(LI:Login PU:PrintUser E:End)--> ");
                String select = sc.next();
                if ("E".equals(select)) {
                    break;
                } else if ("LI".equals(select)) {
                    System.out.print("UserId --> ");
                    String userid = sc.next();
                    System.out.print("Password --> ");
                    String pass = sc.next();
                    logmanager.login(userid, pass);
                    System.out.println("Login is Successful.");
                } else if ("PU".equals(select)) {
                    User user = logmanager.getLoginUser();
                    if (user != null) {
                        System.out.println("UserId : " + user.getUserid());
                    } else {
                        System.out.println("Not login");
                    }
                }
                System.out.println();
            } catch (Exception e) {
                System.out.println(e.getMessage());
            }
        }
    }
}

class User {
    private final String userid; // ユーザID
    private final String password; // パスワード

    User(String userid, String password) {
        this.userid = userid;
        this.password = password;
    }
    String getUserid() { return this.userid; }
    boolean check(String userid, String password) {
        return this.userid.equals(userid) && this.password.equals(password); 
    }
}

class LoginManager {
    // ログイン中のユーザ
    private User loginUser = null;
    // 登録済みユーザの配列
    private User[] userList = { 
            new User("user01", "Denshi"),
            new User("user05", "Qazwsx"), 
            new User("user12", "PoiUytRewq"), 
            new User("user21", "ZxcVbnm")
     };

    User getLoginUser() {
        return loginUser;
    }
    void logout() { this.loginUser = null; }
    void login(String userid, String password) throws IllegalArgumentException {
        // 引数で受け取ったユーザのID/passwordの組み合わせが
        // 登録済みなら, ログイン成功としログイン中のユーザをセットする。
        // 登録済みでないときは、ログイン中のユーザをnullにして
        // ("UserId or Password Error")のエラーメッセージをセットした例外をスローする。
        logout();
        for (User user : userList ) {
            if (user.check(userid, password)) {
                this.loginUser = user;
                return;
            }
        }
        throw new IllegalArgumentException("UserId or Password Error");
    }
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.38%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • Java

    16504questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。