入力した整数の数アスタリスクを表示

Question

初心者ですが宜しくお願いします。
学校での課題ですが、こちらのコード以外に簡単なコードまたは正しいコードがあれば案を頂きたいです。

入力された整数に対して下記のように入力された整数の数だけ@を表示したいです。
整数4
＠＊＊＊
＊＠＊＊
＊＊＠＊
＊＊＊＠

制作中ですが、コードは下記です。

import java.util.Scanner;

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

    Scanner scanner = new Scanner(System.in);
    System.out.println("整数を入力");

    int number = scanner.nextInt();
    for (int i = 0; i < number; i++) {
      for (int j = 0; j < number; j++) {
        if(i == j) {
            System.out.print("@");
        } else {
            System.out.print("＊");
        }
      }
      System.out.println();
    }
    scanner.close();
  }
}

Comments

[y_waiwai]

その提示のコードではどういう動作となるんでしょうか。

[arcxor]

質問文は Markdown を適切に使って記述してください。

コードは ``` で囲んでコードブロックにしてください。

[maisumakun]

> 正しいコード

何をもって「正しい」と呼ぶのでしょうか？

[mesii]

このサイトを始めて利用するので、うまく利用できず申し訳ありません。
動作としましては、整数の4を入力すると上記概要にあります。@と＊を交えた出力内容にしたいと言うことです。
休憩時間の合間に携帯での質問をさせて頂いており。ブラウザ上で結果を確認できるサイトが無かった為憶測のようなコードでもうしわけありません。

[jimbe]

既存コードの動作に問題は無く、他の方法などを知りたいということなら、QA では無く意見交換を利用するのが良いと思います。

プログラムとしては最小になっていると思います。

[arcxor]

動作上の問題よりも、インデントを適切に付けたり演算子前後の空白類を整えたりと、ソースコードの可読性に意識を向けたほうがよいのではないでしょうか。

それと、ソースコードは一部ではなく動作可能な完全な形で提示していただいたほうが好ましいです。

[mesii]

@jimbe
貴重なご意見ありがとう御座います。

[mesii]

携帯でもコンパイル可能なサイトを見つけました。確認したところ修正箇所が複数あり正しく修正しました。
他の方に意見を聞いたところif文の方が好まれることもあるそうなので3項演算子を変更しています。
貴重なお時間ありがとうございました。

[jimbe]

>if文の方が好まれることもあるそうなので

好みと言ってしまえば否定は出来ませんが、 if と三項演算子では表現が違い、読み手に与える認識が異なります。
if では当然 if(i==j) が先に来ますので「処理が分かれる」ことが先に認識されるのに対し、三項演算子では System.out.print が先ですので「表示する」ことが先に認識されます。

コードを読む人に対して「処理が分かれる」ことと「表示する」ことのどちらをより伝えたいかで記述を選ぶという判断基準もあります。

Answer 1

質問のコードは、"@" が半角、"＊" が全角になっていますが、
これは正しいコードですか？

質問のコードでは、System.out.print を number x number 回呼び出し、
System.out.println を number 回呼び出します。
if (i == j) の判定も number x number回実行します。

次のようにすれば、System.out.println を n 回呼び出すだけで済みます。

import java.util.*; // Scanner, Arrays

class Main {
	public static void main(String[] args) {
		Scanner sc = new Scanner(System.in);
		System.out.println("整数を入力");
		int n = sc.nextInt();
		char[] a = new char[n];
		Arrays.fill(a, '*');
		for (int i = 0; i < n; i++) {
			a[i] = '@';
			System.out.println(a);
			a[i] = '*';
		}
	}
}

Answer 2

• nextInt() が使えます。
• 外側のループの最後で改行を表示するとよいでしょう。

Comments

[mesii]

nextInt()について調べたところ、整数のみ入力が可能なものでこちらの方がシンプルでしたので、修正更新しました。
携帯でのコード入力の為お時間がかかり申し訳ありません。
実行結果を確認したいのですが、あいにく携帯での実行結果を確認できるサイトがなく憶測での記述になっています。