Opdatering af GUI

tror du skal have fat i observer pattern ..

http://www.patterndepot.com/put/8/observer.pdf

Muligvis behøver du ikek selv kode det helt fra scratch.

Hvis du bruger JList/JTable og du har en referance til den bagvedliggende
data struktur og du kan kalde fire-et-eller-andet, så skulle den være hjemme.

Det sidste fik jeg ikke helt fat i

Inden jeg kaster mig ud i en lang forklaring: bruger du JList/JTable
i main vinduet ?

Næsten - Jeg bruger List fra AWT'en, men hvis det andet gør det lettere så...

Med AWT List skal du kalde remove metoden på den.

Typisk vil du vel kalde constructor til din persondata frame
med en referance til hoved frame, så kan du i persondata frame
kalde en public metode i hoved frame som kalder remove på den
list.

Swing JList er ihvertfald anderledes end AWT List.

Med referance mener du så at jeg skal sende hovedvinduet med som parameter?

Ja.

Det jeg gør er som følger:
I mit personData vindue har jeg en actionListener på en knap
ActionListener Slet = new ActionListener ()  {
        public void actionPerformed(ActionEvent  e)  {
            int t = list1.getSelectedIndex();
            hv.gh.sletSpiller(t);
            hv.gh.save();
            list1.remove(t);
            hv.list1.remove(t);
        }
    };
list1.remove(t) fjerner den fra personData listen, og min mening er da at hv.list1.remove(t) skal fjerne den fra hovedvinduets liste som desværre hedder det samme - men nåh ja det er jo ligegyldigt

Ok så ikke lige dit "ja"...men men men hvordan sender jeg et vindue med som parameter?

Hvis hv er dit hoved vindue og list1 er listen så bliver den vel fjernet, så
skal du bare have hoved vinduet gentegnet.

hv.repaint() måske ?

hv er rigtigt mit hovedvindue, men i mit personData vindue starter jeg med at lave Hovedvindue hv = new Hovedvindue(). Det betyder jo at det er denne instans der bliver repaintet ikke sandt og ikke den der kører i forvejen

hv.repaint() repainter hv ikke current frame.  repaint() må repainte current.

Ikke forstået

hv.repaint() bør da repiant hoved vinduet.

Men det er meget muligt at jeg ikke har forstået hele logikken.

Det gør den ikke - Det jeg mener er at hv jo er en referance til en instans variabel jeg har lavet øverts i personData vinduet som vist herunder

public class PersonData extends JFrame {
   
    Hovedvindue hv = new Hovedvindue();
   
  List list1 = new List();
  JButton rediger = new JButton();
  JButton slet = new JButton();
osv.

senere kommer så
ActionListener Slet = new ActionListener ()  {
        public void actionPerformed(ActionEvent  e)  {
            int t = list1.getSelectedIndex();
            hv.gh.sletSpiller(t);
            hv.gh.save();
            list1.remove(t);
            hv.list1.remove(t);
        }
    };
og det er her jeg mener at hv jo refererer til min instans variabel og ikke det hovedvindue der i forvejen er åben

Det er korrekt.

Så du skal nok have noget a la:

public class PersonData extends JFrame {
 
    Hovedvindue hv;

    public PersonData(HovedVindue hv) {
        this.h = hv;
    }

og i hoved vinduet:

  PersonData pd = new PersonData(this);

Det giver desværre følgende compilerfejl
cannot resolve symbol - constucter PersonData (<anonymous java.awt.event.ActionListener>)

Jeg skal måske lige fortælle at PersonData vinduet bliver kaldt i en ActionListener

Jeg antog at:
  PersonData pd = new PersonData(this);
blev udført i hoved JFrame'en selv.


Men du har en seperat action listener klasse ?

Så må du jo have hoved vinduer over i dens constructor så den
kan kalde PersonData med den.

Om jeg har en seperat actionListener klasse mener jeg ikke. Mine actionListener er lavet efter min instansvariabler på følgende måde:
ActionListener PersonData = new ActionListener ()  {
        public void actionPerformed(ActionEvent  e)  {
            PersonData personData  = new PersonData(hv);
            personData.setSize(410,480);
            personData.setVisible(true);
        }
    };

Så hvor vil du have jeg skal putte Hovedvindue ind i den?

Tag dig ikke af hv i new PersonData(hv) har blot glemt at fjerne det - cut and paste fejl

Du har faktisk en anonym klasse.

Ja men hvor skal jeg putte Hovedvindue hv ind????

Hm.

Godt spørgsmål.

Har du overvejet at snyde ?

PersonData personData  = new PersonData(this);
personData.setVisible(false);

udenfor action listener og så:

personData.setSize(410,480);
personData.setVisible(true);

indeni ?

Nej det har jeg ikke overvejet, men det tror jeg lige jeg gør, men det er jo ikke den bedste løsning

Næ.

Men jeg kan ikke lige finde ud af hvordan man finder owner instansen
af outer klassen i en instans af en anonym inner class.

Den med at snyde kan jeg faktisk ikke få til at virke. Compilerfejl - får <identifier> expected når jeg laver
PersonData personData  = new PersonData(this);
personData.setVisible(false);

udenfor actionListeneren - fatter jeg faktisk ikke helt

Kan du lige poste et kode fragment ? 5 linier før og 5 linier efter ?

Af "snydekoden" eller hva???

public class Hovedvindue extends JFrame //implements Observer
{

  public static final GemHent gh = new GemHent();

  JLabel spillere = new JLabel();
  JLabel antalBaner = new JLabel();
  List list1 = new List();
  JComboBox comboBaner = new JComboBox();
  //snip
    JButton personData = new JButton();
  PersonData personData = new PersonData(this);
  personData.setVisible(false);
 
  ActionListener PersonData = new ActionListener ()  {
        public void actionPerformed(ActionEvent  e)  {
            //PersonData personData  = new PersonData();
            personData.setSize(410,480);
            personData.setVisible(true);
        }
    };

//snip betyder bare at jeg har pillet noget ud

Om igen har to personData, men det hjælper ikke - har prøvet

personData.setVisible(false);

skal være inden i en metode !

Put den f.eks. ind  HovedVindue constructor.

Det gør desværre det at den så ikke kan ses i actionListeneren

Ikke begge linier:

PersonData personData = new PersonData(this);
personData.setVisible(false);

kun den nederste !

Perfekt - nu kører det pointene er dine - men inden du får dem. Du skrev tidligere at det jo er at snyde og at du ville se om du kunne finde ud af hvordan man finder owner instansen
af outer klassen i en instans af en anonym inner class. Mener du stadig det??? :o)

så vil jeg ligge et svar

:-)

Jeg har læst en del og der synes ikke at være en pæn måde at gøre det på.

HovedVindue xthis; // instans medlem
...
xthis = this; // HovedVinsue constructor
...
PersonData personData = new PersonData(xthis); // action listener

er ikke meget pænere.

Ja ok -betyder det at du ikke finder ud af det andet?

Det pæneste vil være at droppe den anonyme indre klasse og lave
en ganske normal klasse med en constructor som tager et HovedVindue
som argument og gemmer det i et instans medlem, som så kan bruges
af actionPerformed.

Ok - tusind tak for hjælpen. Det var sgu fedt :o)