Durch ändern eines Buttons String füllen

[keeplaughing]

Hallo,

ich hab mit meinem Programm ein kleines Problemchen, das ich irgenwie nicht gelöst bekomme.

ich möchte, dass sich durch die Auswahl eines bzw. mehrerer RadioButtons, ein String, der über ein JLabel ausgegeben wird, ändert
mein bisheriger Ansatz

[java=42]import java.awt.*;
import javax.swing.*;


public class CoatCalculator extends JFrame{

public JPanel p = new JPanel();

String code;

public String setCode(){
if(foxRB.isSelected()){code += " ee"}
else if(heteroblackRB.isSelected()){code += " Ee";}
else if(homoblackRB.isSelected()){code += " EE";}
return code;
}


JLabel genotype = new JLabel("Genotyp: " + code);

JRadioButton foxRB = new JRadioButton("ee");
JRadioButton heteroblackRB = new JRadioButton("Ee");
JRadioButton homoblackRB = new JRadioButton("EE");

public CoatCalculator(){
setTitle("CoatColorCalculator by keeplaughing.deviantart.com");
p.setLayout(null);
add(p);

p.add(coatcolor);
coatcolor.setBounds(10,10,100,30);

p.add(genotype);
genotype.setBounds(10,30,250,30);

//ee, Ee, EE
p.add(foxRB);
foxRB.setBounds(10,60,50,25);
p.add(heteroblackRB);
heteroblackRB.setBounds(60,60,50,25);
p.add(homoblackRB);
homoblackRB.setBounds(110,60,50,25);

ButtonGroup e = new ButtonGroup();
e.add(foxRB);
e.add(heteroblackRB);
e.add(homoblackRB);
foxRB.setSelected(true);

setPreferredSize(new Dimension(800, 600));

setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
pack();


}

public static void main(String[] args) {
// TODO Auto-generated method stub
EventQueue.invokeLater(new Runnable() {

public void run() {
CoatCalculator myCoat = new CoatCalculator();
myCoat.setLocationRelativeTo(null);
myCoat.setVisible(true);
}
});

}

}[/code]

bei Genotyp steht nicht der code, sonden null

was mache ich falsch?

LG keeplaughing

 

[Volvagia]

Du musst den RadioButtons Listeners zuweißen (Item, Action gehen glaub ich auch), und den Label dann den entsprechenden String setzen.

btw. Am besten
Variablen
Konstruktor(s)
Methoden

Und Strings sind unveränderbar.

 

[keeplaughing]

ah, actionlistener, da hätte ich auch drauf kommen können, danke

 

[keeplaughing]

noch ein problemchen
also das lesen der RadioButtons geht wunderbar mit der ItemStateChanged-Funktion und dann ändert sich auch der code, was ich in der konsole sehen kann, ABER der neue Code wird nicht in meinem JLabel ausgegeben

[java=42]import java.awt.*;
import java.awt.event.*;
import javax.swing.*;


public class Test extends JFrame{

public JPanel p = new JPanel();

String code;

/*public String setCode(){
if(foxRB.isSelected()){code += " ee";}
else if(heteroblackRB.isSelected()){code += " Ee";}
else if(homoblackRB.isSelected()){code += " EE";}
return code;
}*/

JLabel genotype = new JLabel("Genotyp: " + code);

JRadioButton foxRB = new JRadioButton("ee");
JRadioButton heteroblackRB = new JRadioButton("Ee");
JRadioButton homoblackRB = new JRadioButton("EE");



public Test(){
setTitle("CoatColorCalculator by keeplaughing.deviantart.com");
p.setLayout(null);
add(p);

p.add(genotype);
genotype.setBounds(10,30,250,30);

ButtonGroup e = new ButtonGroup();
e.add(foxRB);
e.add(heteroblackRB);
e.add(homoblackRB);
foxRB.setSelected(true);

//ee, Ee, EE
p.add(foxRB);
foxRB.setBounds(10,60,50,25);
foxRB.addItemListener(new ItemListener(){
public void actionPerformed(ItemEvent e){
System.out.println("foxRB gewählt!");
}

@Override
public void itemStateChanged(ItemEvent arg0) {
if(foxRB.isSelected()==true){
code = "ee";
}
//System.out.println(code);
}
});
p.add(heteroblackRB);
heteroblackRB.setBounds(60,60,50,25);
heteroblackRB.addItemListener(new ItemListener(){
public void actionPerformed(ItemEvent e){
System.out.println("heteroblackRB gewählt!");
}

@Override
public void itemStateChanged(ItemEvent arg0) {
if(heteroblackRB.isSelected()==true){
code = "Ee";
}
//System.out.println(code);
}
});
p.add(homoblackRB);
homoblackRB.setBounds(110,60,50,25);
homoblackRB.addItemListener(new ItemListener(){
public void actionPerformed(ItemEvent e){
System.out.println("homoblackRB gewählt!");
}

@Override
public void itemStateChanged(ItemEvent arg0) {
if(homoblackRB.isSelected()==true){
code = "EE";
}
//System.out.println(code);
}
});
System.out.println(code);

setPreferredSize(new Dimension(200, 200));

setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
pack();
}

public static void main(String[] args) {
// TODO Auto-generated method stub
EventQueue.invokeLater(new Runnable() {

public void run() {
Test myCoat = new Test();
myCoat.setLocationRelativeTo(null);
myCoat.setVisible(true);
}
});

}

}[/code]

 

[ARadauer]

nur weil du den String änderst ändert sich nicht das Label du musst den text neu setzen...

 

[Mischka]

Wie wärs mit:

public void itemStateChanged(ItemEvent arg0) {
    jLabel1.setText("blabla");
}

 

[keeplaughing]

den text neu setzen?
tut mir leid, ich steh grade auf dem schlauch

meinst du ich soll den text auf dem JLabel komplett neu aufsetzen?
muss ich dafür eine Methode schreiben oder reicht ein einfaches ansprechen der variable?

 

[keeplaughing]

ich hab später nicht nur 3 buttons von denen 1 ausgewählt werden kann, sondern insgesamt 18 buttongroups die frei kombinierbar sein sollen, daher ist setText() irgendwie unpraktisch

ich bräuchte eher was das das bisherige label ergänzt

 

[Mischka]

Nagut, dann:

public void itemStateChanged(ItemEvent arg0) {
    jLabel1.setText(jLabel1.getText()+" blabla");
}

 

[keeplaughing]

das funktioniert schonmal, nur jetzt setzt das programm natürlich jedesmal, wenn ich einen anderen button anklicke den text dazu und es soll in der buttongroup immer nur eine auswahl möglich sein, d.h. wenn ich einen anderen button auswähle soll nichts dazu geschrieben, sondern ersetzt werden

sprich wähle button1 steht im label "abc"
wähle ich button2 steht "xyz" anstelle von "abc"

kann ich das JLabel auf den ursprungswert zurücksetzen?

EDIT: habs schon geschafft
[java=42]JLabel1.setText("ursprungstext");[/code]

 

[Mischka]

Also wenn du Gruppen von Buttons hast, die jeweils einen eigenen bereich im jlabel haben würde ich dir empfehlen entweder für jede Gruppe ein eigenes jlabel zu erstellen, oder etwas komplizierter: Ein array mit den momentan ausgewählten werten füttern und dann den inhalt des jlabels zu generieren.

Aber das ganze programm scheint mir eine nicht besonders creative lösung fürs problem. Wenn du mir das problem schliderst, könnte ich dir dabei helfen eine bessere lösung zu finden.

 

[keeplaughing]

die idee mit den arrays hatte ich auch schon, weil das ganze programm gerade dazu neigt ungefähr 1 million zeilen quellcode zu bekommen xD

ich erklär dir mal was das ganze irgendwann mal werden soll

es geht um genetik bei pferden und vererbung
das fenster soll in drei teile gesplittet sein (Mutter (links), Vater (rechts), Fohlen (unten)) und die bereiche von der Mutter und dem Vater sollen identisch aufgebaut sein
1 label auf dem mutter bzw vater steht, damit man weis wessen fellfarbe man grade auswählt
1 label für den namen der fellfarbe
1 label für den genetischen code der fellfarbe
und dann für jedes gen eine buttongroup (pro gen 3 auswahlmöglichkeiten (bei manchen nur 2, bei einem sogar 7) insgesamt 18 verschiedene gene

dann ein button um die gene von mutter und vater miteinander zu kombinieren
hier sollen wahrscheinlichkeit, jeweiliger gencode sowie dazugehöriger name alle möglichkeiten angezeit werden
ein button für die zufällige auswahl einer dieser möglichkeiten
ein button für die zufällige auswahl ob männlein oder weiblein

soweit die idee

im moment bin ich dabei den bereich mutter bzw vater zu schreiben

 

[Mischka]

Also statt dem haufen buttons würde ich da eher zu ner combo-box, also nem drop down menu raten. dann wird das ganze übersichtlicher und du brauchst kein label mehr, dass verändert wird, da das ausgewählte combo-box feld ja angezeigt wird.

alles in allem sollten dafür ca. 500-600zeilen reichen.

dann würde ich noch mutter+vater und fohlen ein eigenes jpanel spendieren, die in eigenen Klassen ausgelagert werden. das bringt zusätzlich ein bisschen übersichtlichkeit in den quellcode.

Das scheint kein großes programm zu sein, also in 1-2 stunden machbar.

falls du noch hilfe brauchst, dann schreib mir eine pm. verfolge relativ selten die beiträge von anderen.

Ach und da du anfänger zu seinen scheinst würde ich dir für die gui netbeans als ide empfehlen. damit ist das schnell gemacht.

 

[keeplaughing]

ich merke grade, das ganze wird sehr kompliziert

neben meinem "großen" programm, in dem ich schon alle RadioButtons und Buttons etc. platziert habe , hab ich ein kleineres programm zum testen der funktionen

ich habe jetzt 2 buttongroups und um die zu kombinieren muss ich in meinen if-schleifen zum ändern der JLabels für jeden einzelnen button eine bedingung schreiben

gehts auch einfacher?

auf diesem weg brauch ich für jeden button einen itemListener mit ItemStateChange in dem ich für jede kombination eine eigene if-schleife habe

Bsp.
[java=42]import java.awt.*;
import java.awt.event.*;
import javax.swing.*;


public class Test extends JFrame{

public JPanel p = new JPanel();

JLabel coatcolor = new JLabel("Coatcolor: ");
JLabel color = new JLabel("Chestnut");
JLabel genotype = new JLabel("Genotype: ");
JLabel code = new JLabel("ee");

JRadioButton foxRB = new JRadioButton("ee");
JRadioButton heteroblackRB = new JRadioButton("Ee");
JRadioButton homoblackRB = new JRadioButton("EE");

JRadioButton noagoutiRB = new JRadioButton("aa");
JRadioButton heterobayRB = new JRadioButton("Aa");

public Test(){
setTitle("CoatColorCalculator by keeplaughing.deviantart.com");
p.setLayout(null);
add(p);

p.add(coatcolor);
coatcolor.setBounds(10,10,70,30);

p.add(genotype);
genotype.setBounds(10,30,70,30);

p.add(color);
color.setBounds(80,10,150,30);

p.add(code);
code.setBounds(80,30,150,30);

p.add(foxRB);
foxRB.setBounds(10,60,50,25);

p.add(heteroblackRB);
heteroblackRB.setBounds(60,60,50,25);

p.add(homoblackRB);
homoblackRB.setBounds(110,60,50,25);

p.add(noagoutiRB);
noagoutiRB.setBounds(10,90,50,25);

p.add(heterobayRB);
heterobayRB.setBounds(60,90,50,25);

ButtonGroup e = new ButtonGroup();
e.add(foxRB);
e.add(heteroblackRB);
e.add(homoblackRB);
foxRB.setSelected(true);

ButtonGroup a = new ButtonGroup();
a.add(noagoutiRB);
a.add(heterobayRB);
noagoutiRB.setSelected(true);

//ee, Ee, EE

foxRB.addItemListener(new ItemListener(){
public void actionPerformed(ItemEvent e){
}

@Override
public void itemStateChanged(ItemEvent arg0) {
if(foxRB.isSelected()==true && noagoutiRB.isSelected() == true){
code.setText("ee");
color.setText("Fuchs");
}
}
});

heteroblackRB.addItemListener(new ItemListener(){
public void actionPerformed(ItemEvent e){
}

@Override
public void itemStateChanged(ItemEvent arg0) {
if(heteroblackRB.isSelected()==true && noagoutiRB.isSelected() == true){
code.setText("Ee");
color.setText("Rappe");
}
else if(heterobayRB.isSelected()==true && heteroblackRB.isSelected()== true){
code.setText("Ee Aa");
color.setText("Brauner");
}
}
});

homoblackRB.addItemListener(new ItemListener(){
public void actionPerformed(ItemEvent e){
}

@Override
public void itemStateChanged(ItemEvent arg0) {
if(homoblackRB.isSelected()==true && noagoutiRB.isSelected() == true){
code.setText("EE");
color.setText("Rappe");
}
else if(heterobayRB.isSelected()==true && homoblackRB.isSelected()== true){
code.setText("EE Aa");
color.setText("Brauner");
}
}
});

noagoutiRB.addItemListener(new ItemListener(){
public void actionPerformed(ItemEvent e){
}

@Override
public void itemStateChanged(ItemEvent arg0) {
if(foxRB.isSelected()==true && noagoutiRB.isSelected() == true){
code.setText("ee");
color.setText("Fuchs");
}
if(heteroblackRB.isSelected()==true && noagoutiRB.isSelected() == true){
code.setText("Ee");
color.setText("Rappe");
}
if(homoblackRB.isSelected()==true && noagoutiRB.isSelected() == true){
code.setText("EE");
color.setText("Rappe");
}
}
});

heterobayRB.addItemListener(new ItemListener(){
public void actionPerformed(ItemEvent e){
}

@Override
public void itemStateChanged(ItemEvent arg0) {
if(heterobayRB.isSelected()==true && heteroblackRB.isSelected()== true){
code.setText("Ee Aa");
color.setText("Brauner");
}
else if(heterobayRB.isSelected()==true && homoblackRB.isSelected()== true){
code.setText("EE Aa");
color.setText("Brauner");
}
else if(heterobayRB.isSelected()==true && foxRB.isSelected()== true){
code.setText("ee Aa");
color.setText("Fuchs");
}
}
});

setPreferredSize(new Dimension(300, 300));

setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
pack();
}

public static void main(String[] args) {
// TODO Auto-generated method stub
EventQueue.invokeLater(new Runnable() {

public void run() {
Test myCoat = new Test();
myCoat.setLocationRelativeTo(null);
myCoat.setVisible(true);
}
});

}

}[/code]

 

[keeplaughing]

ich hab grade ein kleines, vielleicht auch größeres problem

anscheinend hab ich meinen ItemStateChange(){...} überladen, bzw den speicher überschritten, habe allerdings noch lange nicht alle möglichkeiten beisammen

wie kann ich das problem beheben?