Mehrere Probleme mit Java's GUI

[Gubbel]

Ich habe mehrere kleine Probleme mit Javas GUI (Hauptsächlich Swing).
Ich bin C# gewöhnt und gehe nun die ersten Schritte mit Java GUI und ich muss ehrlich sagen, ich komm nicht drauf klar

Hier erstmal mein Programmcode

package gui;

import javax.swing.*;

import java.awt.*;

public class CodecGUI extends JFrame{
	
	Container c;
	
	public CodecGUI(){
		c = getContentPane();
		
		JLabel losungswort1, losungswort2, codewort1, codewort2;
		JTextArea wortVerschluesselt, wortEntschluesselt;
		JTextField wortLosung1, wortLosung2;
		JButton encrypt, decrypt;
		
		JPanel main = new JPanel(new BorderLayout()), top = new JPanel(new GridLayout(4,2,10,0)), bottom = new JPanel();
		
		codewort1 = new JLabel("Klartext: ");
		wortEntschluesselt = new JTextArea(3,100);
		
		codewort2 = new JLabel("Verschlüsselt: ");
		wortVerschluesselt = new JTextArea(3,100);
		
		losungswort1 = new JLabel("Losungswort 1");
		wortLosung1 = new JTextField(15);
		losungswort2 = new JLabel("Losungswort 2");
		wortLosung2 = new JTextField(15);
		
		encrypt = new JButton("Encrypt");
		decrypt = new JButton("Decrypt");
		
		
		top.add(losungswort1);
		top.add(losungswort2);
		top.add(wortLosung1);
		top.add(wortLosung2);
		top.add(codewort1);
		top.add(codewort2);
		top.add(wortVerschluesselt);
		top.add(wortEntschluesselt);
		
		bottom.add(encrypt);
		bottom.add(decrypt);
		
		main.add(top, BorderLayout.NORTH);
		main.add(bottom, BorderLayout.SOUTH);
		main.add(bottom, BorderLayout.SOUTH);
		c.add(main);

	}

public static void main(String[] args) {
		
		CodecGUI fenster = new CodecGUI();
		
		fenster.setSize(500,300);
		fenster.setTitle("Geheim ! || Eyes Only");
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		fenster.setVisible(true);
		

	}

}

Das Ergebnis sieht nicht so toll aus ... Ich möchte das TextField einzeilig halten und den Text mit dem ich dann abreiten muss in mehreren Spalten als TextArea halten, da das Losungswort, was zur verschlüsselung dient nur aus einem Wort besteht, wohingegen der "Klartext" mehrere Zeilen umfassen kann.

Außerdem macht meine TextArea keinen automatischen Zeilnumbruch. Wie Formatiere ich also die TextArea damit dies geschieht ? :bahnhof:

Danke für die Hilfe schonmal

 

[vanny]

#setLineWrap(true); könnte helfen.

an sonsten schauste mal in die API !?:rtfm:

 

[Gubbel]

Ich sehe grade das ich es ins famlsche forum gepostet habe ... tschuldigung

werds im richtigen nochmal einstellen, in der hoffnung das mir dort geholfen wird

 

[Marco13]

Ansonsten... allgemein sollte man den Gültigkeitsbereich von Variablen möglichst klein halten, und nicht alles oben in die Methode klatschen. Man sollte auch nur die als Fields einfügen, die man wirklich braucht (vermutlich wirst du die TextFields- und Areas als Fields brauchen). JLabels braucht man nur dann als Lokale Variable oder Field, wenn man den Text noch ändern will.

Ich persönlich finde es übersichtlicher, den Code so zu gruppieren, dass er "Topologieerhaltend" auf das GUI mappt - also nicht oben einen Haufen Objekte erstellen, und unten mit 20 "add"-Aufrufen zusammenklatschen.

Am konkreten Beispiel kann man mit einem weiteren Panel das Layout noch "schöner" machen, in dem Sinne, dass die TextAreas den überschüssigen Platz beim vergrößern des Fensters bekommen.

Als Alternativvorschlag also

import javax.swing.*;
 
import java.awt.*;
 
public class CodecGUI extends JFrame
{
    public CodecGUI()
    {
        JPanel main = new JPanel(new BorderLayout());
        
        JPanel top = new JPanel(new GridLayout(2,2,10,0)); 
        
        top.add(new JLabel("Losungswort 1"));
        top.add(new JLabel("Losungswort 2"));

        JTextField wortLosung1 = new JTextField(15);
        top.add(wortLosung1);

        JTextField wortLosung2 = new JTextField(15);
        top.add(wortLosung2);

        main.add(top, BorderLayout.NORTH);
        
        JPanel center = new JPanel(new BorderLayout());
        
        JPanel centerTop = new JPanel(new GridLayout(1,2,5,5));
        centerTop.add(new JLabel("Klartext: "));
        centerTop.add(new JLabel("Verschlüsselt: "));
        center.add(centerTop, BorderLayout.NORTH);

        JPanel centerCenter = new JPanel(new GridLayout(1,2,5,5));
        JTextArea wortVerschluesselt = new JTextArea(3,100);
        centerCenter.add(wortVerschluesselt);
        JTextArea wortEntschluesselt = new JTextArea(3,100);
        centerCenter.add(wortEntschluesselt);
        center.add(centerCenter, BorderLayout.CENTER);

        main.add(center, BorderLayout.CENTER);
        
        JPanel bottom = new JPanel();

        JButton encrypt = new JButton("Encrypt");
        JButton decrypt = new JButton("Decrypt");
        bottom.add(encrypt);
        bottom.add(decrypt);
        
        main.add(bottom, BorderLayout.SOUTH);

        getContentPane().add(main);
 
    }
 
    public static void main(String[] args) 
    {
        // Swing components should be created
        // on the Event Dispatch Thread
        SwingUtilities.invokeLater(new Runnable()
        {
            @Override
            public void run()
            {
                createAndShowGUI();
            }
        });
    }
 
    private static void createAndShowGUI()
    {
        CodecGUI fenster = new CodecGUI();
        fenster.setSize(500,300);
        fenster.setTitle("Geheim ! || Eyes Only");
        fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        fenster.setVisible(true);
    }
}

 

[Gubbel]

Hat bestens geklappt. Unser Prof hat da so seine Eigenart wie er es gerne hätte. Ich werde mir den Quellcode mal genau anschauen und analysieren

danke !


[EDIT]Ich bekomme nun vom Eclipse Compiler folgende Warnung

"The serializable class CodecGUI does not declare a static final serialVersionUID field of type long"

Was soll ich mir darunter vorstellen ? habe noch ein wenig den Programmcode hin und hergeschoben und die Main Methode wieder in eine andere Klasse verfrachtet.[/EDIT]

 

[Marco13]

Die sollte vorher auch schon gekommen sein. Eine Websuche nach der Meldung sollte genug Ergebnisse bringen. Wenn nicht, nochmal bescheid sagen.

 

[Gubbel]

Problem behoben. Obwohl ich immer noch nicht genau verstehe warum diese GUI Klasse eine einzigartige "Serial" braucht