1

Estoy tratando de imprimir la imagen correspondiente de una provincia seleccionada en el jCombobox con el evento ItemStateChanged.

¿Por qué me da error del tipo StackOverflowError?

Error:

run:
Exception in thread "AWT-EventQueue-0" java.lang.StackOverflowError
    at java.security.AccessController.doPrivileged(Native Method)
    at java.net.URLClassLoader.findClass(URLClassLoader.java:361)
    at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
    at java.lang.ClassLoader.loadClass(ClassLoader.java:411)
    at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
    at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
    at com.mysql.jdbc.Util.handleNewInstance(Util.java:422)
    at com.mysql.jdbc.ConnectionImpl.getInstance(ConnectionImpl.java:400)
    at com.mysql.jdbc.NonRegisteringDriver.connect(NonRegisteringDriver.java:327)
    at java.sql.DriverManager.getConnection(DriverManager.java:664)
    at java.sql.DriverManager.getConnection(DriverManager.java:247)
    at bbdd.Conexiones.conexion_a_BBDD(Conexiones.java:30)
    at bbdd.Conexiones.cargar_provincias(Conexiones.java:82)
    at interfaz.Combobox_Anidado.combo_provinciasItemStateChanged(Combobox_Anidado.java:228)
    at interfaz.Combobox_Anidado.access$200(Combobox_Anidado.java:20)

Quiero que cuando se seleccione una Comunidad, me imprima también la primera provincia ubicada en el getSelectedIndex(0). Y una vez que se cambie el valor del jCombobox_provincia se actualice la provincia.

Código:

public class Combobox_Anidado extends javax.swing.JFrame {
    public Combobox_Anidado() {
        initComponents();

        //Cargamos las comunidades en el jComboBox llamado "combo_comunidades".
        ArrayList<String> comunidades = new ArrayList<String>();
        comunidades = Conexiones.cargar_comunidades();
        //Creamos un iterator para recorrer el ArrayList.
        Iterator<String> i = comunidades.iterator();
        //Dato inicial del combo_comunidades.
        combo_comunidades.addItem("Seleccione una comunidad...");
        //Recorremos el iterator "i".
        while(i.hasNext()){
            //Añadimos a los items cada comunidad.
            combo_comunidades.addItem(i.next());
        }
    }

    private void combo_comunidadesItemStateChanged(java.awt.event.ItemEvent evt) {                                                   
        //Guardamos en una variable string el valor seleccionado del combo_comunidades.
        String comunidad = (String) combo_comunidades.getSelectedItem();
        String capital = null;

        //Si el index del combo_comunidades es cero, NO existe comunidad: deshabilitamos.
        if(combo_comunidades.getSelectedIndex()==0) {
            combo_provincias.setEnabled(false);
            //Borramos todos los items del combobox.
            combo_provincias.removeAllItems();
            textfield_capital.setEnabled(false);
            textfield_capital.setText("");

            //Agregar una imagen a un JLabel.
            URL url_insertar = this.getClass().getResource("/imagenes/España.jpg");
            ImageIcon ico_insertar = new ImageIcon(url_insertar);
            lbl_fotoComunidad.setIcon(new ImageIcon(ico_insertar.getImage().getScaledInstance(lbl_fotoComunidad.getWidth(), lbl_fotoComunidad.getHeight(), Image.SCALE_SMOOTH)));
        }else if (evt.getStateChange() == ItemEvent.SELECTED) {
            //Habilitamos.
            combo_provincias.setEnabled(true);
            textfield_capital.setEnabled(true);
            lbl_fotoComunidad.setEnabled(true);

            //Creamos un ArrayList de String para las provincias.
            ArrayList<String> provincias = new ArrayList<String>();
            //Devolvemos las provincias dada una comunidad.
            provincias = Conexiones.cargar_provincias(comunidad);

            //Recorremos el ArrayList de provincias.
            Iterator<String> i = provincias.iterator();
            //Limpiamos los valores anteriores del combo_provincias.
            combo_provincias.removeAllItems();
            while (i.hasNext()) {
                //Añadimos a los items con cada provincia.
                combo_provincias.addItem(i.next());
            }

            //Devolvemos las provincias dada una comunidad.
            capital = Conexiones.capital_de_comunidad(comunidad);
            textfield_capital.setText(capital);

            //Ponemos la foto de la comunidad seleccionada.
            ImageIcon foto = new ImageIcon(getClass().getResource("/imagenes/"+comunidad+".png"));
            ImageIcon foto_xyz = new ImageIcon(foto.getImage().getScaledInstance(lbl_fotoComunidad.getWidth(), lbl_fotoComunidad.getHeight(), Image.SCALE_SMOOTH));
            lbl_fotoComunidad.setIcon(foto_xyz);
        }
    }                                                  

    private void combo_provinciasItemStateChanged(java.awt.event.ItemEvent evt) {                                                  
        String comunidad = (String) combo_comunidades.getSelectedItem();
        String provincia = (String) combo_provincias.getSelectedItem();

        if (evt.getStateChange() == ItemEvent.SELECTED) {
            //Creamos un ArrayList de String para las provincias.
            ArrayList<String> provincias = new ArrayList<String>();
            //Devolvemos las provincias dada una comunidad.
            provincias = Conexiones.cargar_provincias(comunidad);

            //Recorremos el ArrayList de provincias.
            Iterator<String> i = provincias.iterator();
            //Limpiamos los valores anteriores del combo_provincias.
            combo_provincias.removeAllItems();
            while (i.hasNext()) {
                //Añadimos a los items con cada provincia.
                combo_provincias.addItem(i.next());
            }

            //Ponemos la foto de la provincia seleccionada.
            ImageIcon foto = new ImageIcon(getClass().getResource("/imagenes/"+provincia+".png"));
            ImageIcon foto_xyz = new ImageIcon(foto.getImage().getScaledInstance(lbl_fotoProvincia.getWidth(), lbl_fotoProvincia.getHeight(), Image.SCALE_SMOOTH));
            lbl_fotoProvincia.setIcon(foto_xyz);
        }    
    }
}

Sin el método combo_provinciasItemStateChanged, funciona perfectamente al visualizar las comunidades, con las provincias me da ese error.

introducir la descripción de la imagen aquí

Código adiccional cargar_provincias():

public static ArrayList cargar_provincias(String comunidad) {
        ArrayList<String> provincias = new ArrayList<String>();
        java.sql.Connection c = (java.sql.Connection) Conexiones.conexion_a_BBDD("spain");
        Statement stm;
        ResultSet rs;
        try {
            //Consulta para sacar todas las provincias de una comunidad.
            stm = c.createStatement();
            String consulta_provincias = "SELECT p.provincia FROM provincias AS p, comunidades AS c WHERE c.comunidad = '"+comunidad+"' AND c.id=p.comunidad_id";
            rs = stm.executeQuery(consulta_provincias);
            //System.out.println("CONSULTA - PROVINCIAS DE X COMUNIDAD: Mostramos todas las provincias.");
            int i = 0;
            while (rs.next()) {
                String provincia = rs.getString("provincia");
                provincias.add(provincia);
                i++;
            }
            //System.out.println(provincias);
            c.close();
        } catch (SQLException e) {
            e.printStackTrace();
            System.out.println("Ha habido un error.");
        }     
        return provincias;
    }
13
  • Las preguntas que buscan ayuda con la depuración deben incluir el comportamiento esperado, un problema específico o error, y el código más corto necesario para reproducirlo en la propia pregunta. Las preguntas sin un planteamiento claro del problema no son útiles para otros lectores. Véase: Cómo crear un ejemplo mínimo verificable el 12 may. 2020 a las 17:29
  • @RuslanLópez Creo que la pregunta está perfectamente explicada y con código todo detalle, y al principio tengo escrito el error: "¿Por qué me da error del tipo StackOverflowError?"
    – omaza1990
    el 12 may. 2020 a las 17:38
  • Entonces por qué no veo el número de línea donde te dá el error? el 12 may. 2020 a las 17:42
  • @RuslanLópez No tengo error sintáxtico ni al compilar, el error lo tengo al elegir una comunidad y entonces me salga el error en el Output. Puse en el código del error en la explicación. Edité.
    – omaza1990
    el 12 may. 2020 a las 17:45
  • 1
    @RuslanLópez Si estaría mal configurado diría que el error sería de la conexión, y si funciona el combobox_comunidades, entonces obviamente la conexión MySQL está bien.
    – omaza1990
    el 12 may. 2020 a las 17:51

1 respuesta 1

2

Como decía en mi comentario "Normalmente este error viene dado por llamadas recursivas sin límite,". Y tengo razón.

Por algún motivo (ahem, copy & paste sin pensar), en tu lista de provincia repites el comportamiento de la lista de comunidades. Si seleccionas una provincia, borras la lista de provincias y la cargas de nuevo.

provincias = Conexiones.cargar_provincias(comunidad);

//Recorremos el ArrayList de provincias.
Iterator<String> i = provincias.iterator();
//Limpiamos los valores anteriores del combo_provincias.
combo_provincias.removeAllItems();
while (i.hasNext()) {
  //Añadimos a los items con cada provincia.
  combo_provincias.addItem(i.next());
}

¿Y qué pasará al hacer esto? Pues que el item que has seleccionado cambiará, con lo cual pasará a ser otro, lo que hará que se invoque de nuevo combo_provinciasItemStateChanged. Que a su vez invocará de nuevo a combo_provinciasItemStateChanged, que a su vez...

Esto queda ocultado porque hay tantas llamadas por enmedio que en el stacktrace no es fácil ver la recursividad.

Solución: No toques la lista de provincias al seleccionar una provincia. Porque además de causar el error, es que no tiene sentido.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.