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

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

ただいまの
回答率

87.79%

暗証番号入力のプログラム

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 5,269

score 5

初めてこのサイトを利用するのとjava初心者なので、もし至らない点があればご指摘ください。

暗証番号をキーボードから入力し、正しければ「暗証番号が確認できました」間違っていれば「暗証番号が確認できません。 もう一度暗証番号を入力してください」というメッセージを表示させ、再入力(正しい暗証番号が入力されるまでループ)させるプログラムを書いているのですが、暗証番号を入力しても正しくても間違っていても数字入力が終わった時点でコンソールが終了されます。
理想
パターン1

暗証番号を入力してください。
//↓ 暗証番号入力受付
1111 //正しい暗証番号
//↓
暗証番号が確認できました。

パターン2
暗証を入力してください。
//↓ 暗証番号入力受付
2222 //間違った暗証番号
//↓
暗証番号が確認できません。もう一度暗証番号を入力してください
//↓  暗証番号入力受付
暗証番号を入力してください。
// 正しい暗証番号が入力されるまでループ

現実
パターン1
暗証番号を入力してください。
//↓ 暗証番号入力受付
1111 //正しい暗証番号
//終了

パターン2
暗証番号を入力してください。
//↓ 暗証番号入力受付
2222  //間違った暗証番号
//終了

素人が無い知恵振り絞って考えてもダメで独学で勉強していて周りに教えてくれる人もいないのでどうか知恵を貸してください。
1個目のソースが現実のパターンになり、2個目のソースは「構文エラーがあります。"while ( Expression ) ;" を挿入して DoStatement を完了してください」が全く解決しなくて実行できてないです。

1個目
import java.util.Scanner;

public class Yooo {
    public static void main(String[] args){
        Scanner man=new Scanner(System.in);
        boolean a;
        int A; //正しい暗証番号 1111
        String an;
        
        an="暗証番号を入力してください";
        a=true;
        
        System.out.println(an);
        A=man.nextInt();
        while(!a){
        System.out.println(an);
        A=man.nextInt();
        if(A==1111){
            System.out.println("暗証番号が確認できました。");
            a=true;
        }
        if(a){
            System.out.println("暗証番号が確認できません。 もう一度"+an);
            A=man.nextInt();
            }
        }
    }
}

2個目
import java.util.Scanner;

public class Hoooo {
    public static void main(String[] args){
        Scanner man=new Scanner(System.in);
        boolean a;
        int A; //正しい暗証番号 1111
        String an;
        
        an="暗証番号を入力してください";
        a=true;
        
        do{
        System.out.println(an);
        A=man.nextInt();
        if(A==1111){
            System.out.println("暗証番号が確認できました。");
            a=true;
        }
        if(a){
            System.out.println("暗証番号が確認できません。 もう一度"+an);
            A=man.nextInt();
            }while(!a);
          }
        }
    }


  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

+4

まず「どのタイミングで何をするのか」ということを考えながら,コードに文章を交えて書いてみましょう.
基本は「正しいパスワードが入力されるまで繰り返し入力を求める」ことなので
do{
    パスワードを入力
}while(入力されたパスワードが間違っている);
これが大枠になります.これをもとに,具体的にどうするのかを考えながら文章・コード化してみてください.
あとどうもブロックがずれていることに気づいていないようなので,ブロックを作るときは"{"に対応する"}"を先に書いてしまう癖を付けましょう.

あるいはdo-whileを使わなくとも通常のwhileで次のような書き方で(Eclipse上では)うまく動作しました.
import java.util.Scanner;

public class Test {

    public static void main(String[] args) {
        Scanner s = new Scanner(System.in);
        System.out.println("入力");
        while (s.nextInt() != 1) {
            System.out.println("エラー!もう一度");
        }
        System.out.println("成功");
        s.close();
    }
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/10/18 00:40

    すみません。
    まだ素人なものでお見苦しいソースで申し訳ないです。
    綺麗に書けるよう精進します。

    キャンセル

+2

以下のような感じでしょうか?
import java.util.Scanner;

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

        System.out.println("暗証番号を入力してください。");
        int A = man.nextInt();

        while (A != 1111) { // 正しい暗証番号 1111
            System.out.println("暗証番号が確認できません。 もう一度暗証番号を入力してください。");
            A = man.nextInt();
        }

        System.out.println("暗証番号が確認できました。");
    }
}

まずは落ち着いて、ozwk様の回答の通り実現したいことをフローチャートに書き起こしてみることをお勧めします。

もし、まだお使いでなければ、プログラミング用のエディタも導入しましょう。
変数名・メソッド名を補完してくれたり、構文エラーを指摘してくれたりするので、格段に効率が上がります。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/10/18 00:39

    プログラム用のエディタというのがあったのですね。
    知りませんでした。
    是非とも導入してみます。

    キャンセル

+1

・2個目
do{
~~~
}while(~~~);
が正しい形ですね。

それを踏まえてコードを見なおして下さい。
whileの位置がおかしくありませんか?

2個めのコードをいろいろ省略して書くと、
do{
    if(){
    }
    if(){
    }while();
}
こうなりますよね


・共通
if(正しかったら){
    a=true;
}
if(a==true){
    もう一度入力させる
}

これだと正しかった場合a=trueになり、
その後if(a==true)を判定するわけですから、
結局もう一度入力を求められますよね?

まずはいきなりコードを書かずに、
紙にフローチャートなどを書くなどして、きちんとロジックを設計することから始めるといいと思います。


do{に対応する}がズレていることに気が付かないということは、
ひょっとしてメモ帳相当のエディタでコードを書いていませんか?
たしかにプログラムはメモ帳でも書けますが、
普通はプログラムを書くことに特化したエディタを使います。
初心者こそちゃんとしたエディタを使いましょう。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/10/17 21:08

    何回見返してもdo whileを調べても分からないです泣
    どうか答えを教えてください。

    キャンセル

  • 2015/10/17 21:52 編集

    while文の前の A=man.nextInt();を消す
    if(A==1111){
    System.out.println("暗証番号が確認できました。");
    a=true;
    } else{
    System.out.println("暗証番号が確認できません。 もう一度"+an);
    }

    とりあえずこれで動くと思いますが可読性を高めるなら
    条件式の中に否定は使わないほうがいいと思います。

    例えば
    初期値はa=falseをいれて
    while(a==false){
    System.out.println(an);
    A=man.nextInt();
    if(A==1111){
    System.out.println("暗証番号が確認できました。");
    a=true;
    } else{
    System.out.println("暗証番号が確認できません。 もう一度"+an);
    }
    }

    こうすれば一致するまでループを抜けないということがはっきりします

    キャンセル

check解決した方法

-2

ご回答くださった皆様方ありがとうございました。
なんとか自己解決することができました!

import java.util.Scanner; 

public class Oha { 
    public static void main(String[] args){ 
        Scanner man=new Scanner(System.in); 
        boolean a,b,c; 
        int A,B,C; //正しい暗証番号 1111 
        String an,kaku,nokaku;
        char j;
         
        j='\n';
        an="暗証番号を入力してください。"; 
        kaku="暗証番号が確認できました。";
        nokaku="暗証番号が確認できません。もう一度";
        a=true; 
        b=true;
        c=true;
         
        System.out.println(an);
        while(a){
            
        A=man.nextInt();
        if(A==1111){ 
            System.out.println(kaku+j+"プログラムを終了します。"+j); 
            a=false; 
        } 
        if(a){ 
            System.out.println(nokaku+an); 
          } 
        } 
        
        System.out.println("プログラム起動"+j+"2つ目の"+an);
        while(b){
            
            B=man.nextInt();
            if(B==2222){
                System.out.println(kaku+j+"2つ目の暗証番号が確認できたのでプログラムを終了します。"+j);
                b=false;
            }
            if(b){
                System.out.println(nokaku+an);
            }
        }
        
        System.out.println("プログラム起動"+j+"3つ目の"+an);
        while(c){
            
            C=man.nextInt();
            if(C==3333){
                System.out.println(kaku+j+"3つ目の暗証番号が確認できたのでプログラムを終了します。");
                c=false;
            }
            if(c){
                System.out.println(nokaku+an);
            }
        }

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2015/10/18 00:42

    なんか最初の仕様からいきなりかけ離れてませんか?
    それから実用上これで問題ないとしても,この書き方は後から読み返したりする際に難解になるので,もう少し書き方考えたほうがいいと思いますよ.
    「正しい暗証番号が入力された状態」がfalseと言うのは直感に反するので.

    キャンセル

  • 2015/10/18 02:54

    汚いコードで、どうもすみません(平謝り)
    まだまだ素人なもので少しでも早く覚えたいので無駄に変数とか入れてます。

    どうせ幼稚園児レベルのゴミクズなのでfalseでしか書けなくてごべん”な”ざい”

    正しい暗証番号が入力された状態」がfalseと言うのは直感に反するので←素人なのでこれが精一杯です。すみません

    今後は綺麗なコードを書けるように心がけていきます。
    どうも、すみませんでしたー(土下座)

    キャンセル

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

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

関連した質問

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