0

No se hacer este método:

public boolean añadirCarta(Carta carta) Añade la carta recibida como argumento a la parte superior de la pila que representa el mazo de cartas. Parameters: carta - La carta que se añade en la cima del mazo. Returns: true si se pudo añadir o false si no se pudo añadir.

Me da siempre el error de: no se puede añadir una carta cuando aun es posible hacerlo.

Mi código es el siguiente:

public class MazoDeCartas {

    private Carta mazoCartas[];

    public MazoDeCartas(int numeroMaximoDeCartas){

        mazoCartas=new Carta[numeroMaximoDeCartas];

    }

    public boolean anadirCarta(Carta carta){
        for(int i = 0; i < mazoCartas.length; i++){
            if(mazoCartas[i] == null){
                mazoCartas[i] = carta;
                return true;
            }
        }
        return false;

    }
    public Carta extraerCarta(){
         Carta cartaExt=null;
         boolean encontrar=false;
         for(int i=mazoCartas.length-1;i>=0;i--){
             if(mazoCartas[i]!=null && (!encontrar)){
                 cartaExt=mazoCartas[i];
                 mazoCartas[i]=null;
                 encontrar=true;
             }
         }
        return cartaExt;

    }
    public void barajar(){

    }
}

El problema esta en el método anadirCarta.

15
  • 1
    no se puede añadir una carta no es un error, cual es el error que recibis? y en linea lo tira? y que esperas que haga exactamente?
    – gbianchi
    el 1 nov. 2017 a las 17:34
  • 1
    Nunca había visto un constructor así: mazoCartas=new Carta[numeroMaximoDeCartas]; usando [] en vez de (). ¿Ese código compila?
    – A. Cedano
    el 1 nov. 2017 a las 17:35
  • En el test que tiene que pasar para considerarlo correcto, me dice que: Se añade una carta cuando no debe ser posible hacerlo.
    – iHack
    el 1 nov. 2017 a las 17:35
  • Si, si deja compilar.
    – iHack
    el 1 nov. 2017 a las 17:36
  • 2

1 respuesta 1

0
package pruebas;

public class MazoDeCartas {

    private static Carta[] mazoCartas;
    private int numeroMaximoDeCartas;
    private static int cartasEnMazo;

    public MazoDeCartas( int nMaximoDeCartas ) {
        System.out.println( "Creando mazo de " + nMaximoDeCartas + " cartas..." );

        MazoDeCartas.mazoCartas = new Carta[ nMaximoDeCartas ];
        this.numeroMaximoDeCartas = nMaximoDeCartas;


        //Reinicia el mazo cada vez que se crea una nueva instancia
        MazoDeCartas.cartasEnMazo = 0;
    }

    public boolean anadirCarta( Carta carta ) {
        System.out.println( "Cartas en el mazo: " + cartasEnMazo );

        if( cartasEnMazo < this.numeroMaximoDeCartas ) {
            System.out.println( "Añadiendo carta: " + carta.getName( ));

            MazoDeCartas.mazoCartas[ cartasEnMazo ] = carta;
            MazoDeCartas.cartasEnMazo++;

            return true;
        }


        return false;
    }

} //class

class Carta {

    private String name = null;

    public Carta( String name ) {

        this.name = name;
    }

    public String getName() {

        return this.name;
    }
} //class

He creado la clase Carta sólo con la propiedad "name", lo mínimo necesario para hacerlo funcionar. He añadido trazas para que puedas ver por donde va el programa. A continuación la clase de test:

package pruebas;

/**
 *
 * @author orici
 */
public class TestMazoDeCartas {

    public static void main( String[] args ){
        System.out.println( "TestMazoDeCartas" );

        MazoDeCartas mdc = new MazoDeCartas( 3 );

        if ( mdc.anadirCarta( new Carta( "a" )))
            System.out.println( "Añadida primera carta" );

        if ( mdc.anadirCarta( new Carta( "b" )))
            System.out.println( "Añadida segunda carta" );

        if ( mdc.anadirCarta( new Carta( "c" )))
            System.out.println( "Añadida tercera carta" );

        if ( mdc.anadirCarta( new Carta( "d" )))
            System.out.println( "Añadida cuarta carta" );
    }

} //class

Está es la salida a la ejecución del test:

run:
TestMazoDeCartas
Creando mazo de 3 cartas...
Cartas en el mazo: 0
Añadiendo carta: a
Añadida primera carta
Cartas en el mazo: 1
Añadiendo carta: b
Añadida segunda carta
Cartas en el mazo: 2
Añadiendo carta: c
Añadida tercera carta
Cartas en el mazo: 3
BUILD SUCCESSFUL (total time: 1 second)

Necesitabas algunas propiedades estáticas y un condicional el lugar de un bucle, también creo que deberías comprobar que la carta que vas a añadir no este ya en el mazo.

Saludos.

4
  • 1
    me encataria saber como adivinaste que queria OP, pq habla de agregar cartas a una pila (que no tiene) y no se entiende pq le da error.
    – gbianchi
    el 1 nov. 2017 a las 23:25
  • Creo que esta claro que la pila de cartas es el array declarado en: private static Carta[] mazoCartas;
    – Orici
    el 2 nov. 2017 a las 8:29
  • Lo que pasa es q lo que escribio habla de la pila que representa el mazo de cartas. No del mazo de cartas a secas, pero puede ser un problema de semantica solamente.
    – gbianchi
    el 2 nov. 2017 a las 12:23
  • No se decirte, cogí lo que entendí de lo que puso y rellene los huecos con lo que yo haría en una situación similar. A ver si el usuario que pregunto se pronuncia...
    – Orici
    el 2 nov. 2017 a las 22:12

Tu Respuesta

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

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