J2ee+Struts 2

[Ruben82]

Buenas, estoy realizando una aplicación web en j2ee y struts 2 y varias dudas me vienen a la cabeza, bueno mas bien, varios errores ocurren en mi aplicación.

Lo primero que voy a hacer es detallar un poco lo que quiero hacer para poneros en situación.

Lo que estoy realizando es una página donde el usuario tenga que rellenar varios formularios.
En principio hacía una validación mediante el Action de que los campos estuviesen llenos pero después intenté hacer una validación mediante ficheros XML, opción que finalmente deseché por no entender muy bien y compicarseme la vida.

Bueno, lo que intento hacer es, primeramente rellenar un formulario, al darle a continuar compruebe que todo sea correcto y cargue un nuevo formulario, aquí hariamos la misma operacion, es decir, al darle a continuar vuelva a comprobar que todo esté correcto y una vez mas carga un tercer formulario, pero esta vez, necesito datos del formulario anterior, porque dependiendo de unos de los campos, carga un formulario u otro.


El caso es que no se muy bien como hacerlo, ni siquiera se muy bien como hacer para las comprobaciones de los campos de los anteriores. Me explico:

En principio tenía una accion para cada formulario pero me daba un error, hasta que decidi trabajar con la misma accion pero con distintos metodos para cada formulario. Ahora no me funcionan las validaciones por que el metodo validate no puede ser comun para ambos formularios...


Bueno espero que me entiendan y que me puedan echar una mano

un saludo y muchas gracias

[Xerelo]

El método validate() (supongo que será el que usas para validar) se ejecuta siempre si tu action hereda de Action (o ActionSupport) y si pasa por el interceptor workflow(defaultstack).

Aunque puedes definir en struts.xml qué métodos llamarán a validate(), en tu caso, si quieres una solución rápida y sencilla (aunque no elegante) te recomendaría lo siguiente.

Ya que vas a llamar a tres formularios desde la misma action, crea en cada uno de ellos un campo oculto con el nombre del formulario, de esta forma tu método validador podrá saber fácilmente qué campos validar según el contenido del campo oculto.

Para pasar los datos de un formulario a otro es igual de sencillo, cuando vuelves de una action, los objetos que estén declarados con sus getter y setter (beans), podrán ser leídos desde la jsp de destino, siempre que les pongas el mismo nombre (name) que a los objetos de la action.

Si quieres que esos datos se vean, usa cajas de textos, si quieres que no, utiliza un campo hidden. Échale un vistazo a las etiquetas "HTML" de struts.

[Ruben82]

Iniciado por Xerelo

El método validate() (supongo que será el que usas para validar) se ejecuta siempre si tu action hereda de Action (o ActionSupport) y si pasa por el interceptor workflow(defaultstack).

Sip, estás en lo cierto. El método validate() es que utilizo para validar, o al menos eso es lo que intento.

En cuanto al workflow no estoy muy seguro de a que te refieres

Iniciado por Xerelo

.... crea en cada uno de ellos un campo oculto con el nombre del formulario, de esta forma tu método validador podrá saber fácilmente qué campos validar según el contenido del campo oculto.

¿te refieres a poner en cada formulario algo del tipo <s:hidden name="Form1"/> y desde el validate recuperar ese nombre?, porque también lo intenté pero no se como recuperar el nombre desde el Action

Iniciado por Xerelo

Para pasar los datos de un formulario a otro es igual de sencillo, cuando vuelves de una action, los objetos que estén declarados con sus getter y setter (beans), podrán ser leídos desde la jsp de destino, siempre que les pongas el mismo nombre (name) que a los objetos de la action.

Aquí si que ya estoy perdido de todo ....

Iniciado por Xerelo

Si quieres que esos datos se vean, usa cajas de textos, si quieres que no, utiliza un campo hidden. Échale un vistazo a las etiquetas "HTML" de struts.

Supongo que será algo del estilo <s:textfield value="Aqui recuperar el texto no se como">

Un saludo Xerelo e graciñas!!!

[Xerelo]

Vale, por lo que veo hay una parte muy concreta que es la que te confunde.

¿te refieres a poner en cada formulario algo del tipo <s:hidden name="Form1"/> y desde el validate recuperar ese nombre?, porque también lo intenté pero no se como recuperar el nombre desde el Action

Los action.class, además de servir para ejecutar tu programa (lógica de negocio), sirve para traspasar valores desde la JSP (capa vista) al código.

Me dejo de teorías y te pongo un ejemplo claro.

Código Javascript:

public tuAction extends ActionSupport {
 
private String formulario;
 
public execute () {
...
}
 
public validate () {
 
if (getFormulario().equals("formu1") {...} else {..}
 
}
 
public void setFormulario(String form){
this.formulario = form;
}
 
 
public String getFormulario(){
return this.formulario;
}
 
}
 

Vale, pues ese atributo formulario, servirá para pasar automáticamene una cadena entre la JSP y el action, y viceversa.

<s:text name="formulario"/> (hay más etiquetas que te sirven)

Muestra en la JSP el contenido del atributo formulario

<s:hidden name="formulario" value="formu1"/> (otras etiquetas te servirían)

Hará que al enviar el formulario que lo contiene, el atributo formulario del action valga formu1. Y ese valor lo puedes leer sin problemas en el validate()

http://www.mkyong.com/struts2/struts...value-example/

Este traspaso de valores es lo básico para empezar a usar Struts2, así que te recomendaría que leyeses un poco sobre le tema.

[Ruben82]

Iniciado por Xerelo

<s:text name="formulario"/> (hay más etiquetas que te sirven)

Muestra en la JSP el contenido del atributo formulario

<s:hidden name="formulario" value="formu1"/> (otras etiquetas te servirían)

He leido el "pequeño" manual que me enviaste y entiendo lo que me quieres decir y lo que significa. De hecho probe y de un jsp a otro lo pasa perfectamente, sin embargo a la hora de leerlo en el action me da un error, concretamente este:

Stacktraces
java.lang.NullPointerException

Un saludo

[Xerelo]

Si te lo pasa de un JSP a otro es que lo está pasando por el action, supongo que te faltará alguna tontería.

¿Cuándo te falla?¿De dónde debería leer el valor que provoca el nullpointer?¿Qué tipo de result usas en el struts.xml? ¿Has creado los getter y setter con mayúscula getFormulario? No puedes utilizar el redirect, ya que con ese creas una nueva request, y por lo tanto pierdes los datos.

Si pones trozos de código sería más fácil decirte qué te falta.

[Ruben82]

Justo me disponía a apagar el ordenador cuando se me dio por ver el foro, pero voy a hacer un esfuerzo.

Este es el formulario de la clase registroC.jsp

Código java:

<s:form action="saveC" method="post">   
        <s:hidden name="formulario" value="form1"/>
        <s:submit key="submit" cssClass="boton" />                  
</s:form>
 

Este es el codigo del action

Código java:

public class registroP  extends ActionSupport {
private String formulario;
 
public void validate(){
    if (getFormulario().equals("form1")){
        if (cBean.getMarca().length()!=0){
            addFieldError( "cBean.marca", "*" );
        }
                ...
          }
}
 
public void setFormulario(String form){ 
    this.formulario = form; 
}
    
public String getFormulario(){  
    return this.formulario; 
}
 

Y es aqui justo en el if(getFormulario()).... donde me da el error mencionado.

No encuentro el fallo.

Un saludo




Edito:Problema solucionado, mñn te cuento como arregle.

Un saludo

[Ruben82]

La culpa era del execute que como utilizo la misma action para validar todos los formulario, también hacía distinción entre distintos executes.

Solucionado pues, aunque ahora tengo otra duda que no se si preguntarla aquí o crear otro hilo nuevo.

Me explico:

Llegado a este punto necesitaría saber algo de conexión a base de datos, selección de campos e inserción bajo struts 2. ¿Alguna ayuda o manual que me pueda guiar?

Un saludo

Edito: hasta ahora en el validate solo compruebo que los campos están vacíos pero ¿sería posible comprobar también que además de contener "algo" que fuera correcto?, es decir que por ejemplo un número de teléfono contenga sólo números y 9 digitos

[Xerelo]

A ver, exagerando un poco, una Action es como una clase cualquiera de Java salvo en su relación con las JSP. Así que puedes validar lo que quieras y cómo quieras, la diferencia es que si no es correcto añades un error de forma que la JSP lo reciba.

Para validar teléfonos, email,... deberías usar expresiones regulares, aunque siempre puedes hacerlo con chapucillas varias.

Expresión regular para email

Código Javascript:

email = "[email protected]";
if (!email.matches(".+@.+\\..+")) {
 
addFieldError( "cBean.marca", "*" );
 
}
 

Con respecto a bases de datos, Struts no te soluciona nada, tendrás que usar JDBC como en cualquier clase de J2SE. Existen frameworks como Ibatis o Hibernate, que se combinan bien con Struts, pero en tu caso creo que sólo te complicaría el asunto.

Edito: Me olvidaba decirte que aunque el execute se ejecuta siempre (si no configuras lo contrario), tus action pueden llamar a un método/función concreta de tu Action.class, de forma que el execute sólo lo dejarías para elementos comunes.

<action name="login1" ... class="tuAction.class" method="funcion1">...</action>

<action name="login2" ... class="tuAction.class" method="funcion2">...</action>

Donde funcion1() y funcion2() son métodos de tuaction.class

[Ruben82]

Iniciado por Xerelo

A ver, exagerando un poco, una Action es como una clase cualquiera de Java salvo en su relación con las JSP. Así que puedes validar lo que quieras y cómo quieras, la diferencia es que si no es correcto añades un error de forma que la JSP lo reciba.

Para validar teléfonos, email,... deberías usar expresiones regulares, aunque siempre puedes hacerlo con chapucillas varias.

Expresión regular para email

Código Javascript:

email = "[email protected]";
if (!email.matches(".+@.+\\..+")) {
 
addFieldError( "cBean.marca", "*" );
 
}
 

Con respecto a bases de datos, Struts no te soluciona nada, tendrás que usar JDBC como en cualquier clase de J2SE. Existen frameworks como Ibatis o Hibernate, que se combinan bien con Struts, pero en tu caso creo que sólo te complicaría el asunto.

Edito: Me olvidaba decirte que aunque el execute se ejecuta siempre (si no configuras lo contrario), tus action pueden llamar a un método/función concreta de tu Action.class, de forma que el execute sólo lo dejarías para elementos comunes.

<action name="login1" ... class="tuAction.class" method="funcion1">...</action>

<action name="login2" ... class="tuAction.class" method="funcion2">...</action>

Donde funcion1() y funcion2() son métodos de tuaction.class

Ok. tomo nota.

La verdad es que estoy un poco verde, pero es que me han metido prisa para presentar un proyecto y no tengo ningun tiempo y me veo a marchas forzadas surgiendo problemas continuamente.

otra pregunta que te queria hacer:

en el tercer formulario que te comente cargo mediante javascript un formulario u otro dependiendo de los valores que tenga. te pongo el codigo pare entendernos mejor

Código java:

<div id="form">         
        <p>¿Opcion 1 o 2?
            <input type="button"  value="1" onclick="show1()"/>
            <input type="button"  value="2" onclick="show2()"/></p>
        </div>
        
        <div id="formulario1" style="visibility:hidden">
            <s:form action="save" method="post">                    
                <s:hidden name="formulario" value="primero"/>       
                <s:textfield key="nombre"/>  
                <s:textfield key="apellido"/>  
                <s:submit key="submit" cssClass="boton" />              
                <s:reset key="Reset" cssClass="boton" />          
            </s:form>
        </div>
            
        <div id="formulario2" style="visibility:hidden">
            <s:form action="save" method="post">                    
                <s:hidden name="formulario" value="segundo"/>       
                <s:textfield key="telefono"/>  
                <s:textfield key="direccion"/>  
                <s:submit key="submit" cssClass="boton" />              
                <s:reset key="Reset" cssClass="boton" />          
            </s:form>
        </div>
</div>
 

Código Javascript:

function show1(){
    var No = document.getElementById("formulario2");
    if (No.style.visibility=="visible"){
        No.style.visibility="hidden";
    }else if (document.getElementById("formulario1").style.visibility=="hidden"){
        No.style.visibility="visible";
    }
}
 
function show2(){
    var Si = document.getElementById("formulario1");
    if (Si.style.visibility=="visible"){
        Si.style.visibility="hidden";
    }else if (document.getElementById("formulario2").style.visibility=="hidden"){
        Si.style.visibility="visible";
    }    
}
 

El caso es que al pasarle esto al action y comprobar si los datos son correctos al volver a cargar otra vez esta pagina aparece por defecto ocultos y lo que quiero yo es que aparezca la que estaba cargada antes. Se te ocurre algo?


Un saludo y muchisimas gracias!!!!!

[Xerelo]

Habría otras formas de hacerlo, pero para no liarte y ya que te manejas con javascript

Solución rápida:

Crea un método en javascript que se cargue en el onload del body (o onready de jquery)

Código Javascript:

<script>
formu = '<s:property value="formulario">';
 
function inicio() {
if (formu=='formulario1'){
show1();}
else if (formu=='formulario2'){
show2();
}
}
</script>
 
<body onload="inicio();">
 

Las JSP se generan en el servidor, mientras que el javascript en el navegador, así cuando la JSP llega al navegador, tendrás que

formu = 'formulario1/formulario2/null' dependiendo de donde vengas.

[Ruben82]

Iniciado por Xerelo

Habría otras formas de hacerlo, pero para no liarte y ya que te manejas con javascript

Solución rápida:

Crea un método en javascript que se cargue en el onload del body (o onready de jquery)

Código Javascript:

<script>
formu = '<s:property value="formulario">';
 
function inicio() {
if (formu=='formulario1'){
show1();}
else if (formu=='formulario2'){
show2();
}
}
</script>
 
<body onload="inicio();">
 

Las JSP se generan en el servidor, mientras que el javascript en el navegador, así cuando la JSP llega al navegador, tendrás que

formu = 'formulario1/formulario2/null' dependiendo de donde vengas.

Entiendo lo que me dices pero no me funciona porque por mucho que le indiques al método onload() al seguir el curso normal cuando llega al div del formulario y ve la equita "hidden" lo oculta. He probado a quitarle esa etiqeta y hacerlo todo mediante javascript pero se ve que el html prevalece sobre él.

¿Alguna solución?


un saludo

[Xerelo]

No me había fijado en como hacías el show(), si quieres ocultar uno y mostar otro, ¿para qué haces una comprobación de en qué estado está cada uno?¿Lo que buscas no es que aparezca uno u otro?

Te recomiendo que en vez del visibility uses el display, así no queda el hueco del formulario no visible.

Código Javascript:

<head>
 
<script>
 
formu = '<s:property value="formulario">';
 
 
function Inicio() {
 
    if (formu=='formulario1')
        show1();
    else if (formu=='formulario2')
        show2();
}
 
function show1(){
 
    document.getElementById("formulario2").style.display="none";
    document.getElementById("formulario1").style.display="";
   
}
 
function show2(){
    document.getElementById("formulario1").style.display="none";
    document.getElementById("formulario2").style.display="";
}
</script>
 
</head>
<body onload="Inicio();">
 
<div id="formulario1" style="display:none">
 
<div id="formulario2" style="display:none">
 

Puedes probar que esto funciona en un html cualquiera, sin usar struts. Bueno, también cambiarías <s:property value="formulario"> por formulario1/formulario2/undefined para simular los regresos de la action.

Para usar javascript sobre elementos HTML, te recomiendo que leas un poco de jQuery, es muy sencillo y te ahorra trabajo.

[Ruben82]

Iniciado por Xerelo

No me había fijado en como hacías el show(), si quieres ocultar uno y mostar otro, ¿para qué haces una comprobación de en qué estado está cada uno?¿Lo que buscas no es que aparezca uno u otro?

Te recomiendo que en vez del visibility uses el display, así no queda el hueco del formulario no visible.

Código Javascript:

<head>
 
<script>
 
formu = '<s:property value="formulario">';
 
 
function Inicio() {
 
    if (formu=='formulario1')
        show1();
    else if (formu=='formulario2')
        show2();
}
 
function show1(){
 
    document.getElementById("formulario2").style.display="none";
    document.getElementById("formulario1").style.display="";
   
}
 
function show2(){
    document.getElementById("formulario1").style.display="none";
    document.getElementById("formulario2").style.display="";
}
</script>
 
</head>
<body onload="Inicio();">
 
<div id="formulario1" style="display:none">
 
<div id="formulario2" style="display:none">
 

Puedes probar que esto funciona en un html cualquiera, sin usar struts. Bueno, también cambiarías <s:property value="formulario"> por formulario1/formulario2/undefined para simular los regresos de la action.

Para usar javascript sobre elementos HTML, te recomiendo que leas un poco de jQuery, es muy sencillo y te ahorra trabajo.

Ok. Lo pruebo y te comento.

Por otro lado he estado intentando hacer alguna insercion en la base de datos pero no hace nada. No muestra error ni registro en la BBDD.....

Lo que he intentado hacer primeramente es desde el propio validate llamar a una funcion por ejemplo insertar() que mediante el objeto rellene la query y luego inserte pero como te dije no hizo nada asi que la segunda opcion fue algo parecido al editservice que explica en la página de struts para validar mediante XML. Lo que hago es crear una interfez y una clase que implemente esa interfaz y ahi un metodo saveObjeto, que al = que antes pilla el objeto y rellena la query de insercion, pero tampoco hace nada, así que viendo la hora que es y que mi despertador suena a las 6 pues me voy para cama.


Muchas gracias por todo.

Un saludo

[Xerelo]

Como ya te dije, el tema de BBDD funciona igual que si no usaras Struts, así que te recomiendo que pruebes las cosas siempre que puedas por separado.

Accede a la BBDD con una clase fuera del proyecto, con su main() y lo que haga falta. Cuando te funcione, traslada el código a Struts.

Y no lo hagas en el validate(), se llama así porque se usa para eso, para validar, y meter datos en la BBDD no lo es.

Te recuerdo que

<action name="guardarDatos" ... class="tuAction.class" method="saveObjeto">
...
</action>

pasará por el validate, el execute, y saveObjeto.

Captura los errores que te pueda dar el acceso a la BD, así sabrás que estás haciendo mal, si la conexión, la consulta, los parámetros...

[Ruben82]

Iniciado por Xerelo

Como ya te dije, el tema de BBDD funciona igual que si no usaras Struts, así que te recomiendo que pruebes las cosas siempre que puedas por separado.

Accede a la BBDD con una clase fuera del proyecto, con su main() y lo que haga falta. Cuando te funcione, traslada el código a Struts.

Y no lo hagas en el validate(), se llama así porque se usa para eso, para validar, y meter datos en la BBDD no lo es.

Te recuerdo que

<action name="guardarDatos" ... class="tuAction.class" method="saveObjeto">
...
</action>

pasará por el validate, el execute, y saveObjeto.

Captura los errores que te pueda dar el acceso a la BD, así sabrás que estás haciendo mal, si la conexión, la consulta, los parámetros...

Ahora que ya había conseguido insertar datos en la BD,jejeje, pero claro lo que tu dices tiene todo el sentido del mundo, voy a probarlo, porque hasta ahora estaba haciendo lo que dice en este pequeño tutorial.

http://www.roseindia.net/struts/struts2/struts-2-mysql.shtml

Pero probaré lo que me dices.

Por otro lado, lo que me indicaste arriba de cargar el formulario cuando al darle a continuar hace la comprobacion de campos vacíos y carga la misma página no me funciona. Supongo que como se cargan las propiedas en el body y luego se carga el div tiene éste último prioridad.
Seguiré probando cosas distintas a ver si me sale.

Muchas gracias por todo amigo

[Xerelo]

Hombre, sigue la máxima, si funciona no lo toques.

De todas formas, lo único que tendrías que hacer sería copiar lo que aparece en el ejemplo de RoseIndian, en su propio método. Ese método lo llamas con una action con lo de method que te expliqué. Simplemente tienes que tener cuidado y pensar que el validate() y el execute() se van a ejecutar también, así que si hay algo en ello que pudiera afectar a tu método, busca la manera de puentearlo con algún if.

Por otro lado, lo que me indicaste arriba de cargar el formulario cuando al darle a continuar hace la comprobacion de campos vacíos y carga la misma página no me funciona.

Igual no entiendo bien lo que quiers hacer. No debes usar la action sólo para validar, lo lógico es que valides antes de insertar los datos, pero en la misma action (recuerda que pasa a narices por el validate). En caso de error (si añades algún addActionError o similar en el validate) por defecto no ejecuta ni el execute ni el method que le hayas indicado a la action, volviendo a la JSP que la lanzó.

Yo haría el siguiente flujo

Cargar la JSP con los dos formularios

El usuario elige uno u otro, no puedes enviar a la vez los datos de dos formularios.

Rellena los datos y le da a enviar, donde tienes indicada una action con su method, pasa por el validate y comprueba que los campos son válidos, y sigue la ruta de execute y tu method, donde se supone que insertará los datos. Todo correcto o error, te devuelve a la JSP.

Si añades esta etiqueta en el JSP,puedes avisar de error en la validación

<s:if test="hasErrors()"><script>alert('Error en validacion');</script></s:if>

y con

<s:if test="hasMessage()"><script>alert('Insertado correctamente');</script></s:if>

y en tu method usando un addMessage(mensaje) puedes confirmar que todo ha ido birn.

Hay otras etiquetas para mostrar exactamente el mensaje que le pasas desde java, tanto para el error como para lo correcto.

Una vez hecho esto, el usuario rellena el segundo formulario, y sigue el mismo proceso, pero con otro method, y teniendo en cuenta que a su paso por validate no debe validar los mismos campos.

[Ruben82]

Iniciado por Xerelo

Hombre, sigue la máxima, si funciona no lo toques.

De todas formas, lo único que tendrías que hacer sería copiar lo que aparece en el ejemplo de RoseIndian, en su propio método. Ese método lo llamas con una action con lo de method que te expliqué. Simplemente tienes que tener cuidado y pensar que el validate() y el execute() se van a ejecutar también, así que si hay algo en ello que pudiera afectar a tu método, busca la manera de puentearlo con algún if.



Igual no entiendo bien lo que quiers hacer. No debes usar la action sólo para validar, lo lógico es que valides antes de insertar los datos, pero en la misma action (recuerda que pasa a narices por el validate). En caso de error (si añades algún addActionError o similar en el validate) por defecto no ejecuta ni el execute ni el method que le hayas indicado a la action, volviendo a la JSP que la lanzó.

Yo haría el siguiente flujo

Cargar la JSP con los dos formularios

El usuario elige uno u otro, no puedes enviar a la vez los datos de dos formularios.

Rellena los datos y le da a enviar, donde tienes indicada una action con su method, pasa por el validate y comprueba que los campos son válidos, y sigue la ruta de execute y tu method, donde se supone que insertará los datos. Todo correcto o error, te devuelve a la JSP.

Si añades esta etiqueta en el JSP,puedes avisar de error en la validación

<s:if test="hasErrors()"><script>alert('Error en validacion');</script></s:if>

y con

<s:if test="hasMessage()"><script>alert('Insertado correctamente');</script></s:if>

y en tu method usando un addMessage(mensaje) puedes confirmar que todo ha ido birn.

Hay otras etiquetas para mostrar exactamente el mensaje que le pasas desde java, tanto para el error como para lo correcto.

Una vez hecho esto, el usuario rellena el segundo formulario, y sigue el mismo proceso, pero con otro method, y teniendo en cuenta que a su paso por validate no debe validar los mismos campos.

Problema del formulario que se oculta solucionado.

Sigo tus consejos, buenos aportes para mi.

Muchas gracias y perdona el retraso en la contestacion. Estoy super liado

Un saludo