Was würdet ihr besser/anders machen?

[Guest]

Hi,
ich bin gerade an der Programmierung eines Taschenrechners beschäftigt. Nun jetzt mal meine Frage, was würdet ihr hier anders machen und warum?

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

public class Taschenrechner extends JFrame
{
    private String anzeige = "0.0";
    
    String[] number = { "7", "8", "9", "4", "5", "6", "1", "2", "3", ".", "0", "+/-" };
    String[] operator = { "=", "CE", "C", "+", "*", "x^x", "-", "/", "sqrt", "%", "Pi", "1/x" };
    
    JTextPane display = new JTextPane();
    
    JPanel panels = new JPanel(new GridLayout(1, 2, 3, 2));
    
    JPanel numberPanel = new JPanel(new GridLayout(4, 3, 1, 1));
    JPanel operatorPanel = new JPanel(new GridLayout(4, 3, 1, 1));
    
    JButton numberButtons[] = new JButton[number.length];
    JButton operatorButtons[] = new JButton[operator.length];
    
    public Taschenrechner()
    {
        setTitle("Taschenrechner");
        setSize(265, 300);
        setResizable(false);
        createGuiAndShow();  
        
        add(display, BorderLayout.NORTH);
        panels.add(numberPanel);
        panels.add(operatorPanel);
        add(panels, BorderLayout.CENTER);
        
        pack();
        display.setEditable(false);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setVisible(true);
    }
    
    public static void main(String[] Args)
    {
        new Taschenrechner();
    }

    private void createGuiAndShow()
    {
        for(int i = 0; i < operator.length; i++)
        {
            operatorButtons[i] = new JButton(operator[i]);
            operatorButtons[i].setMargin(new Insets(5, 5, 5, 5));
        }
        for(int i = 0; i < operator.length; i++)
            operatorPanel.add(operatorButtons[i]);
        for(int i = 0; i < number.length; i++)
        {
            numberButtons[i] = new JButton(number[i]);
            numberButtons[i].setMargin(new Insets(5, 5, 5, 5));
        }
        for(int i = 0; i < number.length; i++)
            numberPanel.add(numberButtons[i]);
    }
}

 

[Marco13]

Ich würde dafür sorgen, dass was passiert, wenn man die Buttons klickt :bae:
Na, bisher gibt's da nicht so viel dran zu bemängeln. Die Panels bräuchten vermutlich keine Instanzvariablen zu sein. Aber sonst...

 

[Guest]

Ich würde dafür sorgen, dass was passiert, wenn man die Buttons klickt :bae:

:lol: werd ich mir merken ;-) xD


Ok, danke, warum sollte man sie denn nicht als Instanzvariablen nutzen? Wäre doch für eventuelle spätere Methodenzugriffe nützlich oder?

MfG
Danke schonmal
Der Gast ;-)

 

[Marco13]

Ich weiß nicht, was du mit "Methodenzugriffe" meinst. Aber diese Klasse stellt (bisher) ja nur die Bedien-Einheit eines Taschenrechners dar. Ich wüßte nicht, WER da WO drauf zugreifen sollte - insbesondere auf die Panels, wo die Buttons drinliegen. Man sollte (IMHO) davon ausgehen können, dass man nicht auf diese Panels zugreifen muss. Und wenn man das nicht tun muss, sind die Panels ja keine "Eigenschaft" der Bedien-Einheit, die von außen ausgelesen (und schon garnicht geändert) werden können sollte, sondern nur ein "Stilmittel", um das gewünschte Layout hinzukriegen. Du würdest ja z.B. auch nicht die Insets
operatorButtons.setMargin(new Insets(5, 5, 5, 5));
als Instanzvariablen speichern: Sie interessieren einfach niemanden.

 

[Gelöschtes Mitglied 5909]

- klassenvariablen mit modifier versehen (es sei denn du hast einen guten grund package protection zu nehmen)
- viel zu viel im konstruktor

 

[Guest]

Danke für die Antworten doch was ist schlimm daran wenn vielzuviel im Konstruktor steht? - Wenn ich statt es in den Konstruktor in 'createGuiAndShow()' schreibe wird diese ja auch vom Konstruktor aufgerufen
Oder gibt es da etwas was ich nicht weiß?

Danke[/code]

 

[Marco13]

Naja, man könnte z.B. den Block
add(display, BorderLayout.NORTH);
panels.add(numberPanel);
panels.add(operatorPanel);
add(panels, BorderLayout.CENTER);
noch in die createAndShowGUI mit reinnehmen.

Ach und noch was: Statt

        for(int i = 0; i < operator.length; i++)
        {
            operatorButtons[i] = new JButton(operator[i]);
            operatorButtons[i].setMargin(new Insets(5, 5, 5, 5));
        }
        for(int i = 0; i < operator.length; i++)
            operatorPanel.add(operatorButtons[i]);

wäre

        for(int i = 0; i < operator.length; i++)
        {
            operatorButtons[i] = new JButton(operator[i]);
            operatorButtons[i].setMargin(new Insets(5, 5, 5, 5));
            operatorPanel.add(operatorButtons[i]); 
        }

besser (weniger code, übersichtlicher....)

 

[happy_robot]

Hi,
ich bin gerade an der Programmierung eines Taschenrechners beschäftigt. Nun jetzt mal meine Frage

ok

, was würdet ihr hier anders machen

einen kaufen

...und warum?

weil weniger aufwändig

 

[happy_robot]

Danke für die Antworten doch was ist schlimm daran wenn vielzuviel im Konstruktor steht? - Wenn ich statt es in den Konstruktor in 'createGuiAndShow()' schreibe wird diese ja auch vom Konstruktor aufgerufen
Oder gibt es da etwas was ich nicht weiß?

Danke[/code]

was machst du wenn du evtl noch einen weiteren konstruktur benötigst (z.b. einen mit einbem anderen parentfenster und anderen titel)?

kopierst du dann den konstruktor?

packe "in sich SINNVOLL geschlossene funktionalitäten" besser in gut-"benannte" methoden.

createGuiAndShow

sollte man z.b. in zwei methoden aufsplitten.

createGUI

und

showGUI

und im Konstruktor hat deren Aufruf nix verloren.

Ein Konstruktor sollte NUR und AUSSCHLIESSLICH das Objekt selber erzeugen!

 

[Marco13]

Naja, createAndShowGUI stammt aus den ganzen Beispielen von sun :wink: ... das "show" besteht ja nur aus einem ...setVisible(true), ... hm :?