1

Estoy intentado crear un pequeño programa que genera apuestas aleatorias a la Quiniela. Me da error en
listaquiniela.add(objeto.generarapuesta());

Clase Quiniela

import java.util.*;

public class Quiniela {
    ArrayList < Apuesta > listaquiniela;

    public Quiniela() {
        listaquiniela = new ArrayList<Apuesta>();
    }

    public ArrayList<Apuesta> generarquiniela (int numero) {
        Apuesta objeto = new Apuesta();

        for (int i=0; i<numero; i++) {
            listaquiniela.add(objeto.generarapuesta());
        }
        return listaquiniela;
    }

    public void mostrarquiniela (){
        Apuesta objeto = new objeto();

        for (int i=0; i<listaquiniela.size(); i++){
                System.out.println(listaquiniela.get(i));
            }
    }
}

Clase Principal

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{
            numero = escaner.nextInt();
            while (numero<2 || numero>8){
                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();
            }
            Quiniela apuesta = new Quiniela();
            apuesta.generarquiniela(numero);
            System.out.println("¿Desea realizar más apuestas?");
            System.out.println("S/N");
            con = escaner.nextLine();
        }while (con.equals("S"));
    }
}

Clase Apuesta

import java.util.*;

public class Apuesta {
    ArrayList<String>listaapuesta;

    public Apuesta(){
        listaapuesta=new ArrayList<String>();
        for (int i=0; i<15; i++){
            listaapuesta.add(" ");
        }
    }

    public ArrayList<String> generarapuesta(){
        int v=0;
        Random objeto = new Random();

        for (int i=0; i<listaapuesta.size(); i++){
            v=objeto.nextInt(3);

            if(v==0){
                listaapuesta.set(i, "x");
            }else{
                listaapuesta.set(i, v+"");
            }
        } return listaapuesta;
    }

    public String mostrarapuesta(){
        String resultado = "";
        for (int i=0; i<listaapuesta.size(); i++){
            resultado = resultado +" " +listaapuesta.get(i);

        }return resultado;
    }
}
8
  • 1
    ¿Has implementado generarapuesta()? El mensaje te está diciendo que de las muchas implementaciones hechas en la clase Apuesta no hay ninguna candidata que encaje con los valores de entrada y salida esperados (entrada void y salida Apuesta). Commented el 22 may. 2017 a las 13:24
  • ¿qué devuelve generarapuesta? Commented el 22 may. 2017 a las 13:26
  • Fíjate que generarapuesta devuelve ArrayList<String> y tu listaquiniela.add(objeto.generarapuesta()) espera recibir un Apuesta. Commented el 22 may. 2017 a las 13:30
  • @Ele, ¿cómo intentas usar Apuesta y Quiniela? ¿Quieres que arreglemos Apuesta para que sólo devuelva un resultado o una secuencia de ellos? ¿O quieres que arreglemos generarquiniela para que obtenga el listado y lo agregue a la quiniela? Commented el 22 may. 2017 a las 13:35
  • Gracias por responder Oscar. Quiero arreglar Apuesta para que devuelva una secuencia de resultados. ¿Puedes mostrarme cómo implementarlo?
    – Ele
    Commented el 22 may. 2017 a las 13:36

4 respuestas 4

1

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.

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"));
    }
}
4
  • 1
    Falta modificar el mostrarquiniela () de Quiniela para que funcione correctamente con este codigo.
    – Lithorell
    Commented el 22 may. 2017 a las 14:03
  • 1
    Cierto. Gracias por recordármelo, ahora estaba arreglando el Principal para que no se salga inmediatamente tras poner el número de apuestas. Commented el 22 may. 2017 a las 14:04
  • Muchísimas gracias, Oscar. Se han solucionado los errores :)
    – Ele
    Commented el 22 may. 2017 a las 14:12
  • De nada @Ele, un placer. Recuerda marcar la respuesta como correcta en el símbolo de marca que hay en la parte superior izquierda de mi respuesta para marcar como solucionada tu pregunta. Gracias. Commented el 22 may. 2017 a las 14:35
1

Tu método generarapuesta() retorna un ArrayList de String y se lo estas pasando a un ArrayList de Apuesta

5
  • Por favor, si no tienes una respuesta específica para la pregunta deberías consultar lo que te falte mediante comentarios. Evita responder con respuestas genéricas o no aplicables. Pásate por: es.stackoverflow.com/help/how-to-answer Gracias Commented el 22 may. 2017 a las 13:29
  • me parece extraño que las dos respuestas que SI solucionan el problema, esten con un voto negativo (no entiendo porque) y que justamente tu @OscarGarcia estes diciendo que hacer. Que es lo malo de ambas respuestas???
    – sioesi
    Commented el 22 may. 2017 a las 13:52
  • Precisamente yo no soy el que pone negativos. Yo explico el motivo por el que las respuestas no son correctas, no las voto, doy la oportunidad de mejorarlas, corregirlas o retirarlas. Tan sencillo es demostrarlo como que me digas dónde quieres que ponga un negativo y te lo pongo. Si yo lo hubiera puesto previamente no podría agregar otro negativo más. Commented el 22 may. 2017 a las 13:59
  • Es un problema de entendimiento, para el la Solucion es hacerle todo el ejercicio, para otros es indicarle donde esta su error y que pueda arreglarlo. Ambas son validas.
    – Lithorell
    Commented el 22 may. 2017 a las 14:04
  • Al menos yo soy un usuario nuevo y todavía no tengo el privilegio de comentar en otros lugares asi que por eso mi respuesta anterior carecía de los requerimientos de ser una buena respuesta, pero despues de que argumentaran mas la pregunta pude dar una respuesta Commented el 22 may. 2017 a las 16:35
1

Es un problema de tipos.

generarApuesta() devuelve un ArrayList de String y lo estas intentando añadir a un ArrayList de Apuesta que es de tipo distinto.

Simplemente define listaQuiniela como ArrayList

ArrayList < String> listaquiniela;

y usa addAll en vez de all.

listaquiniela.addAll(objeto.generarapuesta());
6
  • Por favor, si no tienes una respuesta específica para la pregunta deberías consultar lo que te falte mediante comentarios. Evita responder con respuestas genéricas o no aplicables. Pásate por: es.stackoverflow.com/help/how-to-answer Gracias Commented el 22 may. 2017 a las 13:33
  • Es la respuesta especifica a su error. No estoy comentando nada, estoy diciendole que es lo que falla en su programa que es lo que justo el OP pregunta
    – Lithorell
    Commented el 22 may. 2017 a las 13:34
  • Eso ya lo puse en los comentarios. El usuario busca una solución a ese problema, tu respuesta no le ayuda en esa tarea. Por favor, pásate por el enlace que te proporcioné: Lee la pregunta cuidadosamente. ¿Que es lo que se esta preguntando específicamente? Asegurarte que tu respuesta incluya la solución a esto. Commented el 22 may. 2017 a las 13:36
  • No, el usuario pregunta porque falla y eso es lo que se le ha respondido. Corregirselo entero seria hacerle el ejercicio y ese no es el proposito de esta pagina.
    – Lithorell
    Commented el 22 may. 2017 a las 13:38
  • @OscarGarcia entiendo tu postura pero estoy de acuerdo con Lithorell, ambos hemos señalado exactamente cual es el fallo, para corregirlo deberíamos volver a hacerle toda la clase Apuesta desde 0 y, por tanto, todo el ejercicio. Esto es lo más cercano a una solución que podemos ofrecerle. Commented el 22 may. 2017 a las 13:43
0

Tu error está en que intentas igualar una ArrayList a una ArrayList y NO es lo mismo.

listaquiniela.add(objeto.generarapuesta());//Esto está mal

listaquiniela solo se puede igualar a una ArrayList. Así como tampoco puedes igualar ningún tipo de objeto a otro que no sea el mismo tipo. Ej: int i = "hola mundo!";

0

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.