Per Buttonklicks einer Liste Wörter hinzufügen - Wie umsetzen?

[BadassPanda]

Hallo Leute,
bin recht neu in der Javaprogrammierung und habe bis her zwei Javabücher gelesen und ein gelungenes Spiel in Android programmiert. Zurzeit habe ich von meinem Chef in der Eisdiele den Auftrag bekommen eine Anwendung zu schreiben, bei der er per Buttonklicks Eissorten auswählen kann, die er an einem Tag gemacht hat, damit diese später auf einem kleinen Zettelchen ausgedruckt und in die Speisekarten geklebt werden. Nun bin ich so weit, dass ich ein Swingframe erstellt habe, welches 35 mit den Eissorten beschriftete Buttons und einen Drucken-Button, enthält. Dazu habe ich noch programmiert, dass wenn sie angeklickt werden, sie ihre Farbe in grau ändern und wieder abwählbar sind. Nun wollte ich noch hinzufügen, dass wenn z.B. der Button "Banane" angeklickt wird, "Banane" einem Textfeld hinzugefügt wird und wenn weitere Buttons geklickt werden, dann sollen die Sorten untereinander eingetragen werden. Da die Buttons auch abwählbar sind, soll die abgewählte Eissorte auch aus der Liste verschwinden. Glaubt ihr es wäre machbar? Und wie müsste ich das umsetzen? Ich habs bisher so versucht, dass ich ein Textfield angelegt habe und falls der Button geklickt wird,

textfield.setText("Banane");

ausgeführt wird. Und beim Abwählen

textfield.setText(null)

aber damit komme ich bei 35 Buttons nicht weit Deswegen wollte ich euch hier um Hilfe bitten. Ich bedanke mich schonmal

 

[Flown]

Sowas in der Art. Ich hab dir das jetzt mit eine JTextArea und mit eine JList (was meiner Meinung nach die Bessere Lösung ist um sowas darzustellen).

Spoiler: Code

import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.List;
import javax.swing.AbstractListModel;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;

public class UITest {

  public static void main(String... args) {
    SwingUtilities.invokeLater(new Runnable() {

      @Override
      public void run() {
        new UITest();
      }
    });
  }

  public UITest() {
    JFrame frame = new JFrame();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setLocationByPlatform(true);
    frame.setSize(900, 600);

    JPanel mainPanel = new JPanel(new BorderLayout());

    JPanel listFieldPanel = new JPanel(new GridLayout(1, 2));
    MyListModel<String> listModel = new MyListModel<>();
    JList<String> list = new JList<>(listModel);

    JTextArea area = new JTextArea();
    listFieldPanel.add(area);
    listFieldPanel.add(list);

    mainPanel.add(listFieldPanel, BorderLayout.CENTER);

    JPanel buttonPanel = new JPanel();
    for (int i = 0; i < 10; i++) {
      String item = "Button " + i;
      JButton btn = new JButton(item);
      btn.addActionListener(new ActionListener() {
        boolean isPressed = false;

        @Override
        public void actionPerformed(ActionEvent e) {
          handle(item, listModel, area, isPressed);
          isPressed = !isPressed;
        }
      });
      buttonPanel.add(btn);
    }
    mainPanel.add(buttonPanel, BorderLayout.SOUTH);

    frame.add(mainPanel);

    frame.setVisible(true);
  }

  private <T> void handle(T s, MyListModel<T> listModel, JTextArea area, boolean delete) {
    if (delete) {
      listModel.remove(s);
    } else {
      listModel.add(s);
    }
    StringBuilder sb = new StringBuilder();
    for (int i = 0; i < listModel.getSize(); i++) {
      sb.append(listModel.getElementAt(i));
      sb.append('\n');
    }
    area.setText(sb.toString());
  }

  class MyListModel<T> extends AbstractListModel<T> {

    private final List<T> data = new ArrayList<>();

    public boolean add(T d) {
      if (!data.contains(d) && data.add(d)) {
        int pos = data.size() - 1;
        fireIntervalAdded(this, pos, pos);
        return true;
      }
      return false;
    }

    public boolean remove(T d) {
      int pos = data.indexOf(d);
      if (pos != -1 && data.remove(d)) {
        fireIntervalRemoved(this, pos, pos);
        return true;
      }
      return false;
    }

    @Override

    public int getSize() {
      return data.size();
    }

    @Override
    public T getElementAt(int index) {
      return data.get(index);
    }

  }
}

 

[BadassPanda]

Genial Klappt echt super! Ich muss mich im Code noch ein bisschen zurecht finden aber ich denke ich schaffe das schon Aber ansonsten ist es so wie ich es mir vorgestellt habe. Dankeschön

 

[BadassPanda]

@Flown
Hey Ich habe in deinem Code versucht eine weitere JTextArea area2 hinzuzufügen und wollte, dass sie den Text von area übergeben bekommt und habe

area2.setText(area.getText());

ausprobiert, jedoch ändert sich area2 nicht, wenn area neue Werte bekommt. Wenn ich aber

area2.setText("Test");

eingebe steht da auch "Test". Woran liegt es und wie könnte ich es beheben? Danke schonmal
*EDIT: Jetzt habe ich die erste Codezeile in den Handler eingefügt und area2 nimmt den Wert von area an, jedoch um einen Buttonklick verzögert. Wenn ich also button1 klicke steht da nichts und wenn ich dann einen beliebigen Button, z.B. button2 klicke, steht da button1. Wie kann ich es zeitgleich ablaufen lassen?

 

[Joose]

Wenn ich also button1 klicke steht da nichts und wenn ich dann einen beliebigen Button, z.B. button2 klicke, steht da button1. Wie kann ich es zeitgleich ablaufen lassen?

Das Problem ist das sich deine Oberfläche neu zeichnen muss um solche Änderungen anzuzeigen. Wenn du nicht explizit dafür sorgst das sie neu gezeichnet wird, passiert das erst wenn mit der UI wieder interagiert wird (daher erst beim 2.Buttonklick).

Die unsaubere Variante die UI neu zu zeichnen wäre das JFrame Objekt per "setVisible(true)" auf sichtbar zu setzen (auch wenn es das schon ist).
Die saubere und bessere Variante wäre mit Hilfe des SwingWorkers!

 

[Flown]

Poste doch mal deinen ganzen Codeversuch, dann kann man dir auch weiterhelfen.

 

[BadassPanda]

Also ich hab halt nur ganz wenig dazu hinzugefügt.. Das hab ich dann mit "NEU" markiert.

import java.awt.BorderLayout;import java.awt.Color;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.List;


import javax.swing.AbstractListModel;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;
 
public class UITest {
     JTextArea area2 = new JTextArea();                                      //NEU
  public static void main(String... args) {
    SwingUtilities.invokeLater(new Runnable() {
 
      @Override
      public void run() {
        new UITest();
      }
    });
  }
 
  public UITest() {
    JFrame frame = new JFrame();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setLocationByPlatform(true);
    frame.setSize(900, 600);
 
    JPanel mainPanel = new JPanel(new BorderLayout());
 
    JPanel listFieldPanel = new JPanel();
    JPanel listFieldPanel2 = new JPanel();                                        //NEU
    listFieldPanel2.setBounds(0, 0, 200, 500);                                    //NEU        
    listFieldPanel2.setBackground(Color.WHITE);                               //NEU
    mainPanel.add(listFieldPanel2);                                                    //NEU
    MyListModel<String> listModel = new MyListModel<>();
    JList<String> list = new JList<>(listModel);
 
    JTextArea area = new JTextArea();
    listFieldPanel.add(area);
    listFieldPanel2.add(area2);                                                    //NEU
    
 
    mainPanel.add(listFieldPanel);
 
    JPanel buttonPanel = new JPanel();
    for (int i = 0; i < 10; i++) {
      String item = "Button " + i;
      JButton btn = new JButton(item);
      btn.addActionListener(new ActionListener() {
        boolean isPressed = false;
 
        @Override
        public void actionPerformed(ActionEvent e) {
          handle(item, listModel, area, isPressed);
          isPressed = !isPressed;
        }
      });
      buttonPanel.add(btn);
    }
    mainPanel.add(buttonPanel, BorderLayout.SOUTH);
 
    frame.add(mainPanel);
 
    frame.setVisible(true);
  }
 
  private <T> void handle(T s, MyListModel<T> listModel, JTextArea area, boolean delete) {
      area2.setText(area.getText());                                     //NEU
    if (delete) {
      listModel.remove(s);
    } else {
      listModel.add(s);
    }
    StringBuilder sb = new StringBuilder();
    for (int i = 0; i < listModel.getSize(); i++) {
      sb.append(listModel.getElementAt(i));
      sb.append('\n');
    }
    area.setText(sb.toString());
  }
 
  class MyListModel<T> extends AbstractListModel<T> {
 
    private final List<T> data = new ArrayList<>();
 
    public boolean add(T d) {
      if (!data.contains(d) && data.add(d)) {
        int pos = data.size() - 1;
        fireIntervalAdded(this, pos, pos);
        return true;
      }
      return false;
    }
 
    public boolean remove(T d) {
      int pos = data.indexOf(d);
      if (pos != -1 && data.remove(d)) {
        fireIntervalRemoved(this, pos, pos);
        return true;
      }
      return false;
    }
 
    @Override
 
    public int getSize() {
      return data.size();
    }
 
    @Override
    public T getElementAt(int index) {
      return data.get(index);
    }
 
  }
}

Wie löse ich das Problem denn mit dem Swingworker? Danke schonmal

 

[Flown]

So ich glaub du hast keine Ahnung von einem bzw. dem Programmfluss. Wie soll es sich denn verhalten und was macht es jetzt gerade?

 

[BadassPanda]

Na deswegen treibe ich mich ja hier auf den Foren rum um es zu lernen
Naja es soll jedenfalls immer der Inhalt von area an area2 übergeben werden. Und wenn ich den Befehl in den Handler platziere, dann wird es erst aktualisiert, wenn ich den nächsten Button gedrückt habe. Es soll aber zeitgleich stattfinden. Wenn ich "Button1" an area übergebe soll area es gleichzeitig an area2 übergeben? Wie mache ich das? Ich bin ganz bestimmt kein Profi, gerade weil ich mich mit Runnable nicht auskenne, aber genau deswegen frage ich hier auch.
LG

 

[Flown]

Ich weiß, das ich das oft sage, aber dir fehlen Grundlagen. Du musst den Programmfluss verstehen, wann wo wie was passiert. Ich möchte eigentlich nicht dir den Ablauf dieses Programm schildern, sondern du sollst konkrete Fragen stellen und du bekommst von mir/uns konkrete Antworten. Aber dir Programmieren beizubringen, zählt hier nicht zu meinen/unseren Aufgaben.

Spoiler: Code

import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.List;
import javax.swing.AbstractListModel;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.SwingUtilities;

public class UITest {

  public static void main(String... args) {
    SwingUtilities.invokeLater(new Runnable() {

      @Override
      public void run() {
        new UITest();
      }
    });
  }

  public UITest() {
    JFrame frame = new JFrame();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setLocationByPlatform(true);
    frame.setSize(900, 600);

    JPanel mainPanel = new JPanel(new BorderLayout());

    JPanel listFieldPanel = new JPanel(new GridLayout(1, 3));
    MyListModel<String> listModel = new MyListModel<>();
    JList<String> list = new JList<>(listModel);

    JTextArea area = new JTextArea();
    JTextArea area2 = new JTextArea(); // NEU
    area2.setDocument(area.getDocument()); // NEU

    listFieldPanel.add(area);
    listFieldPanel.add(list);
    listFieldPanel.add(area2); // NEU

    mainPanel.add(listFieldPanel, BorderLayout.CENTER);

    JPanel buttonPanel = new JPanel();
    for (int i = 0; i < 10; i++) {
      String item = "Button " + i;
      JButton btn = new JButton(item);
      btn.addActionListener(new ActionListener() {
        boolean isPressed = false;

        @Override
        public void actionPerformed(ActionEvent e) {
          handle(item, listModel, area, isPressed);
          isPressed = !isPressed;
        }
      });
      buttonPanel.add(btn);
    }
    mainPanel.add(buttonPanel, BorderLayout.SOUTH);

    frame.add(mainPanel);

    frame.setVisible(true);
  }

  private <T> void handle(T s, MyListModel<T> listModel, JTextArea area, boolean delete) {
    if (delete) {
      listModel.remove(s);
    } else {
      listModel.add(s);
    }
    StringBuilder sb = new StringBuilder();
    for (int i = 0; i < listModel.getSize(); i++) {
      sb.append(listModel.getElementAt(i));
      sb.append('\n');
    }
    area.setText(sb.toString());
  }

  class MyListModel<T> extends AbstractListModel<T> {

    private final List<T> data = new ArrayList<>();

    public boolean add(T d) {
      if (!data.contains(d) && data.add(d)) {
        int pos = data.size() - 1;
        fireIntervalAdded(this, pos, pos);
        return true;
      }
      return false;
    }

    public boolean remove(T d) {
      int pos = data.indexOf(d);
      if (pos != -1 && data.remove(d)) {
        fireIntervalRemoved(this, pos, pos);
        return true;
      }
      return false;
    }

    @Override

    public int getSize() {
      return data.size();
    }

    @Override
    public T getElementAt(int index) {
      return data.get(index);
    }

  }
}

Hier ist ein Code der dir das was du - wie ich glaube - haben möchtest.

 

[BadassPanda]

Achsoo es lag also am Anfang daran, dass ich statt setDocument, setText benutzt habe Aber ja, macht genau das was ich wollte. Besten Dank dafür

 

[Flown]

Es geht auch mit dem #setText(...) aber du musst den Befehl an der richtigen Stelle setzen!