Fehlermeldung obwohl Variablen bereits deklariert sind?

[doenis9]

Hey Com.

ich bin gerade dabei Schere, Stein, Papier zu programmieren. Ich dachte daran, dass Benutzer 1 und Benutzer 2 über die verschiedenen Buttons ihren Wunschgegenstand auswählen können. Mit dem Klick auf den letzten Button soll dann if/else den Gewinner ermitteln. Nur leider kommt dabei eine Fehlermeldung. Wäre toll wenn mal jemand darüber schauen könnte und mir sagen würde, was dabei falsch ist 🙂

JButton btn1 = new JButton("Schere");
btn1.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
int spieler1 = 1;
txt1.setText("Gespeichert!");

}
});
btn1.setFont(new Font("Tempus Sans ITC", Font.PLAIN, 11));
btn1.setBounds(10, 150, 89, 23);
contentPane.add(btn1);

JButton btn2 = new JButton("Stein");
btn2.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
int spieler1 = 2;
txt1.setText("Gespeichert!");
}
});
btn2.setFont(new Font("Tempus Sans ITC", Font.PLAIN, 11));
btn2.setBounds(126, 150, 89, 23);
contentPane.add(btn2);


JButton btn3 = new JButton("Papier");
btn3.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
int spieler1 = 3;
txt1.setText("Gespeichert!");
}
});
btn3.setFont(new Font("Tempus Sans ITC", Font.PLAIN, 11));
btn3.setBounds(246, 150, 89, 23);
contentPane.add(btn3);



JLabel lblSpielerl = new JLabel("Spieler 2:");
lblSpielerl.setFont(new Font("Tempus Sans ITC", Font.BOLD, 12));
lblSpielerl.setBounds(10, 191, 89, 14);
contentPane.add(lblSpielerl);

JButton btn4 = new JButton("Schere");
btn4.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
int spieler2 = 1;
txt2.setText("Gespeichert!");

}
});
btn4.setFont(new Font("Tempus Sans ITC", Font.PLAIN, 11));
btn4.setBounds(10, 216, 89, 23);
contentPane.add(btn4);

JButton btn5 = new JButton("Stein");
btn5.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
int spieler2 = 2 ;
txt2.setText("Gespeichert!");
}
});
btn5.setFont(new Font("Tempus Sans ITC", Font.PLAIN, 11));
btn5.setBounds(126, 216, 89, 23);
contentPane.add(btn5);

JButton btn6 = new JButton("Papier");
btn6.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
int spieler2=3;
txt2.setText("Gespeichert!");
}
});
btn6.setFont(new Font("Tempus Sans ITC", Font.PLAIN, 11));
btn6.setBounds(246, 216, 89, 23);
contentPane.add(btn6);

txt3 = new JTextField();
txt3.setBounds(143, 333, 379, 20);
contentPane.add(txt3);
txt3.setColumns(10);

JButton btn7 = new JButton("Ausf\u00FChren");
btn7.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {

if (spieler1 == 1 & spieler2 == 1) {
txt3.setText("Unentschieden!");
}
}

 

[Cromewell]

Zum einen wäre die Fehlermeldung hilfreich und zum anderen die Nutzung von Code-Tags 😀

 

[Robat]

if (spieler1 == 1 & spieler2 == 1) {
txt3.setText("Unentschieden!");
}

In deinem Code ist nicht ersichtlich warum spieler1 und/oder spieler2 an dieser Stelle bekannt sein sollten.
Bis jetzt sind die spieler Variablen alles nur als lokale Variablen in den ActionListener definiert.

 

[Cromewell]

if (spieler1 == 1 & spieler2 == 1) {

Und es sollte && sein

 

[thecain]

Sollte, führt aber nicht zu einem Fehler

 

[doenis9]

In deinem Code ist nicht ersichtlich warum spieler1 und/oder spieler2 an dieser Stelle bekannt sein sollten.
Bis jetzt sind die spieler Variablen alles nur als lokale Variablen in den ActionListener definiert.

Das hört sich schonmal nach was an 😀 hast du mir vielleicht ein Tipp wie ich weiter machen kann?

 

[Robat]

Stichwort Globale Variable

 

[doenis9]

Stichwort Globale Variable

Okey vielen Dank für deine Hilfe, ich probiere es mal aus 🙂

 

[doenis9]

Stichwort Globale Variable

Ich habe es jetzt mit dem Code private static final int versucht, aber dann kommts zu einer Fehlermeldung.

JButton btn1 = new JButton("Schere");
btn1.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
static private int spieler1 = 1; -> Illegal modifier for parameter spieler1; only final is permitted
}
});

 

[Cromewell]

Du musst es als Klassenvariable deklarieren 😉
bzw., wenn du es nur innerhalb einer Methode brauchst als local.

public class Foo {
    private int ply1 = ...;

    //more code...
}

Bzw:

public class Foo {
    public static void main(String[] args){
        int ply1 = ...;

        //more code...
    }
}

Ich weiß ja nicht genau, wie du es brauchst (mit welchen Modifiern etc.)

 

[Robat]

static private int spieler1 = 1; -> Illegal modifier for parameter spieler1; only final is permitted

So legt man auch keine globale Variable an. Wo hast du das denn her?

 

[Steinschnüffler]

Ein

@Override

über dem ActionPerformed würde auch helfen...

 

[Robat]

über dem ActionPerformed würde auch helfen...

Was? 😀
Wer sagt dass man das schreiben muss?

 

[Steinschnüffler]

Was? 😀
Wer sagt dass man das schreiben muss?

Also bei mir wird die Methode sonst nicht als die ActionPerformed Methode vom ActionListener erkannt und wird deswegen bei einem Knopfdruck dann auch nicht ausgeführt

 

[thecain]

Das ist Quatsch, dann machst du sonst was falsch