Combobox Struts Java

[arwen39]

Hallo an die Experten,
ich bin blutige Anfängerin und habe jetzt da ein Problem mit dem Combobox (passiert in der Action). Die Combox wird aus der Datenbank gefüllt, aber es wird falsch vorselektiert. Was mache ich da falsch?
Danke schon mal im voraus!!

Hier der Struts-Code:

</td>
          	  <TD class="<%= styleClass %>">
                  <html:select name="row" property="status" value="<%= row.getStatuskurz() %>">
                    <html:optionsCollection name="statuskurz" value="key" label="value"/>
                  </html:select>
              </td>

und hier der Code aus der Action:

  public void prepareStatusKurzDropDown(HttpServletRequest request) {

    TreeMap statuskurz = new TreeMap();

    ArrayList statusKurzListe = (ArrayList) comboListenServiceImpl.selectStatusListe();
    for (int i = 0 ; i < statusKurzListe.size(); i++)
      statuskurz.put(i, ((StatusListeOutput) statusKurzListe.get(i)).getStatusBezeichnung());

    request.getSession().setAttribute("statuskurz", statuskurz);
  }

 

[SlaterB]

die Vorbelegung soll
<%= row.getStatuskurz() %>
sein, nehme ich an?

und was ist nun deine Frage? was row ist steht nirgendwo,
schon gar nicht was row.getStatuskurz() ist
und noch weniger was daran falsch ist
und noch viel weniger was nun wirklich dahin soll?

soll das was mit
request.getSession().setAttribute("statuskurz", statuskurz);
zu tun haben?

 

[arwen39]

Vielleicht drücke ich mich falsch aus. Wie gesagt, bin Anfängerin.

In der Jsp (struts) ist die Collection definiert. row ist die Zeile, wo die Collection angesprochen wird.

In der Action wird dann für dann die ComboBox wird komplett aus der Datenbank gefüllt. Für jede zeile soll aber schon der richtige Status vorselektiert werden. Tut es aber nicht. Es wird immer um einen Index höher selektiert.
Tabelle Status
idStatus StatusBezeichnung
1 ok
2 fehlerhaft
3 in Bearbeitung

Wenn die Zeile die Bezeichnung fehlerhaft, sollte dann auch die ComboBox auf fehlerhaft selektiert sein. Ist es aber nicht. Es ist dann auf in Bearbeitung selektiert. In StatusKurz ist die Bezeichnung.


Durch request.getSession().setAttribute("statuskurz", statuskurz) wird es dann am Frontend gesetzt.

Wie gesagt, bin Anfängerin.

 

[SlaterB]

> In der Jsp
[..]
>In der Action wird dann

eine JSP kommt nie vor einer Action in der Verarbeitung ran?!
oder geht es um den nächsten Request?

> Durch request.getSession().setAttribute("statuskurz", statuskurz) wird es dann am Frontend gesetzt.

das passiert ja in der Action, was ist dann das Frontend? nicht die JSP?
oder kommt die JSP also doch nach der Action wie es richtig ist?

ein möglichst eindeutige Beschreibung von Problemen und Vorgängen ist wichtig,
gerade als Anfänger(in)
------------

der Rest deines Postings ist nahezu ohne Belang,
immerhin gibst du nun erstmals die Information das es wohl irgendwie um mehrere Zeilen geht,

> Es wird immer um einen Index höher selektiert.

falls es irgendwo um Zahlen/ Indexe geht, gibt es allgemein in Programmiersprachen das Problem,
dass manche Zählungen (z.B. der Felder eines Arrays/ einer Liste) mit 0 beginnen statt mit 1,

insofern ist durchaus häufiger mal 'minus 1' nicht verkehrt,
falls von einer Darstellung '1 bis 3' nach '0 bis 2' gewechselt werden muss,

keine Ahnung ob das mit deinem Problem zu tun hat


----------

warum ein falscher Wert vorselektiert ist, ist genauso schleierhaft wie vorher,

ich habe schon gesagt, das es wohl was mit
value="<%= row.getStatuskurz() %>
zu tun hat,

was dieses row.getStatuskurz() ist, erklärst du aber nicht
(Ansätze wie ' row ist die Zeile, wo die Collection angesprochen wird. ' überlese ich dabei freundlicherweise),
welche Art von Rat möchtest du nun?

 

[arwen39]

ich kann dir gerne per mail mein problem schildern und den ganzen code dazufügen. ich bein kein frontend-entwickler, sondern mache eher die businessschicht. leider sehe ich deine email-adresse nicht. hab dir aber per forum dir eine nachricht geschickt, damit du meine email-adresse siehst.

 

[SlaterB]

kam irgendwie nicht an, slaterb@gmx.de wenn du meinst dass das dann besser geht,
aber ich bin ja eignetlich nicht dazu da, deinen Code zu ändern wenn du nicht mal das Problem beschreiben kannst?

lasse am besten erstmal die ganze Tabelle, Selectbox usw. alles weg,

dann mache eine simple Ausgabe der drei Werte
und dazu noch des ausgewählten Wertes in irgendeine JSP,
du hast ja noch nicht mal gesagt ob das Strings oder Zahlen oder was auch immer ist

lass auf der Action-Seite alles was mit DB usw. zu tun hat weg,
mach einfach eine Collection mit 3 Beispielswerte von welchen Datentyp auch immer und den ausgewählten Wert,
von letzten war wie gesagt bisher noch nix zu sehen,

 

[arwen39]

Hallo SlaterB,
sorry, daß ich mich erst jetzt melde. Komisch, daß die Mail nicht angekommen ist, aber egal. Ich habe mich halt kurz gefaßt, da es sonst einen "Roman" daraus würde. Ich selbst habe die Action nicht geschrieben. Hab das "geerbt" bekommen und bin jetzt halt dabei die Fehler zu fixen bzw. das Fehlende einzufügen. Die Action ist ziemlich wild durcheinander codiert und man muß halt erstmal in die Logik eindenken, was der Entwickler da produziert hat. Zu Refactern habe ich keine Zeit dafür, da ich ziemlich unter Termindruck stehe. Ok, lange Rede kurzer Sinn. Ich kann dir gerne die Info's usw. schicken. Wenn nicht, auch gut und ich danke dir für deine rasche Antworten.

 

[SlaterB]

ich dachte du meinst du hättest mir eine Forum-PM geschickt,

nach Emails hatte ich gar nicht geschaut, nun aber schon,
ja die Email ist heute morgen natürlich angekommen

also sende mir gerne soviel Code wie du möchtest, ist aber fraglich, ob mir das weiterhilft,

-------

andersrum wäre einfaches Testen meiner Erfahrung nach fast immer ein sehr effektiver Weg zum Finden beliebiger Fehler,


also nochmal weil ich mich so gerne Reden lese:
schreibe dir eine ganz neue Action (eine TestAction mit tst.jsp kann man immer mal wieder brauchen)
die nur drei Elemente + ein ausgewähltes Element in den Request schreibt,
in der JSP eine simpelst mögliche Ausgabe:
for (Collection)
Wert: wert
}

Wert ausgewählt: ausgewählter Wert

nicht in irgendwelche Tabellen oder so, sondern einfach nur ausgeben,
und schaue dir an ob das so ist wie du willst oder nicht,

dann versuche mit diesen Test-Werten eine ComboBox zu bauen,
die endlich mal den richtigen Wert selektiert hat,

erst wenn du das einmal testweise geschafft hast,
dann kann man sich höheren Anwendungen zuwenden