Mensajes

Que tal iamendez

Echale un ojo a esto: https://gist.github.com/rockdrigo/930454050af12642c5a8be6a0ae1fe38


Saludos.

[juligarc10]

Que tal juligarc10

En efecto, hay que tomar en cuenta los comentarios de ivancea96.

Adicional a ello, veo unos temas:

• La variable jugadores deberia estar antes de la declaracion de tu arreglo de jugadores.
• Esta variable, al ser estatica y al no darle valor, toma un valor por defecto de 0
• Tu arreglo de jugadores tiene una capacidad de cero elementos (Por el punto anterior)

Mi recomendacion es que utilices mejor una lista (ArrayList) para almacenar a tus jugadores, el ArrayList es un arreglo dinamico, lo cual indica que no importa los elementos que vayas a guardar, esta lista "crece bajo demanda" asi no tienes que preocuparte por darle el tamaño especifico.

Sin embargo, puedes hacer lo que comenta ivancea96, hacer el new una vez que tengas el numero de jugadores, aunque con mi ejemplo ya no seria necesario.

Tu codigo podria quedar algo mas o menos asi:

Código (java) [Seleccionar] Expandir


public static List<Jugador> njugadores = new ArrayList<>();
   
    public static void inicioPartida() {
        int jugadores = ES.pideNumero("¿Cuántas personas jugaran?");

        for (int i = 0; i < jugadores; i++) {
            String nombre = ES.pideCadena("Introduzca el nombre del jugador");
            njugadores.add(new Jugador(nombre, 0, 0));
        }
    }
   
    //En caso de que quieras iterar la lista de Jugadores para obtener sus valores.
    public static void iterarSobreJugadores(List<Jugador> jugadores) {
        for (Jugador jugador: jugadores) {
            System.out.println(jugador.getNombre());
            System.out.println(jugador.getOtroAtributo());
        }
    }



Saludos.

[ivancea96]

Como bien dice ivancea96 lo puedes hacer de esa manera.

O bien, siguiendo la forma en la cual lo estas haciendo, igual puede quedar asi:

Código (java) [Seleccionar] Expandir


System.out.print("\nArreglo 2: ");
        for (int i = 0; i < 4; i++) {  //En lugar de ponerlo a 5, lo pones a 4
            System.out.print(array[i]+", ");
        }
        System.out.print(array[5]); //Y aqui imprimes el ultimo


Sin embargo podrias darle un enfoque diferente, piensa en esto, si el arreglo cambia de longitud, tendras que volver a escribir tu codigo para "adaptarlo" a la nueva longitud.

Quizas podriamos hacer esto, un metodo exclusivo que reciba  2 numeros y retorne un arreglo con estos 2 numeros intercambiados.

Despues lo mandas a llamar dentro de un for el cual recorrera el arreglo original hasta n-1, algo como esto:

Código (java) [Seleccionar] Expandir


public static void main(String[] args) {
        int[] array = {32, 54, 78, 96, 12};
       
        for (int i = 0;i<array.length-1;i++) {
            intercambia(array[i], array[i+1]);
        }
        System.out.println(Arrays.toString(array));
    }

    public static int[] intercambia(int a, int b) {
        return new int[]{b,a};
    }
}


Puede ser que exista una forma mas optimizada, en la programacion todo puede pasar.


Saludos.

Amigo, aqui http://www.allitebooks.com puedes encontrar un sin fin de libros sobre IT.


Saludos.

Amigo, hice muchas pruebas, pero llega un punto en el cual ninguna los resultados no son iguales que tu ejemplo, creo que el ejemplo o bien a la descripcion le hace falta algo.


Saludos.

Que tal mariaelhacker

Antes de todo, me parece que la jerarquia que estas creando deberia ser diferente, ya que la clase Aire no es un Aparato, por lo cual no tendria que ser hija de Aparato, mas bien deberia ser una interfaz.

Por otra parte, las validaciones que haces en el metodo "mostrarAparatos" (Los if's con instance of) para asignar la "Descripcion" estarian de mas, ya que se pierde el concepto de polimorfismo, deberia solo bastar con que hagas:

Código [Seleccionar] Expandir

inte.getDescripcion()

Por ultimo, sobre la linea que comentas que te esta dando el error, yo pensaria que primero deberias ver si esta linea te esta regresando algo

Código [Seleccionar] Expandir

mostrarAparatos((Habitacion)ComboBoxHabitaciones.getSelectedItem());

No mire todo el codigo a detalle, pero espero que mis comentarios puedan ayudarte con algo.


Saludos.