Klammer Setzung Problem

[experto2000]

Hallo,

Könnt ihr mir bitte helfen irgendwie finde ich mein flasche Klammer Setzungen:

//package dumb;
 
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
 
public class game { // Erster Buchstabe einse Klassennamens sollte groß geschrieben sein
    
    public static void main(String args[]) {
        final JFrame oMainWindow = new JFrame("World of War"); // muss "final" sein, damit es im ActionListener benutzt werden kann.
        oMainWindow.setSize(500, 200);
        oMainWindow.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        final JLabel hallo = new JLabel("World of War");
        hallo.setHorizontalAlignment(JLabel.CENTER);
        hallo.setVerticalAlignment(JLabel.TOP);     
        hallo.setFont(new Font("Flubber", Font.BOLD + Font.ITALIC, 60));
        // hallo.setVisible(true);   braucht nicht extra aufgerufen werden
        oMainWindow.add(hallo,BorderLayout.NORTH);
        
        final JButton login = new JButton("Zum Login");
        login.setPreferredSize(new Dimension(17, 17));
        login.setBorderPainted(false);
        oMainWindow.add(login,BorderLayout.CENTER);
        login.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent a) {
                login.setVisible(false);
		login.remove(oMainWindow);
                final JComboBox Server = new JComboBox();
                Server.addItem("S1 Neptun");
                Server.addItem("S2 Erde");
                oMainWindow.add(Server);
		final JButton Weiter = new JButton("Weiter");
		oMainWindow.add(Weiter);
                oMainWindow.add(Server,BorderLayout.CENTER);
                oMainWindow.add(Weiter,BorderLayout.SOUTH); 
		Weiter.addActionListener(new ActionListener() {
		    public void actionPerformed(ActionEvent a) {
			Server.setVisible(false);
			Weiter.setVisible(false);
			hallo.setVisible(false);
			String serverwahl = (String)Server.getSelectedItem ();
			if(serverwahl == "S1 Neptun")
			{
			    JLabel z = new JLabel("Server 1 Neptun");
			    oMainWindow.add(z);
			    final JTextField username = new JTextField("Username",30);
			    final JPasswordField passwort = new JPasswordField("Passwort",30);
			    final JButton fertig = new JButton("Login");
			    JPanel panel = new JPanel();
			    panel.setLayout(new BoxLayout(panel, BoxLayout.Y_AXIS));
			    oMainWindow.add(panel);
			    oMainWindow.add(panel,BorderLayout.SOUTH);
			    oMainWindow.add(z,BorderLayout.NORTH);
			    z.setHorizontalAlignment(JLabel.CENTER);
			    z.setVerticalAlignment(JLabel.TOP); 
			    z.setFont(new Font("Flubber", Font.BOLD + Font.ITALIC, 60));
			    panel.add(username);
			    panel.add(passwort);
			    panel.add(fertig);
			 }
			 else
			 {
			    final JLabel y = new JLabel("Server 2 Erde");
			    oMainWindow.add(y);
			    final JTextField username = new JTextField("Username",30);
			    final JPasswordField passwort = new JPasswordField("Passwort",30);
			    final JButton fertig = new JButton("Login");
			    final JPanel panel = new JPanel();
			    panel.setLayout(new BoxLayout(panel, BoxLayout.Y_AXIS));
			    //panel.setSize(1280,80);
			    oMainWindow.add(y,BorderLayout.NORTH);
			    y.setHorizontalAlignment(JLabel.CENTER);
			    y.setVerticalAlignment(JLabel.TOP); 
			    oMainWindow.add(panel);
			    oMainWindow.add(panel,BorderLayout.SOUTH);
			    panel.add(username);
			    panel.add(passwort);
			    panel.add(fertig);
			    y.setFont(new Font("Flubber", Font.BOLD + Font.ITALIC, 60));
			    fertig.addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent g) {
				    
				    if(richtig == 1)
				    {
					username.setVisible(false);
					passwort.setVisible(false);
					fertig.setVisible(false);
					panel.setVisible(false);
					y.setVisible(false);
				    }
				    {
					JOptionPane fehler = JOptionPane.showMessageDialog(null, "Login felgeschlagen.Bitte Versuche es Erneut", "Error", JOptionPane.ERROR_MESSAGE);
				    }
			      }
			});
			oMainWindow.setSize(1280, 768);
		    }
		}
            });
        }
	oMainWindow.add(login,BorderLayout.CENTER);
        oMainWindow.setVisible(true); // sollte am Schluss aufgerufen werden
    });
}

 

[ARadauer]

           Server.addItem("S2 Erde");
                oMainWindow.add(Server);
            }
            });
        final JButton Weiter = new JButton("Weiter");
        oMainWindow.add(Weiter);

ähmn anonyme inner class oder?

} //methode zu
} //inner classe zu
); //addActionListener Methode zu...

 

[experto2000]

was meinste damit
es geht alles nur irgenwas stimmt mit der Klammer setzung nicht weil als ich das mit der MsgBox hinzugefügt habe ging das net mehr kann mir bitte einer helfen.
Fehler:

game.java:101: <identifier> expected                                              
        oMainWindow.add(login,BorderLayout.CENTER);                               
                       ^                                                          
game.java:101: <identifier> expected                                              
        oMainWindow.add(login,BorderLayout.CENTER);                               
                             ^                                                    
game.java:101: <identifier> expected                                              
        oMainWindow.add(login,BorderLayout.CENTER);                               
                                                 ^                                
game.java:102: <identifier> expected                                              
        oMainWindow.setVisible(true); // sollte am Schluss aufgerufen werden      
                              ^                                                   
game.java:102: illegal start of type
        oMainWindow.setVisible(true); // sollte am Schluss aufgerufen werden
                               ^
game.java:104: reached end of file while parsing
}▒
 ^
6 errors

 

[Aske]

Zeile 99: });}
Zeile 100: });

Zumindest würde es dann kompilieren, wenn Du vorher noch den Boolean "richtig" deklarierst und Zeile 92 nach
JOptionPane.showMessageDialog(null, "Login felgeschlagen.Bitte Versuche es Erneut", "Error", JOptionPane.ERROR_MESSAGE); änderst.

Habs mal mit Eclipse formatiert, sieht übel aus oO

package dumb;

//package dumb;

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

public class game { // Erster Buchstabe einse Klassennamens sollte groß
					// geschrieben sein

	public static void main(String args[]) {
		final JFrame oMainWindow = new JFrame("World of War"); // muss "final"
																// sein, damit
																// es im
																// ActionListener
																// benutzt
																// werden kann.
		oMainWindow.setSize(500, 200);
		oMainWindow.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
		final JLabel hallo = new JLabel("World of War");
		hallo.setHorizontalAlignment(JLabel.CENTER);
		hallo.setVerticalAlignment(JLabel.TOP);
		hallo.setFont(new Font("Flubber", Font.BOLD + Font.ITALIC, 60));
		// hallo.setVisible(true); braucht nicht extra aufgerufen werden
		oMainWindow.add(hallo, BorderLayout.NORTH);

		final JButton login = new JButton("Zum Login");
		login.setPreferredSize(new Dimension(17, 17));
		login.setBorderPainted(false);
		oMainWindow.add(login, BorderLayout.CENTER);
		login.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent a) {
				login.setVisible(false);
				login.remove(oMainWindow);
				final JComboBox Server = new JComboBox();
				Server.addItem("S1 Neptun");
				Server.addItem("S2 Erde");
				oMainWindow.add(Server);

				final JButton Weiter = new JButton("Weiter");
				oMainWindow.add(Weiter);
				oMainWindow.add(Server, BorderLayout.CENTER);
				oMainWindow.add(Weiter, BorderLayout.SOUTH);

				Weiter.addActionListener(new ActionListener() {
					public void actionPerformed(ActionEvent a) {
						Server.setVisible(false);
						Weiter.setVisible(false);
						hallo.setVisible(false);
						String serverwahl = (String) Server.getSelectedItem();
						if (serverwahl == "S1 Neptun") {
							JLabel z = new JLabel("Server 1 Neptun");
							oMainWindow.add(z);
							final JTextField username = new JTextField(
									"Username", 30);
							final JPasswordField passwort = new JPasswordField(
									"Passwort", 30);
							final JButton fertig = new JButton("Login");
							JPanel panel = new JPanel();
							panel.setLayout(new BoxLayout(panel,
									BoxLayout.Y_AXIS));
							oMainWindow.add(panel);
							oMainWindow.add(panel, BorderLayout.SOUTH);
							oMainWindow.add(z, BorderLayout.NORTH);
							z.setHorizontalAlignment(JLabel.CENTER);
							z.setVerticalAlignment(JLabel.TOP);
							z.setFont(new Font("Flubber", Font.BOLD
									+ Font.ITALIC, 60));
							panel.add(username);
							panel.add(passwort);
							panel.add(fertig);
						} else {
							final JLabel y = new JLabel("Server 2 Erde");
							oMainWindow.add(y);
							final JTextField username = new JTextField(
									"Username", 30);
							final JPasswordField passwort = new JPasswordField(
									"Passwort", 30);
							final JButton fertig = new JButton("Login");
							final JPanel panel = new JPanel();
							panel.setLayout(new BoxLayout(panel,
									BoxLayout.Y_AXIS));
							// panel.setSize(1280,80);
							oMainWindow.add(y, BorderLayout.NORTH);
							y.setHorizontalAlignment(JLabel.CENTER);
							y.setVerticalAlignment(JLabel.TOP);
							oMainWindow.add(panel);
							oMainWindow.add(panel, BorderLayout.SOUTH);
							panel.add(username);
							panel.add(passwort);
							panel.add(fertig);
							y.setFont(new Font("Flubber", Font.BOLD
									+ Font.ITALIC, 60));
							fertig.addActionListener(new ActionListener() {
								public void actionPerformed(ActionEvent g) {

									if (richtig) {
										username.setVisible(false);
										passwort.setVisible(false);
										fertig.setVisible(false);
										panel.setVisible(false);
										y.setVisible(false);
									}
									{
										JOptionPane
												.showMessageDialog(
														null,
														"Login felgeschlagen.Bitte Versuche es Erneut",
														"Error",
														JOptionPane.ERROR_MESSAGE);
									}
								}
							});
							oMainWindow.setSize(1280, 768);
						}
					}
				});
			}
		});
		oMainWindow.add(login, BorderLayout.CENTER);
		oMainWindow.setVisible(true); // sollte am Schluss aufgerufen werden
	};
}

 

[experto2000]

Warum
Geht aber immer noch nicht bei der if(richtig)

 

[Aske]

ja, weil es keine Variable gibt, die "richtig" heisst. Du mußt noch "int richtig;" oben hinschreiben und den auch irgendwo auf 1 setzen.

 

[experto2000]

ok
aber warum siehts übel aus

 

[Aske]

Mit "übel" meinte ich nur, daß da sehr viel ineinander verschachtelt ist . Die Listener hätte ich eventuell ausgelagert, für die Lesbarkeit. Aber macht ja nix, wenn Du das alles noch verstehst.

Den Namen der Java Klasse könntest Du noch groß schreiben, also "Game"...das ist so der Standard in Java.

 

[experto2000]

Aber dann muss ich die Datei auch groß schreiben sonst gehts nicht.
Noch ne Frage ich habe gegooglet aber nicht egfunden wie man ein Inhalt einer PasswordField in MD5 verwandelt.

 

[Aske]

Ist ja auch latte, Hauptsache es klappt.

 

[experto2000]

Bitte siehe mein Edit nochmal an

 

[Aske]

Nicht mehr heute Abend, sry

 

[experto2000]

wie schick mir mindestens ein Link xD

 

[experto2000]

Hallo,
Habe jetzt mein Code ergänzt und es kommt ein Fehler allerdings kappier ich in nicht:

game.java:97: cannot find symbol
symbol: variable richtig
                                    if (richtig=="ja") {
                                        ^
Note: game.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
1 error

ich habe den Code um
[JAVA=93]
String pass = passwort.getText();
if(pass=="abc"){
String richtig = "ja";
}
[/code]
ergänzt

 

[diggaa1984]

du musst du Strings mit der equals-Methode prüfen sonst kommt was anderes aus als du denkst

edit:

if(pass=="abc"){
    String richtig = "ja"; //nur innerhalb des if-blocks bekannt
}

=>>

String richtig = "";
if(pass=="abc"){
    richtig = "ja"; //nur innerhalb des if-blocks bekannt
}

if ("ja".equals(richtig))
    fooo();

//------------
if (richtig) {
} else
   //falsch ... warum nimmst nicht gleich ein boolean? die bezeichnung der Variable lässt 
    //vermuten das auch ein boolean ausreicht mit seinem wertebereich von true und false

 

[experto2000]

So der Ausschnitt
[JAVA=92]
public void actionPerformed(ActionEvent g) {
String pass = passwort.getText();
if(pass=="abc"){
String richtig = "ja";
}
if (richtig=="ja") {
username.setVisible(false);
passwort.setVisible(false);
fertig.setVisible(false);
panel.setVisible(false);
y.setVisible(false);
}
{
JOptionPane
.showMessageDialog(
null,
"Login felgeschlagen.Bitte Versuche es Erneut",
"Error",
JOptionPane.ERROR_MESSAGE);
}
[/code]

 

[diggaa1984]

hab nochma editiert bei mir, hatte was übersehen !!

 

[Murray]

1. Die lokale Variable richtig ist nur innerhalb des Scopes sichtbar, in dem sie deklariert ist (also grob gesagt bis zur nächsten schließenden geschweiften Klammer. Wenn Du die Variable außerhalb dieses Scopes brauchst, musst Du sie auch außerhalb deklarieren
Bsp:

String richtig = null;
if ( /* ... */) {
  richtig = "xyz";
}

2. Wenn man Strings mit "==" vergleicht, muss man schon sehr genau wissen, was man tut. In mindestens 99,9% der Fälle ist man besser beraten, equals zu verwenden
3. Wenn Du speichern willst, ob irgendeine Bedingung erfüllt ist, dann wäre eine boolean-Variable vermutlich besser geeignet als ein String.

String pass = passwort.getText();
boolean richtig ="abc".equals( pass);
if ( richtig) {
 /* ... */
}

 

[Aske]

Aber dann muss ich die Datei auch groß schreiben sonst gehts nicht.
Noch ne Frage ich habe gegooglet aber nicht egfunden wie man ein Inhalt einer PasswordField in MD5 verwandelt.

Du holst Dir den Wert aus dem PasswordField mit "getPassword", ich hab jetzt kein javadoc hier, aber da sollte ein byte array zurückkommen.
Dann machst Du folgendes damit:

MessageDigest md5 = MessageDigest.getInstance("MD5");
md5.reset();
md5.update(passwordfield.getPassword());
byte[] result = md5.digest();

/* Hexadezimalen Wert in StringBuffer packen, oder was auch immer Du brauchst*/
StringBuffer hexString = new StringBuffer();
for (int i=0; i<result.length; i++) {
    hexString.append(Integer.toHexString(0xFF & result[i]));
}