Swing ActionListener überschreibt einen Wert aus der Hauptklasse nicht

[PinkMuffin]

Hallo,
ich bin noch ein Java-Anfänger, daher weiß ich nicht 100%, ob mein Fehler überhaupt mit dem Layout zu tun hat, oder ob ich ein Problem mit der
Erreichbarkeit der Variablen habe..


Hier hab ich ein "Kartendeck" erzeugt (ich will mit dem CardLayout arbeiten, daher der Name). Es geht mir um die Variable "punkte".
Eigentlich müssten andere Klassen doch auf die Variable zugreifen können? (Ich habe es auch schon mit einem getter/setter probiert), jedoch bleibt die Variable aus mir unerklärlichen Gründen immer auf 0 eingestellt.



Ich erstelle hier ein paar Panels, die aber für die Frage nicht relevant sind, bis auf das Panel "ranking" und das dazugehörige rankingLabel.
Evtl noch das Frage-Panel, hier sind die Buttons, mit einem ActionListener, der eigentlich den Punktestand überschreiben soll, dass eben, wenn das ranking-Label aufgerufen wird, dort der aktuellste Punktestand angezeigt wird.



Hier ist jetzt mein ActionListener. Eigentlich kann er auf Variablen der Deck-Klasse zugreifen, er hat "card" ja auch daraus entnommen. Auf die Fragen-Klasse kann er prinzipiell auch zugreifen, er holt sich daraus die ArrayList "buttons.
Ich habe schon verschiedene Varianten probiert, die Ausgabe ist immer "Ihr Highscore ist: 0". Auch wenn ich das Ranking-Label erst im ActionListener beschriftet oder sogar erst dort geaddet habe. Ich hab vorsichtshalber auch mal "repaint()" ausprobiert (auch wenn ich als Anfänger noch nicht ganz sicher weiß, was diese Methode macht, geholfen hat sie auf jeden Fall nicht).
Also irgendwie kann mein ActionListener nicht meinen Punktestand überschreiben, das Label lässt sich nämlich im Listener verändern, es müsste also evtl an meinen modifiern liegen?

Ich wäre um jede Hilfe dankbar,
LG
Lena

EDIT: Die Bezeichnung "Hauptklasse" in meinem Fragetitel ist falsch. Ich meinte damit die Deck-Klasse. Meine Hauptklasse mit der main-Methode ist eine andere, die erstellt aber nur ein neues Objekt der Deck-Klasse.

 

[mrBrown]

Code bitte immer als Code direkt hier im Forum und Code-Tags ([code=java]//code...[/code]) und nicht als Screenshot!

 

[mrBrown]

Vermutlich hast du entweder zwei Instanzen von Deck (und eine davon wird angezeigt und in der anderen wird es geändert) oder zwei unabhängige Variablen die zufällig beide punkte heißen, aber sonst nichts miteinander zu tun haben.
Steht der Actionlistener direkt in der Deck-Klasse?

 

[PinkMuffin]

Okay, dann nochmal direkt als Code, hier ist die Deck-Klasse. Mein Listener steht direkt in der Deck-Klasse
(ich weiß, dass es suboptimal ist, dass auch noch die Klasse Fragen nicht ausgelagert ist, allerdings hatte ich sonst noch andere Probleme, die ich auch nicht wirklich anders zu lösen wusste). Ein Objekt der Deck-Klasse wird nur einmal in der Start-Klasse erzeugt, danach nicht mehr und ich bin jetzt noch einmal drüber gegangen, habe aber eigentlich nur eine Variable, die "punkte" heißt.

public class Deck
{
    JFrame frame;
    JPanel deck, login, fragePanel, ranking;
    CardLayout card;
    JButton highscore, restart, play, exit;
    int punkte=0;
    String nutzername;
    JLabel rankingLabel = new JLabel();

    public Deck()
    {
        frame = new JFrame("Swing - Quiz");
        frame.setLocationRelativeTo(null);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);


        deck = new JPanel();
        DerHandler handler = new DerHandler();

        //Buttons erzeugen
        play = new JButton("Play");

        play.addActionListener(handler);

        exit = new JButton("Exit");
        exit.addActionListener(new ActionListener()
        {
        @Override
        public void actionPerformed(ActionEvent e)
        {
            System.exit(0);
        }
        });

        highscore = new JButton("Highscore");
        highscore.addActionListener(handler);

        //An dieser Stelle werden die anderen Panels erzeugt
        login = new Login();
        login.add(exit);
        login.add(play);

        ranking = new JPanel();
        rankingLabel.setText(login.getNutzer() + "Ihr Highscore beträgt: " + punkte);
        ranking.add(rankingLabel);card = new CardLayout();

        fragePanel = new Fragen();

        //Panels werden dem CardLayout zugefügt
        deck.setLayout(card);
        deck.add(login, "Login");
        deck.add(fragePanel, "Fragen");
        deck.add(ranking, "Ranking");

        frame.add(deck);
        frame.setVisible(true);
        frame.pack();
    }


    public class Fragen extends JPanel
    {
        ArrayList<String> fragen;
        ArrayList<String> antwort;
        ArrayList<String> f1,f2,f3;
        JButton b1,b2,b3,b4;
        JLabel frage;
        JPanel br1, br2;
        ArrayList<JButton> buttons;
        JButton rAntwort;

        public Fragen()
        {
            this.setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
            frage = new JLabel();
            fragen = new ArrayList<String>();
            fragen.add("Wie viele Pflanzen stehen bei Sabrina?");
            fragen.add("Wie heißt der Hund von Ralf Weinmann?");
            fragen.add("Welche Farbe hat das cortility Logo (Außer schwarz)?");
            antwort = new ArrayList<String>();
            antwort.add("8");
            antwort.add("Zoey");
            antwort.add("magenta");

            f1 = new ArrayList<String>();
            f1.add("5");
            f1.add("Martha");
            f1.add("gelb");

            f2 = new ArrayList<String>();
            f2.add("3");
            f2.add("Marley");
            f2.add("blau");

            f3 = new ArrayList<String>();
            f3.add("6");
            f3.add("Bello");
            f3.add("Es ist nur schwarz.");

            b1 = new JButton();
            b2 = new JButton();
            b3 = new JButton();
            b4 = new JButton();

            buttons = new ArrayList<JButton>();
            buttons.add(b1);
            buttons.add(b2);
            buttons.add(b3);
            buttons.add(b4);
            zuordnen();
            br1 = new JPanel();
            br2 = new JPanel();

            this.add(frage);
            this.add(br1);
            this.add(br2);
            br1.add(b1);
            br1.add(b2);
            br2.add(b3);
            br2.add(b4);

            /*this.add(buttons.get(0));
            this.add(buttons.get(1));
            this.add(rAntwort);
            this.add(buttons.get(2));*/



        }


        public void zuordnen()
        {
            DerHandler handler = new DerHandler();
            int random = (int) (Math.random()*3);
            int frageRandom = (int) (Math.random()*3);
            String antwort1 = antwort.get(frageRandom);
            String falsch1 = f1.get(frageRandom);
            String falsch2 = f2.get(frageRandom);
            String falsch3 = f3.get(frageRandom);
            String frageTitel = fragen.get(frageRandom);

            frage.setText(frageTitel);

            rAntwort = buttons.get(random);
            buttons.remove(random);
            rAntwort.setText(antwort1);

            buttons.get(0).setText(falsch1);
            buttons.get(1).setText(falsch2);
            buttons.get(2).setText(falsch3);

            rAntwort.addActionListener(new ActionListener()
            {
                @Override
                public void actionPerformed(ActionEvent e) {
                    deck.remove(fragePanel);
                    fragePanel = new Fragen();
                    deck.add(fragePanel, "Fragen");
                    card.show(deck, "Fragen");
                    punkte++; //schon mit einem getter/setter ausprobiert, passiert das gleiche
                    
                }
            });


            for (int i=0; i<buttons.size(); i++)
            {
                buttons.get(i).addActionListener(new ActionListener()
                {
                    @Override
                    public void actionPerformed(ActionEvent e)
                    {

                        card.show(deck, "Ranking");

                    }
                });

            }

        }


    }


    public class DerHandler implements ActionListener
    {

        @Override
        public void actionPerformed(ActionEvent event)
        {
            if (event.getSource()== highscore)
            {

                card.show(deck, "Ranking");
            }
            else if (event.getSource()== play)
            {
                card.show(deck, "Fragen");
            }

        }
    }
}

 

[mrBrown]

Die Anzeige der Punkte wird an keiner Stelle aktualisiert

 

[PinkMuffin]

Die Anzeige der Punkte wird an keiner Stelle aktualisiert

Dankeschön. Ich hätte schwören können, dass ich das schon mal aktualisiert hatte, aber offensichtlich nicht, denn jetzt funktioniert es 😄