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][1]. 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. 


  [1]: http://codejavu.blogspot.mx/2013/06/ejemplo-modelo-vista-controlador.html