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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Java

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

Q&A

解決済

2回答

318閲覧

エンターを一度押しただけで、Round1とRound2の二つの結果が表示されます。

退会済みユーザー

退会済みユーザー

総合スコア0

Java

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

0グッド

0クリップ

投稿2019/06/19 04:29

編集2019/06/19 04:40

前提・実現したいこと

https://mrradiology.hatenablog.jp/entry/20160709/1468022084

上記のサイトのSaikoro.javaを参考にコードを書いたのですが、
エンターを一度押しただけで、Round1とRound2の二つの結果が表示されます。

初心者のためもともとそういうコードなのか、わたしが何が間違えているのか分かりません。

希望としてはエンター、Round1、エンター、Round2...のように一つずつ表示させたいです。

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

エンターを一度押しただけで、Round1とRound2の二つの結果が表示されます。
一度のエンターで二つずつRoundが表示されてしまいます。

該当のソースコード

以下が私の書いたものになります。(Java)

public class Saikoro{ public static int score = 100; public static int bet = 5; public static int round = 1; public static String[] diceStrings = { "| * |", "| : |", "|...|", "|: :|", "|:.:|", "|:::|"}; public static void throwDices() { int me1 = (int)(Math.random()*6.0); int me2 = (int)(Math.random()*6.0); showDices(me1, me2); checkPoints(me1, me2); } public static void showDices(int a, int b){ System.out.println(""); System.out.println(diceStrings[a] + " " + diceStrings[b]); System.out.println(""); } public static void checkPoints(int a, int b){ int c = a + b + 2; if(c % 2 ==0){ if(a == b){ score += bet * 2; System.out.println("You win" + bet*2 + "$"); } else{ score += bet; System.out.println("You win" + bet + "$"); } } else{ if(c == 7){ score -= bet * 2; System.out.println("You lose" + bet*2 + "$"); } else{ score -= bet; System.out.println("You lose" + bet + "$"); } } System.out.println("You have" + score + "$"); } public static void keyInput(){ try{ int c = System.in.read(); } catch(Exception e){} } public static void main(String[] args){ while(score > 0){ System.out.println(""); System.out.println("Hit Enter"); keyInput(); System.out.println(""); System.out.println("Round" + round); throwDices(); round++; } System.out.println("You have no money, gameover"); } }

試したこと

コードを見比べてみましたが、サイトのものと同じに思えます。

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

ここにより詳細な情報を記載してください。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

y_waiwai

2019/06/19 04:32

このままではコードが見づらいので、質門を編集し、<code>ボタンで、出てくる’’’の枠の中にコードを貼り付けてください
退会済みユーザー

退会済みユーザー

2019/06/19 06:41

インデントおかしい
退会済みユーザー

退会済みユーザー

2019/06/19 06:44

サイトのコードをそのまま書き写して実行しただけなので私によく分かりませんが、コンパイルと実行は問題なくできました。もしかしたら、コピペした時に変にペーストされてしまったのかもしれません。
guest

回答2

0

ベストアンサー

仕様だと思います。
System.in.readは
「入力ストリームからデータの次のバイトを読み込みます」らしいので
改行の(CR)(LF)の2つの入力を受けた判定で
2度実行されているのだと思います。

参照

投稿2019/06/19 04:47

yayakona

総合スコア69

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2019/06/19 06:35

コメントありがとうございます。 これは変えることはできないのでしょうか? Round1 |:::| |...| You lose5$ You have95$ Hit Enter この形にするにはどうすればよろしいでしょうか?
yayakona

2019/06/19 13:10 編集

・一行読み込んだ時実行 ・(LF)を読み込んだ時実行 のどっちかで実現できるはずです。 javaの入出力は詳しくないため何とも言えませんが、 下のような感じに修正すれば動くと思います。 実行確認していないので動かなかったらごめんなさい。 ```java try{ int c = System.in.read(); } ``` ↓ ```java try{ while((char)System.in.read() != '\n'); } ``` while((char)System.in.read() == '\n'); から修正しました。
退会済みユーザー

退会済みユーザー

2019/06/19 14:32

できました!本当にありがとうございます。
guest

0

Java

1public static void showDices(int a, int b){ 2 System.out.println(""); 3 System.out.println(diceStrings[a] + " " + diceStrings[b]); 4 System.out.println(""); 5}

このメソッドが得られた値をコンソールに出力している処理です。
メソッドの引数が a, b の2つあり、それぞれをprintlnに渡しているので、もともとそういうコードです。

1エンターで1つの数値だけを表示したいのなら、

System.out.println(diceStrings[a]);// int b は無視する

とするだけで要件を満たせそうです。

エンターを一度押しただけで、Round1とRound2の二つの結果が表示されます。

正確にいうなら、各Roundで2つのダイスを表示しています。

追記
申し訳ありません。
質問を取り違えていたみたいですね。
こちらの回答は誤りです。

投稿2019/06/19 04:43

編集2019/06/19 04:52
dsuzuki

総合スコア1682

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2019/06/19 04:52

コメントありがとうございます。 しかし、私の欲しい結果は得られませんでした。 Round1 |:::| |...| You lose5$ You have95$ Hit Enter Round2 |:::| |: :| You win5$ You have100$ Hit Enter 一度のエンターでこうなってしまうのを、 Round1 |:::| |...| You lose5$ You have95$ Hit Enter こう1ラウンド毎にエンターを入力する形にしたいのです。 質問が悪かったです。すみません。 もしよろしければ加えて回答をお願いします。
dsuzuki

2019/06/19 04:56

確認作業をしていただいたのに申し訳ありません。 yayakonaさんの回答を見て、間違いに気付きました。 yayakonaさんの回答を参照してください。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問