[SOLUCIONADO] linkedlist dentro de una lista desplegable no funciona

[aurelioenriquez]

Buen dia

Tengo una conexion a una bd de oracle y funciona bien dentro de una linkedlist
ahora quiero armar una lista desplegable utilizando la lista y no reconoce la variable donde guardo el registro...gracias por su orientacion.

tengo el siguiente codigo :

<%
LinkedList<Registros> lista = Conexion.getContactos();

%>
<form method="POST">
<p>Selecciona una Sucursal y pulsa el botón
<select name="D1" size="lista.size()" >
<c:forEach var="regi" items="${lista}">
<option value=${regi}></option>");
</c:forEach>
</select>
<input type="button" value="Seleccionar";>
</p>
</form>

[aurelioenriquez]

Bien, pues resolvi en parte mi problema utilizando el metodo ITERATOR ...pero cuando despliega los registros de la lista con (ciclox.next()) me trae unos valores muy raros..pensaria que en hexadecimal ..pero no son los datos de la lista...por que?...que estoy haciendo mal?

<form method="POST">
<p>Selecciona una Sucursal y pulsa el botón

<%
LinkedList<Registros> lista = Conexion.getContactos();
Iterator ciclox = lista.iterator();
out.println("<select name=D1>");

while(ciclox.hasNext())
{
out.println("<option>"+ciclox.next()+"</option>");
}
out.println("</select>");

%>
<input type="button" value="Seleccionar"
onclick="alert('La opción seleccionada es: '+D1.options[D1.selectedIndex].text);">
</p>
</form>

[jomaruro]

Hola:

Iniciado por aurelioenriquez

..pensaria que en hexadecimal ..pero no son los datos de la lista...por que?...que estoy haciendo mal?

Porque estás mostrando el objeto, no las propiedades del objeto. Me explico.

Tienes una lista de tipo Registros:

Código Java:

LinkedList<Registros> lista = Conexion.getContactos();
 

La forma de resolverlo sería, extraer los datos de esos objetos Registros, para eso deberías usar los correspondientes métodos get:

Código Java:

Iterator ciclox = lista.iterator();
out.println("<select name=D1>");
 
while(ciclox.hasNext()) {
    Registros reg = ciclox.next();
 
    out.println("<option>"+reg.getLoQueSea()+"</option>");
    }
    out.println("</select>");
 

[aurelioenriquez]

Jomauro, gracias por tu apoyo.

Pero si utilizo el metodo lista.get(index).getNombre necesito un valor para "index", pero como lo estoy haciendo dentro de el metodo ITERATOR, no se como asignarle un valor de indice a lista.get(index)... no se si es claro mi problema, por eso pense que con Iterator ciclox = lista.iterator podia momerve dentro de los registros sin necesidad de un contador...Gracias por su ayuda

[jomaruro]

Hola:

No necesitas el index. Fíjate que yo no he puesto el index por ningún lado.

Código Java:

out.println("<option>"+reg.getLoQueSea()+"</option>");
 

el reg.getLoQueSea() se refiere a los métodos getter/setter de la clase Registro que tú indicas en el LinkedList

Código Java:

LinkedList<Registros> lista = Conexion.getContactos();
 

No me refiero al método get del LinkedList que sí necesita un index.

Código Java:

get(int index)
 

Vamos a ver si me explico mejor:

1.- Tienes la clase Registros, voy a suponer que tiene estos datos.

Código Java:

public class Registros {
    private int id = 0;
    private String nombre = "";
 
 
    public int getId() {
        return id;
    }
 
    public void setId(int id) {
        this.id = id;
    }
 
    public String getNombre() {
        return nombre;
    }
 
    public void setNombre(String nombre) {
        this.nombre = nombre;
    }
}
 

2.- Una vez creado el LinkedList y el Iterator

Código Java:

LinkedList<Registros> lista = Conexion.getContactos();
Iterator ciclox = lista.iterator();
 

recorres el objeto Iterator como te he indicado antes, para visualizar en el jsp por ejemplo el Nombre (recuerda que en este caso el objeto Registros lo he creado yo y no tiene que coincidir con el tuyo).

Código Java:

while(ciclox.hasNext()) {
    Registros reg = ciclox.next();
    out.println("<option>"+reg.getNombre()+"</option>");
}
 

Saludos.

[aurelioenriquez]

Muchas Gracias por tu ayuda JOMARURO.

Entendi perfecto e hice la correccion del caso. De hecho escribi la linea tal como me escribiste:

Registros reg = ciclox.next();

y el objeto "reg" me muestra los metodos propios de mi lista. (por ejemplo "getValor")

Pero me genera un error en la linea "Registros reg = ciclox.next();"
Me dice : "Incompatible types required Registros"

Aqui pego el codigo con los ajustes...

LinkedList<Registros> lista = Conexion.getContactos();
Iterator ciclox = lista.iterator();
out.println("<select name=D1>");

while(ciclox.hasNext()){
Registros reg = ciclox.next();
out.println("<option>" + reg.getValor() + "</option>");
}
out.println("</select>");

Gracias por sus aportes.

[jomaruro]

Hola:

¿Qué datos recoges con getContactos()?

Saludos.

[aurelioenriquez]

La Clase Contactos tiene una conexion a una BD de Oracle y para esta lista desplegable traigo unas ciudades.

Me parece extraño que despues de declarar "Registros reg = ciclox.next();"
la variable "reg" hereda los metodos de la clase "Registros" -->
"out.println("<option>" + reg.getValor() + "</option>");"

Pero me saca el error en la linea -->
"Registros reg = ciclox.next();"
Es decir en la linea anterior...

[aurelioenriquez]

Si sirve de algo me esta mostrando en la lista desplegable :

Conexion.Registros@144caa4
Conexion.Registros@5f0e0c
Conexion.Registros@935af2
etc....

y deberia sacar Nombres de ciudades

En otra clase utilizo un for i=0 etc... y con get(i) los muestra bien, pero como necesito meterlos en la lista y me quedo grande con el for pewnse que con el iterator era mas facil...y ahora estoy patinando en este problema.

[jomaruro]

Hola:

Por favor, muéstrame el código la clase Registros y la clase Conexion.

Saludos.

[aurelioenriquez]

Gracias jomaruro ahi van...llevo hoy toda la mañana rompiendome el coco....

clase Registros
===========
package Conexion;

/**
*
* @author 79056652
*/
public class Registros {

int id;
int ramo;
String nombre;
double valor;

public int getId() {
return id;
}
public void setId(int id) {
this.id = id;
}
public String getNombre() {
return nombre;
}
public void setNombre(String nombre) {
this.nombre = nombre;
}
public double getValor() {
return valor;
}
public void setValor(double valor) {
this.valor = valor;
}
public int getRamo() {
return ramo;
}
public void setRamo(int ramo) {
this.ramo = ramo;
}

}

clase Conexion
============

package Conexion;

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.LinkedList;

public class Conexion
{
public static LinkedList<Registros> getContactos()
{
LinkedList<Registros> listaContactos = new LinkedList<Registros>();
try
{
Class.forName("oracle.jdbc.driver.OracleDriver");
try (Connection conexion1 = DriverManager.getConnection("jdbc:oracle:thin:@10. 10.6.152:1521:intrac",
"XxsV2344", "Xxsv2344")) {
Statement st = conexion1.createStatement();
ResultSet rs = st.executeQuery("select t.epol_jefe_inmediato dir, t.epol_tipo_negocio tp, t.epol_ramo_contable ramon, sum(t.epol_facturado_mes) vlr from LFV_EXT_POLIZAS t where t.epol_ramo_contable in (1,23,34,40) and t.epol_cod_loc_neg=1003 and t.epol_jefe_inmediato=9937 and to_char(t.epol_ano)=2013 and to_char(t.epol_mes)=8 group by t.epol_jefe_inmediato, t.epol_tipo_negocio, t.epol_ramo_contable order by tp, ramon " );
while (rs.next())
{

Registros contacto = new Registros();
contacto.setId(rs.getInt("dir"));
contacto.setNombre(rs.getString("tp"));
contacto.setRamo(rs.getInt("ramon"));
contacto.setValor(rs.getDouble("vlr"));
listaContactos.add(contacto);
}
rs.close();
st.close();
}
}
catch (ClassNotFoundException | SQLException e)
{
}
return listaContactos;
}
}

[jomaruro]

Hola:

El problema está, creo, en que en el LinkedList necesitas un objeto Registro y estás recibiendo un objeto Conexion. Yo lo haría de otra manera.

En la clase conexión dejaría solo la conexión con la base de datos:

Código Java:

public class Conexion {
 
    /*
     *      La URL con la cual nos conectarémos a la base de datos.
     */
    private static String url = "jdbc:oracle:thin:@10.10.6.152:1521:intrac";
    /*
     *      El usuario de la base de datos.
     */
    private static String user = "XxsV2344";
    /*
     *      La clave del usuario de la base de datos.
     */
    private static String password = "Xxsv2344";
 
    /*
     *      Método para obtener la conexión con la base de datos.
     */
    public static synchronized Connection getConexion() {
        Connection cn;
        try {
            /*
             *      Cargamos el driver
             */
            Class.forName("oracle.jdbc.driver.OracleDriver");
            /*
             *      Obtenemos la conexión
             */
            cn = DriverManager.getConnection(url, user, password);
        } catch (SQLException e) {
            System.out.println("Error --> " + e);
            cn = null;
        } catch (Exception e) {
            System.out.println("Error --> " + e);
            cn = null;
        }
        return cn;
    }
}
 

La clase Registro la dejaría tal y como está. El siguiente paso es que crearía otra clase, por ejemplo DBRegistros para todas las operaciones con la clase Registros, donde incluiría la inserción de registros, actualización de registros, borrado de registros y consulta de registros.

Por ejemplo:

Código Java:

public class DBRegistros {
    public LinkedList<Registros> getContactos() {
        LinkedList<Registros> listaContactos = new LinkedList<Registros>();
        Connection conn = null;
        try {
            conn = Conexion.getConexion();
        } catch (Exception e) {
            System.out.println(e.getMessage());
        }
        if (conn != null) {
            PreparedStatement ps;
            String consulta;
            consulta = "select t.epol_jefe_inmediato dir, t.epol_tipo_negocio tp, t.epol_ramo_contable ramon, sum(t.epol_facturado_mes) vlr from LFV_EXT_POLIZAS t where t.epol_ramo_contable in (1,23,34,40) and t.epol_cod_loc_neg=1003 and t.epol_jefe_inmediato=9937 and to_char(t.epol_ano)=2013 and to_char(t.epol_mes)=8 group by t.epol_jefe_inmediato, t.epol_tipo_negocio, t.epol_ramo_contable order by tp, ramon ";
            try {
                ps = conn.prepareStatement(consulta);
                ResultSet rs = ps.executeQuery();
                while (rs.next()) {
                    Registro contacto = new Registro();
                    contacto.setId(rs.getInt("dir"));
            contacto.setNombre(rs.getString("tp"));
            contacto.setRamo(rs.getInt("ramon"));
                    contacto.setValor(rs.getDouble("vlr"));
            listaContactos.add(contacto);
                }
            } catch (SQLException e) {
                System.out.println(e.getMessage());
            } catch (Exception e) {
                System.out.println(e.getMessage());
            } finally {
                try { // cierro la conexion con la base de datos
                    if (conn != null && !conn.isClosed()) {
                        conn.close();
                    }
                } catch (SQLException e) {
                    System.out.println(e.getMessage());
                } catch (Exception e) {
                    System.out.println(e.getMessage());
                }
            }
        }
        return listaContactos;
    }
}
 

Y por último, donde haces el iterator cambias esta línea:

Código Java:

LinkedList<Registros> lista = Conexion.getContactos();
 

por esta otra:

Código Java:

LinkedList<Registros> lista = DBRegistros.getContactos();
 

Y si no me equivoco debería funcionarte.

Saludos.



P.D.: Es posible que en algunos casos me haya equivocado y haya puesto Registro donde debería poner Registros, supongo que te darás cuenta.

[aurelioenriquez]

Eres un Maestro, muchas gracias.

No entiendo por que funciono asi, y no trabajando directamente la combinacion con Registros dentro de la misma clase, pero bueno, funciono.

[jomaruro]

Hola:

Porque al ejecutar el:

Código Java:

LinkedList<Registros> lista = Conexion.getContactos();
 

El constructor de la clase Conexión te crea un objeto conexión y tu creabas un LinkedList con un objeto Registros.

De ahí que te diera el error:

"Incompatible types required Registros"

Es decir, los tipos eran incompatibles, necesitaba un objeto Registros y estaba recibiendo un objeto Conexion.

De ahí que te visualizara los objetos

Conexion.Registros@144caa4
Conexion.Registros@5f0e0c
Conexion.Registros@935af2

pero no sus valores.

Saludos.