【Java】ソースコードの書き方について質問です。

Question

下記のコンソール結果になるようなソースを作っています。
■
■■
■■■
■■■■
■■■■■
public static void main(String[] args) {
String s = ("■");
for(int i = 0; i<5; i++){
for(int j = 0; j<5; j++){
if(条件){
System.out.print(s +" ");
}
System.out.println();
}
}
}

ifの条件（else ifなど）を設定することで上記結果になると考えましたが、見当違いでしょうか？

Answer 1

問題無いと思います

求めたい結果の行と四角の個数の関係は

１行目で１個
２行目で２個
・・・
５行目で５個

なので、 i と j を使った条件を書くことになりますね

ただ、無駄なループもしてしまうので j の for文を少し工夫すれば、if(条件)を書かなくてよくなります

Answer 2

３つの方法を作ってみました。
sub1() 質問文のコードをベースにしたもの
sub2() sub1() ではメソッド中で stdout に出力しています。
junit などでテストしやすくするため、sub2() では 三角形の文字列全体を生成しています。
組み立てた結果をSystem.out.print() で一気に出力します。
sub3() sub2() での文字列の組み立て方を変更しています。for ループのネストを除去しています。
汎用性を高めるため、三角形の大きさをパラメータで渡すようにもしています。

public class Triangle {
    // 改行コード
    static private String line_sep = System.getProperty("line.separator");

    public static void main(String[] args) {
        sub1();

        String triangle = sub2();
        System.out.print(triangle);

        triangle = sub3(5);
        System.out.print(triangle);
    }

    static void sub1() {
        String str = "■";
        for (int i = 1; i <= 5; i++) {
            for (int j = 0; j < i; j++) {
                System.out.print(str);
            }
            System.out.println();
        }
    }

    // 直接 stdout に出力せず、String を組み立てる。
    static String sub2() {
        String str = "■";
        String buf = "";
        for (int i = 1; i <= 5; i++) {
            for (int j = 0; j < i; j++) {
                buf += str;
            }
            buf += line_sep;
        }
        return buf;
    }

    // String の組み立てかたを工夫する。
    // See
    // http://stackoverflow.com/questions/1235179/simple-way-to-repeat-a-string-in-java
    static String sub3(int height) {
        String str = "■";
        StringBuilder buf = new StringBuilder();
        for (int i = 1; i <= height; i++) {
            String line = new String(new char[i]).replace("\0", str);
            buf.append(line).append(line_sep);
        }
        return buf.toString();
    }
}

実行例：

■
■■
■■■
■■■■
■■■■■
■
■■
■■■
■■■■
■■■■■
■
■■
■■■
■■■■
■■■■■

Answer 3

ソースを推察すると、■を先頭で定義したいから
for文を2回重ねたという感じでしょうか？

そうでしたら、以下のように書くとスマートです。

String square = "■"; // 定数として使う
String display = square; // 表示用の変数として使う
for(int i=0; i<5; i++){
 System.out.println(display);
 display += square;
}

で、表示すると以下のようになります。

■
■■
■■■
■■■■
■■■■■

また、提示されたソースを元に書くとすれば、
以下のような感じでしょうか。

String s = ("■"); 
for(int i = 0; i<5; i++){ 
 for(int j = 0; j<5; j++){ 
  if(i>=j){
   System.out.print(s); 
  } 
 }
 System.out.println(); 
}

if文の中身ですが、

i=0 のとき j=0 (1回) -> "■"
i=1 のとき j=0,1 (2回) -> "■■"
i=2 のとき j=0,1,2 (3回) -> "■■■"
i=3 のとき j=0,1,2,3 (4回) -> "■■■■"
i=4 のとき j=0,1,2,3,4 (5回) -> "■■■■■"

というロジックです。

Comments

[swordone]

この後半のコードだと，■があってもなくても改行が入り，望む形にならないのでは？

[SKYYFISH]

よくよく見たらその通りですね。改行を入れる位置を修正しました。

Answer 4

ある条件によって、５個の棒グラフを描画するのですから、以下ではないでしょうか。

public static void main(String[] args) {
String s = ("■");
String outs; // 出力用文字列
for(int i = 0; i<5; i++){
outs = ""; // 出力文字列の初期化
for(int j = 0; j<5; j++){
if(条件) {
outs = outs + s; //棒を作成する
}
}
System.out.println(outs); //棒を出力する
}
}

上記の条件に「i >= j」を入れると、例にある出力になります。

Answer 5

forを使って書くならこのようになるでしょう．

String str = "■";
for(int i = 1; i <= 5; i++){
    for(int j = 0; j < i; j++){
        System.out.print(str);
    }
    System.out.println();
}

Comments

[nishimy]

ありがとうございました！！
解決です。

Answer 6

それで問題ありません。

ただ、例えば5行を10行に変更しようと思った時に
i,j両方の終了条件を書き換える必要があります。
面倒ですね。

jの繰り返し範囲は5である必要はありませんよね？

そもそものやりたいことは、
「i行目に、■をi+1個(スペースで区切って)書く」
ことかと思います。

for(int i=0; i<5; i++){
    writeSquare(i+1);
}

こう書けたら嬉しいですけどwriteSquareなんてそんな都合のいいもの存在しません。
ないなら作ってみるのも面白いと思います。

Answer 7

変数i,jとbreakを使えばうまくいくと思います。
がもう少しうまくやる方法がありそうです。