Skip to main content
se añadieron 924 caracteres en el cuerpo
Origen Enlace
Legna
  • 6.6k
  • 4
  • 20
  • 48
import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JFrame;

public class Vista extends JFrame{
Controlador controlador;
    
    //ESTOS TE PERMITEN ACCEEDER AL CONTROLADOR QUE SE INICIO EN EL MAIN.
    public Controlador getControlador() {
        return controlador;
    }

    public void setControlador(Controlador controlador) {
       //ESTO ESTA TERRIBLEMENTE MAL. ¿TU CONTROLADOR ES UN KEYLISTENER????
       //DEBES DE CAMBIAR ESTO. NO TIENE NADA QUE VER CON MVC.
        this.addKeyListener(controlador);
        this.controlador = controlador;
    }




public Vista() {
    this.setVisible(true);
    this.setBounds(100, 100, 500, 500);
    //Esto no funciona asi POR QUE NO TIENES COMO ACCEDER A ESTE CONTROLADOR
    // Y AL QUE ESTA EN EL MAIN. SON DOS OBJETOS DIFERENTES. 
//    this.controlador = new Controlador();
    //ESTO ESCUCHA LAS TECLAS QUE ESTEN EN EL FRAME. 
    this.addKeyListener(controlador);
}
//

@Override
public void paint(Graphics g) { // g es un objeto gráfico y contiene los métodos de pintado
    g.setColor(Color.WHITE);
    g.fillRect(0, 0, 500, 500); // para crear un rectángulo
    g.setColor(Color.blue);
    if (controlador.getLetra()!=null) {
    //ESTA ES UNA DE LAS PARTES QUE TE INTERESA. FIJATE COMO ESTOY LLAMANDO
    //A LA INSTANCIA DE controlador y DE AHI AL STRING QUE LUEGO CONVIERTO EN 
    //CHAR. ESTO LO HICE ASI POR COMODIDAD MIA. QUEDATE CON LO QUE TE SIRVA. 
    g.drawChars (controlador.getLetra().toCharArray(), 0, controlador.getLetra().toCharArray().length,10, 60);
        
    }
 }
}
import java.util.Random;
import java.util.Scanner;

public class Modelo {

    static Scanner teclado = new Scanner(System.in);
    Controlador controlador;

    public Controlador getControlador() {
        return controlador;
    }

    public void setControlador(Controlador controlador) {
        this.controlador = controlador;
    }
    
    

    /*** DECLARACIÓN DE VARIABLES ***/

    String[] listaDeportes = {"futbol, baloncesto, balonmano, tenis, ski, padel, ciclismo, natacion, petanca, hipica"};
    int errores;
    char[] estadoPalabra;
    char[] letrasErroneas;
    String palabraSecreta;
    Random numAlea = new Random();
    //ESTA NO SE OCUPA POR QUE SE ACCEDE ATRAVEZ DEL CONTROLADOR. 
    
//    Vista view;

    /*** CONSTRUCTOR ***/
    //ESTO NO VA A FUNCIONAR ASI. 
//
//    public Modelo(int errores, String palabraSecreta, char[] estadoPalabra) {
//        this.errores = errores;
//        this.palabraSecreta = listaDeportes[(int) (numAlea.nextDouble() * 10)];
//        for (int i = 0; i < palabraSecreta.length(); i++) {
//            this.estadoPalabra[i] = '_';
//        }
//        
//        
//    }
    
    //AQUI LLAMAMOS A LA FUNCION INICIAR DATO. 
    //COMO NO SE QUE QUIERES HACER AQUI NO LO ARREGLE PERO ASÍ TE VA A 
    //FUNCIONAR MEJOR. SI UTILIZAS EL CONSTRUCTOR TENDRIAS QUE CREAR UNA 
    //CLASE MODELO NUEVA CADA VEZ Y NO QUEREMOS ESO. 
    //TU ARREGLAS ESTA PARTE. 
    public void iniciarDato(int errores, String palabraSecreta, char[] estadoPalabra){
     this.errores = errores;
        this.palabraSecreta = listaDeportes[(int) (numAlea.nextDouble() * 10)];
        for (int i = 0; i < palabraSecreta.length(); i++) {
            this.estadoPalabra[i] = '_';
        }
    }

    public char[] getEstadoPalabra() {
        return estadoPalabra;
    }

    public void verificar(char letra) {

        for (int i = 0; i < palabraSecreta.length(); i++) {
            if (palabraSecreta.toCharArray()[i]==(letra)) {
                estadoPalabra[i] = letra;
            }
            else
                errores++;
        }
    }
}

Cosas que te deben quedar claras:

POO
Que es MVC
Paquetes en java
Paquetes en netbeans

import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JFrame;

public class Vista extends JFrame{
Controlador controlador;
    
    //ESTOS TE PERMITEN ACCEEDER AL CONTROLADOR QUE SE INICIO EN EL MAIN.
    public Controlador getControlador() {
        return controlador;
    }

    public void setControlador(Controlador controlador) {
       //ESTO ESTA TERRIBLEMENTE MAL. ¿TU CONTROLADOR ES UN KEYLISTENER????
       //DEBES DE CAMBIAR ESTO. NO TIENE NADA QUE VER CON MVC.
        this.addKeyListener(controlador);
        this.controlador = controlador;
    }




public Vista() {
    this.setVisible(true);
    this.setBounds(100, 100, 500, 500);
    //Esto no funciona asi POR QUE NO TIENES COMO ACCEDER A ESTE CONTROLADOR
    // Y AL QUE ESTA EN EL MAIN. SON DOS OBJETOS DIFERENTES. 
//    this.controlador = new Controlador();
    //ESTO ESCUCHA LAS TECLAS QUE ESTEN EN EL FRAME. 
    this.addKeyListener(controlador);
}
//

@Override
public void paint(Graphics g) { // g es un objeto gráfico y contiene los métodos de pintado
    g.setColor(Color.WHITE);
    g.fillRect(0, 0, 500, 500); // para crear un rectángulo
    g.setColor(Color.blue);
    if (controlador.getLetra()!=null) {
    g.drawChars (controlador.getLetra().toCharArray(), 0, controlador.getLetra().toCharArray().length,10, 60);
        
    }
 }
}
import java.util.Random;
import java.util.Scanner;

public class Modelo {

    static Scanner teclado = new Scanner(System.in);
    Controlador controlador;

    public Controlador getControlador() {
        return controlador;
    }

    public void setControlador(Controlador controlador) {
        this.controlador = controlador;
    }
    
    

    /*** DECLARACIÓN DE VARIABLES ***/

    String[] listaDeportes = {"futbol, baloncesto, balonmano, tenis, ski, padel, ciclismo, natacion, petanca, hipica"};
    int errores;
    char[] estadoPalabra;
    char[] letrasErroneas;
    String palabraSecreta;
    Random numAlea = new Random();
    //ESTA NO SE OCUPA POR QUE SE ACCEDE ATRAVEZ DEL CONTROLADOR. 
    
//    Vista view;

    /*** CONSTRUCTOR ***/
    //ESTO NO VA A FUNCIONAR ASI. 
//
//    public Modelo(int errores, String palabraSecreta, char[] estadoPalabra) {
//        this.errores = errores;
//        this.palabraSecreta = listaDeportes[(int) (numAlea.nextDouble() * 10)];
//        for (int i = 0; i < palabraSecreta.length(); i++) {
//            this.estadoPalabra[i] = '_';
//        }
//        
//        
//    }
    
    //AQUI LLAMAMOS A LA FUNCION INICIAR DATO. 
    public void iniciarDato(int errores, String palabraSecreta, char[] estadoPalabra){
     this.errores = errores;
        this.palabraSecreta = listaDeportes[(int) (numAlea.nextDouble() * 10)];
        for (int i = 0; i < palabraSecreta.length(); i++) {
            this.estadoPalabra[i] = '_';
        }
    }

    public char[] getEstadoPalabra() {
        return estadoPalabra;
    }

    public void verificar(char letra) {

        for (int i = 0; i < palabraSecreta.length(); i++) {
            if (palabraSecreta.toCharArray()[i]==(letra)) {
                estadoPalabra[i] = letra;
            }
            else
                errores++;
        }
    }
}
import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JFrame;

public class Vista extends JFrame{
Controlador controlador;
    
    //ESTOS TE PERMITEN ACCEEDER AL CONTROLADOR QUE SE INICIO EN EL MAIN.
    public Controlador getControlador() {
        return controlador;
    }

    public void setControlador(Controlador controlador) {
       //ESTO ESTA TERRIBLEMENTE MAL. ¿TU CONTROLADOR ES UN KEYLISTENER????
       //DEBES DE CAMBIAR ESTO. NO TIENE NADA QUE VER CON MVC.
        this.addKeyListener(controlador);
        this.controlador = controlador;
    }




public Vista() {
    this.setVisible(true);
    this.setBounds(100, 100, 500, 500);
    //Esto no funciona asi POR QUE NO TIENES COMO ACCEDER A ESTE CONTROLADOR
    // Y AL QUE ESTA EN EL MAIN. SON DOS OBJETOS DIFERENTES. 
//    this.controlador = new Controlador();
    //ESTO ESCUCHA LAS TECLAS QUE ESTEN EN EL FRAME. 
    this.addKeyListener(controlador);
}
//

@Override
public void paint(Graphics g) { // g es un objeto gráfico y contiene los métodos de pintado
    g.setColor(Color.WHITE);
    g.fillRect(0, 0, 500, 500); // para crear un rectángulo
    g.setColor(Color.blue);
    if (controlador.getLetra()!=null) {
    //ESTA ES UNA DE LAS PARTES QUE TE INTERESA. FIJATE COMO ESTOY LLAMANDO
    //A LA INSTANCIA DE controlador y DE AHI AL STRING QUE LUEGO CONVIERTO EN 
    //CHAR. ESTO LO HICE ASI POR COMODIDAD MIA. QUEDATE CON LO QUE TE SIRVA. 
    g.drawChars (controlador.getLetra().toCharArray(), 0, controlador.getLetra().toCharArray().length,10, 60);
        
    }
 }
}
import java.util.Random;
import java.util.Scanner;

public class Modelo {

    static Scanner teclado = new Scanner(System.in);
    Controlador controlador;

    public Controlador getControlador() {
        return controlador;
    }

    public void setControlador(Controlador controlador) {
        this.controlador = controlador;
    }
    
    

    /*** DECLARACIÓN DE VARIABLES ***/

    String[] listaDeportes = {"futbol, baloncesto, balonmano, tenis, ski, padel, ciclismo, natacion, petanca, hipica"};
    int errores;
    char[] estadoPalabra;
    char[] letrasErroneas;
    String palabraSecreta;
    Random numAlea = new Random();
    //ESTA NO SE OCUPA POR QUE SE ACCEDE ATRAVEZ DEL CONTROLADOR. 
    
//    Vista view;

    /*** CONSTRUCTOR ***/
    //ESTO NO VA A FUNCIONAR ASI. 
//
//    public Modelo(int errores, String palabraSecreta, char[] estadoPalabra) {
//        this.errores = errores;
//        this.palabraSecreta = listaDeportes[(int) (numAlea.nextDouble() * 10)];
//        for (int i = 0; i < palabraSecreta.length(); i++) {
//            this.estadoPalabra[i] = '_';
//        }
//        
//        
//    }
    
    //AQUI LLAMAMOS A LA FUNCION INICIAR DATO. 
    //COMO NO SE QUE QUIERES HACER AQUI NO LO ARREGLE PERO ASÍ TE VA A 
    //FUNCIONAR MEJOR. SI UTILIZAS EL CONSTRUCTOR TENDRIAS QUE CREAR UNA 
    //CLASE MODELO NUEVA CADA VEZ Y NO QUEREMOS ESO. 
    //TU ARREGLAS ESTA PARTE. 
    public void iniciarDato(int errores, String palabraSecreta, char[] estadoPalabra){
     this.errores = errores;
        this.palabraSecreta = listaDeportes[(int) (numAlea.nextDouble() * 10)];
        for (int i = 0; i < palabraSecreta.length(); i++) {
            this.estadoPalabra[i] = '_';
        }
    }

    public char[] getEstadoPalabra() {
        return estadoPalabra;
    }

    public void verificar(char letra) {

        for (int i = 0; i < palabraSecreta.length(); i++) {
            if (palabraSecreta.toCharArray()[i]==(letra)) {
                estadoPalabra[i] = letra;
            }
            else
                errores++;
        }
    }
}

Cosas que te deben quedar claras:

POO
Que es MVC
Paquetes en java
Paquetes en netbeans

Origen Enlace
Legna
  • 6.6k
  • 4
  • 20
  • 48

Aqui te va un ejemplo de tu código modificado. Estudialo e implementa lo que te sirva.

PRINCIPAL

    public class Principal {
        public static void main(String[] args)  {
            Vista view = new Vista();
            Controlador c = new Controlador();
            //INICIALIZA LAS VARIABLES PARA QUE LAS INSTANCIAS FUNCIONES. 
            //ESTAS CREANDO VARIAS INSTANCIAS DE LO MISMO Y POR ESO NO PUEDES HACER 
            //REFERENCIA A ELLO. 
            
            view.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            
            c.setView(view);
            
            //AQUI SETEA LAS INSTANCIAS PARA QUE SOLO EXISTAN UNAS. 
            view.setControlador(c);
            view.setVisible(true);
            
        }
   

VISTA

import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JFrame;

public class Vista extends JFrame{
Controlador controlador;
    
    //ESTOS TE PERMITEN ACCEEDER AL CONTROLADOR QUE SE INICIO EN EL MAIN.
    public Controlador getControlador() {
        return controlador;
    }

    public void setControlador(Controlador controlador) {
       //ESTO ESTA TERRIBLEMENTE MAL. ¿TU CONTROLADOR ES UN KEYLISTENER????
       //DEBES DE CAMBIAR ESTO. NO TIENE NADA QUE VER CON MVC.
        this.addKeyListener(controlador);
        this.controlador = controlador;
    }




public Vista() {
    this.setVisible(true);
    this.setBounds(100, 100, 500, 500);
    //Esto no funciona asi POR QUE NO TIENES COMO ACCEDER A ESTE CONTROLADOR
    // Y AL QUE ESTA EN EL MAIN. SON DOS OBJETOS DIFERENTES. 
//    this.controlador = new Controlador();
    //ESTO ESCUCHA LAS TECLAS QUE ESTEN EN EL FRAME. 
    this.addKeyListener(controlador);
}
//

@Override
public void paint(Graphics g) { // g es un objeto gráfico y contiene los métodos de pintado
    g.setColor(Color.WHITE);
    g.fillRect(0, 0, 500, 500); // para crear un rectángulo
    g.setColor(Color.blue);
    if (controlador.getLetra()!=null) {
    g.drawChars (controlador.getLetra().toCharArray(), 0, controlador.getLetra().toCharArray().length,10, 60);
        
    }
 }
}

CONTROLADOR

import javax.swing.*;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

public class Controlador implements KeyListener {

private Vista view;
private Modelo modelo;
String palabraSecreta;
char[] estadoPalabra;
int errores;
private String letra;

    public String getLetra() {
        return letra;
    }

    public void setLetra(String letra) {
        this.letra = letra;
    }

    

    public Vista getView() {
        return view;
    }

    public void setView(Vista view) {
        this.view = view;
    }

    public Modelo getModelo() {
        return modelo;
    }

    public void setModelo(Modelo modelo) {
        this.modelo = modelo;
    }




    @Override
    public void keyTyped(KeyEvent e) {
        letra = letra + e.getKeyChar();
        view.repaint();
    }

    @Override
    public void keyPressed(KeyEvent e) {
    }

    @Override
    public void keyReleased(KeyEvent e) {

    }
}

MODELO

import java.util.Random;
import java.util.Scanner;

public class Modelo {

    static Scanner teclado = new Scanner(System.in);
    Controlador controlador;

    public Controlador getControlador() {
        return controlador;
    }

    public void setControlador(Controlador controlador) {
        this.controlador = controlador;
    }
    
    

    /*** DECLARACIÓN DE VARIABLES ***/

    String[] listaDeportes = {"futbol, baloncesto, balonmano, tenis, ski, padel, ciclismo, natacion, petanca, hipica"};
    int errores;
    char[] estadoPalabra;
    char[] letrasErroneas;
    String palabraSecreta;
    Random numAlea = new Random();
    //ESTA NO SE OCUPA POR QUE SE ACCEDE ATRAVEZ DEL CONTROLADOR. 
    
//    Vista view;

    /*** CONSTRUCTOR ***/
    //ESTO NO VA A FUNCIONAR ASI. 
//
//    public Modelo(int errores, String palabraSecreta, char[] estadoPalabra) {
//        this.errores = errores;
//        this.palabraSecreta = listaDeportes[(int) (numAlea.nextDouble() * 10)];
//        for (int i = 0; i < palabraSecreta.length(); i++) {
//            this.estadoPalabra[i] = '_';
//        }
//        
//        
//    }
    
    //AQUI LLAMAMOS A LA FUNCION INICIAR DATO. 
    public void iniciarDato(int errores, String palabraSecreta, char[] estadoPalabra){
     this.errores = errores;
        this.palabraSecreta = listaDeportes[(int) (numAlea.nextDouble() * 10)];
        for (int i = 0; i < palabraSecreta.length(); i++) {
            this.estadoPalabra[i] = '_';
        }
    }

    public char[] getEstadoPalabra() {
        return estadoPalabra;
    }

    public void verificar(char letra) {

        for (int i = 0; i < palabraSecreta.length(); i++) {
            if (palabraSecreta.toCharArray()[i]==(letra)) {
                estadoPalabra[i] = letra;
            }
            else
                errores++;
        }
    }
}

Como dato. Estudia este link. Lo que estás haciendo no tiene nada que ver con el MVC. Solo estas poniendo sus nombres pero no es así como se lleva a cabo. La manera en que trabajas te va a traer muchos problemas despues. Sería bueno que fueran entrando un poco más en detalle de este u otros modelo que te ayudarán a organizarte mejor.