Ordnen des Programms

[Kontraproduktiv]

nachdem ich mein letztes programm aufgegeben habe, weil ich einfach die Übersicht verloren habe (einfach alles in main() gequetscht), versuche ich jetzt erneut, aber diesmal mit Struktur. Das ist aber gar nicht so einfach, weil ich immer nur tutorials gelesen habe, in denen nie langer quelltext sondern immer nur codeschnipsel vorkamen. Eine Erklährung wie man nun mit langem Quelltext hantiert gab es dementsprechend nicht. Mein (erstes) Problem:

package projekt2;
import javax.swing.*;

public class Main {

public static menu(){
    JMenuBar menubar = new JMenuBar();
    JMenu menu = new JMenu("Hilfe");
    JMenuItem about = new JMenuItem("Über...");
    
    menubar.add(menu);
    menu.add(about);
    return menubar;
}

    public static void showwindow(){
JFrame frame = new JFrame("Hallo");
frame.setSize(200, 200);
frame.setVisible(true);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
JPanel panel = new JPanel();
frame.add(panel);
frame.setJMenuBar(menu());
    }


public static void main(String[] args) {
       showwindow();
}

Ich versuche die methode menu in die methode showwindow zu integrieren, bekomme aber die ganze zeit fehler. In der Java-Doku steht es aber so.

Fehler:

-neben public menu() steht: invalid method declaration; return type required
-neben return menubar; steht: cannot return a value from method whose result type is void
-neben frame.setJMenuBar(menu()); steht: void type not allowed here

 

[André Uhres]

Versuch's mit "result type", etwa so: public static JMenuBar menu(){...}

 

[JanHH]

Jede Funktion benötigt einen Rückgabewert. Der Typ für "keinen Rückgabewert" ist void (=null und nichtig). Eine Funktion kann nur einen Wert zurückgeben, wenn sie nicht den Rückgabetyp "void" hat. Wenn eine Funktion bspws. einen String zurückgeben soll, muss sie als Rückgabetyp "String" haben (ist doch logisch, oder)?

 

[André Uhres]

ist doch logisch, oder?

Ja, aber kontraproduktiv :lol:

 

[Kontraproduktiv]

ah sehr schön, JMenuBar hats gebracht, jetzt meine Frage: Wieso musste ich das davor setzten?

 

[Leroy42]

ah sehr schön, JMenuBar hats gebracht, jetzt meine Frage: Wieso musste ich das davor setzten?

Wie die anderen schon schrieben: Weil die Definition einer Methode den Rückgabewert benötigt;
oder eben void falls sie keinen Rückgabewert hat.

 

[Kontraproduktiv]

in diesem fall ist also JMenuBar der rückgabewert? Ich dachte der rückgabewert muss immer mittels return definiert werden.

 

[Leroy42]

in diesem fall ist also JMenuBar der rückgabewert?

Ja!

Ich dachte der rückgabewert muss immer mittels return definiert werden.

Nein, durch die return-Anweisung wird nichts definiert!:shock:

 

[Gast2]

in diesem fall ist also JMenuBar der rückgabewert? Ich dachte der rückgabewert muss immer mittels return definiert werden.

nein ... mittels return gibt Du den Rückgabewert zurück

<visibility> [static] <returnvalue> <methodname>([<parameter>])

in Deinem Fall

public static JMenuBar menu()

setzt Dich mal bitte mit dem Thema Methoden und Methoden-Signaturen auseinander ... das sind mit die wichtigsten Grundlagen

hand, mogel

 

[Kontraproduktiv]

ah ok, methoden sind bekannt, von methoden-signaturen höre ich zum ersten Mal...

 

[Kontraproduktiv]

Bevor ich jetzt nen neuen Thread aufmache:

ich habe mir ein (klitze)kleines Adessbuch geschrieben, bekomme aber 2 Dinge nicht so wirklich hin:

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

public class Main {

    public static JFrame aboutwindow(){
    JFrame aboutwindow = new JFrame("Über");
    JPanel aboutpanel = new JPanel();
    JLabel aboutlabel = new JLabel("Version 1.0");
    
    aboutwindow.setVisible(true);
    aboutwindow.setSize(100, 100);
    aboutwindow.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
    aboutwindow.add(aboutpanel);
    aboutpanel.add(aboutlabel);
    return aboutwindow;
}


    static class aboutaction implements ActionListener{
     public void actionPerformed(ActionEvent e){
        aboutwindow();
    }
     }


    static public JMenuBar menu(){
    JMenuBar menubar = new JMenuBar();
    JMenu menu = new JMenu("Hilfe");
    JMenuItem about = new JMenuItem("Über...");

    menubar.add(menu);
    menu.add(about);

    about.addActionListener(new aboutaction());


    return menubar;

     }


    static public JTextField Name(){
        JTextField Name = new JTextField("Name");
        return Name;
    }

    static public JTextField Nachname(){
        JTextField Nachname = new JTextField("Nachname");
        return Nachname;
    }

    static public JTextField Telefon(){
        JTextField Telefon = new JTextField("Telefonnummer");
        return Telefon;
    }



    static public JTextArea Kontakte(){
        JTextArea Kontakte= new JTextArea(5, 20);
        

        Kontakte.setEditable(false);
        Kontakte.setMargin(new Insets(5,5,5,5));

        return Kontakte;
        }

    static public JScrollPane KontakteScroll(){
        JScrollPane KontakteScroll = new JScrollPane(Kontakte());
        return KontakteScroll();
    }

    static public void hinzufuegen(){
       String name = Name().getText();
       String nachname = Nachname().getText();
       String telefon = Telefon().getText();
       String newline = "/n";

       Kontakte().append(name + newline);
       Kontakte().append(nachname + newline);
       Kontakte().append(telefon + newline);

       Name().selectAll();
       Nachname().selectAll();
       Telefon().selectAll();

       Kontakte().setCaretPosition(Kontakte().getDocument().getLength());
 }
  
    static class hinzuaction implements ActionListener{
        public void actionPerformed (ActionEvent e){
            hinzufuegen();
        }
        
    }

    static public JButton Hinzu(){
        JButton Hinzu = new JButton("Hinzufügen");

        Hinzu.addActionListener(new hinzuaction());
        return Hinzu;
    }

    static public void showwindow(){
JFrame frame = new JFrame("Hallo");
JPanel panel = new JPanel();

frame.setSize(400, 400);
frame.setVisible(true);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.add(panel);
frame.setJMenuBar(menu());
panel.add(Name());
panel.add(Nachname());
panel.add(Telefon());
panel.add(Hinzu());
panel.add(Kontakte());
panel.add(KontakteScroll());
    }



public static void main(String[] args) {
       showwindow();
}
}

1.) wenn ich auf den Button Hinzufügen drücke tut sich gar nichts. Sinn dieses Buttons ist es aber, den text in den feldern Name Nachname und Telefon in die TextArea zu kopieren.

2.) wie binde ich diese verdammte Scrollpane ein. Ich habe schon alles Versucht, alle Dokus gelesen, aber ich bekomms einfach nicht hin.

 

[Leroy42]

...
public static void main(String[] args) {
       showwindow();
}
}

;(

Was sollen denn deine ganzen static-Deklarationen?

Das ist ja grausam.

 

[Kontraproduktiv]

bekomme sonst immer fehlermeldungen^^, inwiefern grausam?

 

[Painii]

inwiefern grausam?

Nicht objektorientiert. Würde sich bei Sachen wie Adressbüchern aber anbieten.

 

[Kontraproduktiv]

Ich wüsste nicht, wie man das programm so unschreiben könnte, dass das static entfällt. Villeicht kann mir jemand anhand 1 methode ein beispiel geben, denn wenn ich nicht OOP programiere hat es lerntechnisch kaum sinn. Wie auch immer wir kommen vom eigentlichen problem ab.

 

[bygones]

im grunde alle statics weg und

public static void main(String[] args) {
       new Main().showwindow();
}

als Objekte erzeugen und mit denen Arbeiten

 

[Kontraproduktiv]

danke, hat gekappt, die beiden anderen Probleme haben aber immer noch Bestand

 

[Kontraproduktiv]

weis keiner lösungen auf meine Fragen?

 

[Landei]

Zeig doch bitte mal den umgemodelten Code, das macht es einfacher!

 

[Kontraproduktiv]

2 Probleme:

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

public class Main {

    public JFrame aboutwindow(){
    JFrame aboutwindow = new JFrame("Über");
    JPanel aboutpanel = new JPanel();
    JLabel aboutlabel = new JLabel("Version 1.0");
    
    aboutwindow.setVisible(true);
    aboutwindow.setSize(100, 100);
    aboutwindow.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
    aboutwindow.add(aboutpanel);
    aboutpanel.add(aboutlabel);
    return aboutwindow;
}


     class aboutaction implements ActionListener{
     public void actionPerformed(ActionEvent e){
        aboutwindow();
    }
     }


     public JMenuBar menu(){
    JMenuBar menubar = new JMenuBar();
    JMenu menu = new JMenu("Hilfe");
    JMenuItem about = new JMenuItem("Über...");

    menubar.add(menu);
    menu.add(about);

    about.addActionListener(new aboutaction());


    return menubar;

     }


        public JTextField Name(){
        JTextField Name = new JTextField("Name");
        return Name;
    }

        public JTextField Nachname(){
        JTextField Nachname = new JTextField("Nachname");
        return Nachname;
    }

        public JTextField Telefon(){
        JTextField Telefon = new JTextField("Telefonnummer");
        return Telefon;
    }



        public JTextArea Kontakte(){
        JTextArea Kontakte= new JTextArea(5, 20);
        

        Kontakte.setEditable(false);
        Kontakte.setMargin(new Insets(5,5,5,5));

        return Kontakte;
        }

        public JScrollPane KontakteScroll(){
        /*1.Problem:
         wie binde ich die JScrollPane richtig in die JTextArea Kontakte ein?*/
        JScrollPane KontakteScroll = new JScrollPane(Kontakte());
        return KontakteScroll();
    }

       public void hinzufuegen(){
       /*2.Problem:
         beim drücken des Buttons soll der Text der Textfelder in die JTextArea 
         kopiert werden*/
       String name = Name().getText();
       String nachname = Nachname().getText();
       String telefon = Telefon().getText();
       String newline = "/n";

       Kontakte().append(name + newline);
       Kontakte().append(nachname + newline);
       Kontakte().append(telefon + newline);
       Kontakte().append(newline);
       
       Name().selectAll();
       Nachname().selectAll();
       Telefon().selectAll();

       Kontakte().setCaretPosition(Kontakte().getDocument().getLength());
 }
  
    class hinzuaction implements ActionListener{
        public void actionPerformed (ActionEvent e){
            hinzufuegen();
        }
        
    }

    public JButton Hinzu(){
        JButton Hinzu = new JButton("Hinzufügen");

        Hinzu.addActionListener(new hinzuaction());
        return Hinzu;
    }

public void showwindow(){
JFrame frame = new JFrame("Hallo");
JPanel panel = new JPanel();

frame.setSize(400, 400);
frame.setVisible(true);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.add(panel);
frame.setJMenuBar(menu());
panel.add(Name());
panel.add(Nachname());
panel.add(Telefon());
panel.add(Hinzu());
panel.add(Kontakte());
panel.add(KontakteScroll());
    }



public static void main(String[] args) {
       new Main().showwindow();
}
}

 

[Kontraproduktiv]

keiner ne idee?

 

[André Uhres]

Versuch's mal so:

import javax.swing.*;
import java.awt.event.*;
import java.awt.*;
public class Main {
    private JTextArea taContacts;
    private JTextField tfName;
    private JTextField tfFirstname;
    private JTextField tfPhone;
    private JMenuBar menubar;
    private JScrollPane contactsScroll;
    private JMenu menuAid;
    private JFrame frame;
    private JButton btAdd;
    private JDialog aboutwindow;
    public JDialog aboutwindow() {
        if (aboutwindow == null) {
            aboutwindow = new JDialog(frame, "Über", true);
            JPanel aboutpanel = new JPanel();
            JLabel aboutlabel = new JLabel("Version 1.0");
            aboutwindow.setSize(100, 100);
            aboutwindow.setLocationRelativeTo(menuAid);
            aboutwindow.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
            aboutwindow.add(aboutpanel);
            aboutpanel.add(aboutlabel);
        }
        aboutwindow.setVisible(true);
        return aboutwindow;
    }
    class aboutaction implements ActionListener {
        public void actionPerformed(ActionEvent e) {
            aboutwindow();
        }
    }
    public JMenuBar getMenubar() {
        if (menubar == null) {
            menubar = new JMenuBar();
            menuAid = new JMenu("Hilfe");
            JMenuItem about = new JMenuItem("Über...");
            menubar.add(menuAid);
            menuAid.add(about);
            about.addActionListener(new aboutaction());
        }
        return menubar;
    }
    public JTextField getName() {
        if (tfName == null) {
            tfName = new JTextField("Name");
        }
        return tfName;
    }
    public JTextField getFirstname() {
        if (tfFirstname == null) {
            tfFirstname = new JTextField("Nachname");
        }
        return tfFirstname;
    }
    public JTextField getPhone() {
        if (tfPhone == null) {
            tfPhone = new JTextField("Telefonnummer");
        }
        return tfPhone;
    }
    public JTextArea getContacts() {
        if (taContacts == null) {
            taContacts = new JTextArea(5, 20);
            taContacts.setEditable(false);
            taContacts.setMargin(new Insets(5, 5, 5, 5));
        }
        return taContacts;
    }
    public JScrollPane getContactsScroll() {
        if (contactsScroll == null) {
            contactsScroll = new JScrollPane(getContacts());
        }
        return contactsScroll;
    }
    public void add() {
        String name = getName().getText();
        String firstname = getFirstname().getText();
        String phone = getPhone().getText();
        String newline = "\n";
        getContacts().append(name + newline);
        getContacts().append(firstname + newline);
        getContacts().append(phone + newline);
        getContacts().append(newline);
        getName().selectAll();
        getFirstname().selectAll();
        getPhone().selectAll();
        getContacts().setCaretPosition(getContacts().getDocument().getLength());
        getName().requestFocusInWindow();
    }
    class addAction implements ActionListener {
        public void actionPerformed(ActionEvent e) {
            add();
        }
    }
    public JButton getAdd() {
        if (btAdd == null) {
            btAdd = new JButton("Hinzufügen");
            btAdd.addActionListener(new addAction());
        }
        return btAdd;
    }
    public void showWindow() {
        frame = new JFrame("Hallo");
        JPanel panel = new JPanel();
        frame.setSize(400, 400);
        frame.setLocationRelativeTo(null);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.add(panel);
        frame.setJMenuBar(getMenubar());
        panel.add(getName());
        panel.add(getFirstname());
        panel.add(getPhone());
        panel.add(getAdd());
        panel.add(getContacts());
        panel.add(getContactsScroll());
        frame.setVisible(true);
    }
    public static void main(final String[] args) {
        Runnable gui = new Runnable() {
            @Override
            public void run() {
                try {
                    UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
                } catch (Exception ex) {
                    ex.printStackTrace();
                }
                new Main().showWindow();
            }
        };
        //GUI must start on EventDispatchThread:
        SwingUtilities.invokeLater(gui);
    }
}

 

[Kontraproduktiv]

danke schonmal, aber was bringen die if-schleifen in den methoden? Bzw. wieso sollte die methode null sein, um ausgeführt zu werden?

 

[André Uhres]

Beim Programmstart sind alle Variablen null. In unserem Fall darf nur dann ein Objekt neu erzeugt werden, wenn die entsprechende Variable noch null ist, sonst würde ja bei jedem folgenden Methodenaufruf das Objekt immer wieder neu erzeugt werden, statt daß einfach nur die Referenz auf das bestehende Objekt zurückgegeben wird.

 

[Landei]

danke schonmal, aber was bringen die if-schleifen in den methoden? Bzw. wieso sollte die methode null sein, um ausgeführt zu werden?

Du hast es nicht anders gewollt: if-schleife.de

 

[Kontraproduktiv]

@Andre

Danke für den gut verständlichen und übersichtlichen Code. Ich musste zwar noch ein paar methoden nachgucken, aber sonst ist mir klar was du da gemacht hast^^. Ich habs gerade compiliert und es hat wunderbar funktioniert. Ich werde an dem Projekt auf jedenfall weiterarbeiten (Datenbank um die Kontakte auch zu speichern wäre sinnvoll, stell ich mir aber schwer vor, zuerst aber werden die Buttons richtig angeordnet) und erweitern.

@Landei

Ich werds mir merken