Java forループでのジグザグ状に表示させる

Question

Java forループでのジグザグ状に表示

Javaを学び始めて3か月が経ち、
『ユーザーに名前と数字を打たせ、その数字に応じてジグザグ状に名前を繰り返し出力し、30回目が出力されたら、折り返す』
という課題が出されたのですが、折り返すためのプログラムがわかりません。

名前: 太郎
回数: x

太郎
　太郎
　　太郎
---------------------------------------------
　　　　　　　太郎
　　　　　　　　太郎　　//ここが三十回目という前提で
　　　　　　　太郎　　　//ここから下を表示させる方法がわからない
---------------------------------------------
　　太郎
　太郎
太郎
　太郎
　　太郎……

イメージとしてはこんな感じです。下のコードを見ていただければわかると思いますが、右方向に30回表示させ、
戻る方向に28回表示させるつもりです。

public class loop {

	public static void main(String args[]) {
		EasyReader console = new EasyReader();

		System.out.print("名前: ");
		String name = console.readLine();
		System.out.print("回数: ");
		int number = console.readInt();

		for (int i = 1; i <= number; i++) {
			for (int j = 1; j <= i - 1; j++) {
				System.out.print(" ");
			}
			System.out.println(name);

			for (int j = 28; j >= i - 1; j--) {
				System.out.print(" ");
			}
			System.out.println("name");

		}
	}
}

コードを書いて、何回も試行錯誤してやってみたのですが、全然成功できず、間に何を入れればわからずじまいです。ヒントでも構わないので、わかる方がいらっしゃいましたら、助言・アドバイスの方をよろしくお願いします。

Comments

[m.ts10806]

コードはJavaScriptではなくJavaですが？区別はきちんとつけたほうが良いです。

[MS7789]

ご指摘ありがとうございます。JavaとJavaScriptの違いは理解していませんが、私がプログラミングしているのはJavaなのは承知しております。今後はこういったミスがないように気を付けたいと思います。

[m.ts10806]

あと、コードにインデントが全くなく読みづらいですし、 スコープが視認困難なのでバグの温床になりやすいです。ご調整ください

[m.ts10806]

きちんと理解しておかないと今後トラブルになりえますので、このさい調べて理解しておくようにしてください

[MS7789]

まだ不慣れな部分はたくさんありますが、慣れるよう頑張ります！

[m.ts10806]

全角スペースはインデントで使ってはいけませんよ。コンパイルエラーになるのでまともに再現確認できません。ご自身で適切に付けるのが難しければIDEのフォーマット機能とか、Web上の整形サービスを利用してください。

[m.ts10806]

インデントについては「質問の体裁をよくするため」ではなく「プログラミングにおける基本技術」として捉えてください。なので、インデントをつけたものをきちんと動作確認してから提示してください。「エラーが発生する」という質問ならまだしも、そうでないなら動かないコードを提示してはいけません。

[MS7789]

すいません…Tabキーでスペースを開けれなかったので全角スペースを使ってしまいました。質問の際にも、全角スペース使ってはいけないんですね…

[m.ts10806]

違います。インデントをつけて正常動作を確認したものを投稿してください。全角スペースがあるとコンパイルエラーになるのはわかりますよね？コンパイルエラーになるコードを提示されても意味がありません。私のひとつ前のコメント通りです。

[MS7789]

私の言い方が悪かったですね…すでに正常に動作したのを確認して提示しましたが、『質問の際に全く同じプログラムを(インデントがしっかりついている&確認済みの状態)を提示する』という意味で言いました。

[m.ts10806]

そういうことでしたか。了解です。「質問の際にも、全角スペースを使ってはいけない」だと「質問編集画面でそのままコードにインデントをつけようとした」という印象を受けますので。

Answer 1

現在のソースは、上りの山と下りの山を**同時（交互に１行づつ）**に書いているので、きれいな模様ができています。

よくループの練習で、三角を作る演習を見かけますが。
※表現上、横幅を高さ、縦幅を長さとしています。

■
■■
■■■

今回の場合、山の最大の高さが３０になります。
たぶん、書けますよね？

次に逆順の山を作るケース
上記の山を書いたあとに以下のような逆順になる山を書きます。
高さは２９が最大です。

■■■
■■
■

これも書けますよね？

最後に今までの全部を入力されたnumber回書きます。
で、黑い四角は全角スペースに、ただの改行は名前＋改行とすれば、望みの通りの結果になるかと。

冗長ですが、これが一番簡単。
あとは、上りか下りかの判定しながら変数設定するなら２重ループでも書けます。（これをnumber回ループ）

int max = 30; // 頂点
int lineCnt = max * 2 + 1; // 上りと下りの長さなので２倍 と、頂点の部分でプラス１
for (int i = 0;i < lineCnt;  i++) {
	// ここを工夫して変数設定してあげれば、２重ループで書けます。
	int num = (i > max) ? i : i;
	for(int j = 0;j < num; j++) {
		System.out.print("■");
	}
	System.out.println("名前");
}

追記
解決したということなので、不必要だと思いますが。一応あとから見る人のために
元のソースのロジックは

for ループ number
 　for ループ 上り山の高さ
 　for ループ 下り山の高さ

と、いう構造でしたが。
山を書くロジックはループが２つ必要なので・・・

for ループ 上り山の長さ
   for ループ 上り山の高さ

あとは、下り山と連続して書くか、１つのループで上りと下り書くかは、ループ演習では気にしなくてよいかと。

Comments

[MS7789]

回答ありがとうございます。
無事、望み通りに動作させることができました！私には先に頂点を決めるという発想はありませんでした。

Answer 2

右方向に30回表示させた**『後に』**戻る方向に28回表示するんですよね？
間に何を入れれば、とありますが、その『間』の位置、合っていますか？

Comments

[MS7789]

回答ありがとうございます。
間にif文を書くのかと思っておりましたので、今回は間ということにしました。もちろん、間ではなくても助言をいただけたら嬉しい限りです。

Answer 3

ロジックだけなのでとりあえずjavascriptで

<style>
#view{white-space:pre}
</style>
<script>
window.addEventListener('DOMContentLoaded', function(e){
  document.querySelector('#btn').addEventListener('click',function(){
    var count=parseInt(document.querySelector('#count').value||"0");
    var name=document.querySelector('#name').value;
    var fold=parseInt(document.querySelector('#fold').value||"1");
    var v="";
    var i=1;
    var j=0;
    while(i<=count){
      if(parseInt(i/fold)%2==1){
        j=(fold*2)-i%(fold*2);
      }else{
        j=i%(fold*2);
      }
      v+=i+":"+" ".repeat(j)+name+"\n";
      i++;
    }
    var name=document.querySelector('#view').textContent=v;
  });
});
</script>
名前:<input type="text" id="name" value="太郎"><br>
回数:<input type="text" id="count" value="200"><br>
折返:<input type="text" id="fold" value="30"><br>
<input type="button" id="btn" value="go">
<div id="view"></div>