ayuda con problema en codigo

[xxcom]

buenas a todos, el problema que tengo es de un juego bingo, aqui pongo el que yo he realizado, no me da errores, pero tiene que tener algun/os fallo/os en la implementacion del codigo, pero a pesar de darle vueltas no lo encuentro.

algunas partes de codigo van comentadas, ya que son partes que he ido realizando y para no borrarlas las he dejado comentada por no perder las ideas.

son dos clases (Bingo y CartonBingo) y el Main que debe ser ese, no se le puede cambiar nada.

aqui va el codigo:

Bingo.java

Código [java]:

package juegobingo;
import java.io.*;
public class Bingo
{
    final int MAXBOLAS=500;
    final int ibol;
    final int vbolas[];
 
     public Bingo()
     {
         this.ibol=0;
         this.vbolas=new int[MAXBOLAS];
     }
     public int SacarBola()
     {
         int bola=(int)(Math.random()*MAXBOLAS);
 
         return bola;
     }
     private boolean YaUsado(int n)
     {
         boolean enc;
 
         enc=false;
         for(int i=0;i<vbolas.length;i++)
         {
             if(vbolas[i]==n)
             {
                 enc=false;
             }
             else
             {
                 enc=true;
             }
         }
        return enc;
     }
     public void MostrarBolas()
     {
         for(int i=0;i<vbolas.length;i++)
         {
             System.out.print("Bolas extraidas: "+vbolas[i]+", ");
         }
     }
     public void GenerarFicheroBolas()
     {
         InputStreamReader isr=new InputStreamReader(System.in);
         BufferedReader br=new BufferedReader(isr);
         //String dato;
 
         File f1;
         FileWriter fw;
 
         try
         {
             f1=new File("Fbolas.txt");
             fw=new FileWriter(f1);
 
             if(f1.exists())
             {
                 f1.createNewFile();
                 this.MostrarBolas();
                 //System.out.println("El fichero GenerarFicheroBolas.txt se ha creado correctamente");
             }
             else
             {
                 System.out.println("No se ha podido crear el fichero GenerarFicheroBolas.txt");
             }
             fw.close();
         }
         catch(Exception ex)
         {
             System.out.println("Errores en la lectura del fichero GenerarFicheroBolas.txt: "+ex);
         }
     }
}
 

CartonBingo.java

Código java:

package juegobingo;
 
import java.io.*;
public class CartonBingo
{
    final int filas=3;
    final int columnas=9;
    final int rango=500;
    private int boleto[][];
    private boolean estadoboleto[][];
    private boolean haylinea;
    private boolean haybingo;
    public int lineapremio;
 
     public CartonBingo()
     {
         this.boleto=new int[filas][columnas];
         this.estadoboleto=new boolean[filas][columnas];
         this.haylinea=false;
         this.haybingo=false;
         this.lineapremio=-1;
     }
     private boolean EsRepetido(int num)
     {
         boolean enc=false;
 
         for(int i=0;i<boleto[filas].length;i++)
         {
             for(int j=0;j<boleto[columnas].length;j++)
             {
                 if(num==boleto[filas][columnas])
                 {
                     enc=true;
                 }
                 else
                 {
                     enc=false;
                 }
             }
         }
         return enc;
     }
     public void RevisarNumero(int num)
     {
         boolean enc;
 
         enc=false;
         for(int i=0;i<boleto[filas].length;i++)
         {
             for(int j=0;j<boleto[columnas].length;j++)
             {
                 if(num==boleto[i][j]/*==estadoboleto[i][j]*/)
                 {
                     estadoboleto[i][j]=true;
                     enc=true;
                     this.ComprobarLineas();
                     this.ComprobarBingo();
                 }
             }
         }
     }
     public boolean ComprobarLineas()
     {
         boolean enc;
 
         enc=false;
         for(int i=0;i<estadoboleto.length;i++)
         {
             if(EsLinea(i))
             {
                 enc=true;
             }
             else
             {
                 enc=false;
             }
         }
         return enc;
     }
     private boolean EsLinea(int fil)
     {
         boolean enc;
 
         enc=false;
         for(int i=0;i<estadoboleto[fil].length;i++)
         {
             if(estadoboleto[fil][i]==enc)
             {
                 enc=false;
             }
             else
             {
                 enc=true;
             }
         }
         return enc;
     }
     public boolean ComprobarBingo()
     {
         boolean enc;
 
         enc=false;
         for(int i=0;i<estadoboleto[filas].length;i++)
         {
             for(int j=0;j<estadoboleto[columnas].length;j++)
             {
                 if(estadoboleto[i][j]==true)
                 {
                     enc=true;
                 }
                 else
                 {
                     enc=false;
                 }
             }
         }
         return enc;
     }
     public boolean HayLinea()
     {
         boolean enc;
 
         enc=false;
         /*for(int i=0;estadoboleto[filas].length;i++)
         {
             if(estadoboleto[i]==true)
             {
                 enc=true;
             }
             else
             {
                 enc=false;
             }
             for(int j=0;j<estadoboleto[columnas].length;j++)
             {
                 enc=true;
             }
         }*/
         return enc;
     }
     public boolean HayBingo()
     {
         boolean enc;
 
         enc=false;
         for(int i=0;i<estadoboleto.length;i++)
         {
             for(int j=0;j<estadoboleto.length;j++)
             {
                 if(estadoboleto[i][j]==true)
                 {
                     enc=true;
                 }
                 else
                 {
                     enc=false;
                 }
             }
         }
         return enc;
     }
     public void MostrarCarton()
     {
         for(int i=0;i<boleto[filas].length;i++)
         {
             System.out.println(boleto[i]);
             for(int j=0;j<boleto[columnas].length;j++)
             {
                 System.out.println(boleto[j]);
             }
         }
     }
     public void GenerarFicheroCarton(int ind)
     {
         InputStreamReader isr=new InputStreamReader(System.in);
         BufferedReader br=new BufferedReader(isr);
         //String dato;
 
         File f1;
         FileWriter fw;
 
         try
         {
             f1=new File("Fich"+ind+".txt");
             fw=new FileWriter(f1);
 
             if(!f1.exists())
             {
                 f1.createNewFile();
                 this.MostrarCarton();
             }
             else
             {
                 System.out.println("El fichero Fichind.txt no se creo correctamente");
             }
             fw.close();
         }
         /*File fichero=new File("Fich"+ind+".txt");
 
         InputStreamReader isr;
         BufferedReader br;
 
         try
         {
             isr=new InputStreamReader(System.in);
             br=new BufferedReader(isr);
 
             fw=FileWriter(File);
 
             if(fichero.createNewFile())
             {
                 this.MostrarCarton();
                 System.out.println("El fichero GenerarFicheroCarton.txt se ha creado correctamente");
             }
             else
             {
                 System.out.println("No se pudo crear el fichero GenerarFicheroCarton.txt");
             }
         }*/
         catch(Exception ex)
         {
             System.out.println("Errores en la lectura del fichero GenerarFicheroCarton.txt: "+ex);
         }
     }
}
 

el main va en un nuevo tema, ya que no lo puedo incluir en este mensaje.

[carlosmarin]

Y cual es el problema?

[xxcom]

esa es la cuestion, que no se cual es el problema, no me da errores, pero no me ejecuta el bingo de manera eficiente.

[Gguer]

Tu main

Código Main:

package juegobingo;
 
public class Main
{
    public static void main(String[] args)
    {
        final int NUMBOLETOS=5;
        Bingo bingo;
        CartonBingo boletos[];
        int j,x,i;
        int bol;
        boolean haylinea=false;
        boolean haybingo=false;
 
        boletos=new CartonBingo[NUMBOLETOS];
        bingo=new Bingo();
 
        try
        {
            for(j=0;j<boletos.length;j++)
            {
                boletos[j]=new CartonBingo();
                System.out.println("Generando Fichero Carton: B"+j);
                boletos[j].MostrarCarton();
                boletos[j].GenerarFicheroCarton(j);
            }
 
            x=0;
            while(!haybingo)
            {
                bol=bingo.SacarBola();
                for(j=0;j<boletos.length;j++)
                {
                    boletos[j].RevisarNumero(bol);
 
                    if(boletos[j].HayLinea() && !haylinea)
                    {
                        System.out.println("\nLINEA CANTADA en BOLETO: B"+j+", en la fila: "+boletos[j].lineapremio+", y en la bola: "+x);
                        boletos[j].MostrarCarton();
                        boletos[j].GenerarFicheroCarton(10);
                        haylinea=true;
                    }
                    if(boletos[j].HayBingo())
                    {
                        haybingo=true;
                        System.out.println("\nBINGO CANTADO en BOLETO: B"+j+", en la bola: "+x);
                        boletos[j].MostrarCarton();
                        //Imprimo todos los boletos.
                        for(i=0;i<boletos.length;i++)
                        {
                            boletos[i].GenerarFicheroCarton(i);
                        }
                        System.out.println("\nGenerando Fichero Bolas Jugadas...");
                        bingo.MostrarBolas();
                        bingo.GenerarFicheroBolas();
                    }
                }
                if(!haybingo)
                {
                    x++;
                }
            }
        }
        catch(Exception ex)
        {
            System.out.println("\nErrores de lectura: "+ex);
        }
    }
}
 

Y cómo es que se ejecuta entonces?

Cuál es el comportamiento inesperado que lo hace ineficiente?

Què es lo que esperas que haga y no hace?

Cuéntanos bien cómo está el asunto, para analizar la situación.

[xxcom]

el main que tiene que ejetucar este codigo esta en un nuevo tema, ya que en este no me permitia incluirlo, mirad eso aver si me os aclara algo

[Gguer]

Mi estimado compañero, he colocado el main que está en el otro post, en este mismo.

No podemos darnos cuenta de qué está mal porque no nos dices qué es lo que esperas que haga y qué es lo que hace. No conocemos tu código y no sabemos qué debe hacer.

Sé más explicito en tu problema. Comprende que queremos ayudarte pero no podemos porque no tenemos idea (buno, al menos yo) de qué es lo que sucede inadecuadamente.

[xxcom]

te voi a pasar lo que tienen que hacer las clases y lo que tiene que hacer el programa en general.

Contenido a Desarrollar:
Se desea crear un programa en Java que simule el comportamiento de un Bingo. El juego
necesitará 3 clases:
• Main.
• Bingo.
• CartonBingo.
Cada una con los siguientes métodos:
• Main: El código de esta clase se adjunta en el fichero Main.class.
• Bingo: Necesitará de las siguientes variables, constantes y métodos de clase:
Las Variables:
• final int MAXBOLAS: Indica el número máximo de bolas que tiene el bombo, que
para nuestro caso.
• int ibol: Contará el número de bolas que llevamos sacadas del bombo.
• int vbolas[]: Se utilizará para ir guardando cada bola que se vaya sacando
del bombo.
Los Métodos:
• public Bingo(): Sería el Constructor de la clase y lo único que haría sería
crear el vector vbolas de tamaño MAXBOLAS.
• public int SacarBola(): Sacaría aleatoriamente una bola del bombo de 500
números. Esto habría que hacerlo con el metodo random del paquete Math. La
expresión (int)(Math.random()*X); genera un numero entero aleatorio entre 1 y X.
Pero OJO, habrá que tener en cuenta que si la bola devuelta por éste método ya
ha salido antes, deberá generar otra nueva, hasta conseguir una bola que no
haya sadido anteriormente. Para conseguir ésto deberá hacer uso del método
siguiente.
• private boolean YaUsado(int n): Este método comprueba que la bola “n” no ha
salido ya en el vector de bolas (vbolas), devolviendo TRUE en ese caso y FALSE en
el contrario.
• public void MostrarBolas(): Este método presenta en pantalla todas las bolas
que han salido hasta el momento de su llamada.
• public void GenerarFicheroBolas(): Este método genera un fichero llamado
Fbolas.txt, donde se han registrado todas las bolas que han salido hasta el
momento de su llamada.
• CartonBingo: Necesitará de las siguientes variables, constantes y métodos de clase:
Las Variables:
• final int filas: Indicará el número de filas del boleto. Para nuestro caso
serán 4.
• final int columnas: Indicará el número de columnas del boleto. Para nuestro
caso serán 6.
• final int rango: Indicará el intervalo de números que formarán parte del
boleto. Para nuestro caso serán 500.
• private int boleto[][]: Será una matriz de filas x columnas de enteros, donde
se guardarán los distintos números del boleto.
• private boolean estadoboleto[][]: Será una matriz de filas x columnas de
booleanos, de forma que si estadoboleto[i][j]=true, indicará que boleto[i][j]
ha salido ya en una de las bolas jugadas.
• private boolean haylinea: Indica si este boleto “HA CANTADO LÍNEA”, es decir, hay una fila en la matriz estadoboleto[][] que tiene todos sus elementos a
true, tomando en ese caso el valor de true.
• private boolean haybingo: Indica si este boleto “HA CANTADO BINGO”, es
decir, la matriz estadoboleto[][] tiene todos sus elementos a true, tomando en ese
caso, el valor de true.
• public int lineapremio: En caso de que este boleto “CANTE LÍNEA”, indicará
el número de la línea que se ha completado. En otro caso almacena el valor -1.
Los Métodos:
• public CartonBingo(): Será el constructor y deberá hacer lo siguiente:
• Crear la matriz boleto[][] de tamaño filas x columnas, generando números
aleatorios distintos, entre 0 y el valor dado por la variable rango.
• Crear la matriz estadoboleto[][] de tamaño filas x columnas, colocando
todos sus elementos a false.
• Inicializa la variable haylinea a false.
• Inicializa la variable haybingo a false.
• Inicializa la variable lineapremio a -1.
• private boolean EsRepetido(int num): Devolverá true si el número num está
repetido en el boleto. En otro caso devolverá false.
• public void RevisarNumero(int num): Si num está en la matriz boleto[i][j],
colocará un true en la matriz estadoboleto[i][j]. Además comprobará si se puede
CANTAR LINEA o BINGO en el boleto.
• public boolean ComprobarLineas(): Comprobará si en el boleto se puede CANTAR
LINEA. Para ello revisará la matriz estadoboleto[][] y devolverá true y hay
alguna fila con todos sus elementos a true. En otro caso devolverá false.
• private boolean EsLinea(int fil): Devuelve true si en la fila fil se puede
CANTAR LINEA. Devolverá false en otro caso.
• public boolean ComprobarBingo(): Comprobará si en el boleto se puede CANTAR
BINGO. Para ello comprobará que todos los elementos de la matriz estadoboleto[][]
están a true.
• public boolean HayLinea(): Devuelve true, si la variable haylinea es true.
Devolverá false en otro caso.
• public boolean HayBingo(): Devuelve true, si la variable haybingo es true.
Devolverá false en otro caso.
• public void MostrarCarton(): Muestra en pantalla todos los números de este
boleto.
• public void GenerarFicheroCarton(int ind): Genera un fichero cuyo nombre
sería “Fich”+ind+".txt", donde estará registrados todos los números de este boleto.

[xxcom]

Iniciado por xxcom

te voi a pasar lo que tienen que hacer las clases y lo que tiene que hacer el programa en general.

Contenido a Desarrollar:
Se desea crear un programa en Java que simule el comportamiento de un Bingo. El juego
necesitará 3 clases:
• Main.
• Bingo.
• CartonBingo.

Cada una con los siguientes métodos:
• Main: El código de esta clase se adjunta en el fichero Main.class.
• Bingo: Necesitará de las siguientes variables, constantes y métodos de clase:

Las Variables:
• final int MAXBOLAS: Indica el número máximo de bolas que tiene el bombo, que para nuestro caso.
• int ibol: Contará el número de bolas que llevamos sacadas del bombo.
• int vbolas[]: Se utilizará para ir guardando cada bola que se vaya sacando
del bombo.

Los Métodos:
• public Bingo(): Sería el Constructor de la clase y lo único que haría sería
crear el vector vbolas de tamaño MAXBOLAS.
• public int SacarBola(): Sacaría aleatoriamente una bola del bombo de 500
números. Esto habría que hacerlo con el metodo random del paquete Math. La
expresión (int)(Math.random()*X); genera un numero entero aleatorio entre 1 y X.
Pero OJO, habrá que tener en cuenta que si la bola devuelta por éste método ya
ha salido antes, deberá generar otra nueva, hasta conseguir una bola que no
haya sadido anteriormente. Para conseguir ésto deberá hacer uso del método
siguiente.
• private boolean YaUsado(int n): Este método comprueba que la bola “n” no ha
salido ya en el vector de bolas (vbolas), devolviendo TRUE en ese caso y FALSE en
el contrario.
• public void MostrarBolas(): Este método presenta en pantalla todas las bolas
que han salido hasta el momento de su llamada.
• public void GenerarFicheroBolas(): Este método genera un fichero llamado
Fbolas.txt, donde se han registrado todas las bolas que han salido hasta el
momento de su llamada.


• CartonBingo: Necesitará de las siguientes variables, constantes y métodos de clase:

Las Variables:
• final int filas: Indicará el número de filas del boleto. Para nuestro caso
serán 4.
• final int columnas: Indicará el número de columnas del boleto. Para nuestro
caso serán 6.
• final int rango: Indicará el intervalo de números que formarán parte del
boleto. Para nuestro caso serán 500.
• private int boleto[][]: Será una matriz de filas x columnas de enteros, donde
se guardarán los distintos números del boleto.
• private boolean estadoboleto[][]: Será una matriz de filas x columnas de
booleanos, de forma que si estadoboleto[i][j]=true, indicará que boleto[i][j]
ha salido ya en una de las bolas jugadas.
• private boolean haylinea: Indica si este boleto “HA CANTADO LÍNEA”, es decir, hay una fila en la matriz estadoboleto[][] que tiene todos sus elementos a
true, tomando en ese caso el valor de true.
• private boolean haybingo: Indica si este boleto “HA CANTADO BINGO”, es
decir, la matriz estadoboleto[][] tiene todos sus elementos a true, tomando en ese
caso, el valor de true.
• public int lineapremio: En caso de que este boleto “CANTE LÍNEA”, indicará
el número de la línea que se ha completado. En otro caso almacena el valor -1.

Los Métodos:
• public CartonBingo(): Será el constructor y deberá hacer lo siguiente:
• Crear la matriz boleto[][] de tamaño filas x columnas, generando números
aleatorios distintos, entre 0 y el valor dado por la variable rango.
• Crear la matriz estadoboleto[][] de tamaño filas x columnas, colocando
todos sus elementos a false.
• Inicializa la variable haylinea a false.
• Inicializa la variable haybingo a false.
• Inicializa la variable lineapremio a -1.
• private boolean EsRepetido(int num): Devolverá true si el número num está
repetido en el boleto. En otro caso devolverá false.
• public void RevisarNumero(int num): Si num está en la matriz boleto[i][j],
colocará un true en la matriz estadoboleto[i][j]. Además comprobará si se puede
CANTAR LINEA o BINGO en el boleto.
• public boolean ComprobarLineas(): Comprobará si en el boleto se puede CANTAR
LINEA. Para ello revisará la matriz estadoboleto[][] y devolverá true y hay
alguna fila con todos sus elementos a true. En otro caso devolverá false.
• private boolean EsLinea(int fil): Devuelve true si en la fila fil se puede
CANTAR LINEA. Devolverá false en otro caso.
• public boolean ComprobarBingo(): Comprobará si en el boleto se puede CANTAR
BINGO. Para ello comprobará que todos los elementos de la matriz estadoboleto[][]
están a true.
• public boolean HayLinea(): Devuelve true, si la variable haylinea es true.
Devolverá false en otro caso.
• public boolean HayBingo(): Devuelve true, si la variable haybingo es true.
Devolverá false en otro caso.
• public void MostrarCarton(): Muestra en pantalla todos los números de este
boleto.
• public void GenerarFicheroCarton(int ind): Genera un fichero cuyo nombre
sería “Fich”+ind+".txt", donde estará registrados todos los números de este boleto.