Как создать мультиселект комбо

Question

Я хочу создать комбинированный блок с множественным выбором в Swing, который отображает элементы, выбранные пользователем, разделенные точкой с запятой или другим символом.

Например:

Выбрать статьи (ы) <- отображает выбор пользователя
Выберите статьи
Нет статьи

Если пользователь выбрал "a" и "the", "a;" будет отображаться вместо "Выбрать статьи (и)".

Я пытался запрограммировать такую ​​комбинацию, но моя проблема в том, что "Выбор статей" не заменяется текущим выбором пользователя.

Вы можете увидеть только что-то вроде:
Выбрать статьи (и) <- отображает выбор пользователя (не заменяется на "a; the")
а;
Нет статьи

Вот мой код:

import java.awt.*;
import java.awt.event.*;
import java.util.*;
import java.util.List;
import javax.swing.*;
import javax.swing.plaf.basic.BasicComboBoxRenderer;

public class MultiSelectionComboBox {

    private DefaultComboBoxModel model;

    private JPanel getContent() {
        Object[] items = { "Select article(s)", "No article", "a", "the" };

        model = new DefaultComboBoxModel(items);
        JComboBox combo = new JComboBox(model);

        SelectionManager manager = new SelectionManager();
        manager.setNonSelectable(items[0]);

        Renderer renderer = new Renderer(manager);
        combo.addActionListener(manager);
        combo.setRenderer(renderer);

        JPanel panel = new JPanel();
        panel.add(combo);
        return panel;
    }

    class SelectionManager implements ActionListener {
        JComboBox combo = null;
        private List<Object> selectedItems = new ArrayList<Object>();
        private Object nonSelectable;

        public void setNonSelectable(Object val) {
            nonSelectable = val;
        }
        public void actionPerformed(ActionEvent e) {
            if (combo == null) {
                combo = (JComboBox) e.getSource();
            }
            Object item = combo.getSelectedItem();
            // Toggle the selection state for item.  
            if (selectedItems.contains(item)) {
                selectedItems.remove(item);
            } else if (!item.equals(nonSelectable)) {
                selectedItems.add(item);
            }

            combo.setSelectedIndex(0);
        }

        public List<Object> getSelectedItems() {
            return selectedItems;
        }
    }

    class Renderer extends BasicComboBoxRenderer {
        SelectionManager selectionManager;

        public Renderer(SelectionManager sm) {
            selectionManager = sm;
        }

        public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected,
                boolean cellHasFocus) {
            setFont(list.getFont());

            if (index == 0) { // first item shows currently selected items delimited by ;
                StringBuffer firstItem = new StringBuffer();
                for (Object sel : selectionManager.getSelectedItems()) {
                    firstItem.append(sel + "; ");
                }
                if (firstItem.toString().endsWith("; ")) {
                    firstItem.deleteCharAt(firstItem.length() - 2);
                }
                setText((value == null) ? "" : firstItem.toString());
            } else {// other items
                setText((value == null) ? "" : value.toString());
            }

            return this;
        }
    }

    public static void main(String[] args) {
        JFrame f = new JFrame();
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.getContentPane().add(new MultiSelectionComboBox().getContent());
        f.setSize(300, 145);
        f.setLocation(200, 200);
        f.setVisible(true);
    }
}

Я знаю, что комбо не предназначено для множественного выбора, но в моем случае я не вижу лучшего элемента пользовательского интерфейса, потому что я хочу поместить такие комбинации в предложениях. Например: "Где находится | a; клавиша?"

Answer 1

В вашем устройстве визуализации ячеек вы предполагаете, что индекс 0 - это выбранное значение, это не так. Это на самом деле -1 (или, точнее, это индекс, используемый для представления значения редактора)

public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected, boolean cellHasFocus) {
    setFont(list.getFont());

    if (index == -1 && selectionManager.getSelectedItems().size() > 0) {
        StringBuffer firstItem = new StringBuffer();
        for (Object sel : selectionManager.getSelectedItems()) {
            firstItem.append(sel + "; ");
        }
        if (firstItem.toString().endsWith("; ")) {
            firstItem.deleteCharAt(firstItem.length() - 2);
        }
        setText((value == null) ? "" : firstItem.toString());
    } else {// other items
        setText((value == null) ? "" : value.toString());
    }

    return this;
}

Answer 2

Основная проблема в вашем коде состоит в том, что вы упаковываете ответственность модели (== решаете о выборе) в ваше представление (== рендерер + код действия). Вместо этого решите их, где они принадлежат, вот модель. Ниже приведена очень упрощенная реализация, просто чтобы указать направление:

@SuppressWarnings({ "unchecked", "rawtypes" })
public static class MyComboBoxModel extends AbstractListModel
    implements ComboBoxModel {
    public static Object NONE = "none"; 
    List values = new ArrayList();
    List selected = new ArrayList();

    public MyComboBoxModel(Object... values) {
        for (Object object : values) {
            this.values.add(object);
        }
    }

    @Override
    public int getSize() {
        return values.size();
    }

    @Override
    public Object getElementAt(int index) {
        return values.get(index);
    }
    @Override
    public void setSelectedItem(Object anItem) {
        if (anItem == null || anItem == NONE) {
            if (selected.isEmpty()) return;
            selected.clear();
        } else {
            boolean removed = selected.remove(anItem);
            if (!removed) {
                selected.add(anItem);
            }
        }
        fireContentsChanged(this, -1, -1);
    }

    @Override
    public Object getSelectedItem() {
        return selected;
    }

}

// use
MyComboBoxModel model = new MyComboBoxModel(MyComboBoxModel.NONE , "a", "the", "other");
model.setSelectedItem(null);
JComboBox box = new JComboBox(model);

Даже с помощью стандартного рендерера мы видим его работоспособность:-) Пользовательский рендерер может дополнительно настроить визуальное представление по мере необходимости - без какой-либо логики, связанной с данными.