静的メソッド生成したら表示されなくなりました

-----実現したいこと---------
実行例にならい、1から10の2つ乱数を発生させ、以下に示し他静的メソッドを用いてそれぞれの数をたて、よこの■の数として、四角形を表示させるプログラムを作りなさい。
実行結果の例
たて：3 ← 発生した乱数を表示
よこ：5 ← 発生した乱数を表示

■■■■■
■■■■■
■■■■■ ← 縦3個、横5個の■マークを表示
ただし、静的メソッドの概要は以下のようにすること。

|関数名|square|

|処理内容|縦、横に指定された数の■マークを表示する。ただし、どちらかの値が0以下の場合は、何も表示しない。|

|引数|整数の値×2（縦、横の値）

|戻り値|なし

以下コードになります。

Java
public class Prob1_3 {

/**
 * @param args
 */
public static void main(String[] args) {
	int a = (int) (Math.random() * 10) + 1; // 1～10の乱数を発生
	int b = (int) (Math.random() * 10) + 1; // 1～10の乱数を発生
	System.out.println("たて" + a);
	System.out.println("よこ" + b);
	System.out.println("");
}

static void square(int a, int b) {
	for (int i = 1; i < a; i++) {
		System.out.println("");
		System.out.print("■");
		for (int j = 1; j < b; j++) {
			System.out.print("■");
		}
	}
	System.out.println("");

}

}

何も表示されず、どこを修正したらよろしいでしょうか？
ご教示していただいたいです。

行動規範の内容に同意します

回答3件

ベストアンサー

他の方々が理由についておっしゃっているので私はそれ以外で気づいたことを述べます。
.メソッドsquareの中身ですが、for文でループする際カウンター変数i,jのスタート地点は1ではなく0からにしてください。このコードだと縦1行足りません

また、乱数の部分はint型の場合rand.nextInt()を使ってもいいでしょう。

java
1import java.util.*;
2
3public class Main {
4    public static void main(String[] args){
5        
6        Random rand = new Random();
7
8        int a = rand.nextInt(10) + 1;
9        int b = rand.nextInt(10) + 1;
10
11        System.out.println("たて" + a);
12        System.out.println("よこ" + b);
13
14        square(a,b);
15    }
16    public static void square(int a, int b)
17    {
18        for(int i = 0; i < a; i++){
19            for(int j = 0; j < b; j++){
20                System.out.print("■");
21                if(j < b - 1){
22                    System.out.print(" ");
23                }
24            }
25            System.out.println();
26        }
27    }
28
29}
30