Array Label

[Guest]

guten morgen ich habe ein arraylabel von 5 und 7 checkboxen nun möchte ich wenn die checkbox selected ist das erst geprüft wird welches label frei ist und dann in das niedrigste reinschreibt, jedoch sollte auch geprüft werden ob der checkboxen text schon in einem anderem label drin steht. habe das nun immer mit einzel vergleichen gemacht jedoch komm ich da auf fast 400 zeilen quellcode (hab alles immer mit IF verglichen) nun suche ich wie das ganze kürzer geht. bin für jede anregung offen danke

 

[Guest]

ok das mit dem sortieren scheint langsam zu klappen nun habe ich eine frage? und zwar ich fülle per checkboxen ein textarea nun wenn ich das häckchen rausmachen soll der text aus dem area verschwinden, was auch funktioniert bis auf den fall das die checkbox unter anderen ist (also wenn sie zwischen zwei anderen ist kann man sie raus löschen) jedoch nicht wenn sie ganz unten ist :? oder hab ich eine abfrage vergessen?

 else if (!cb.isSelected()) {
	
			if (ta.getText().equals(cb.getText())) {
				ta.setText(ta.getText().replaceFirst(cb.getText(), ""));
			}

			else {
				System.out.println("Es wurde der " + cb.getText()
						+ " deaktiviert!");
				ta.setText(ta.getText().replaceFirst(cb.getText() + "\n", ""));
			}
		}

 

[SlaterB]

> 400 zeilen quellcode

mit Schleifen und Arrays und/ oder Unteroperationen für gleiche Codeteile arbeiten,
gib mal bisschen Code-Beispiele, falls noch aktuell, aber nicht mehr als 40 Zeilen

> nun habe ich eine frage?

vielleicht ein Fehler von dir beim Einfügen oder vielleicht verschluckt die Textarea das letzte \n oder ähnliche unvorhersehbare Dinge

wenn du es selber machen willst:
schaffe dir eine Situation, in der er es nicht klappt und untersuche dann ta.getText(),
gibt z.B. jeden Buchstaben einzeln aus,
teste replace ohne das \n am Ende usw.

wenn es andere machen sollen:
poste ein kleines vollständiges Testprogram nur mit der Textarea und 1-3 Checkboxen

 

[Guest]

so bin nun auf label umgestiegen, jedoch seht ihr schon das ich da schon sehr viele vergleiche für 3 labels machen muss eine frage kann man das ganz auch verkürzen?

package Tests;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;

import javax.swing.JCheckBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JTextArea;

public class beispiel extends JFrame implements ItemListener, ActionListener {
	private final JTextArea ta = new JTextArea();

	static JCheckBox[] cb = new JCheckBox[3];

	static JLabel[] lb = new JLabel[3];

	public beispiel() {
		super();
		for (int i = 0; i < 3; ++i) {
			lb[i] = new JLabel();
			lb[i].setBounds(10, 110 + (i * 20), 100, 25);
			add(lb[i]);
		}
		for (int i = 0; i < 3; ++i) {
			cb[i] = new JCheckBox("Feld " + i);
			cb[i].setBounds(10, i * 20, 100, 25);
			cb[i].addItemListener(this);
			add(cb[i]);
			addWindowListener(new WindowAdapter() {
				public void windowClosing(WindowEvent event) {
					System.exit(0);
				}
			});
		}
	}

	public void prüfen(ItemEvent e) {
		JCheckBox cb = (JCheckBox) e.getSource();
		if (cb.isSelected()) {
			if (lb[0].getText().equals("")) {
				lb[0].setText(cb.getText());
			} else if (lb[1].getText().equals("")) {
				lb[1].setText(cb.getText());
			} else if (lb[2].getText().equals("")) {
				lb[2].setText(cb.getText());
			}
		}
		if (!cb.isSelected()) {
			if (cb.getText().equals(lb[0].getText())) {
				lb[0].setText("");
				if (lb[0].getText().equals("") && !lb[1].getText().equals("")) {
					lb[0].setText(lb[1].getText());
					lb[1].setText("");
					if (lb[1].getText().equals("")
							&& !lb[2].getText().equals("")) {
						lb[1].setText(lb[2].getText());
						lb[2].setText("");
					}
				}
			}
			if (cb.getText().equals(lb[1].getText())) {
				lb[1].setText("");
				if (lb[1].getText().equals("") && !lb[2].getText().equals("")) {
					lb[1].setText(lb[2].getText());
					lb[2].setText("");
				}
			}
			if (cb.getText().equals(lb[2].getText())) {
				lb[2].setText("");
			}
		}
	}

	public void itemStateChanged(ItemEvent e) {
		JCheckBox cb = (JCheckBox) e.getSource();
		if (cb.isSelected()) {
			prüfen(e);
		} else if (!cb.isSelected()) {
			prüfen(e);
		}
	}

	public static void main(String[] args) throws Exception {
		beispiel wnd = new beispiel();
		wnd.setBounds(250, 250, 150, 250);
		wnd.setLayout(null);
		wnd.setVisible(true);
		wnd.repaint();
	}

	public void actionPerformed(ActionEvent arg0) {
		// TODO Auto-generated method stub
		
	}
}

 

[SlaterB]

das ist in der Tat recht sinnloser Code,
solche Reihenfolgen kann man sicherlich auf verschiedene Weisen lösen, hier ist eine:

public class beispiel
    extends JFrame
    implements ItemListener, ActionListener
{
    private final JTextArea ta = new JTextArea();

    static int anzahl = 5;
    static JCheckBox[] cb = new JCheckBox[anzahl];
    static JLabel[] lb = new JLabel[anzahl];
    static List<String> visibleStrings = new ArrayList<String>(anzahl);

    public beispiel()
    {
        super();
        for (int i = 0; i < anzahl; ++i)
        {
            lb[i] = new JLabel();
            lb[i].setBounds(10, 110 + (i * 20), 100, 25);
            add(lb[i]);
        }
        for (int i = 0; i < anzahl; ++i)
        {
            cb[i] = new JCheckBox("Feld " + i);
            cb[i].setBounds(10, i * 20, 100, 25);
            cb[i].addItemListener(this);
            add(cb[i]);
            addWindowListener(new WindowAdapter()
                {
                    public void windowClosing(WindowEvent event)
                    {
                        System.exit(0);
                    }
                });
        }
    }

    public void prüfen(ItemEvent e)
    {
        JCheckBox cb = (JCheckBox)e.getSource();
        if (cb.isSelected())
        {
            visibleStrings.add(cb.getText());
        }
        else
        {
            visibleStrings.remove(cb.getText());
        }

        if (visibleStrings.size() > anzahl)
        {
            throw new RuntimeException("bug");
        }

        for (int i = 0; i < anzahl; i++)
        {
            if (i < visibleStrings.size())
            {
                lb[i].setText(visibleStrings.get(i));
            }
            else
            {
                lb[i].setText("");
            }
        }
    }

    public void itemStateChanged(ItemEvent e)
    {
        JCheckBox cb = (JCheckBox)e.getSource();
        if (cb.isSelected())
        {
            prüfen(e);
        }
        else if (!cb.isSelected())
        {
            prüfen(e);
        }
    }

    public static void main(String[] args)
        throws Exception
    {
        beispiel wnd = new beispiel();
        wnd.setBounds(250, 250, 150, 250);
        wnd.setLayout(null);
        wnd.setVisible(true);
        wnd.repaint();
    }

    public void actionPerformed(ActionEvent arg0)
    {
    }
}

 

[Guest]

danke für deine hilfe jedoch kann ich dein quellcode net nutzen! die funktion bzw. die variable visibleStrings visibleStrings ist immer falsch

 

[SlaterB]

alle Vorkommnisse von <String> entfernen

und bei visibleStrings.get(i) auf String casten