JRadioButton aktualisieren nicht...

[Gelöschtes Mitglied 36296]

Hallo alle zusammen,

ich hab hier im Forum nach Lösungen gesucht und auch eine gefunden, die bei mir aber nicht wirklich funktioniert.
Deswegen schildere hier nochmal mein Problem und hoffe auf Hilfe

Ich hab ein Panel mit RadioButtons. In einem anderen Fenster kann ich die Namen ändern. Diese Namen sind in einer LinkedList<JLabel> gespeichert. (Das mit den JLabels brauch ich aus anderen Gründen, nich wundern ) Anhand der Labels sollen auch die RadioButtons benannt werden.
Hier mal die Methode:

public void updateRadioButtons(LinkedList<JLabel> labels,
			LinkedList<String> mod) {

		for (int i = 0; i < mod.size(); i++) {
			switch (mod.get(i)) {
			case "d":
				radioButton_panel.remove(radioButtonList.get(i));
				radio_button_group.remove(radioButtonList.get(i));
				radioButtonList.remove(i);
				mod.remove(i);
				i--;
				break;
			case "r":
				radioButtonList.get(i).setText(labels.get(i).getText());
				radioButtonList.get(i).validate();
				radioButtonList.get(i).updateUI();
				mod.set(i, "");
				break;
			case "n":
				radioButtonList.add(i,
						new JRadioButton(labels.get(i).getText()));
				radio_button_group.add(radioButtonList.get(i));
				radioButton_panel.add(radioButtonList.get(i));
				radioButtonList.get(i).addItemListener(this);
				mod.set(i, "");
				break;
			default:
				break;
			}
		}
		radioButton_panel.validate();
		radioButton_panel.updateUI();
		radioButton_panel.repaint();
		validate();
		updateUI();
		repaint();

	}

Die Methode bekommt die oben genannte Lables-Liste und eine LinkedList<String> mod. In der mod-Liste steht entweder
d für deleted (also gelöscht)
r für renamed (also Name geändert) oder
n für new (also neues Label angelegt)

das Proble ist, egal was ich auch versuche (sieht man vielleicht schon unten in der Methode mit allen möglichen repaint()- und validate()-Aufrufen), die RadioButtons ändern sich nicht...

ich hoffe es kann mir jemand helfen
Wenn weiterer Code gebraucht wird, einfach melden

Vielen Dank schon mal.
Boldi

 

[SlaterB]

unnötig komplizierte Methode, die letztlich doch nichts schafft,
ein sinnvoller Test ist etwas in Richtung

removeAll();
vielleicht noch
add(Label mit Zufallszahl/ aktueller Systemzeit als Inhalt);

und dann die validate()s usw. hinterher

alles hat aber gar keine Auswirkung, falls du dich in einem Panel befindest, welches gar nicht in der GUI enthalten ist,
schnell sind zwei unterschiedliche erstellt

vielleicht ist aber auch die Liste leer oder es wird genau der vorherige Zustand nachgebaut,
man kann nichts ablesen, man weiß es einfach nicht

ohne vollständiges Testprogramm ist wenig mehr zu sagen

 

[bERt0r]

Was afaik auch nicht gut geht ist während man per for schleife über eine Liste iteriert von dieser Liste Elemente zu entfernen.

 

[jgh]

bau ein KSKB das deinen geschilderten Fehler reproduziert...imho hast du zwei verschiedene Listen!

Hast du denn mal deine Liste von RadioButtons ausgegeben, also ob die nicht null sind, ob der Text auch da ist..etc?

 

[Gelöschtes Mitglied 36296]

Hallo SlaterB,

danke für die schnelle Antwort.
Vielleicht gebe ich mal noch weiter Erklärungen zum Thema
Diese RadioButtons sind ein Auswahlkriterium für eine spätere SQL-Abfrage an einen Server mit den Daten. Je nach RadioButton müssen dann verscheidene Datensammlungen in einer Tabelle angezeigt werden. Die Infos, welche Datensammlung dann auch gebraucht wird muss ich i-wo speichern. Wenn ich die RadioButtons jedes mal lösche und wieder neu aufsetze, geht mir diese Verbindung verloren... Das wollte ich mit dieser Methode verhindern

Das Panel ist das richtige und wird in ein JTabbedPane eingebunden. Dort erscheint es auch inklsuive der Initialisierungs-RadioButtons (queries-Bild). Wie gesagt, nur das updaten funktioniert nicht...
Gibt es noch andere Möglichkeiten zum updaten??

Auf dem Bild queries, hab ich jetzt einfach 8 Test-RadioButtons.
(Das "Country" und "Filter" könnt ihr ignorieren )
Das Bild Settings zeigt das Fenster, in dem ich die Dinger dann verwalten kann.

 

[Gelöschtes Mitglied 36296]

Hallo jgh,

hab in Eclipse mit dem Debugger die Listen (alle) genau angeschaut.
Die Listen stimmen, nur er ändert es bei den RadioButtons nich...

 

[Gelöschtes Mitglied 36296]

Was afaik auch nicht gut geht ist während man per for schleife über eine Liste iteriert von dieser Liste Elemente zu entfernen.

Ja ich weiß, die Liste wird danach aber nicht mehr gebraucht, die wird dann neu afgesetzt, sobald ich bei Settings "Ok" gedrückt hab. ich rbauch die nur temporär für die Änderungen die in Settings gerade vorgenommen wurden.
Danach wird sie wieder neu gemacht

 

[bERt0r]

Meine Kristallkugel flüsterte mir jetzt einfach mall dass die mods "r", "d" und "n" rename, delete und new darstellen sollen. Jetzt die Frage, warum so kompliziert?
Ich würde eine Liste Radiobuttons gleich deinem Settings Dialog übergeben und direkt im Action Listener des Buttons den jeweiligen Radiobutton ändern.

 

[Gelöschtes Mitglied 36296]

Meine Kristallkugel flüsterte mir jetzt einfach mall dass die mods "r", "d" und "n" rename, delete und new darstellen sollen.

Das steht so unter meinem Java Quellcode


Stimmt, dann könnte ich mir das mit der Liste sparen.
Ich werde das mal ausprobieren und dann Bericht erstatten

Vielen Dank schon mal an eure Hilfe

Boldi

 

[SlaterB]

> Das Panel ist das richtige und wird in ein JTabbedPane eingebunden.

solche Bekenntnisse stimmen nur so lange bis sie mal nicht stimmen,
gerne kann auch das Panel 'radioButton_panel' richtig sein, aber das Objekt, this, ist doppelt zu einem anderen usw.

die Möglichkeiten zur Kontrolle sind leicht zu durchdenken,
kein Debugger sondern z.B. ein System.out.println("neu") im Konstruktor zeigt messerscharf, wenn zwei erzeugt werden

jedenfalls für andere im Moment nicht gut nachvollziehbar

 

[Gelöschtes Mitglied 36296]

So hallo nochmal,

tut mir leid für die lange Wartezeit.
Hab das Problem jetzt behoben, wie ihr es mir geraten habt.
Hier mal der Ausschnitt aus dem Programm, um den es ging.
(Hab es als eigenständiges Projekt erzeugt)


Startklasse mit main():

package execution;
import radioButtonsPackage.RadioButtons;

public class StartingClass {

	public static void main(String[] args) {
		new RadioButtons();
		

	}

}

Frame, in dem die RadioButtons sind:

package radioButtonsPackage;

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;
import java.io.LineNumberReader;
import java.util.LinkedList;
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.JPanel;
import javax.swing.JRadioButton;
import javax.swing.JButton;

import settings.Settings;

@SuppressWarnings("serial")
public class RadioButtons extends JFrame implements ActionListener {

	private Settings theSettings;

	private LinkedList<JRadioButton> radioButtons = new LinkedList<JRadioButton>();
	private LinkedList<String> initialiseList = new LinkedList<String>();

	private ButtonGroup radio_button_group = new ButtonGroup();

	private JButton settings = new JButton("Settings");

	private JPanel radio_button_panel = new JPanel();

	public RadioButtons() {
		super();
		setTitle("Hauptframe");
		setLayout(new BoxLayout(this.getContentPane(), BoxLayout.Y_AXIS));
		initialiseList();
		theSettings = new Settings(this);
		add(initialiseRadioButtons());
		addSettingsButton();
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		pack();
		setVisible(true);

	}

	private void addSettingsButton() {
		add(settings);
		settings.addActionListener(this);
	}

	private void initialiseList() {
		try {
			initialiseList = readOutOfFile("Test.txt");
		} catch (FileNotFoundException e) {

			e.printStackTrace();
		}
	}

	private JPanel initialiseRadioButtons() {
		radio_button_panel.setLayout(new GridLayout(0, 3, 5, 5));
		for (int i = 0; i < initialiseList.size(); i++) {
			radioButtons.add(new JRadioButton(initialiseList.get(i)));
			radio_button_group.add(radioButtons.get(i));
			radio_button_panel.add(radioButtons.get(i));

		}
		radioButtons.get(0).setSelected(true);
		return radio_button_panel;

	}

	public void renameButtonAction(int index, String s) {
		radioButtons.get(index).setText(s);
		radioButtons.get(index).repaint();
		repaint();
	}

	public void deleteButtonAction(int index) {
		radio_button_panel.remove(radioButtons.get(index));
		radio_button_group.remove(radioButtons.get(index));
		radioButtons.remove(index);
		repaint();
		pack();
	}

	public void addNewButtonAction(String s) {
		radioButtons.add(new JRadioButton(s));
		radio_button_panel.add(radioButtons.getLast());
		radio_button_group.add(radioButtons.getLast());
		repaint();
		pack();
	}

	private LinkedList<String> readOutOfFile(String fn)
			throws FileNotFoundException {
		LinkedList<String> list = new LinkedList<String>();
		LineNumberReader in = null;
		try {
			in = new LineNumberReader(new FileReader(fn));
			String line;
			while ((line = in.readLine()) != null) {
				if (!line.trim().equals("") && !line.trim().equals("\n")) {
					list.add(line);
				}
			}
			in.close();
		} catch (IOException e) {

			Logger.getLogger(this.getName()).log(Level.SEVERE, null, e);
		}

		return list;
	}

	public LinkedList<String> getInitList() {
		return initialiseList;
	}

	@Override
	public void actionPerformed(ActionEvent e) {

		Object o = e.getSource();

		if (o.equals(settings)) {
			theSettings.setVisible(true);

		}

	}

}

Der Einstellungsframe:

package settings;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import java.util.LinkedList;

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;

import radioButtonsPackage.RadioButtons;

@SuppressWarnings("serial")
public class Settings extends JFrame implements ActionListener {

	private RadioButtons theRadioButtons;
	private RenameFrame theRenameFrame;

	private JPanel label_button_panel = new JPanel();
	private JPanel bottom_button_panel = new JPanel();

	private LinkedList<JLabel> labels = new LinkedList<JLabel>();
	private LinkedList<JButton> renameButtons = new LinkedList<JButton>();
	private LinkedList<JButton> deleteButtons = new LinkedList<JButton>();

	private JButton new_button = new JButton("New");
	private JButton ok_button = new JButton("Ok");

	private JTextField new_textfield = new JTextField(10);

	private int index;

	public Settings(RadioButtons rb) {
		super();
		setTitle("Settingsframe");
		theRadioButtons = rb;
		theRenameFrame = new RenameFrame(this);
		setLayout(new BoxLayout(this.getContentPane(), BoxLayout.Y_AXIS));
		buttonRegister();
		add(getLabelButtonPanel(theRadioButtons.getInitList()));
		add(getBottomButtonPanel());
		setDefaultCloseOperation(DISPOSE_ON_CLOSE);
		pack();
		setVisible(false);
	}

	private void buttonRegister() {
		ok_button.addActionListener(this);
		new_button.addActionListener(this);
		registerNewTextfield();

	}

	private JPanel getLabelButtonPanel(LinkedList<String> list) {
		label_button_panel.setLayout(new GridLayout(0, 3, 5, 5));

		for (int i = 0; i < list.size(); i++) {
			labels.add(new JLabel(list.get(i)));
			label_button_panel.add(labels.get(i));
			renameButtons.add(i, new JButton("Rename"));
			label_button_panel.add(renameButtons.get(i));
			renameButtons.get(i).addActionListener(this);
			deleteButtons.add(i, new JButton("Delete"));
			label_button_panel.add(deleteButtons.get(i));
			deleteButtons.get(i).addActionListener(this);

		}
		return label_button_panel;
	}

	private JPanel getBottomButtonPanel() {
		bottom_button_panel.add(new_button);
		bottom_button_panel.add(ok_button);

		return bottom_button_panel;
	}

	protected void renameButtonAction(String s) {
		if (!labels.get(index).getText().equals(s)) {
			labels.get(index).setText(s);
			theRadioButtons.renameButtonAction(index, s);

		}
	}

	private void deleteButtonAction() {
		label_button_panel.remove(labels.get(index));
		label_button_panel.remove(renameButtons.get(index));
		label_button_panel.remove(deleteButtons.get(index));
		renameButtons.remove(index);
		deleteButtons.remove(index);
		labels.remove(index);
		theRadioButtons.deleteButtonAction(index);
		pack();

	}

	private void registerNewTextfield() {
		new_textfield.addKeyListener(new KeyAdapter() {
			@Override
			public void keyPressed(KeyEvent e) {
				Object o = e.getKeyCode();

				if (o.equals(KeyEvent.VK_ENTER)) {
					String s = new_textfield.getText();
					label_button_panel.remove(new_textfield);
					labels.add(new JLabel(s));
					label_button_panel.add(labels.getLast());
					renameButtons.add(new JButton("Rename"));
					label_button_panel.add(renameButtons.getLast());
					deleteButtons.add(new JButton("Delete"));
					label_button_panel.add(deleteButtons.getLast());
					deleteButtons.getLast().addActionListener(Settings.this);
					renameButtons.getLast().addActionListener(Settings.this);
					repaint();
					pack();
					theRadioButtons.addNewButtonAction(s);

				}
			}
		});
	}

	@Override
	public void actionPerformed(ActionEvent e) {
		Object o = e.getSource();

		if (o.equals(ok_button)) {
			dispose();

		} else if (o.equals(new_button)) {
			label_button_panel.add(new_textfield);
			new_textfield.setText("New Name");
			new_textfield.selectAll();
			new_textfield.requestFocusInWindow();
			pack();
		} else {

			for (int i = 0; i < labels.size(); i++) {
				if (o.equals(renameButtons.get(i))) {
					theRenameFrame.setTextFieldText(labels.get(i).getText());
					theRenameFrame.setVisible(true);
					index = i;

					break;
				} else if (o.equals(deleteButtons.get(i))) {
					index = i;
					deleteButtonAction();
					break;
				}
			}

		}

	}
}

Der kleine Frame zum umbenennen:

package settings;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;




@SuppressWarnings("serial")
public class RenameFrame extends JFrame implements ActionListener{
	private JButton ok_button = new JButton("Ok");
	private JTextField textfield = new JTextField(20);
	private JLabel label = new JLabel("Type in Name");
	private JPanel help_panel = new JPanel();
	Settings theSettings;

	protected RenameFrame(Settings s) {
		super();
		theSettings = s;
		setLayout(new BoxLayout(this.getContentPane(), BoxLayout.Y_AXIS));
		registerTextfield();
		add(label);
		add(getHelpPanel());
		ok_button.addActionListener(this);
		setDefaultCloseOperation(DISPOSE_ON_CLOSE);
		pack();
		setVisible(false);
	}

	private void registerTextfield() {
		textfield.addKeyListener(new KeyAdapter() {
			@Override
			public void keyPressed(KeyEvent e) {
				Object o = e.getKeyCode();

				if (o.equals(KeyEvent.VK_ENTER)) {
					dispose();
					theSettings.renameButtonAction(textfield.getText());
					theSettings.repaint();
					theSettings.pack();
				}
			}
		});
	}

	private JPanel getHelpPanel() {
		help_panel.setLayout(new BoxLayout(help_panel, BoxLayout.X_AXIS));
		help_panel.add(textfield);
		help_panel.add(ok_button);

		return help_panel;
	}

	public void setTextFieldText(String s) {
		textfield.setText(s);
	}

	@Override
	public void actionPerformed(ActionEvent e) {
		Object o = e.getSource();

		if (o.equals(ok_button)) {
			dispose();
			theSettings.renameButtonAction(textfield.getText());
			theSettings.repaint();
			theSettings.pack();

		}

	}
}

kein Debugger sondern z.B. ein System.out.println("neu") im Konstruktor zeigt messerscharf, wenn zwei erzeugt werden

Ja das ist eine sehr gute Möglichkeit, die ich auch immer wieder nutze

Danke für die Hilfe hier

P.S.: In der Test.txt sind einfach nur test-Strings Habe sie trotzdem mal mit angehängt
Und das runnable jar-File gibt's auch noch

P.S.No.2: Wenn es noch Stilverbesserungsvorschläge gibt, bin ich gerne offen für konstruktive Kritik

Boldi