Problema con JMenu

[Kritik]

Bueno, por fin he terminado mi juego de tetris.

En la versión que tengo actualmente nada más abrir el programa se pone a jugar, con las fichas ya bajando... y eso es porque este es mi void main:

Código:

public static void main(String[] args){
	new base();
	juego juego1=new juego();
	juego1.yajuego();
}

en mi clase base tengo todos los elementos del tetris, paneles, marcadores, etc.
Y la clase juego es la clase que tiene toda la logística y los métodos que hacen funcionar el tetris.

Ahora he añadido en la clase base un menú, y le he añadido el actionlistener a dicho menú.

Y este es mi action Listener:

Código:

public void actionPerformed(ActionEvent arg0) {
	System.out.println("Menú seleccionado: \""+arg0.getActionCommand()+"\"");
}

Así pues, el menú lo único que hace cuando le doy al menú "Jugar" es decirme en la consola: Menú seleccionado: "Jugar"

Ahora bien, y he aquí el problema que estoy teniendo.

Si muevo las líneas:

Código:

juego juego1=new juego();
juego1.yajuego();

desde el void main a dentro del ActionListener de la siguiente manera:

Código:

public void actionPerformed(ActionEvent arg0) {
	if (arg0.getActionCommand()=="Salir")System.out.println("Saliendo");System.exit(0);
	if (arg0.getActionCommand()=="Jugar"){
		juego juego1=new juego();
		juego1.yajuego();
	}
}

Entonces al iniciar el juego no hace nada, y al ir al menú y seleccionar el menú jugar se me queda colgado.

Sin embargo, el otro menú "Salir" si me lo hace bien. Me escribe en la consola "Saliendo" y luego sale del programa. Por esto último creo que no es un problema del actionListener y no se me ocurre donde pueda estar el problema.

[Profesor_Falken]

Buenas,

arg0.getActionCommand()=="Salir"

En java los objetos (como String) se comparan utilizando el metodo equals. Cuando comparas con == lo que estas haciendo es comparar las referencias (las direcciones de memoria).

Código Java:

public void actionPerformed(ActionEvent arg0) {
    if ("Salir".equals(arg0.getActionCommand()))System.out.println("Saliendo");System.exit(0);
    if ("Jugar".equals(arg0.getActionCommand())){
        juego juego1=new juego();
        juego1.yajuego();
    }
}
 

Un saludo

[Kritik]

No es un problema del menú, y tampoco del método de comparación, pues he probado ambas formas tanto en menú como en un botón... y los 2 reaccionan de la misma manera. Se cuelga el programa.

Sin embargo, si lo ejecuto directamente poniéndolo en el void main entonces no se cuelga. No entiendo la razón.

[Profesor_Falken]

Buenas,

Yo solo puedo comentar sobre lo que veo. Esas comparaciones no son correctas como te comento. Otra cosa es que puedan funcionar porque la implementacion de tu maquina virtual implemente un pool de literales, pero eso es otra historia.

La otra cosa que veo es que no inicializas la aplicacion swing (entiendo que es Swing por los eventos que has puesto) correctamente.

A mi me tiene pinta de que estas dejando colgado el EDT y por eso no responde, pero sin ver el codigo es imposible de adivinar.


Un saludo

[Kritik]

Como se inicializan las aplicaciones swing? Podría ser eso y estar funcionando poniendo la clase con su método en el void main?

[Profesor_Falken]

La lectura de este tutorial es fundamental para entender Swing:
http://docs.oracle.com/javase/tutori...y/initial.html

Como ahí se indica la forma correcta de iniciar es:

Código Java:

SwingUtilities.invokeLater(new Runnable() {
    public void run() {
        createAndShowGUI();
    }
});
 

Con lambdas sería:

Código Java:

EventQueue.invokeLater(() -> {createAndShowGUI();});
 

Todos los eventos de la interfaz gráfica de una aplicación Swing se ejecutan en un thread especial llamado el EDT (Event Dispatch Thread). Si por algún motivo este thread se colapsa haciendo alguna tarea pesada, la aplicación se congelará y no responderá a los eventos.

Un saludo