[SOLUCIONADO] Error sin localizar en programa

[enderzote]

Hola, soy programador novato en Java, aunque tengo bastante experiencia en TBasic, Pascal y Assembler.

Estoy diseñando una aplicación a modo de prueba para implementar la estructura en otro proyecto de mayores dimensiones. La idea es trabajar con 2 ventanas idénticas, las cuales están provistas de 3 botones:

- botonIncrementar: añade 1 a la variable contador1 o al 2, según esté activa la ventana 1 ó 2
- botonSalir: Sale de la aplicación
- botonCambia: Cierra la ventana actual y activa la otra

Se podría añadir a cada ventana 2 displays con los contadores acumulados, pero aún estoy buscando la forma.

El caso es que Eclipse me detecta un error en la línea:

Pantalla1 frame1 = new Pantalla1();

Concretamente me subraya "new Pantalla1()", mientras que no señala error en la siguiente línea de código, que es su gemela. Seguro que hay más errores de concepto que quizá influyan (o no) en ese error u otros, pero ya dije que soy novato en Java. El error específico que me marca Eclipse es:

"No enclosing instance of type CambiaPantalla is accessible. Must qualify the allocation with an enclosing instance of type CambiaPantalla (e.g. x.new A() where x is an instance of CambiaPantalla)"

Les agradecería cualquier tipo de orientación para la solución del error. Les adjunto el código a continuación.

Saludos.

Código Java:

package Default;
 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
 
public class CambiaPantalla {
    
    Pantalla1 frame1;
    Pantalla2 frame2;
    public int contador1 = 0;
    public int contador2 = 0;
            
    public static void main (String[] args) {
        Pantalla1 frame1 = new Pantalla1(); // AQUÍ INDICA EL ERROR
        Pantalla2 frame2 = new Pantalla2();
        frame2.setVisible(false);
        frame1.setVisible(true);
    }
    
    public class Pantalla1 extends JFrame implements ActionListener {
        
        private JPanel contentPane;
        
        public Pantalla1() {
            setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            contentPane = new JPanel();
            contentPane.setLayout(null);
            setBounds(10,10,640,480);
            setContentPane(contentPane);
            JButton botonSalir = new JButton("SALIR");
                botonSalir.setBounds(500,300,100,50);
                botonSalir.setActionCommand("salir");
                botonSalir.addActionListener(this);
                contentPane.add(botonSalir);
            JButton botonIncrementar = new JButton("SUMAR");
                botonIncrementar.setBounds(50,100,100,50);
                botonIncrementar.setActionCommand("incr1");
                botonIncrementar.addActionListener(this);
                contentPane.add(botonIncrementar);
            JButton botonCambia1 = new JButton("CAMBIA");
                botonCambia1.setBounds(50,300,100,50);
                botonCambia1.setActionCommand("cambia2");
                botonCambia1.addActionListener(this);
                contentPane.add(botonCambia1);
        }
        
        public void incrementa1() {
            ++contador1;
        }
        
        @Override
        public void actionPerformed(ActionEvent arg1) {
            if (arg1.getActionCommand().equals("salir")) {
                System.exit(0);}
            
            else if (arg1.getActionCommand().equals("incr1")) {
                incrementa1();
                System.out.println("El Contador1 acumula: "+contador1);}
            
            else if (arg1.getActionCommand().equals("cambia2")) {
                frame1.setVisible(false);
                frame2.setVisible(true);}
        }
    }
    
    public class Pantalla2 extends JFrame implements ActionListener {
 
        private JPanel contentPane;
        
        public Pantalla2() {
            setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            contentPane = new JPanel();
            contentPane.setLayout(null);
            setBounds(100,100,640,480);
            setContentPane(contentPane);
            JButton botonSalir = new JButton("SALIR");
                botonSalir.setBounds(500,300,100,50);
                botonSalir.setActionCommand("salir");
                botonSalir.addActionListener(this);
                contentPane.add(botonSalir);
            JButton botonIncrementar = new JButton("SUMAR");
                botonIncrementar.setBounds(50,100,100,50);
                botonIncrementar.setActionCommand("incr2");
                botonIncrementar.addActionListener(this);
                contentPane.add(botonIncrementar);
            JButton botonCambia2 = new JButton("CAMBIA");
                botonCambia2.setBounds(50,300,100,50);
                botonCambia2.setActionCommand("cambia1");
                botonCambia2.addActionListener(this);
                contentPane.add(botonCambia2);
        }
        
        public void incrementa2() {
            ++contador2;
        }
 
        @Override
        public void actionPerformed(ActionEvent arg2) {
            if (arg2.getActionCommand().equals("salir")) {
                System.exit(0);}
            
            else if (arg2.getActionCommand().equals("incr2")) {
                incrementa2();
                System.out.println("El Contador2 acumula: "+contador2);}
            
            else if (arg2.getActionCommand().equals("cambia1")) {
                frame2.setVisible(false);
                frame1.setVisible(true);}
        }
    }
}
 

[Xerelo]

Eso se debe a que tu código está mal, estás declarando una clase dentro de otra clase.

Aunque existe ese concepto, ni se hace de la forma en que has puesto ni se usa para lo que quieres.

Saca las clases Pantalla1 y Pantalla 2 de CambiaPantalla, y aún mejor, mete cada una en su propio archivo.

Creo que deberías leer algún tutorial básico, porque te ahorrarás errores como este.

[enderzote]

Gracias por responder.

Lo hice en módulos de inicio y me daba otros errores que tampoco sabía resolver. Siguiendo tus indicaciones lo he vuelto a modular tal que:

Código Java:

package Default2;
 
public class CambiaPantalla {
    
    public static void main (String[] args) {
 
        Pantalla1 frame1 = new Pantalla1();
        Pantalla2 frame2 = new Pantalla2();
        frame1.setVisible(false);
        frame2.setVisible(true);
    }
}
 

y la clase Pantalla1:

Código Java:

package Default2;
 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
 
public class Pantalla1 extends JFrame implements ActionListener {
    
    private JPanel contentPane;
    Pantalla1 frame1;
    Pantalla2 frame2;
    int contador1 = 0;
    
    public Pantalla1() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        contentPane = new JPanel();
        contentPane.setLayout(null);
        setBounds(10,10,640,480);
        setContentPane(contentPane);
        JButton botonSalir = new JButton("SALIR");
            botonSalir.setBounds(500,300,100,50);
            botonSalir.setActionCommand("salir");
            botonSalir.addActionListener(this);
            contentPane.add(botonSalir);
        JButton botonIncrementar = new JButton("SUMAR");
            botonIncrementar.setBounds(50,100,100,50);
            botonIncrementar.setActionCommand("incr1");
            botonIncrementar.addActionListener(this);
            contentPane.add(botonIncrementar);
        JButton botonCambia1 = new JButton("CAMBIA");
            botonCambia1.setBounds(50,300,100,50);
            botonCambia1.setActionCommand("cambia2");
            botonCambia1.addActionListener(this);
            contentPane.add(botonCambia1);
    }
    
    public void incrementa1() {
        ++contador1;}
    
    public void activaVentana2() {
        frame2.setVisible(true);
        frame1.setVisible(false);}
    
    @Override
    public void actionPerformed(ActionEvent arg1) {
        if (arg1.getActionCommand().equals("salir")) {
            System.exit(0);}
        
        else if (arg1.getActionCommand().equals("incr1")) {
            incrementa1();
            System.out.println("El Contador1 acumula: "+contador1);}
        
        else if (arg1.getActionCommand().equals("cambia2")) {
            activaVentana2();
        }
    }
}
 

La clase Pantalla2 no la reproduzco porque es simétrica a Pantalla1.

En este caso sólo indica error en la consola de Eclipse cuando intento cambiar de ventana con el botón que arranca el método activaVentana2.

Agradeceré cualquier sugerencia. Saludos

[Xerelo]

Normal, dentro de activaVentana2 estás cambiando las propiedades de frame1 y frame2.

¿Pero quién es frame1? El atributo/propiedad que hay al principio de la clase Pantalla1

¿Tiene algún valor frame1? No, en ningún momento le asignas un valor, por lo que es null.

Eso te lo está diciendo la consola, lee siempre los errores que te salen, están para que sepas qué es lo que sucede y en qué línea de código.

Estás intentando aprender java por prueba y error, y así poco avanzarás. No sabes nada sobre ámbitos de variables, modificadores de acceso, etc, y son cosas básicas e imprescindibles.

[enderzote]

Bueno, no me riñas tanto que soy muy sensible Tranquilo, sólo es una pequeña broma. Soy autodidacta y me alimento a base de vídeos, manuales y, como bien dices, de tropiezos... "enriquecedores".

Respecto al valor de frame1, en la línea 9 del main genero el objeto frame1 heredado de Pantalla1, cambio sus atributos mediante setVisible() y ahí no se queja. De hecho me permite acumular en contador1 con botonIncrementar. También se supone que frame1 debe contener los mismos atributos que Pantalla1, por lo que me parece redundante declararlo de nuevo, ya que no cambian.

Corrígeme si me equivoco en lo anterior. Por otro lado te agradecería que me indicaras en qué parte del código meto la pata u omito algo y si tiene una solución inmediata que no pase por matricularme en ingeniería informática Por mi parte seguiré atacando manuales para seguir aprendiendo, porque sé que Java es muy muy muy sofisticado y precisa mucha más dedicación de la que llevo acumulada, pero las prisas ajenas por ver resultados tangibles me traicionan.

Gracias por tu tiempo y saludos.

PD: me encargaron desarrollar en Java un simulador del software de una máquina determinada, que conllevaría el trabajo de meses de un equipo de programadores. No tengo límite de tiempo (todavía), pero estoy atascado en el cambio de una pantalla a otra y es algo frustrante.

[Xerelo]

Respecto al valor de frame1, en la línea 9 del main genero el objeto frame1 heredado de Pantalla1, cambio sus atributos mediante setVisible() y ahí no se queja. ... También se supone que frame1 debe contener los mismos atributos que Pantalla1, por lo que me parece redundante declararlo de nuevo, ya que no cambian.

Ese es el problema, aunque se llamen igual, frame1 del main y frame1 de CambiaPantalla no es la misma referencia. El objeto frame1 está siendo creado dentro del main, por lo que sólo existe dentro de ese método. Eso es a lo que me refiero cuando digo que no conoces el ámbito de las variables.

Por otro lado te agradecería que me indicaras en qué parte del código meto la pata u omito algo y si tiene una solución inmediata que no pase por matricularme en ingeniería informática

Solución, claro que la tiene y muy sencilla, prácticamente cualquiera del foro puede hacer lo que buscas en 5 minutos, por eso recalco la importancia de que leas un tutorial básico.

http://proyectosimio.wordpress.com/2...n-java-ambito/

El problema es que al desconocer lo básico, para corregirte o te lo resuelvo yo (no es mi estilo) o tengo que explicar lo básico de java, y eso está explicado en miles de tutoriales y mejor de lo que yo haría.

Pierde un par de horas leyendo un tutorial, avanzarás más.

http://www.matematica.ciens.ucv.ve/f...%20primero.pdf

Empieza por el capítulo 2

[enderzote]

Gracias, pero prefiero el AprendaJavaComo... sólo para consultas puntuales. Son mejores los manuales del estilo de tu referencia anterior al tema "ámbito", con explicaciones más coloquiales y ejemplos claros, aunque me quedo perplejo ante la imposiblidad de utilizar un atributo public en cualquier parte del programa, lo que parece un contrasentido. Tendré que seguir insistiendo...

Saludos

[Xerelo]

aunque me quedo perplejo ante la imposiblidad de utilizar un atributo public en cualquier parte del programa, lo que parece un contrasentido.

No es que no se pueda hacer, es que no lo sabes hacer.

Cualquier variable public que sea atributo de instancia o clase puede ser accedido desde cualquier clase o método, pero una variable creada dentro de una función no. Si creas un objeto dentro de un método, o lo asignas a una referencia accesible, o la pasas como parámetro/return de una función, de lo contrario desaparece al salir del método.

Seguramente tu mayor problema es pasar de programación estructurada a objetos, es necesario entender que una cosa es la referencia y otra el objeto. En estructurado variable y valor van unidos, en OOP no.

Gracias, pero prefiero el AprendaJavaComo... sólo para consultas puntuales.

A tu gusto, las búsquedas puntuales están bien cuando ya sabes, pero cuando no es difícil buscar lo que ni siquiera sabes que existe.

[Xerelo]

Tu código tocando dos cosas

Código Javascript:

public class Proceso {
    
    
     public static void main(String[] args) {
        
         Pantalla frame1 = new Pantalla("1");
         Pantalla frame2 = new Pantalla("2");
         frame1.setAlternativa(frame2);
         frame2.setAlternativa(frame1);
         frame1.setVisible(true);
         
    }
}
 

Y solo una clase pantalla

Código Javascript:

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
 
public class Pantalla extends JFrame implements ActionListener {
    
    private JPanel contentPane;
    int contador = 0;
    private Pantalla alternativa;
    private String nombre;
    
    public void setAlternativa (Pantalla alternativa){
        this.alternativa = alternativa;
    }
    
    public Pantalla(String nombre) {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.nombre = nombre;
        contentPane = new JPanel();
        contentPane.setLayout(null);
        setBounds(10,10,640,480);
        setContentPane(contentPane);
        JButton botonSalir = new JButton("SALIR");
            botonSalir.setBounds(500,300,100,50);
            botonSalir.setActionCommand("salir");
            botonSalir.addActionListener(this);
            contentPane.add(botonSalir);
        JButton botonIncrementar = new JButton("SUMAR");
            botonIncrementar.setBounds(50,100,100,50);
            botonIncrementar.setActionCommand("incr1");
            botonIncrementar.addActionListener(this);
            contentPane.add(botonIncrementar);
        JButton botonCambia1 = new JButton("CAMBIA");
            botonCambia1.setBounds(50,300,100,50);
            botonCambia1.setActionCommand("cambia");
            botonCambia1.addActionListener(this);
            contentPane.add(botonCambia1);
    }
    
    public void incrementa1() {
        ++contador;}
    
    public void activaVentana2() {
        alternativa.setVisible(true);
        this.setVisible(false);}
    
    @Override
    public void actionPerformed(ActionEvent arg1) {
        if (arg1.getActionCommand().equals("salir")) {
            System.exit(0);}
        
        else if (arg1.getActionCommand().equals("incr1")) {
            incrementa1();
            System.out.println("El Contador " + nombre + " acumula: "+contador);}
        
        else if (arg1.getActionCommand().equals("cambia")) {
            activaVentana2();
        }
    }
}
 

[enderzote]

Iniciado por Xerelo

No es que no se pueda hacer, es que no lo sabes hacer.

Efectivamente, no lo sé hacer desde el momento en que me encuentro frases lapidarias como: "cuando declaro un atributo o método de tipo public todas las clases tienen acceso a él". Patinaré si me tomo esa frase literalmente. A esa frase le falta matizar, relacionándola con tu segundo párrafo

Iniciado por Xerelo

Cualquier variable public que sea atributo de instancia o clase puede ser accedido desde cualquier clase o método, pero una variable creada dentro de una función no. Si creas un objeto dentro de un método, o lo asignas a una referencia accesible, o la pasas como parámetro/return de una función, de lo contrario desaparece al salir del método.

Eso aclara el tema definitivamente.

Iniciado por Xerelo

A tu gusto, las búsquedas puntuales están bien cuando ya sabes, pero cuando no es difícil buscar lo que ni siquiera sabes que existe.

No me malinterpretes. El AprendaJavaComo... posiblemente sea el más completo de los manuales de Java, pero es un texto demasiado farragoso para leérmelo entero. Me explicaré con un ejemplo: en la universidad tuve algún profesor que se limitaba a copiar en la pizarra sus apuntes desde unas hojas amarillas (por antiguas). Otros me parecían más amenos simplemente por su forma coloquial de explicar los temas. En las clases de los primeros me limitaba a copiar pizarras, para luego intentar razonarlo por mi cuenta. En las clases de los segundos salía habiendo comprendido.

Con ésto quiero decir que prefiero una explicación inicial comprensible, para después desarrollar; antes que empezar de cero con una explicación tan rigurosa como tortuosa. Avanzo más deprisa con el primer método. Preferencias personales...

En cualquier caso te agradezco tu tiempo y esfuerzo, sinceramente. Ahora me queda razonar por mi cuenta los cambios que has realizado en el código.

Saludos

[Xerelo]

La frase es la que es y describe la realidad correcta y literalmente.

Lo que te chirría precisamente es la falta de conocimiento sobre modificadores de acceso, porque dentro de un método no puedes declarar variables public, da un error de compilación. Por lo tanto es totalmente cierto que un método/variable public es accesible (si la instancia/clase lo es también) desde cualquier sitio.

Y repito, es muy importante que entiendas la diferencia entre referencia/variable y objeto al que apunta.

El código que he modificado es simplemente para que funcione, se puede hacer de muchas formas, seguramente cuando vayas definiendo exactamente lo que quieres hacer tendrás que modificarlo.

[enderzote]

Por si a alguien que estuviera en mi misma situación le interesara, expongo el código a partir del cual conseguí hacer funcionar mi programa como quería: desde el main escojo la ventana a la que ir mediante botones, cerrando y abriendo respectivamente, y pudiendo volver a la ventana inicial desde otro botón en la secundaria.

Hasta yo lo entiendo (es broma, Xerelo)

El main:

Código:

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

public class Login extends JFrame implements ActionListener {
	
	private JPanel panelLogin;
	
    Login(){
        panelLogin = new JPanel();
        panelLogin.setLayout(null);
    	setBounds(50,50,400,400);
        setTitle("Ventana de Login");
        setContentPane(panelLogin);
        JButton btnLogin1 = new JButton("Ventana 1");
	        btnLogin1.setBounds(150,100,100,50);
	        btnLogin1.setActionCommand("ventana1");
	        btnLogin1.addActionListener(this);
	        panelLogin.add(btnLogin1);
        JButton btnLogin2 = new JButton("Ventana 2");
	        btnLogin2.setBounds(150,250,100,50);
	        btnLogin2.setActionCommand("ventana2");
	        btnLogin2.addActionListener(this);
	        panelLogin.add(btnLogin2);
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.setVisible(true);
    }
    
    public static void main(String[] args) {
        new Login(); 
    }

    public void actionPerformed(ActionEvent e) {
        if (e.getActionCommand().equals("ventana1")) {
        	new Ventana1(this);
        }
        else if (e.getActionCommand().equals("ventana2")) {
        	new Ventana2(this);
        }
    }
}

La clase Ventana1, idéntica a Ventana2 con las salvedades evidentes, es decir, sustituir los "1" por "2" y listo:

Código:

import java.awt.event.WindowEvent;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.WindowAdapter;

class Ventana1 extends JFrame implements ActionListener {    
    JFrame ventanaLogin;
    private JPanel panel1;
    
    Ventana1(JFrame ventanaLogin) {
        this.ventanaLogin = ventanaLogin;
        ventanaLogin.setVisible(false);
        
        panel1 = new JPanel();
        setTitle("Ventana 1");
        panel1.setLayout(null);
    	setBounds(200,200,400,400);
    	setContentPane(panel1);
        JButton btnP1 = new JButton("Salir 1");
            btnP1.addActionListener(this);
            panel1.add(btnP1);
            btnP1.setBounds(150,200,100,50);
        this.setVisible(true);
        // Con la linea sig. saldría de la aplicación
        // Sin la linea sig. vuelve a Login si clic en X, igual que botón "salir"
        // this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.addWindowListener(new Ventana1.ManejadorDeVentana1()); 
    }

    private void activarVentanaLogin() {
        this.setVisible(false);
        ventanaLogin.setVisible(true); 
    }

    public void actionPerformed(ActionEvent e) {
        activarVentanaLogin(); 
    }

    class ManejadorDeVentana1 extends WindowAdapter{
        public void windowClosing(WindowEvent e) {
            activarVentanaLogin(); 
        } 
    }
}

[Xerelo]

Pues mucho no has entendido cuando tienes dos clases exactamente iguales, ya te dije cuando te puse el código que sólo es necesaria una. Estás yendo contra los principios de la orientación a objetos.

Aparte de eso estás haciendo una cosa muy mal, que es crear una nueva instancia de ventana cada vez que haces un cambio, así lo único que vas a conseguir es petar la memoria.

El hecho de que las pongas a visible=false no significa que desaparezcan, sólo que no las ves.

Es un ejercicio sencillísimo, y te está llevando un montón de tiempo por simple cabezonería.

[enderzote]

Ya dije al principio del hilo que sólo buscaba la manera de cambiar de ventana mediante un botón, que sólo necesitaba averiguar el modo para aplicarlo a otro proyecto de mayores dimensiones con multitud de pantallas DISTINTAS. Lo de crear 2 clases iguales sólo fue por simplificar las pruebas.

Es un ejercicio sencillísimo, y te está llevando un montón de tiempo por simple cabezonería.

Agradezco el interés y las molestias, amigo, pero de sencillísimo no tiene nada. Lo averiguaré, pero estoy convencido de que el software que quiero emular lo diseñó un equipo de programadores expertos a lo largo de muchos meses... o quizá varios años, mientras que yo estoy sólo, no puedo dedicarme al completo, ni siquiera me pagan y en breve me pedirán resultados; de forma que me gustaría poder mostrar algo palpable. No es lo mismo enseñar algo prometedor y que entre por la vista, que excusarme diciendo que prefiero antes estudiar Java como es debido. Lo haré sobre la marcha, como solemos hacer los ingenieros. A veces es más conveniente tomar un atajo, aunque te encuentres algunas zarzas.

Saludos

PD: Por supuesto que en un próximo refinamiento añadiré la orden "dispose()" para liberar la memoria de las ventanas que abandono, no te preocupes.