Hilfe bei EventListening

[Johnny990]

Also ich hab zwar schon mal mein Problem erklärt, aber nur sehr schwammig! versuchs mal genauer zu beschreiben...

(Wär echt sehr dringend die benötigte Hilfe)

Ich hab folgendes Applet(führt es ruhig mal aus damit ihr auch wisst was ich meine:

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

public class Umfrageversuch1 extends Applet implements ItemListener
{
 //Dies ist die klasse die altersstartwerte beinhaltet
//MannFrau in = new MannFrau();
  // ERSCHAFFUNG DER KOMPONENTEN
  
     //Choice Boxen
    Choice c1 = new Choice();  //Mann oder frau
    Choice c2 = new Choice();   //Ja oder nein frage 1(übergweichtig?)
    Choice c3 = new Choice(); //Ja oder nein frage 2 (fettarmes essen)
    Choice c4 = new Choice();
    //Labels
    Label l0 = new Label ("Sind Sie ein Mann oder eine Frau ?");
    Label l1 = new Label("Waren/Sind Sie übergewichtig ? ");
    Label l2 = new Label("Essen Sie täglich möglichst fettarm ?");
    Label l3 = new Label ("Bewegen Sie sich dreimal pro Woche ?");
    
    //Panels für Fragenumrahmung
    Panel p0 = new Panel();
    Panel p1 = new Panel();
    Panel p2 = new Panel();//Frage fettarm
    Panel p3 = new Panel(); // Frage bewegung
    
    //Textfield
    TextField tf1 = new TextField();
 //--------------------------------------------------------------------------------------------------
    //Initialsierungen des Applets
  public void init()
  {
   //Hintergrund
   setBackground(Color.lightGray);
   p0.setBackground(Color.red);
   p1.setBackground(Color.orange);
   p2.setBackground(Color.orange);
   p3.setBackground(Color.orange);
   setLayout(null);
   //--------------------- Initialsierungen im Applet von Komponenten
     //Auswahl zwischen männlich und weiblich
   c1.addItem(" ");
   c1.addItem("Mann");
   c1.addItem("Frau");
   c1.setBounds(300,5,200,60);
   c1.addItemListener(this);
   add(c1);
   
   //Auswahl (CHOICE) ja oder nein für Frage zu übergewicht
   c2.addItem("");
   c2.addItem("Ja");
   c2.addItem("Nein");
   c2.setBounds(350,100,100,30);
   add(c2);
   
   //Auswahl ja oder nein zu Frage fettarm
   c3.addItem(" ");
   c3.addItem("Ja");
   c3.addItem("Nein");
   c3.setBounds(350,140,100,30);
   add(c3);
   
   //Auswahl ja oder nein zu Frage bewegung
   c4.addItem(" ");
   c4.addItem("Ja");
   c4.addItem("Nein");
   c4.setBounds(350,180,100,30);
   add(c4);
   
   //-----------FRAGEN-----------------------------------------------------------
   //Frage 1 ob Mann oder Frau
   p0.setBounds(5,5,250,30);
   p0.add(l0);
   add(p0);
   //--- 1. frage--mit übergewicht
   p1.setBounds(0,100,250,30);
   p1.add(l1);
   add(p1);
   
     //--Frage 2--fettwarmes essen
     p2.setBounds (0,140,250,30);
     p2.add(l2);
     add(p2);
     
     //Frage 3 -- bewegung
     p3.setBounds(0,180,250,30);
     p3.add(l3);
     add(p3);
   
   
   
   // Textfield add()
   tf1.setBounds(350,400,200,30);
   add(tf1);
   
  }
  
  //Action Listening
  public void itemStateChanged(ItemEvent i)
  {
     if(i.getItem().equals("Mann"))
       setBackground(Color.blue);

       // Wie gehts hier weiter?????????????????????????????????????????????? Bitte Code! Vielen vielen Dank

  }
}

So und nun zum Problem:

ich will so ne Art Altersrechner programmieren, der ausgibt wie alt man (laut statistik ) wird.

Von bedeutung ist hier ob die person männlich oder weiblich ist....

Also angenommen ich wähle "Mann" dann muss ich ja andere Berechnungen bei den nachfolgenden Fragen anstellen als bei frauen.

Die erste frage lautet zB.: Sind oder waren sie übergewichtig? Wenn der mann Ja wählt werden ihm von seinem Durchschnittsalter(--> 77,73 Jahre) zwei JAhre abgezogen!

Nun, wie bring ich das am simpelsten zusammen, dass wenn "Mann" gewählt wurde, dass jeweils bei den entsprechenden Antworten zu den fragen (in denChoice boxen) entweder jahre hinzugezählt oder abgezogen werden???Und wie mach ich das , dass zum schluss das Alter in dem textfield ausgegeben wird???
und wie sprech ich genau die Choice Komponete( in meinem fall c1 und c2) an das zu der Frage gehört?

(ich hab jetzt hier nur mal eine Frage! Der test setzt sich natürlich aus etlichen mehr fragen zusammen!)

Bitte helft mir!!! 🙂

 

[Marco13]

Hmja. Wenn du das jetzt so weitermachst, wie bisher (mit mehr Fragen) dann hast du irgendwann solchen Code wie

    Choice c1 = new Choice();  //Mann oder frau
    Choice c2 = new Choice();   //Ja oder nein frage 1(übergweichtig?)
    ...
    ...
    ...
    ...
    ...
    Choice c42 = new Choice();  // Bist du Programmierer? Wenn "ja", 10 Jahre abziehen

Und das ist häßlich. Abgesehen davon kommst du in Teufels Küche, wenn du mit dem Layout durcheinaderkommst: Wenn du irgendwann nichtmehr weißt, für welches Panel du welche Bounds angeben mußt, damit es neben der gewünschten Choice erscheint.

Wie genau der Ablauf sein soll und so hast du jetzt auch noch nicht gesagt, und über die Struktur des ganzen könnte man sich bestimmt lange Gedanken machen. Ohne jetzt den kompletten abstraktions-Overkill propagieren zu wollen - vielleicht kannst du ja sowas machen wie

    private float predictedAge = -1;

    void init()
    {
        predictedAge = 77.73f;
        setLayout(new GridLayout(0,1));

        add(createChoice("Mann oder Frau?", new String[]{"Mann", "Frau"}, new float[]{-5, 0}));
        add(createChoice("Fettes Essen?", new String[]{"Ja", "Nein"}, new float[]{-3, 0}));
        add(createChoice("Hobbies", new String[]{"Sport", "Essen", "Schlafen"}, new float[]{-25, -10, -3}));
    }

    private Component createChoice(String title, final String choices[], final float changes[])
    {
        Panel panel = new Panel(new GridLayout(1,2);
        panel.setBackground(Color.orange);
        panel.add(new JLabel(title));

        final Choice choice = new Choice();
        choice.add(" ");
        for (int i=0; i<choices.length; i++)
        {
            choice.add(choices[i]);
        }
        choice.addItemListener(new ItemListener()
        {
            public void itemStateChanged(ItemEvent event)
            {
                if (event.getStateChange() == ItemEvent.SELECTED)
                {
                    int index = choice.getSelectedIndex();
                    if (index >= 1)
                    {
                        predictedAge += changes[index-1];
                    }
                }
            }
        });
        panel.add(choice);
        return panel;
    } 
}

Nur sinngemäß - es geht darum, dass du dann mit EINER einzigen(!) Zeile eine neue Frage hinzufügen kannst. In der ItemStateChanged-Methode der (anonymen) Listener kannst du dann jeweils ein TextField mit dem Alter updaten, oder das am Ende machen.

Das ist jetzt natürlich noch nicht sauber - wenn man z.B. mehrmals etwas wählt, wird mehrmals etwas vom Alter abgezogen usw, aber das kann man noch ausbauen. Man sollte aber solchen Spagehtti-Code vermeiden, der sich schon bei den ersten 4 Fragen abgezeichnet hat....

P.S.: Ich würde dir empfehlen, nicht AWT sondern Swing zu verwenden.
http://java.sun.com/docs/books/tutorial/uiswing/

 

[Campino]

Marco13: das ist durchaus richtig, aber bringt ihn nur bedingt weiter...

Eine Choice-Box spricht man natürlich über ihren Namen an, also c1 mit c1 usw. Im ItemState-Listener kannst du über ItemEvent.getSource() abfragen, von welcher Box das Event kommt. Einfach die Rückgabe dieser Funktion mit der equals-Funktion mit den Boxen vergleichen.

Etwas in einem TextField ausgeben natürlich mit der setText()-Methode des TextFields.

Ansonsten würde ich eine double oder float-Variable (da Kommazahl) für das Alter anlegen und da dann immer was abziehen.

Ich hoffe das hilft dir,
campino

 

[Marco13]

Ob es ihn weiter bringt oder nicht, hat er ja bisher noch nicht gesagt :roll:

Aber bei 40 ComboBoxen (oder so) für jede eine Klassenvariable anzulegen, die dann einzeln per Hand zu befüllen und per Hand irgendwo zu platzieren, um dann im ItemListener alle einzeln mit if(...equals) abzufragen ist imho einfach nur schlecht. Wenn die wenigstens (!) in einem Array liegen würden, OK. Aber ... was spricht denn gegen die Lösung, die ich vorgeschlagen hatte? War nur sinngemäß, und könnte man noch schöner machen, aber damit hat man die ganze Verwaltung einer Frage mit allen Auswahlmöglichkeiten und Alters-Abzügen in EINER Zeile - das ist praktisch, leicht wartbar, leicht erweiterbar... (Ich bin ja für Gegenarguemnte offen 😀 mit selbst fällt aber gerade keins ein :roll: )