JLabel

[JairLizcano]

Código:

package yohen;
import javax.swing.*;
import java.awt.*;
import javax.swing.event.*;

/**
 *
 * @author Edison Jair Lizcano
 */

public class Main {

    public static void main(String[] args) {
        
        JFrame ventana = new JFrame();
        ventana.setLocation(200, 300);
        ventana.setLayout(new GridBagLayout());
        GridBagConstraints limite = new GridBagConstraints();
        
        final JLabel cartas []  = new JLabel[5];
        
        String cambio = "";
        for(int i = 0; i<=4; i++){
            
            cambio = cambio.valueOf(i);
            cartas[i].setName(cambio);
            cambio = "/build/classes/yohen/mallet/"+cambio+".jpg";
            cartas[i].setIcon(new ImageIcon(cambio));
            
            limite.gridx = i;
            limite.gridy = 0;
            limite.gridheight = 1;
            limite.gridwidth = 1;
            
            ventana.add(cartas[i], limite);
            
        }
        
        ventana.show();
        ventana.pack();
        
    }

}

...lo que busco con el anterior codigo es que se agreguen en orden horizontal las 5 cartas llamadas: 0,1,2,3 y 4; pero necesito agregarlas de la forma mas optimizada posible...y hasta ahora he logrado nada...:S
...alguien puede recalcar una idea o una correccion sobre mi codigo?
...o bien no se si el vector de JLabel lo pueda utilizar de dicha forma...:S

[chuidiang]

Hola:

No estaría de más que indicaras qué quiere decir "de la forma más óptima posible" y qué es lo que estás obteniendo ahora que no se parece a cómo lo quieres.

Se bueno.

[JairLizcano]

Código:

package yohen;
import javax.swing.*;
import java.awt.*;
import javax.swing.event.*;

/**
 *
 * @author Edison Jair Lizcano
 */

public class Main {

    public static void main(String[] args) {
        
        JFrame ventana = new JFrame();
        ventana.setLocation(200, 300);
        ventana.setLayout(new GridBagLayout());
        GridBagConstraints limite = new GridBagConstraints();
        
        final JLabel cartas []  = new JLabel[5];
        
        String cambio = "";
        for(int i = 0; i<=4; i++){

            cambio = cambio.valueOf(i);
            cartas[i].setName(cambio);
            cambio = "/build/classes/yohen/mallet/"+cambio+".jpg";
            cartas[i].setIcon(new ImageIcon(cambio));
            
            JLabel prueba = new JLabel();
            cartas[i] = prueba;

            limite.gridx = i;
            limite.gridy = 0;
            limite.gridheight = 1;
            limite.gridwidth = 1;
            
            ventana.add(cartas[i], limite);
            
        }
        
        ventana.show();
        ventana.pack();
        
    }

}

...el problema solo era mi falta de referencia al array cartas[]...

[JairLizcano]

..."de la forma más óptima posible"; ello quieres decir, ese mismo codigo hubiese podido ser de por lo menos 15 o 20 lineas para cada carta, pero como puedes ver implemento un ciclo que me identifica cada carta de diferente forma...aprox. solo en 30 lineas para todas ...(Y)

[megajandro]

Iniciado por JairLizcano

..."de la forma más óptima posible"; ello quieres decir, ese mismo codigo hubiese podido ser de por lo menos 15 o 20 lineas para cada carta, pero como puedes ver implemento un ciclo que me identifica cada carta de diferente forma...aprox. solo en 30 lineas para todas ...(Y)

pero creo que aca la idea no es ahorrar lineas de codigo sino que primero
pienses un poco en el POO. Para eso primero deberias identificar un poco
cuales son los objetos con los que tenes que trabajar. En el codigo que tiraste no
identificaste ninguno. Despues Hacer un poco de uso de collecciones quizas te serviria un poco para tu ejercicio. Y por ultimo si queres utilizar awt o swing
ver bien como manejar los distintos componentes.