Verbesserung für JRadioButton

[Mikesh]

Hey an alle,

vorweg ich bin quasi kompletter Anfänger was das Programmieren angeht, daher ist der folgende Code für viele Fortgeschrittene bestimmt nicht sonderlich schön anzusehen.

Problem: Ich habe mit Swing einen BMI-Rechner programmiert bei dem die Möglichkeit besteht auszuwählen ob man direkt den BMI-Wert möchte oder eine Erläuterung, z.B. bei einem BMI von 32 "leichtes Übergewicht". Wie gesagt der Rechner läuft, allerdings bin ich mit meiner Lösung nicht wirklich zufrieden, es kommt mir alles etwas umständlich vor. Hätte mir vielleicht jemand einen Tipp wie ich z.B die Sache mit den RadioButtons besser lösen könnte?

import javax.swing.*;

import java.awt.*;
import java.awt.event.*;

public class BmiRechner extends JFrame {

    // Attribute

    private JPanel jPanel1 = new JPanel();

    private JLabel jLabelGE = new JLabel();
    private JLabel jLabelGR = new JLabel();
    private JLabel jLabelBMI = new JLabel();

    private JTextField txtGroesse = new JTextField(4);
    private JTextField txtGewicht = new JTextField(4);
    private JTextField txtBMI = new JTextField(15);

    private JButton btnBerechnung = new JButton();

    private JRadioButton jRbZahl = new JRadioButton();
    private JRadioButton jRbText = new JRadioButton();
    private ButtonGroup gruppe = new ButtonGroup();

    double gewicht;
    double groesse;
    double bmi;

    // Konstruktor

    public BmiRechner() {
        initUI();
    }

    // Methoden

    private void initUI() {

        jPanel1.setLayout(new FlowLayout());
        this.setTitle("BMI-Rechner");

        jLabelGE.setText("Gewicht in Kg: ");
        jPanel1.add(jLabelGE);
        txtGewicht.setText("");
        jPanel1.add(txtGewicht);
        jLabelGR.setText("Größe in cm: ");
        jPanel1.add(jLabelGR);
        txtGroesse.setText("");
        jPanel1.add(txtGroesse);
        jRbZahl.setText("Zahlenwert");
        jPanel1.add(jRbZahl);
        gruppe.add(jRbZahl);
        jRbText.setText("Erläuterung");
        jPanel1.add(jRbText);
        gruppe.add(jRbText);
        btnBerechnung.setText("Berechnung");
        jPanel1.add(btnBerechnung);
        jLabelBMI.setText("BMI");
        jPanel1.add(jLabelBMI);
        txtBMI.setText("");
        jPanel1.add(txtBMI);

        this.getContentPane().add(jPanel1);
        pack();
        setDefaultCloseOperation(EXIT_ON_CLOSE);

        jRbText.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent ae) {
                // TODO Auto-generated method stub
                jLabelBMI.setText("Erläuterung");
                txtBMI.setText("");
                pack();

            }

        });
        jRbZahl.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent ae) {
                // TODO Auto-generated method stub
                jLabelBMI.setText("BMI");
                txtBMI.setText("");
                pack();

            }

        });

        btnBerechnung.addActionListener(new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                try {
                    double gewicht = Double.parseDouble(txtGewicht.getText());
                    double groesse = Double.parseDouble(txtGroesse.getText());
                    double bmi;
                    bmi = gewicht / (groesse * groesse);
                    if (jRbText.isSelected() == true && bmi < 20) {
                        txtBMI.setText("Untergewicht");
                    } else if (jRbText.isSelected() == true && bmi >= 20
                            && bmi < 25) {
                        txtBMI.setText("Normalgewicht");

                    } else if (jRbText.isSelected() == true && bmi >= 25
                            && bmi < 30) {
                        txtBMI.setText("Übergewicht (Grad I)");

                    } else if (jRbText.isSelected() == true && bmi >= 30
                            && bmi < 40) {
                        txtBMI.setText("Starkes Übergewicht (Grad II)");

                    } else if (jRbText.isSelected() == true && bmi > 40) {
                        txtBMI.setText("Extremes Übergewicht (GradIII)");

                    } else if (jRbZahl.isSelected() == true) {
                        txtBMI.setText("" + bmi);

                    }
                } catch (NumberFormatException er) {
                    txtBMI.setText("###FEHLER###");
                }
            }

        });

    }

}

Bin über jeden Tipp dankbar! War aber trotzdem schon froh, dass ich das Ding selbst zum laufen bekommen hab
Viele Grüße und schönes Wochenende

 

[Blender3D]

vorweg ich bin quasi kompletter Anfänger was das Programmieren angeht, daher ist der folgende Code für viele Fortgeschrittene bestimmt nicht sonderlich schön anzusehen.

Ich habe Dir Deinen Code etwas in Form gebracht, indem ich:
1) die Logik des BMI in eine eigene Klasse gepackt habe.
2) Man sollte nicht von JFrame erben. Ich erbe von einem JPanel den ich in den JFrame einsetze.
3) Ich habe die RadioButtons entfernt und zeige beide Werte ständig an. Macht mehr Sinn.
4) Ok Button habe ich auch entfernt. Die Neuberechnung steuere ich mit einem Fokuslistener. Das bedeutet immer wenn Du auf die andere Eingabe wechselst wird neu berechnet.
Du brauchst so nur mehr 7 anstatt 9 Komponenten für die Anzeige.
5) Der BMI Wert wird auf 2 Kommastellen angezeigt.


Hier der Code dazu :

public class BodyMassIndex {
    private double weight = 0;

    private double size = 0;

    /**
     * @param weight
     *            in kilogram
     * @param size
     *            in meter
     */
    public BodyMassIndex(double weight, double size) {
        this.weight = weight;
        this.size = size;
    }

    public BodyMassIndex(String weight, String size) throws NumberFormatException {
        this.weight = Double.parseDouble(weight);
        this.size = Double.parseDouble(size);
    }

    public double getBMI() {
        return weight / (size * size);
    }

    public String getFormattedStringBMI(int numDecimal) {
        return (String.format("%." + numDecimal + "f", getBMI()));
    }

    public double getSize() {
        return size;
    }

    public double getWeight() {
        return weight;
    }

    @Override
    public String toString() {
        double bmi = getBMI();
        if (bmi < 20)
            return "Untergewicht";
        if (bmi < 25)
            return "Normalgewicht";
        if (bmi < 30)
            return "\"Übergewicht (Grad I)\"";
        if (bmi < 40)
            return "Starkes Übergewicht (Grad II)";
        return "Extremes Übergewicht (GradIII)";
    }

}

mport java.awt.Color;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.event.FocusEvent;
import java.awt.event.FocusListener;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.SwingConstants;
import javax.swing.border.LineBorder;

@SuppressWarnings("serial")
public class BMIPanel extends JPanel implements FocusListener {
    private JLabel labelBMIValue = new JLabel("", SwingConstants.CENTER);
    private JLabel labelBMIText = new JLabel("", SwingConstants.CENTER);
    private JTextField txtSize = new JTextField(6);
    private JTextField txtWeight = new JTextField(6);
    BodyMassIndex bmi = new BodyMassIndex(70, 1.75); // default value

    public BMIPanel(int width, int height) {
        super.setPreferredSize(new Dimension(width, height));
        setLayout(new FlowLayout());
        initLayout(width);
        updateBMIValuesDisplay();
        txtSize.addFocusListener(this);
        txtWeight.addFocusListener(this);
    }

    @Override
    public void focusGained(FocusEvent e) {
        updateValues();
    }

    @Override
    public void focusLost(FocusEvent e) {
        updateValues();
    }

    private double getdoubleValue(String value, double oldValue) {
        try {
            oldValue = Double.parseDouble(value.trim());
        } catch (NumberFormatException e) {
        }
        return oldValue;
    }

    private void initLayout(int width) {
        JLabel labelSize = new JLabel("  Größe in m  ");
        JLabel labelWeight = new JLabel(" Gewicht in kg  ");
        JLabel labelBMI = new JLabel("BMI ==", SwingConstants.CENTER);
      
        labelSize.setBorder(new LineBorder(Color.RED));
        labelWeight.setBorder(new LineBorder(Color.RED));
        labelBMI.setBorder(new LineBorder(Color.RED));
        labelBMIValue.setBorder(new LineBorder(Color.BLACK));
        int h = 2 * getFont().getSize();
        labelBMI.setPreferredSize(new Dimension(width/4, h));
        labelBMIValue.setPreferredSize(new Dimension(width / 5, h));
        labelBMIText.setBorder(new LineBorder(Color.BLACK));
        labelBMIText.setPreferredSize(new Dimension(width / 2, h));
        add(labelSize);
        add(txtSize);
        add(labelWeight);
        add(txtWeight);
        add(labelBMI);
        add(labelBMIValue);
        add(labelBMIText);
    }

    private void updateBMIValuesDisplay() {
        txtSize.setText(bmi.getSize() + "");
        txtWeight.setText(bmi.getWeight() + "");
        labelBMIValue.setText(bmi.getFormattedStringBMI(2));
        labelBMIText.setText(" " + bmi);
    }

    private void updateValues() {
        double size = getdoubleValue(txtSize.getText(), bmi.getSize());
        double weight = getdoubleValue(txtWeight.getText(), bmi.getWeight());
        bmi = new BodyMassIndex(weight, size);
        updateBMIValuesDisplay();
    }
}

import javax.swing.JFrame;

public class start {
    public static void main(String[] args) {  
        JFrame frame = new JFrame("BMI Rechner");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.add(new BMIPanel(400, 60));
        frame.setResizable(false);
        frame.pack();
        frame.setVisible(true);
    }
}

 

[Mikesh]

Super, ja so sieht das alles natürlich schon besser aus, Danke!Denke da hilft nur programmieren, programmieren und programmieren. Bin mal gespannt ob ich mein Snake hinbekomme^^. Gibt es zwar schon fertig im Netz aber das ist ja nicht der Sinn der Sache

 

[Blender3D]

Danke!Denke da hilft nur programmieren, programmieren und programmieren.

Stimmt. Mit jedem Stück Code, das man schreibt lernt man dazu.

 

[MoxxiManagarm]

@Blender3D Nach fast 350 Beiträgen, bitte bitte bitte, setze doch den Code auch mal in Java Tags. Das ist nicht schwer und macht es für alle leslicher.
[code=java][/code]

 

[Blender3D]

setze doch den Code auch mal in Java Tags.

Hab ich doch oder meinst Du = Java?

 

[Xyz1]

Nach fast 350 Beiträgen, bitte bitte bitte, setze doch den Code auch mal in Java Tags

Das habe ich ihm schon gesagt, diesbezüglich ist er aber lernresistent. Nicht mein paar Schuhe.

 

[Blender3D]

Das habe ich ihm schon gesagt, diesbezüglich ist er aber lernresistent

Hab den Hinweis nicht wirklich verstanden gehabt. Da ich die vom Forum angebotene Funktion benutze.
Wäre aber eine Idee den angebotenen Tag von

automatisch auf

einzustellen, da wir hier ein Java Forum haben.

 

[MoxxiManagarm]

du meinst einfach nur das =java in dem öffnenden Tag ergänzen. Dann wirds Java

 

[Xyz1]

du meinst

du musst....

 

[Blender3D]

du meinst einfach nur das =java in dem öffnenden Tag ergänzen. Dann wirds Java

Genau als Defaultwert.
Ich habe die Code Tagfunktion relativ schnell nach einem Hinweis benutzt.
Den Tipp mit den Java Tag hab ich erst jetzt realisiert.
Grund. Ich habe in den 80er Jahren zu programmieren begonnen. Damals hat man teilweise noch mit Hex-Editoren programmiert. Auf meinem C64 habe ich so einen eigenen Macro Assemlber geschrieben.
Von daher bin ich es gewohnt Code auch ohne Formatierung zu lesen. Und ich ging davon aus, dass die Code Funktion das macht. Mein Fehler. Sorry.
Aber wie schon gesagt -> sollte vielleicht per Default so eingestellt sein.