Erste Schritte mit SWING

[simlmf]

Hallo zusammen,
ich habe mich heut mal ein paar Stunden hingehockt und ne kleine MiniGUI "gecodet"
ich kenn mich da leider noch nicht aus mit Regeln und gutem Programmierstil, also wollte ich euch einfach mal meine Schnipsel posten. ausführbar sind sie und es funzt auch alles wie ich es will. nur wären verbesserungen gern gesehen.

gerade all diese JMenus und JMenuItems von Hand zu erzeugen sieht irgendwie seltsam aus.
naja. macht einfach mal n paar vernichtende kommentare dazu^^

mfg simon und vielen dank

package GUI;

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

public class GraphicalUserInterface extends JFrame implements ActionListener
{
	public void actionPerformed(ActionEvent e)
	{
		
	}
	
	public GraphicalUserInterface()
	{
		JFrame mainFrame = new JFrame("Kontaktverwaltung");		
		
		JMenuBar mainMenuBar = new JMenuBar();
		
		JMenu file = new JMenu ("File");
		JMenu edit = new JMenu ("Edit");
		JMenu search = new JMenu ("Search");
		JMenu view = new JMenu ("View");
		JMenu about = new JMenu ("About");
		
		JMenuItem newContact = new JMenuItem ("New Contact");
		JMenuItem save = new JMenuItem ("Save");
		JMenuItem close = new JMenuItem ("Close");
		JMenuItem editContact = new JMenuItem ("Edit Contact");
		JMenuItem deleteContact = new JMenuItem ("Delete Contact");
		JMenuItem searchContact = new JMenuItem ("Search Contact by");
		JMenuItem showDetails = new JMenuItem ("Show Details on/off");
		JMenuItem toggleRaster = new JMenuItem ("Toggle Raste on/Off");
		JMenuItem info = new JMenuItem ("Info about SIMON");
		
		mainFrame.setJMenuBar(mainMenuBar);
		mainFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		mainFrame.setSize(800, 600);
		mainFrame.setVisible(true);
		
		mainMenuBar.add(file);
		mainMenuBar.add(edit);
		mainMenuBar.add(search);
		mainMenuBar.add(view);
		mainMenuBar.add(about);
		
		edit.add(editContact);
		edit.add(deleteContact);
		search.add(searchContact);
		view.add(showDetails);
		view.add(toggleRaster);
		about.add(info);
		file.add(newContact);
		file.add(save);
		file.add(close);
		
		close.addActionListener(new ActionListener()
		{
			public void actionPerformed(ActionEvent e)
			{
				if(e.getActionCommand().equals("Close"))
				{
					System.out.println("Window Closed");
					dispose();
					System.exit(0);
				}
			}
		});
	}

	public static void main(String[] args)
	{
		GraphicalUserInterface frame = new GraphicalUserInterface();	
	}
}

 

[Illuvatar]

Also zum Menü - das musst du eben irgendwann zusammenbauen, und dafür braucht man eben eine Liste an Codezeilen die recht repetitiv sind, wird man kaum ändern können.

Ansonsten kommt es mir relativ gut vor. Coding Conventions sind eingehalten, Variablennamen in Englisch, alles gut lesbar. Eine kleinere Anmerkung, die ich jetzt einfach noch in den Absatz mitreinpacke: setVisible ist meistens besser, erst ganz am Schluss zu machen. Sonst kriegt der User erst die unfertige Oberfläche zu sehen, die dann noch verändert werden muss, und manche Änderungen danach können dann auch nicht mehr richtig dargestellt werden (such mal im Forum nach "validate", diese Probleme gibts oft )

Was ich dann noch sagen will: das Ganze scheint ja zumindest kein ganz kleines Programm mehr zu werden. Bei sowas solltest du dir vorher erstmal überlegen, was für Klassen du denn alles brauchst. Das ist mir persönlich zwar auch der am wenigsten liebe Teil, aber er hilft enorm... Von deinem Code her denke ich zumindest, du solltest da in der Gegend von den ActionListenern nochmal überlegen. Anonyme Listener... sind zwar komfortabel, aber nicht immer optimal Und warum implementiert deine Hauptklasse denn ActionListener? Schau auch mal nach dem Stichwort "MVC".

 

[SlaterB]

die Leerzeichen bei "new JMenu ("File");" vor der ( stören,
macht das dein Formatter?

in jedem Fall nicht konsequent: "new JFrame("Kontaktverwaltung");"

 

[Marco13]

Der Zweite Schritt könnte oder sollte vielleicht sein, das alles jetzt in eine Methode "createMenu" oder so zu packen :wink:

 

[Gast]

Und zu guter letzt würde ich das Programm explizit im Event Dispatcher Thread starten.

public static void main(String[] args){
   SwingUtilities.ionvokeLater(new Runnable(){
      public void run(){
         GraphicalUserInterface frame = new GraphicalUserInterface();
      }
   });
}

 

[tobbbbi]

Und zu guter letzt würde ich das Programm explizit im Event Dispatcher Thread starten.

public static void main(String[] args){
   SwingUtilities.ionvokeLater(new Runnable(){
      public void run(){
         GraphicalUserInterface frame = new GraphicalUserInterface();
      }
   });
}

Vielleicht könntest du bitte noch kurz dazu erklären, wozu man das braucht, ist mir bisher nämlich nicht klar geworden. Danke

 

[0001001]

Hanz einfach:

Wenn du eine Swing Anwendung hast, also eine mit GUI, dann läuft alles, was mit der GUI zu tun hat (Buttons zeichnen, Panels zeichnen, usw.) in einem dafür vorgesehenen Thread, dem Event Dispatcher Thread (EDT).

Durch den Aufruf oben stellst du sicher, dass die GUI Anwendung (das ist sie ja, da du JFrame erweiterst) in diesem EDT läuft. Das ist zwar auch ohne den Aufruf oben meist der Fall, kann aber u.U. auch mal nicht so sein und dann startet deine GUI nicht. Wildcard kann dir dazu glaub ich was erzählen =).

 

[simlmf]

SlaterB
BeitragVerfasst am: 15. 10. 2008, 19:06 Titel:
die Leerzeichen bei "new JMenu ("File");" vor der ( stören,
macht das dein Formatter?

in jedem Fall nicht konsequent: "new JFrame("Kontaktverwaltung");"

jo, da hab ich wohl was übersehen, tschuldige

Und zu guter letzt würde ich das Programm explizit im Event Dispatcher Thread starten.

public static void main(String[] args){
   SwingUtilities.ionvokeLater(new Runnable(){
      public void run(){
         GraphicalUserInterface frame = new GraphicalUserInterface();
      }
   });
}

vielen dank, das is durchaus ein guter einwand

Illuvatar
BeitragVerfasst am: 15. 10. 2008, 19:02 Titel:
Also zum Menü - das musst du eben irgendwann zusammenbauen, und dafür braucht man eben eine Liste an Codezeilen die recht repetitiv sind, wird man kaum ändern können.

Ansonsten kommt es mir relativ gut vor. Coding Conventions sind eingehalten, Variablennamen in Englisch, alles gut lesbar. Eine kleinere Anmerkung, die ich jetzt einfach noch in den Absatz mitreinpacke: setVisible ist meistens besser, erst ganz am Schluss zu machen. Sonst kriegt der User erst die unfertige Oberfläche zu sehen, die dann noch verändert werden muss, und manche Änderungen danach können dann auch nicht mehr richtig dargestellt werden (such mal im Forum nach "validate", diese Probleme gibts oft icon_wink.gif)

Was ich dann noch sagen will: das Ganze scheint ja zumindest kein ganz kleines Programm mehr zu werden. Bei sowas solltest du dir vorher erstmal überlegen, was für Klassen du denn alles brauchst. Das ist mir persönlich zwar auch der am wenigsten liebe Teil, aber er hilft enorm... Von deinem Code her denke ich zumindest, du solltest da in der Gegend von den ActionListenern nochmal überlegen. Anonyme Listener... sind zwar komfortabel, aber nicht immer optimal icon_smile.gif Und warum implementiert deine Hauptklasse denn ActionListener? Schau auch mal nach dem Stichwort "MVC".

vielen dank für die umfangreiche antwort. was das eventhandling angeht hab ich nur mal bissel rumprobiert. hab da ja noch net viel erfahrung.
was das mit den klassen angeht, das stimmt schon somal das projekt wirklich nicht ganz klein wird
die änderungen wurden soweit vorgenommen.


Vielen dank für eure mithilfe. ich werde soblad ich neuigkeiten habe, updaten.
mfg simon