Swing Idealgewicht anzeigen per Radiobuttonaktivierung

[dariusberlin]

Hallo es handelt sich hierbei um Kapitel 63 Programmieraufgaben Aufgabe 1.

Ich weiß das man den Frame eigentlich von innen mit der Methode Pack() anpassen sollte. Ich verstehe aber noch nicht genau wie man das macht. Kann mir jemand das erklären?

Außerdem müssen ja "Paare" zwischen female und einer größe gebildet werden und eben zwischen male und einer größe. Wie ich das machen kann, dazu habe ich noch keine Idee. Kann mir jemand einen Tipp geben?

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

import javax.swing.*;

public class IdealWeight extends JFrame implements ActionListener
{
  JRadioButton genderM, genderF;
  ButtonGroup  genderGroup;
  JPanel       genderPanel;

  JRadioButton heightA, heightB, heightC, heightD, heightE;
  ButtonGroup  heightGroup;
  JPanel       heightPanel;

  JTextField   resultText;
  JLabel       resultLabl;
  JPanel       resultPanel;

  public  IdealWeight()
  {
    setTitle( "Your Ideal Weight" );
    setDefaultCloseOperation( EXIT_ON_CLOSE );

    // Gender-Gruppe
    genderM = new JRadioButton("Male", true );
    genderF = new JRadioButton("Female", false );
   
    genderM.setActionCommand("genderM");
    genderM.addActionListener(this);
    
    genderF.setActionCommand("genderF");
    genderF.addActionListener(this);
    
    genderGroup = new ButtonGroup();
    genderGroup.add( genderM );
    genderGroup.add( genderF );
    genderPanel = new JPanel();
    genderPanel.setLayout( new BoxLayout( genderPanel, BoxLayout.Y_AXIS ) );
    genderPanel.add( new JLabel("Your Gender") );
    genderPanel.add( genderM );  genderPanel.add( genderF );

    // Height-Gruppe
    heightA = new JRadioButton("60 to 64 inches", true );
    heightB = new JRadioButton("64 to 68 inches", false );
    heightC = new JRadioButton("68 to 72 inches", false );
    heightD = new JRadioButton("72 to 76 inches", false );
    heightE = new JRadioButton("76 to 80 inches", false );

    heightGroup = new ButtonGroup();
    heightGroup.add( heightA ); heightGroup.add( heightB );
    heightGroup.add( heightC ); heightGroup.add( heightD );
    heightGroup.add( heightE );

    heightPanel = new JPanel();
    heightPanel.setLayout( new BoxLayout( heightPanel, BoxLayout.Y_AXIS ) );
    heightPanel.add( new JLabel("Your Height") );
    heightPanel.add( heightA ); heightPanel.add( heightB );
    heightPanel.add( heightC ); heightPanel.add( heightD );
    heightPanel.add( heightE );
    
    heightA.setActionCommand("heightA");
    heightA.addActionListener(this);
    heightB.setActionCommand("heightB");
    heightB.addActionListener(this);
    heightC.setActionCommand("heightC");
    heightC.addActionListener(this);
    heightD.setActionCommand("heightD");
    heightD.addActionListener(this);
    heightE.setActionCommand("heightE");
    heightE.addActionListener(this);

    // Result-Panel
    resultText  = new JTextField(13);
    resultText.setEditable( false );
    resultLabl  = new JLabel("Ideal Weight");
    resultPanel = new JPanel();
    resultPanel.add( resultLabl );
    resultPanel.add( resultText );

    // Frame: verwendet den Standard Layoutmanager
    add( genderPanel, BorderLayout.WEST );
    add( heightPanel, BorderLayout.EAST );
    add( resultPanel, BorderLayout.SOUTH  );
    
    
    }

  double    idealM  (double H){
	return (H*H/28);}
  
  double idealW(double H){
      return (H*H/30);
  }

  public static void main ( String[] args )
  {
    IdealWeight weightApp  = new IdealWeight() ;
    weightApp.setSize( 250, 225 );
    weightApp.setResizable( false );
    weightApp.setVisible( true );
  }

@Override
public void actionPerformed(ActionEvent eve) {
   
if (eve.getActionCommand()=="genderM" && eve.getActionCommand()=="heightA"){
	resultText.setText(idealM(62)+"");
}
 
}
}

 

[strußi]

du musst nur bei der auslösung der Action die ausgewählten Radiobuttons auslesen und dadurch dein idealgewicht auslesen

vlt mit einer methode in der du abfragst if( male){ auslesen der größe und berechnen } else{ auslesen der größe und berechnen} und anschließend das Textfeld befüllen

 

[dariusberlin]

Klingt erstmal logisch....aber ich verstehe es im Detail noch nicht, bzw. weiß immernoch nicht wie ich es umsetzen soll. Ich muss ja im Prinzip 2 buttons auslesen und diese Werte 2 verschiedenen Variablen zuweisen. Und das kriege ich rein praktisch grade nicht hin.

 

[strußi]

so könntest du das machen oder du fragst das auslösende objekt ab und bildest daraus deine abfrage/deine berechnung

if( genderM.isSelected()){
        if( heightA.isSelected()){
            resultText.setText( Double.toString(idealM( 60)));
        } else if( heightB.isSelected()){
            //...
        }
    } else{
        if( heightA.isSelected()){
            resultText.setText( Double.toString(idealW( 60)));
        } else if( heightB.isSelected()){
            //...
        }
    }

 

[Foxei]

Hallo dariusberlin,
ich bin kein Freund von solchen Online-Tutoriums weil sie oft eine sehr fragwürdige Art zu programmieren erklären :/

Erstmal zu deinem Code:
Du befolgst einige Programmier-Standards nicht:

• Man erbt nicht von einer JFrame!!
• Man verwendet keine variablenen und Methoden ohne "private, public, final static" Deklarationen

Des weiteren würde ich dir eine bessere Methode vorschlagen dein Programm auf zubauen.

Im Konstruktor erzeugst du alle Instanzen.
In einer zweiten Methode ich nenne sie jetzt "init()" werden alle aufgaben abgearbeitet die notwendig sind damit das UI angepasst ist.
Wahlweite noch eine zweite Methode in der du alle Listener setzt "addListener()"
Und dann eine Methode die aufgerufen wird wenn du alles Sichtbar machst.

Das hat den Vorteil das dein Programm wenn du später das MVC anwendest vom Controller aus viel besser angesteuert werden kann.

Nun zu deinem Problem:
Ich gebe den Button für das geschlehct den ActionCommand "f" bzw. "m". Und den Radiobutton für das gewicht immer das Gewicht für das sie stehen also bei "68 to 72 inches" "70" usw.
Dann setzte ich die RadioButton nicht mit "setSelected()" ausgewählt sondern mit "doClick()" so wird der ActionListener ausgeführt. Dann Speicher ich das Geschlecht in einer Variabel zwischen und die Größe und rufen dann einer simpelen unterscheidungs Methode auf.

private void checkWeight() {
        if (selectedGender.equals("f")) {
            resultText.setText(formatter.format(idealW(size)));
        } else {
            resultText.setText(formatter.format(idealM(size)));
        }
    }

Ich habe deinen ganzes Programm überarbeitet und es dir hier eingestellt:

Spoiler: Code

IdealWeight.java

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.text.DecimalFormat;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.BoxLayout;
import javax.swing.ButtonGroup;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JRadioButton;
import javax.swing.JTextField;
import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;
import javax.swing.border.EmptyBorder;

public class IdealWeight implements ActionListener {

    private final JFrame frame;
    private final JRadioButton genderM, genderF;
    private final ButtonGroup genderGroup;
    private final JPanel genderPanel;

    private final JRadioButton heightA, heightB, heightC, heightD, heightE;
    private final ButtonGroup heightGroup;
    private final JPanel heightPanel;

    private final JTextField resultText;
    private final JLabel resultLabl;
    private final JPanel resultPanel;

    private String selectedGender;
    private int size;
    private final DecimalFormat formatter;

    public IdealWeight() {
        frame = new JFrame();
        genderM = new JRadioButton();
        genderF = new JRadioButton();
        genderGroup = new ButtonGroup();
        genderPanel = new JPanel();
        heightA = new JRadioButton();
        heightB = new JRadioButton();
        heightC = new JRadioButton();
        heightD = new JRadioButton();
        heightE = new JRadioButton();
        heightGroup = new ButtonGroup();
        heightPanel = new JPanel();
        resultText = new JTextField();
        resultLabl = new JLabel();
        resultPanel = new JPanel();
        formatter = new DecimalFormat("0.00");
    }

    public void init() {
        frame.setTitle("Your Ideal Weight");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setResizable(false);

        // Gender-Gruppe
        genderM.setText("Male");
        genderM.setActionCommand("m");
        genderM.addActionListener(this);
        genderM.doClick();

        genderF.setText("Female");
        genderF.setActionCommand("f");
        genderF.addActionListener(this);

        genderGroup.add(genderM);
        genderGroup.add(genderF);

        genderPanel.setLayout(new BoxLayout(genderPanel, BoxLayout.Y_AXIS));
        genderPanel.setBorder(new EmptyBorder(5, 5, 5, 5));
        genderPanel.add(new JLabel("Your Gender"));
        genderPanel.add(genderM);
        genderPanel.add(genderF);

        // Height-Gruppe
        heightGroup.add(heightA);
        heightGroup.add(heightB);
        heightGroup.add(heightC);
        heightGroup.add(heightD);
        heightGroup.add(heightE);

        heightPanel.setLayout(new BoxLayout(heightPanel, BoxLayout.Y_AXIS));
        heightPanel.setBorder(new EmptyBorder(5, 5, 5, 5));
        heightPanel.add(new JLabel("Your Height"));
        heightPanel.add(heightA);
        heightPanel.add(heightB);
        heightPanel.add(heightC);
        heightPanel.add(heightD);
        heightPanel.add(heightE);

        heightA.setText("60 to 64 inches");
        heightA.setActionCommand("62");
        heightA.addActionListener(this);
        heightA.doClick();
        heightB.setText("64 to 68 inches");
        heightB.setActionCommand("66");
        heightB.addActionListener(this);
        heightC.setText("68 to 72 inches");
        heightC.setActionCommand("70");
        heightC.addActionListener(this);
        heightD.setText("72 to 76 inches");
        heightD.setActionCommand("74");
        heightD.addActionListener(this);
        heightE.setText("76 to 80 inches");
        heightE.setActionCommand("78");
        heightE.addActionListener(this);

        // Result-Panel
        resultText.setEditable(false);
        resultText.setColumns(20);
        resultLabl.setText("Ideal Weight");
        resultPanel.add(resultLabl);
        resultPanel.add(resultText);

        // Frame: verwendet den Standard Layoutmanager
        frame.add(genderPanel, BorderLayout.WEST);
        frame.add(heightPanel, BorderLayout.EAST);
        frame.add(resultPanel, BorderLayout.SOUTH);
    }

    private void show() {
        frame.pack();
        frame.setLocationRelativeTo(null);
        frame.setVisible(true);
    }

    private double idealM(double H) {
        return (H * H / 28);
    }

    private double idealW(double H) {
        return (H * H / 30);
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        if (e.getActionCommand().equals("f") || e.getActionCommand().equals("m")) {
            selectedGender = e.getActionCommand();
        } else {
            size = Integer.parseInt(e.getActionCommand());
        }
        checkWeight();
    }

    private void checkWeight() {
        if (selectedGender.equals("f")) {
            resultText.setText(formatter.format(idealW(size)));
        } else {
            resultText.setText(formatter.format(idealM(size)));
        }
    }

    public static void main(String[] args) {
        loadLnF();
        IdealWeight weightApp = new IdealWeight();
        weightApp.init();
        weightApp.show();
    }

    public static void loadLnF() {
        try {
            UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
        } catch (ClassNotFoundException | InstantiationException | IllegalAccessException | UnsupportedLookAndFeelException ex) {
            Logger.getLogger(IdealWeight.class.getName()).log(Level.SEVERE, null, ex);
        }
    }
}

Grüße Simon Schäfer

Freu mich auf Antwort

 

[dariusberlin]

Danke erstmal für die aufwendige Antwort und die Mühe, warum man nicht von einem JFrame erben sollte ist mir jetzt erstmal kla geworden und einige andere Dinge auch. Aber das hier hab ich nicht verstanden.... [Java]
@Override
public void actionPerformed(ActionEvent e) {
if (e.getActionCommand().equals("f") || e.getActionCommand().equals("m")) {
selectedGender = e.getActionCommand();
} else {
size = Integer.parseInt(e.getActionCommand());
}
[/Java]
In diesem Fall ist es doch garnicht möglich das der Else-Zweig ausgeführt wird, oder? Und genau das verstehe ich an dieser If-anweisung nicht.

 

[Foxei]

Gib dir mal den actionCommand aus

 

[dariusberlin]

Danke, jetzt ist mir das schon etwas klarer geworden. Auf welche Weise hast Du programmieren gelernt, was war für Dich nützlich?

 

[Foxei]

Die Grundlagen wurden in der Schule gelegt if beziehungen Schleifen, Methoden sowas. Und danach habe ich mir alles selber beigebracht Einfach ein projekt gesucht und gelöst dann nächstes Projekt. Ich hatte nie eine Ausbildung

Gruß Simon