Pasar de código estructurado a Orientado a Objetos

[link01]

La semana pasada tuvimos que hacer el juego del Ahorcado (hangman), pero el profesor me corrigió porque hice un diseño muy estructurado. Solo cree un una clase con un JFrame y ahí metí todos los métodos, eventos, gui, todo junto.

Ahora estoy modificando el código para que quede Orientado a Objetos, pero tengo problemas para comunicar los objetos entre sí, sobre todo los métodos y eventos con la interfaz gráfica.



Lo que hice fue dividir cada panel (palabra, letra, intentos y mensajes), para cada panel cree una clase, después cree una clase llamada Ventana en la que cree dos paneles, izquierdo y derecho, en el de la izquierda puse las instancias de las clases que ya mencioné y en el de la derecha la instancia de la clase del panel imagen. Y está todo bien.

Pero ahora no sé donde poner los métodos y los eventos. El profesor dijo algo como que había que crear un clase con los métodos aparte y después crear una instancia para comunicar los métodos con la interfaz, pero estoy teniendo problemas.
Por ejemplo:

Tengo este método para mostrar tantos guiones como letra tiene la palabra (gato = _ _ _ _ )

Código Java:

public void presentarRayas(){
        for(int i=0; i<palabraElegida.length(); i++){
            guionesBajosExternos +="_ ";
        }
        lbl_palabra.setText(guionesBajosExternos);
        tamanoG = lbl_palabra.getFont();
        lbl_palabra.setFont(new Font(tamanoG.getFontName(),tamanoG.getStyle(), 20));
    }
 

Ese método usa la etiqueta lbl_palabra, que se encuentra en otra clase: la clase PanelPalabra, cuando declaré el JLabel le puse el método de acceso protected y después lo cambié a public, pero como quiera me sigue marcando error lbl_palabra can´t be resolved.

No sé que tengo que hacer, para ese método por ejemplo podría ponerlo en la clase PanelPalabra y ya no me daría error, pero hay métodos en los que tengo que usar dos JLabels distintos y no sé entonces como podría "acomodarlo", además no estoy seguro de que esa sea la mejor forma....

[Fuzzylog]

Tengo problemas para entender lo que quieres decir.

1º Se pueden meter todos los textos en una clase de mensajes no instanciable, de forma que para llamarlo sería: Mensajes.nombreMensaje (Clase.atributo de clase)
2º La GUI debe ir en otra clase separada, los paneles bien puedes meterlos en clases distintas cada uno con su constructor, pero también puedes hacer un constructor general si hay parámetros comunes, y jugar con esos parámetros para ubicar los paneles.
3º El thread o main lo que uses para hacer funcionar en la aplicación iria en otra clase separaqda.
4º Si quieres, la clase que obtiene las palabras la puedes poner aparte. Una clase muy bonita sería una capaz de leer de un txt las palabras indicadas separadas por comas, contar el numero de palabras incluidas, y sacar una aleatoriamente.
5º La clase que gestiona las imagenes del ahorcado podría ser otra diferente.

Realmente puedes tener tantas clases como quieras, pero no tienes q llegar al extremo de crear clases completamente idénticas. Que cada clase tenga su propia funcíonalidad.

De ti depende el límite, el profesor no creo que te exija muchas clases, solo que sepas separar el modelo (los datos) de las vistas (la GUI) del control (en tu caso tu main o thread). Si quieres añadir un par de ellas a mayores para nota adelante :D pero que sea justificado.

[forumtecnico]

Cuesta trabajo cambiar un programa de estructurado a orientado a objetos, pero lo que más cuesta es cambiar la "mente" del programador para que piense en objetos. Es un problema más bien filosófico, salvando las distancias.
Cuando haces un programa estructurado, los elementos del mismo son herramientas inertes a las que tú les dices en cada momento qué hacer. Sin embargo, en la orientación a objetos cada elemento del programa es un "actor" que sabe qué tiene que hacer y tiene lo datos necesarios para hacerlo. Es por eso por lo que te cuesta tanto acceder desde una clase a los datos de otra. La solución no es abrirle paso a la fuerza para que pueda acceder, más bien eso debe darte la pista de que te has equivocado colocando ese dato fuera de donde realmente se usa.
Sé que es un poco rollo pero es así.
En tu caso veo que hay tres "actores": el selector de la letra elegida por el usuario, el controlador que lleva la cuenta de intentos-aciertos-fallos y por último el que muestra la imagen del ahorcado. El flujo de información es unidireccional:

selector--->controlador--->imagen

Si te fijas, cada uno sólo necesita conocer sus propios datos y avisar al siguiente (llamada a un método) de que tiene que actualizarse. Por ejemplo, el controlador es el único que conoce la palabra buscada y únicamente tiene que pasarle a la imagen cuál es la cuenta de aciertos actual. La imagen no tiene que saber cómo el controlador ha calculado ese dato. En el caso del selector, sólo tiene que conocer la lista del alfabeto y pasarle al controlador qué letra es elegida en cada momento. No sabe absolutamente nada del juego.
En el problema que dices con el JLabel, evidentemente es que el JLabel debe pertenecer a la clase que va a usarlo, no a otra.
Por supuesto, como dice Fuzzylog, una parte importante del trabajo es separar la interfaz (el GUI) del programa propiamente dicho. Cada clase le dirá a "su" parte de la interfaz lo que hacer.
Saludos
www.forumtecnico.com

[link01]

Estoy todo enredado, esto es lo que tengo:

Código Java:

//Clase para el Menu
package Ahorcaito;
 
import javax.swing.JMenu;
 
import javax.swing.JMenuBar;
 
import javax.swing.JMenuItem;
 
 
 
 
 
public class Menu extends JMenuBar{
 
    private JMenuBar mb = new JMenuBar();
 
    private JMenu menu = new JMenu();
 
    static JMenuItem reiniciar = new JMenuItem();
 
    static JMenuItem salir = new JMenuItem();
 
    
 
    public Menu(){
 
        mb = this;
 
        menu.setText("Menú");
 
        reiniciar.setText("Reiniciar");
 
        menu.add(reiniciar);
 
        menu.addSeparator();
 
        salir.setText("Salir");
 
        menu.add(salir);
 
        this.add(menu);
 
    }
 
 
 
}
 

Código Java:

//Clase para el panel que muestra los guiones bajos o rayitas
package Ahorcaito;
 
 
 
import java.awt.Color;
 
import java.awt.FlowLayout;
 
import java.awt.Font;
 
 
 
import javax.swing.BorderFactory;
 
import javax.swing.JLabel;
 
import javax.swing.JPanel;
 
import javax.swing.border.TitledBorder;
 
 
 
 
 
public class PanelPalabras extends JPanel{
 
    private JPanel panelPalabras = null;
 
    private TitledBorder titulo;
 
    private String palabraElegida = "", guionesBajosExternos = "", string = "", solucion = "", letraEscogida = "", guionesBajos = "";
 
    private Font tamanoG;
 
    private boolean chequearif;
 
    private char[] letra;
 
    private char[] arrayGuiones;
 
    
 
    public PanelPalabras(){
 
        panelPalabras = this;
 
        this.setLayout(new FlowLayout());
 
        this.setSize(275,100);
 
        this.setBackground(Color.lightGray);
 
        titulo = BorderFactory.createTitledBorder("Palabras");
 
        this.setBorder(titulo);
 
    }
 
}
 

Código Java:

//se muestra las letras utilizadas
package Ahorcaito;
 
import java.awt.Color;
 
import java.awt.FlowLayout;
 
import javax.swing.BorderFactory;
 
import javax.swing.JLabel;
 
import javax.swing.JPanel;
 
import javax.swing.border.TitledBorder;
 
 
 
 
 
public class PanelIntentos extends JPanel {
 
    private JPanel panelIntentos;
 
    private TitledBorder titulo;
 
    Textos texto = new Textos();
 
    
 
    public PanelIntentos(){
 
        panelIntentos = this;
 
        this.setLayout(new FlowLayout());
 
        this.setSize(275,100);
 
        this.setBackground(Color.lightGray);
 
        titulo = BorderFactory.createTitledBorder("Palabras");
 
        this.setBorder(titulo);
 
        this.add(texto.lbl_img);
 
        
 
    }
 
}
 

Código Java:

//Clase para el panel donde se escoge la letra
package Ahorcaito;
 
import java.awt.Color;
 
import java.awt.FlowLayout;
 
import javax.swing.JButton;
 
import javax.swing.BorderFactory;
 
import javax.swing.JComboBox;
 
import javax.swing.JPanel;
 
import javax.swing.border.TitledBorder;
 
 
 
 
 
public class PanelLetras extends JPanel{
 
    private JPanel panelLetras;
 
    private TitledBorder titulo;
 
    static String[] abcdario = {"A","B","C","D","E","F","G","H","I","J","K","L","M","N","O","P","Q","R","S","T","U","V","W","X","Y","Z"};
 
    static JComboBox jcb = new JComboBox(abcdario);
 
    static JButton boton = new JButton();
 
    
 
    
 
    public PanelLetras(){
 
        panelLetras = this;
 
        this.setLayout(new FlowLayout());
 
        this.setSize(275,100);
 
        this.setBackground(Color.lightGray);
 
        titulo = BorderFactory.createTitledBorder("Letras");
 
        this.setBorder(titulo);
 
        boton.setText("Aceptar");
 
        this.add(jcb);
 
        this.add(boton);
 
    }
 
 
 
}
 

Código Java:

//muestra los intentos que faltan para perder, si gano o perdio
package Ahorcaito;
 
import java.awt.Color;
 
import java.awt.FlowLayout;
 
 
 
import javax.swing.BorderFactory;
 
import javax.swing.JLabel;
 
import javax.swing.JPanel;
 
import javax.swing.border.TitledBorder;
 
 
 
 
 
public class PanelMensajes extends JPanel{
 
    private JPanel panelMensajes = new JPanel();
 
    private TitledBorder titulo;
 
    private JLabel lbl_msj = new JLabel("Le quedan 6 intentos");
 
    
 
    public PanelMensajes(){
 
        panelMensajes = this;
 
        this.setLayout(new FlowLayout());
 
        this.setSize(175,100);
 
        this.setBackground(Color.lightGray);
 
        titulo = BorderFactory.createTitledBorder("Mensajes");
 
        this.setBorder(titulo);
 
        this.add(lbl_msj);
 
    }
 
}
 

Código Java:

//cambia las imagenes
package Ahorcaito;
 
import java.awt.Color;
 
import java.awt.FlowLayout;
 
 
 
import javax.swing.Icon;
 
import javax.swing.ImageIcon;
 
import javax.swing.JLabel;
 
import javax.swing.JPanel;
 
 
 
 
 
public class PanelImg extends JPanel{
 
    private JPanel panelImg;
 
    Icon[] imagen = {new ImageIcon(getClass().getResource("1.gif")), new ImageIcon(getClass().getResource("2.gif")),
 
                            new ImageIcon(getClass().getResource("3.gif")), new ImageIcon(getClass().getResource("4.gif")),
 
                            new ImageIcon(getClass().getResource("5.gif")), new ImageIcon(getClass().getResource("6.gif")),
 
                            new ImageIcon(getClass().getResource("7.gif"))}; 
 
    
 
    public PanelImg(){
 
        panelImg = this;
 
        this.setLayout(new FlowLayout());
 
        this.setSize(175,100);
 
        this.setBackground(Color.lightGray);
 
        
 
    }
 
}
 

Código Java:

//junto todas las clases que tienen paneles en una ventana
package Ahorcaito;
 
import java.awt.GridLayout;
 
 
 
import javax.swing.ImageIcon;
 
import javax.swing.JFrame;
 
import javax.swing.JPanel;
 
 
 
 
 
public class Ventana extends JFrame{
 
    private JPanel panelIzq = new JPanel();
 
    private JPanel panelDer = new JPanel();
 
    
 
    public Ventana(){
 
        Menu menu = new Menu();
 
        PanelPalabras panelPalabras = new PanelPalabras();
 
        PanelIntentos panelIntentos = new PanelIntentos();
 
        PanelLetras panelLetras = new PanelLetras();
 
        PanelMensajes panelMensajes = new PanelMensajes();
 
        PanelImg panelImg = new PanelImg();
 
        Funciones funciones = new Funciones();
 
        Textos texto = new Textos();
 
        
 
        
 
        texto.lbl_img.setIcon(imagen[0]);
 
        PanelImg.add(texto.lbl_img);
 
        
 
        this.setTitle("Ahorcaito");
 
        this.setSize(570,470);
 
        this.setLayout(new GridLayout(1,2));
 
        this.setDefaultCloseOperation(EXIT_ON_CLOSE);
 
        this.setIconImage(new ImageIcon("icono.gif").getImage());
 
        this.setJMenuBar(menu);
 
        
 
        panelIzq.setSize(280,4210);
 
        panelIzq.setLayout(new GridLayout(4,1));
 
        panelDer.setSize(280,410);
 
        panelDer.setLayout(new GridLayout(1,1));
 
        
 
        panelIzq.add(panelPalabras);
 
        panelIzq.add(panelIntentos);
 
        panelIzq.add(panelLetras);
 
        panelIzq.add(panelMensajes);
 
        
 
        panelDer.add(panelImg);
 
        
 
        this.add(panelIzq);
 
        this.add(panelDer);
 
    }
 
 
 
}
 

[link01]

Código Java:

//Todas los metodos
package Ahorcaito;
 
 
 
import java.awt.Font;
 
import java.awt.event.ActionEvent;
 
 
 
 
 
 
 
public class Funciones {
 
    private Font tamano, tamanoG;
 
    private double posicionDEpalabra;
 
    private char[] arrayGuiones;
 
    private char[] letra;
 
    private String[] agregar = new String[10];
 
    private boolean chequearif;
 
    private int fallos=0, k=0;
 
    private String palabraElegida = "", letraEscogida = "", auxLetra = "", guionesBajos = "", guionesBajosExternos = "", string = "", solucion = "";
 
    private String[] lista_de_palabras = {"perro","vaca","chivo","oveja","canguro","gallina", "pato", "conejo", "aguila","avestrus",
 
              "caracol","abeja","babosa","caballo","delfin","elefante","halcon","jabali","koala","lagarto","mosca"};
 
    Textos texto = new Textos();
 
    PanelImg pimg = new PanelImg();
 
    
 
            public void elegirPalabra(){
 
                posicionDEpalabra = Math.random()*21;
 
                palabraElegida = lista_de_palabras[(int)posicionDEpalabra];
 
            }
 
 
 
            public void presentarRayas(){
 
                for(int i=0; i<palabraElegida.length(); i++){
 
                    guionesBajosExternos +="_ ";
 
                }
 
                texto.lbl_palabra.setText(guionesBajosExternos);
 
                tamanoG = texto.lbl_palabra.getFont();
 
                texto.lbl_palabra.setFont(new Font(tamanoG.getFontName(),tamanoG.getStyle(), 20));
 
            }
            public void NumRayasInterno(){
 
                for(int i=0; i<palabraElegida.length(); i++){
 
                    guionesBajos += "_";
 
                }
 
                arrayGuiones = guionesBajos.toCharArray();
 
            }
 
            public void actionPerformed(ActionEvent e) {
 
                if(e.getSource() == Menu.salir)
 
                    System.exit(0);
 
                if(e.getSource() == PanelLetras.boton){
 
                    letraEscogida = (((String) PanelLetras.jcb.getSelectedItem()).toLowerCase());
 
                    agregar[k] = letraEscogida.toUpperCase();
 
                    k++;
 
                    PanelLetras.jcb.removeItem(PanelLetras.jcb.getSelectedItem());
 
                    auxLetra += letraEscogida;
 
                    auxLetra = auxLetra.toUpperCase();
 
                    texto.lbl_letras.setText(auxLetra+=",  ");
 
                    buscarLetra();
 
                }
 
                if(e.getSource() == Menu.reiniciar)
 
                    reiniciar();
 
            }
            public void buscarLetra(){
 
                chequearif = false;
 
                string = "";
 
                solucion = "";
 
                letra = letraEscogida.toCharArray();
 
                int index = 0;
 
                for(int i=0;i<(palabraElegida.length());i++){
 
                    if(palabraElegida.substring(i, i+1).equals(letraEscogida) == true){
 
                        if(chequearif == true){
 
                            string = "";
 
                            solucion = "";
 
                            index = palabraElegida.indexOf(letraEscogida, index+1);
 
                            arrayGuiones[index] = letra[0];
 
                            for(int j=0;j<arrayGuiones.length;j++){
 
                                string += arrayGuiones[j]+" ";
 
                                solucion += arrayGuiones[j];
 
                            }
 
 
 
                            texto.lbl_palabra.setText(string);
 
                        }
 
                        if(chequearif == false){
 
                            index = palabraElegida.indexOf(letraEscogida);
 
                            arrayGuiones[index] = letra[0];
 
                            for(int j=0;j<arrayGuiones.length;j++){
 
                                string += arrayGuiones[j]+" ";
 
                                solucion += arrayGuiones[j];
 
                            }
 
                            texto.lbl_palabra.setText(string);
 
                            chequearif = true;
 
                        }
 
                    }
 
                }
 
                
 
                ganarPerder();
 
            }
            public void ganarPerder(){
                if(solucion.equals(palabraElegida) == true){
 
                    texto.lbl_msj.setText("¡HA GANADO! ¡FELICIDADES!");
 
                    tamano = texto.lbl_msj.getFont();
 
                    texto.lbl_msj.setFont(new Font(tamano.getFontName(),tamano.getStyle(),18));
 
                    PanelLetras.jcb.setEnabled(false);
 
                    PanelLetras.boton.setEnabled(false);
 
                }
                if(chequearif == false){
 
                    fallos=fallos+1;
 
                    texto.lbl_img.setIcon(pimg.imagen[fallos]);
 
                    texto.lbl_msj.setText("Le quedan " +(6-fallos)+ " intentos");
 
                    if(fallos == 6){
 
                        texto.lbl_msj.setText("¡HA PERDIDO!\n La palabra era: " +palabraElegida);
 
                        tamano = texto.lbl_msj.getFont();
 
                        texto.lbl_msj.setFont(new Font(tamano.getFontName(),tamano.getStyle(),13));
 
                        PanelLetras.jcb.setEnabled(false);
 
                        PanelLetras.boton.setEnabled(false);
 
                    }
 
                }
 
            }
            public void reiniciar(){
 
                PanelLetras.jcb.setEnabled(true);
 
                PanelLetras.boton.setEnabled(true);
 
                fallos = 0;
 
                texto.lbl_img.setIcon(pimg.imagen[0]);
 
                texto.lbl_msj.setText("Le quedan 6 intentos");
 
                texto.lbl_palabra.setText("");
 
                texto.lbl_letras.setText("");
 
                auxLetra = "";
 
                solucion= "";
 
                string = "";
 
                chequearif = false;
                guionesBajos = "";
                guionesBajosExternos = "";
                elegirPalabra();
                presentarRayas();
                NumRayasInterno();
                for(int i=0;i<k;i++){
                    PanelLetras.jcb.addItem(agregar[i]);
                }
                k=0;
            }        
}