ItemListener, itemStateChanged

Hvorfor sætter du så ikke en seperat listener på hver af de to JComboBoxe ??

class test implements ActionListener
{
  public void test()
  {
    JComboBox 1 = new JComboBox();
    JComboBox 2 = new JComboBox();

    1.addActionListener( this );
    2.addActionListener( this );
  }

  public voic actionPerformed( Action Event e )
  {
    if ( e.getSource == 1 )
    {
    // den ene
    }
    elseif ( e.getSource == 2 )
    {
      // den anden
    }
  }
}

kun en skitse

Jeg er ikke helt med på hvad du mener -- her er koden for tilføjelse af ItemListener\'s

jcbMaxLaan.addItemListener(new java.awt.event.ItemListener() {
            public void itemStateChanged(ItemEvent e) {
                jcbMaxLaan_itemStateChanged(e);
            }
        });



jcbMaxKredit.addItemListener(new java.awt.event.ItemListener() {
            public void itemStateChanged(ItemEvent e) {
                jcbMaxKredit_itemStateChanged(e);
            }
        });

OK -- nu har jeg valg at bruge ItemListener, men jeg har jo i princippet gjort det samme som du foreslår

jcbMaxLaan.addItemListener(new java.awt.event.ItemListener() {
            public void itemStateChanged(ItemEvent maxlaan) {
                jcbMaxLaan_itemStateChanged(maxlaan);
            }
        });



jcbMaxKredit.addItemListener(new java.awt.event.ItemListener() {
            public void itemStateChanged(ItemEvent maxkredit) {
                jcbMaxKredit_itemStateChanged(maxkredit);
            }
        });


Har du forsøgt at ændre på ItemEvent \"variabel\" navn ??

Så vidt jeg kan se har de forskelligt scope, så hvordan skulle det kunne hjælpe?

Har du noget kode, som man kunne teste med?

Det kan desværre ikke lade sig gøre :(

Det vil normalt ikke være muligt det du beskriver, så jeg bliver lidt i tvivl om du ved et uheld har fået forkerte referencer det forkerte sted hen (ala jcbMaxKredit = jcbMaxLaan), for så ville det være det samme.

Tag getSource() fra dit event, og se om de kommer fra samme komponent.

Jeg har testet ved at tilføje følgende:

    void jcbMaxKredit_itemStateChanged(ItemEvent e) {       
        Object s = e.getSource();
        System.out.println(\"max KREDIT event source = \"+s);
        System.out.println((kc++)+\"-------------------------------\");
       
        model.getAftale().setProdukttype((ProduktType) e.getItem());
    }

    void jcbMaxLaan_itemStateChanged(ItemEvent e) {
        Object s = e.getSource();
        System.out.println(\"max LÅN    event source = \"+s);
        System.out.println((lc++)+\"-------------------------------\");

        model.getAftale().setProdukttype2((ProduktType) e.getItem());
    }

variablene kc og lc er counters.
Jeg kan se at når jeg ændre i combo boxen jcbMaxKredit bliver
jcbMaxKredit_itemStateChanged kaldt og kun den.

når jeg ændre i combo boxen jcbMaxLaan bliver
jcbMaxLaan_itemStateChanged kaldt og kun den.

Dvs. hver combo box genererer et specifikt event.
Jeg skriver mit event object ud, og kan se at det er
det rigtige item i den rigtige combo box der bliver valgt.

Hvis du så stadig får sat både produkttype og produkttype2 hvergang, må dine problemer ligge i dit aftaleobject. (Jeg mener, metoderne ovenfor siger du selv er ok).

private BizObjReference produkttype  = new BizObjReference(this, \"produkttype\");
private BizObjReference produkttype2 = new BizObjReference(this, \"produkttype\");


public void setProdukttype(ProduktType type) {
    try{
        produkttype.setBO(type);
    } catch(BusinessRuleException e) {
        e.printStackTrace();
    }
}
   
public void setProdukttype2(ProduktType type) {
    try {
        produkttype2.setBO(type);
    } catch(BusinessRuleException e) {
        e.printStackTrace();
    }
}

Her er de vaske ægte set metoder -- de burde virke.

Jamen, har du kigget på hvor de metoder bliver kaldt fra?

Du kan evt. (hvis du er tilfreds med system.out) lave en:

Thread.currentThread().dumpStack();

Så får du en exception udskrevet på skærmen med et stack trace, om hvilke metoder, der har kaldt hvilke. De burde jo i princippet være isoleret, i forhold til dit oprindelige problem, men.

metoderne bliver kaldt fra disse to, der implementere itemStateChanged
for de to forskellige combo box\'e

void jcbMaxKredit_itemStateChanged(ItemEvent e) {       
    model.getAftale().setProdukttype((ProduktType) e.getItem());
}

void jcbMaxLaan_itemStateChanged(ItemEvent e) {
    model.getAftale().setProdukttype2((ProduktType) e.getItem());
}

Jeg tror imidlertid jeg dropper denne implementering og forsøger mig
med actionPerformed --

Jamen, prøvede du at kigge på et stacktrace, fordi med de to kodestumper, du har leveret er der ingen fejl...?

Og et spørgsmål i forbindelse med dine produktklasser. Jeg kender selvfølgelig ikke BizObjReference objekter, men du opretter to på nøjagtig samme måde. Hvis det er en wrapper over for et andet lag, kan man spekulere på om de to new metoder rent faktisk relaterer til det samme på det bagved liggende system. Er det to unikke objekter med separat lager, og hvis de refererer til bagvedliggende objekter er disse så fælles eller en ren deep copy.

Jeg kan godt foranledes til at tro noget omkring det sidste. Et tilsvarende problem har jeg nemlig haft tidligere.

logical>> Nu må jeg rødmende erkende at jeg ikke er helt stiv i det
med stack trace :( jeg burde nok have lært at gå før jeg begyndte at
løbe, men sådan skulle det ikke være.

En white box test har afsløret at metoderne bliver kaldt som de skal,
data bliver hentet fra GUI\'en som de skal, de havner nogen gange rigtigt
i datamodellen, men bliver altid gemt forkert i databasen. Derfor blev
jeg nødt til at undersøge JFrameworket (desværre nåede jeg ikke at se
dit svar 10:38)

Da min datamodel nedarver fra en BizObj model, der både
er i stand til at reflektere simple attributter, men også en til mange
relationer vha. containere, var det selvfølgelig oplagt at undersøge om
frameworket \'havde noget imod\' at to referencer blev \'hardcoded\' istedet
for at blive repræsenteret som en container.

Det viste sig at når man gør det fortolker JFrameworket det fejlagtigt,
og sætter den første reference til at pege på sidste references \'lager rum\'.
Hvis man vil have mere end en reference til den samme type objekt er man
altså nødt til at bruge en container.



Grundet \'samtidigheds problemer\' i vores ideer håber jeg det er ok vi deler?

Fint for mig, jeg har bare det, at jeg hader problemer, og jeg vil løse dem.

Det har givet mig en del øvelse med tiden, så det plus stor viden om java giver ret gode muligheder for at identificere problemer.

De største \"hurdles\", jeg arbejder med, er fejl i application servere og race conditions i flertrådssystemer. Der er det lidt sværere at finde fejl :-)