Skip to main content
Agregados los cambios en mostrarquiniela, gracias @Lithorell. Agregadas correcciones a Principal
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Por otro lado, habría que arreglar también mostrarquiniela de la siguiente manera:

public void mostrarquiniela (){
    /* Por cada apuesta de la lista llamamos a su método "mostrarapuesta" */
    for (int i=0; i<listaquiniela.size(); i++){
            System.out.println("Quiniela Nº" + i);
            System.out.println(listaquiniela.get(i).mostrarapuesta());
        }
}

Y, por último, te queda adaptar Principal para que el texto que pregunta el número de apuestas salga antes de preguntarte el número y que tras generar la quiniela se muestre su contenido:

import java.util.*;

public class Principal{
    public static void main (String [] args){
        int numero = 0;
        String con = "S";
        Scanner escaner = new Scanner(System.in);
        do{
            do {
                System.out.println("Elija el número de apuestas");
                System.out.println("El mínimo son 2 y el máximo 8");
                numero = escaner.nextInt();
            } while (numero < 2 || numero > 8);
            Quiniela apuesta = new Quiniela();
            /* Generamos "numero" apuestas */
            apuesta.generarquiniela(numero);
            /* Las mostramos */
            apuesta.mostrarquiniela();
            System.out.println("¿Desea realizar más apuestas?");
            System.out.println("S/N");
            con = escaner.nextLine();
        }while (con.equals("S"));
    }
}

Por otro lado, habría que arreglar también mostrarquiniela de la siguiente manera:

public void mostrarquiniela (){
    /* Por cada apuesta de la lista llamamos a su método "mostrarapuesta" */
    for (int i=0; i<listaquiniela.size(); i++){
            System.out.println("Quiniela Nº" + i);
            System.out.println(listaquiniela.get(i).mostrarapuesta());
        }
}

Y, por último, te queda adaptar Principal para que el texto que pregunta el número de apuestas salga antes de preguntarte el número y que tras generar la quiniela se muestre su contenido:

import java.util.*;

public class Principal{
    public static void main (String [] args){
        int numero = 0;
        String con = "S";
        Scanner escaner = new Scanner(System.in);
        do{
            do {
                System.out.println("Elija el número de apuestas");
                System.out.println("El mínimo son 2 y el máximo 8");
                numero = escaner.nextInt();
            } while (numero < 2 || numero > 8);
            Quiniela apuesta = new Quiniela();
            /* Generamos "numero" apuestas */
            apuesta.generarquiniela(numero);
            /* Las mostramos */
            apuesta.mostrarquiniela();
            System.out.println("¿Desea realizar más apuestas?");
            System.out.println("S/N");
            con = escaner.nextLine();
        }while (con.equals("S"));
    }
}
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Tal y como te puse en los comentarios el mensaje de error viene provocado porque no encuentra ninguna implementación del método generarapuesta dentro de tu clase Apuesta que se ajuste a los valores de entrada (void) y de salida (Apuesta).

La forma en la que implementas la creación de Apuestas y las guardas es incorrecta. Deberías hacerlo de la siguiente manera:

public ArrayList<Apuesta> generarquiniela (int numero) {
    for (int i=0; i<numero; i++) {
        /* Por cada quiniela rellena creo una instancia de "Apuesta" */
        Apuesta objeto = new Apuesta();
        /* Genero la apuesta */
        objeto.generarapuesta();
        /* Guardo la apuesta en nuestra lista de apuestas */
        listaquiniela.add(objeto);
    }
    /* Esto es innecesario, pero te lo mantengo por ahora */
    return listaquiniela;
}

Al finalizar la ejecución de generarquiniela se habrán guardado n apuestas en la lista listaquiniela.

Pero tal y como te pongo el código, necesitarás una pequeña modificación en la clase Apuesta.

En lugar de:

public ArrayList<String> generarapuesta(){

Usa:

public void generarapuesta(){

No necesitamos devolver la apuesta porque ésta se guarda internamente en la instancia, en la propiedad listaapuesta.