Question

Me ha salido el siguiente error:
https://www.anony.ws/i/2016/08/10/error.png

Tengo el siguiente código:
-Content.xhtml:

<h:dataTable value="#{controller.MenuSelecions}" var="c"    styleClass="order-table"    headerClass="order-table-header"    rowClasses="order-table-odd-row,order-table-even-row">
        <h:column>
           <f:facet name="header"></f:facet>
            <h:outputText value="#{c.nombre}"/>
        </h:column>
    </h:dataTable>

-Controller:

@ManagedBean(name="controller")
@SessionScoped
public class Controller implements Serializable{
    public List<MenuSelecion> MenuSelecions() throws SQLException {
        Statement st = conex.createStatement();
        try {
            String cadSQL = "select * from menuSeleccion";
            rs = st.executeQuery(cadSQL);
            List<MenuSelecion> list = new ArrayList<MenuSelecion>();
            while(rs.next()){
                MenuSelecion MS = new MenuSelecion();
                MS.setNombre(rs.getString("nombre"));
                MS.setId(rs.getInt("id"));
                MS.setRutaArchivo(rs.getString("rutaArchivo"));
                MS.setTitle(rs.getNString("title"));
                //store all data into a List
                list.add(MS);
            }
            return list;
        }catch (SQLException e) {
            System.out.print(e);
        } finally {
            //conex.close();
        }
        return null;
    }

No entiendo porque pasa y aunque he googleado no me queda claro la forma de solucionarlo, estoy usando NetBeans

Answer 1

Cuando se utiliza EL expression para asignar una propiedad a un value, esta no se hace directamente con el nombre del método, sino directamente con el nombre de la propiedad.

Ejemplo:

//controller
private List<MenuSeleccion> menusSeleccion;

private List<MenuSeleccion> getMenusSeleccion(){
return menusSeleccion;
}

//view

<h:dataTable value="#{controller.menusSeleccion}"...

Por defecto, si no se añade un paréntesis al final de la expresión JSF entiende que se trata da una propiedad de la clase y busca el getter asociado a dicha propiedad.

Teniendo esto en cuenta estas dos expresiones son equivalentes:

<h:dataTable value="#{controller.menusSeleccion}"...

<h:dataTable value="#{controller.getMenusSeleccion()}"... 

Al añadir los parentesis entiende que lo que buscamos es ejecutar un metodo no una propiedad.

Además de lo dicho NUNCA se debe añadir lógica de negocio en los métodos getter o setter de un controller ya que estos se ejecutan muchas veces, cada vez que se renderiza un componente o tras realizar un update en el front, muchísimo menos meter en ellos una llamada a la base de datos,y tampoco es buen diseño que estos lancen excepciones, ya que estos aparecerán directamente en la pagina que ve el usuario y provocaran fallos visibles en tu aplicación:

Un diseño adecuado para lo que buscas hacer seria añadir la creación de la lista en el método con la anotación postConstruct, que se ejecutara una sola vez tras el constructor así:

@ManagedBean(name="controller")
@SessionScoped
public class Controller implements Serializable{
    private List<MenuSeleccion> menusSeleccion;

    @PostConstruct
    public void init() {
        Statement st = conex.createStatement();
        try {
            String cadSQL = "select * from menuSeleccion";
            rs = st.executeQuery(cadSQL);
            List<MenuSelecion> list = new ArrayList<MenuSelecion>();
            while(rs.next()){
                MenuSelecion MS = new MenuSelecion();
                MS.setNombre(rs.getString("nombre"));
                MS.setId(rs.getInt("id"));
                MS.setRutaArchivo(rs.getString("rutaArchivo"));
                MS.setTitle(rs.getNString("title"));
                //store all data into a List
                list.add(MS);
            }
            menusSeleccion = list;
        }catch (SQLException e) {
            System.out.print(e);
        } finally {
            //conex.close();
        }

    }

    public List<MenuSeleccion> getMenusSeleccion(){
        if(menusSeleccion == null) init();
        return menusSeleccion;
    }
    //si sabes que los datos de la base de datos han cambiado invalidas la lista con este método y la próxima vez que la lista sea requerida se volverá a crear:

   public void reset(){
      menusSeleccion = null;
   }

De esta manera la llamada a la base de datos solo se ejecuta una vez, y cuando sabemos que los datos no son validos porque han cambiado hacemos que la propiedad sea null, así la próxima vez que se solicite mediante el getter se volverá a crear.

Un saludo