java 全ての文字列を表示させる

Question

javaで 5文字の文字列を5回入力し、5回目の入力終了後に全ての文字列を表示させるコードが書けません。

キーボードからの文字列入力とfor文とifを使うい書きましたが、どうしても上手くいきません。
教えて頂きたいです。どうか宜しくお願い致します。

public class Keisan {
public static void main(String[] args) {

for(int i = 1; i <= 10; i++) {
String n = new java.util.Scanner(System.in).nextLine();
String name += n;
if(i == 10) {
System.out.println(name + "です。");
}
}
}

}

Comments

[kazuma-s]

> キーボードからの文字キーボードからの文字列入力とfor文とifを使うい書きましたが、どうしても上手くいきません。列入力とfor文とifを使うい書きましたが、どうしても上手くいきません。

その上手く行かないコードを質問に追記してください。
コードの添付方法は teratail のヘルプをよく読まないと失敗しますよ。

[kazuma-s]

コードの追記、ありがとうございます。
しかし、コードの投稿に失敗しています。ヘルプは読んでいないのですか？

問題には 5 という値が何度も出てくるのに、コードにはそれがなく、
10 という値が何度も出てくるのを変だとは思いませんか？

[Crimson_Tide]

どういう処理の流れを想定し、各行でどういう処理をするか説明できますか？
恐らくできないでしょう。Javaの基礎知識が圧倒的に不足しています。
体系的に学習し直すべきです。
プログラミング完全初心者がネットでかじった知識程度でまともに動くわけがありません。

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

[yusuke_3_17_yu]

ご指摘頂きありがとうございます。
学習方法を見直してみます。
またご不明な点がございましたら、ご質問させていただきますので宜しくお願い致します。

Answer 1

teratail の「ヘルプ」の「質問するときのヒント」を読むと、
「3-4. 発生している問題・エラーメッセージを書きましょう」と書いてあります。

エラーメッセージから、ソースのどこが悪いのかが分かります。
String name += n;
ここです。
変数 name を宣言するんだったら、String name:
または、初期化を含めて、String name = ""; です。
name に n を連結するなら、name += n; です。
String name = ""; を for文の前に置いて、
for文の中は name += n; だけにしてみてください。

キーボードから

aaaaa
bbbbb
ccccc
ddddd
eeeee

と入力すると、aaaaabbbbbcccccdddddeeeee と表示されます。

ところが、入力をファイルに切り替えると、そのような表示もされません。
それは、Scanner の使い方を間違っているからです。

追記
一つの入力である System.in に対して、ループの中で
Scanner を何度も new するのはよくありません。
Scanner を new すると、生成された Scannerオブジェクトは
それぞれが 8KB 程度のバッファを保持します。
標準入力の System.in がキーボードなら、入力は行単位なので、
各Scannerオブジェクトのバッファに 1行ずつ入るだけで、
思い通りの処理がされますが、
javac Keisan <data.txt
のように標準入力をファイルに切り替えると、最初の Scannerオブジェクトが
自分の持っているバッファ一杯にファイルの内容を読み込んでしまいます。
nextLine で 1行取得しますが、残りの 4行はバッファ内に残ります。
2番目の Scannerオブジェクトが続きを読もうと思っても、
もうファイルの終わりに達しているので読み込めません。

ということで、次のように Scanner の new は 1回だけにしましょう。

import java.util.Scanner;

public class Keisan {
	public static void main(String[] args) {
		Scanner sc = new Scanner(System.in);
		String name = "";
		for(int i = 0; i < 5; i++)
			name += sc.nextLine();
		System.out.println(name + "です。");
	}
}

もちろん、入力が複数のファイルだとか、複数の文字列で、それに対して
Scannerオブジェクトを複数生成するのは構いません。

Comments

[yusuke_3_17_yu]

ありがとうございます。
コンパイル、実行できました。
しかしScannerの使い方が違うとかどういうことなのでしょうか？

[yusuke_3_17_yu]

ありがとうございます。
理解できました。