Warum geht das nicht? if-else usw..... compilieren geht nicht -- HILFE!!!

[javalion]

Siehe Überschrift...

Kurz zum Inhalt: Ich versuche ein Programm zu erstellen, in das eine Nummer eingegeben werden kann und diese überprüft/verglichen wird mit Hilfe von if-else und einer internen Variabel "r = 123". Das Ausgabefeld gibt dann aus, ob die eingegebene Nummer "a" mit "r" richitg oder falsch ist.

Hier der Quellcode und darunter die Fehlermeldung:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;



public class codeschloss extends JFrame {

  private JLabel ueberschrift = new JLabel();
  private JLabel Jeingabe = new JLabel();
  private JButton check = new JButton();
  private JLabel Jausgabe = new JLabel();
  private JTextField ausgabe = new JTextField();
 
 
  public codeschloss(String title) {
   
    super(title);
    setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
    int frameWidth = 297;
    int frameHeight = 300;
    setSize(frameWidth, frameHeight);
    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2;
    setLocation(x, y);
    setResizable(false);
    Container cp = getContentPane();
    cp.setLayout(null);
   
   
    ueberschrift.setBounds(8, 8, 267, 41);
    ueberschrift.setText("Bitte Zahlencode eingeben!");
    ueberschrift.setOpaque(false);
    ueberschrift.setHorizontalTextPosition(SwingConstants.CENTER);
    ueberschrift.setHorizontalAlignment(SwingConstants.CENTER);
    ueberschrift.setFont(new Font("Calibri", Font.BOLD, 18));
    cp.add(ueberschrift);
    cp.setBackground(new Color(0xC0C0C0));
    Jeingabe.setBounds(8, 64, 51, 33);
    Jeingabe.setText("Eingabe:");
    Jeingabe.setBackground(Color.WHITE);
    Jeingabe.setOpaque(true);
    Jeingabe.setHorizontalAlignment(SwingConstants.CENTER);
    cp.add(Jeingabe);
    check.setBounds(8, 104, 265, 33);
    check.setText("C  H  E  C  K  !");
    check.setMargin(new Insets(2, 2, 2, 2));
    check.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        check_ActionPerformed(evt);
      }
    });
    check.setBackground(Color.RED);
    check.setForeground(Color.WHITE);
    check.setFont(new Font("Calibri", Font.BOLD + Font.ITALIC, 24));
    cp.add(check);
    Jausgabe.setBounds(8, 144, 57, 33);
    Jausgabe.setText("Ausgabe:");
    Jausgabe.setHorizontalAlignment(SwingConstants.CENTER);
    Jausgabe.setBackground(Color.WHITE);
    Jausgabe.setOpaque(true);
    cp.add(Jausgabe);
    ausgabe.setBounds(72, 144, 201, 33);
    ausgabe.setHorizontalAlignment(SwingConstants.CENTER);
    cp.add(ausgabe);
    eingabe.setHorizontalAlignment(SwingConstants.CENTER);

   
    setVisible(true);
  }
 
 
  public static void main(String[] args) {
    new codeschloss("codeschloss");
  }
 
  public void check_ActionPerformed(ActionEvent evt) {
    int a = eingabe.getInt();
    int r = 123;
   
    if (a==r) {
      ausgabe.setText("richtig!");
    } else {
      ausgabe.setText("falsch!");
    }
   
  }

 
}

Compiler-Fehler:

Compiliere "C:\Users\PC\Desktop\Codeschloss\codeschloss.java" mit Java-Compiler
codeschloss.java:74:5: error: cannot find symbol
    eingabe.setHorizontalAlignment(SwingConstants.CENTER);
    ^
  symbol:   variable eingabe
  location: class codeschloss
codeschloss.java:87:13: error: cannot find symbol
    int a = eingabe.getInt();
            ^
  symbol:   variable eingabe
  location: class codeschloss
2 errors

 

[Cromewell]

Du schreibst die ganze Zeit "Jeingabe" - du hast keine "eingabe" deklariert, soweit ich das sehe.


->>
EDIT: Ich glaube du hast dich oben beim Deklarieren verschrieben 😀

 

[KaffeeFan]

Kann es sein, dass du den Java-Editor benutzt?😕

int a = eingabe.getInt();

Was ist denn deiner Meinung nach "eingabe"?

Gruß
Luk

 

[thet1983]

Weil es "eingabe" nicht gibt

Upps zu spät

 

[VfL_Freak]

Moin,

um das zuvor gesagte mal zusammenzufassen:
Du solltest DRINGENST (!!!) über die Benennung deiner Variablen nachdenken!!

  private JLabel ueberschrift = new JLabel();  // das ist noch ok ;-)
  private JLabel Jeingabe = new JLabel();    // was soll hier das 'J' - zumal wenn du es dann in der gennaten Zeile 74 vergisst ??
  private JButton check = new JButton();     // das ist auch noch ok ;-)

  // 'Jausgabe' und 'ausgabe'?? Glaubst du, du weißt nach ein paar Wochen noch, was was ist ???
  private JLabel Jausgabe = new JLabel();
                                                                 
  private JTextField ausgabe = new JTextField();

Ist weiß, dass es vielfach verpönt ist, aber wenn es denn schon ein Präfix sein soll, dann nutze die Ungarische Notation
https://de.wikipedia.org/wiki/Ungarische_Notation

Gruß Klaus

 

[thet1983]

Ungarische Notation...?
Sorry aber das würde ich nicht machen.

Ich würde es so machen:

//in Swing
JLabel jlbName
JTextfied jtfName..

 

[JStein52]

Über Namen kann man lange "streiten" aber verschreiben kann man sich immer mal, egal welchen Namen man nimmt.

 

[VfL_Freak]

Moin,

aber verschreiben kann man sich immer mal, egal welchen Namen man nimmt

schon klar, aber dann sollte man es (nicht zuletzt aufgrund der Fehlermeldung) auch merken, oder ??
Daher die Hinweise .....
Gruß Klaus

 

[JStein52]

dann sollte man es (nicht zuletzt aufgrund der Fehlermeldung) auch merken

Ja klar. Stimmt schon. Wobei ich aber denke er hat sich noch nicht mal verschrieben sondern einfach vergessen analog zu "ausgabe" ein entsprechendes Textfeld für "eingabe" anzulegen.

 

[dzim]

Also die ungarische Notation würde ich nicht mal mit Lötkolben und Kneifzange anfassen wollen!
Ich denke, wichtig ist es erst einmal, der Konvetion zu folgen, Camel Case zu verwenden (klein, wenn Variable; gross, wenn Klasse). Der Rest ist Geschmackssache. Ich habe schon "labelName" genauso wie "nameLabel" verwendet. Ich finde es zum Beispiel ausgeschrieben meist einfacher lesbar, als abgekürzt. Wenn ich Obfuscation will, dann nutze ich ProGuard...

 

[RalleYTN]

Ungarische Notation...?
Sorry aber das würde ich nicht machen.

Ich würde es so machen:

//in Swing
JLabel jlbName
JTextfied jtfName..

ich schreibe immer alles aus.

JLabel nameLabel;
JTextField nameTextField;

Da kommt nie Verwirrung auf.
Sollte man aber nur machen wenn man mit ner IDE und Auto-Completition arbeitet.
Ansonsten wird das zu viel Schreibarbeit auf Dauer.