Struts ActionForms / Mehrere auf einer Seite

[RaoulDuke]

Hallo,

ich bin auf folgendes Problem gestossen:

Ich habe eine ActionForm Bean, beispielsweise mit dem Namen "SearchForm". Diese Bean die Felder "von", "bis", "text". Nun habe ich 2 Actions die beide dieses Form benutzen, z.B. SucheA und SucheB.

Auf einer Webseite habe ich 2 Formulare, einmal für Action SucheA und einmal für SucheB.

Wenn jetzt aber bei der Form Validierung Fehler auftreten und die Seite erneut angezeigt wird schaffe ich es nicht mehr die Errors dem richtigen Formular zuzuordnen. Da in beiden Forms die Felder gleich heissen zeig er die Errors dann an den Feldern beider Forms an.

Die Errors scheinen nicht an das Form gebunden zu sein sondern allgemein im Request Context zu stecken.

Kann mir da jemand helfen?

 

[RaoulDuke]

Hier mal das ganze als Code Beispiel:

Die Konfiguration der Actions in der Struts Config:

        <action input="/testforms.jsp" 
                name="SearchForm" 
                path="/SearchB" 
                scope="request" 
                type="app.struts.actions.SearchB"
                attribute="SearchFormB"
                >
            <forward name="success" path="/ok.jsp"/>
        </action>
        
        <action input="/testforms.jsp" 
                name="SearchForm" 
                path="/SearchA" 
                scope="request" 
                type="app.struts.actions.SearchA"
                attribute="SearchFormA"
                >
            <forward name="success" path="/ok.jsp"/>
        </action>

Hier die beiden Formulare auf einer JSP Seite:

    <html:form action="SearchA">
        
<html:text property="von"/><html:errors property="von"/>
        
<html:text property="bis"/><html:errors property="bis"/>
        
<html:text property="text"/><html:errors property="text"/>
        
<html:submit/>
    </html:form>

    
    <html:form action="SearchB">
        
<html:text property="von"/><html:errors property="von"/>
        
<html:text property="bis"/><html:errors property="bis"/>
        
<html:text property="text"/><html:errors property="text"/>
        
<html:submit/>
    </html:form>

Das SearchForm hat in der Validate Methode fest eingetragen das es immer das Feld "text" anmeckert.

Der Output sieht dann so aus:

   <form name="SearchFormA" method="post" action="/spielwiese/do/SearchA">
        
<input type="text" name="von" value="testtext">
        
<input type="text" name="bis" value="">

        
<input type="text" name="text" value=""><UL><LI>Text ungueltig</LI>[/list]
        
<input type="submit" value="Submit">
    </form>

    
    <form name="SearchFormB" method="post" action="/spielwiese/do/SearchB">
        
<input type="text" name="von" value="">
        
<input type="text" name="bis" value="">
        
<input type="text" name="text" value=""><UL><LI>Text ungueltig</LI>[/list]

        
<input type="submit" value="Submit">
    </form>

 

[HLX]

Was spricht dagegen die Parameter/Properties des zweiten Formulars umzubenennen?

 

[RaoulDuke]

Was spricht dagegen die Parameter/Properties des zweiten Formulars umzubenennen?

Dann müsste ich 2 unterschiedliche FormBean Klassen haben, die genau das gleiche tun, die gleichen Felder haben, nur anders benannt. Nennt man das dann Wiederverwendbarkeit?

Ich hab es bisher genau so gelöst, weil ich mir nicht anders zu helfen wusste, aber das ist einfach eine schlechte Lösung.

 

[SlaterB]

soweit ich das verstehe, sind die Errors losgelöst vom allem anderen als Request-Attribut org.apache.struts.action.ERROR abgelegt,
dieses Attribut enthält grob gesagt eine List von Elementen property + error-message,

du hast also wirklich nur die Information "bei property 'text' Feher 'xy'",
weder mit einem Form, einer JSP oder sonstwas verknüpft,

Vorteil:
volle Kontrolle,
kann unabhängig von Struts/ Forms in jeder Action an jeder Stelle gesetzt werden,
kann überall in der JSP ausgegeben werden, oder in einem Framework von JSPs auf einer speziellen Fehlerteilseite, die gar nix von dem Form weiß, oder oder

Nachteil: siehe dein Programm

die Anzeige ist bei dir recht manuell, du gibst selber an wie die Fehler heißen sollen,
das gibt dir die Möglichkeit, z.B. 2x ><html:errors property="text"/>
durch
<html:errors property="a.text"/>
und
<html:errors property="b.text"/>
zu ersetzen,

dann müsstest du aber im Form-Objekt auch manuell die Fehler setzen,
tust du das oder macht Struts dies automatisch?

um im Form zu wissen, ob a oder b vorliegt könnte entweder kompliziert die aktuelle URL erfragt werden,
oder etwas einfacher im Formular ein hidden-field mit dem Typ des Form-Objektes abgelegt werden

im Idealfall müsste für die Form-Definition in der JSP nur ein weiteres Feld gesetzt werden:

set typ= a oder b;
<html:form action="SearchA oder b">
// hidden field typ

<html:text property="von"/><html:errors property="typ.von"/>

<html:text property="bis"/><html:errors property="typ.bis"/>

<html:text property="text"/><html:errors property="typ.text"/>

<html:submit/>
</html:form>


das erfordert wohl etwas höhere JSP-Sprachen,
sagt dir
<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c"%>
<c:set var="x" value="${y}" />
was?
mit solchen Befehlen ginge es vielleicht

----------

vielleicht auch in den Actions ein Flag im Request setzen und dann in den JSPs erkennen, ob im Moment Form a oder Form b verwendet wird,
und entsprechend manuell nur bei einem der Forms die Fehler anzeigen,

ist in deinem Beispiel bisschen schlecht, da viele einzelne Zeilen zu ändern wären,
mit einem etwas modularem Aufbau (Form-Code in einem Tag oder einer separaten JSP erzeugen, welches 2x aufgerufen wird) ginge es leichter,

aber da istwohl das letztere, der modulare Aufbau, das größere Hinderhis

----------

ein anderer Ansatz wäre, ganz auf die Struts-Errors zu verzichten,
und selber zu bauen, dann könnte man sie z.B. direkt in den Form-Objeken ablegen und könnte sie besser zuordnen

aber wohl noch komplizierter, zumal man dann auch die Darstellung selber machen muss
-------

in jedem Fall viel Eigenarbeit, das generelle Verfahren versagt da anscheinend

 

[RaoulDuke]

Also die Idee mit einem Hidden Field ist vermutlich eine gute Idee. Dann können die Felder der Bean so bleiben wie sie sind, es wird nur ein Feld Typ hinzugefügt. Und bei den Errors wird dann der Wert des Hidden Feldes als Prefix benutzt.

<html:form action="SearchA">
<html:hidden property="typ" value="searcha."/>

<html:text property="von"/><html:errors property="searcha.von"/>

<html:text property="bis"/><html:errors property="searcha.bis"/>

<html:text property="text"/><html:errors property="searcha.text"/>

<html:submit/>
</html:form>

Das hält den Aufwand deutlich in Grenzen.

Das die Errors unabhängig von der Bean im Request Kontext gespeichert werden habe ich auch gemerkt, es wunderte mich nur etwas, weil es doch eigentlich viel besser wäre in der Oberklasse ActionForm einen Container für Errors zu haben. Dann wären die Errors korrekt an das entsprechende Form gebunden, und da man den Namen der Form Bean kennt kommt man auch von Überall an diese Werte heran.


--------

mit einem etwas modularem Aufbau (Form-Code in einem Tag oder einer separaten JSP erzeugen, welches 2x aufgerufen wird) ginge es leichter,

Wenn ich das Form durch einen Custom Tag oder ein separates JSP erzeuge wird alles noch umständlicher, schliesslich wird es in HTML eingebettet und soll evtl. auf jeder Seite anders ausssehen.

Es wäre perfekt wenn die Errors in der FormBean wären und ich html:errors den Namen übergeben konnte unter dem die Form Bean im Context gespeichert ist.