1) Por favor, antes de publicar algo, léanse un toque las reglas. Si vas a colocar código Java, utiliza las etiquetas GeSHi.

Primero, tu código está mal. La forma de llenar una matríz es la siguiente:

for(int row=0; row<numbers.length; row++) {
    for(int col=0; col<numbers[row].length; col++) {
        numbers[row][col] = generateRandomNumber();
    }
}

El método generateRandomNumber() que usaremos para generar los números de las cartillas es un método recursivo:

private Integer generateRandomNumber() {
    Integer number = 1 + RANDOM_GEN.nextInt(90);
    if(isRepeated(number)) {
        number = generateRandomNumber();
    }
    return number;
}

Que llama al método isRepeated que evalúa si un número ya existe dentro de la matríz:

private Boolean isRepeated(final Integer number) {
    Boolean isRepeated = false;
    for(Integer[] row : numbers) {
        for(Integer col : row) {
            if(col == number)
                isRepeated = true;
         }
     }
     return isRepeated;
}

Luego, puedes obtener la cartilla:

public Integer[][] getCard() throws IllegalAccessException {
    if(numbers[0][0] == null) {
        throw new IllegalAccessException("Primero llene la cartilla");
    }
    return numbers;
}

Y, el método main sería así:

public static void main(String[] args) {
    Bingo bingo = new Bingo().generateNumbers();
    try {
        for(Integer[] row : bingo.getCard()) {
            for(Integer col : row) {
                System.out.println(col);
             }
         }
     } catch(IllegalAccessException e) {
         javax.swing.JOptionPane.showMessageDialog(null,e.getMessage());
     }
}

Nota que en el método getCard() lanzamos una IllegalAccessException si tratamos de obtener la cartilla y ésta está vacía. Ésta excepción la capturamos en el método main y mostramos el mensaje en un JOptionPane.

El código, quedaría así:

import java.util.Random;
 
public class Bingo {
    private Integer[][] numbers = new Integer[9][3];
    private final Random RANDOM_GEN = new Random(System.nanoTime());
 
    public Integer[][] getCard() throws IllegalAccessException {
        if(numbers[0][0] == null) {
            throw new IllegalAccessException("Primero llene la cartilla");
        }
        return numbers;
    }
    public Bingo generateNumbers() {
        for(int row=0; row<numbers.length; row++) {
            for(int col=0; col<numbers[row].length; col++) {
                numbers[row][col] = generateRandomNumber();
            }
        }
        return this;
    }
    private Integer generateRandomNumber() {
        Integer number = 1 + RANDOM_GEN.nextInt(90);
        if(isRepeated(number)) {
            number = generateRandomNumber();
        }
        return number;
    }
    private Boolean isRepeated(final Integer number) {
        Boolean isRepeated = false;
        for(Integer[] row : numbers) {
            for(Integer col : row) {
                if(col == number)
                    isRepeated = true;
            }
        }
        return isRepeated;
    }
    public static void main(String[] args) {
        Bingo bingo = new Bingo().generateNumbers();
        try {
            for(Integer[] row : bingo.getCard()) {
                for(Integer col : row) {
                    System.out.println(col);
                }
            }
        } catch(IllegalAccessException e) {
            javax.swing.JOptionPane.showMessageDialog(null,e.getMessage());
        }
    }
}

Saludos.

Hola DarK_FirefoX,

Sip, es exactamente eso. Es lo mismo que hacer:

for(int row=0; row<numbers.length; row++) {
    for(int col=0; col<numbers[row].length; col++) {
        if(numbers[row][col] == number)
            isRepeated = true;
    }
}

Incluso se puede otpimizar un poco, para que rompa el loop principal al detectar un número repetido:

private Boolean isRepeated(final Integer number) {
    Boolean isRepeated = false;
    mainloop:
    for(Integer[] row : numbers) {
        for(Integer col : row) {
            if(col == number) {
                isRepeated = true;
                break mainloop;
            }
        }
    }
    return isRepeated;
}

Un saludo.

No te preocupes, no pasa nada ^^. Un saludo y dale duro a la carrera.

Ya veo, algo como el algoritmo Bucket Sort. Sip, en arrays grandes puede llegar a ser más eficiente, aunque en la VM actual no lo creo. Por ejemplo, for y while tardan 234 milisegundos en recorrer un array de 15 millones, osea .234 segundos, Iterator 340. Puedes ver el benchmark aquí.

Como puedes observar, es poquísimo tiempo para tan exorbitante muestra.


Saludos.