JComboBox getSelectedItem

Question

新来java和我无法看到为什么我的动作侦听器不工作的jcombobox。我想我已经跟着网上的其他例子getSelectedItem，但没有发生。 仅供参考，我的项目是一个单位转换器（使用MVC ..喜欢，但这不是我的优先事项）。 任何援助非常感谢。 谢谢，西蒙。

import java.awt.event.ActionEvent; 
    import java.awt.event.ActionListener; 

    import javax.swing.*; 


    public class UnitConverterView extends JFrame{ 

    //variables and components 
    private static final long serialVersionUID = -4673040337179571462L; 
    private JComboBox<String> unitCategory; 

    private JTextField fromValue = new JTextField(7); 
    private JComboBox<String> convertFrom; 
    private JLabel equalsLabel = new JLabel(" = "); 

    private JTextField toValue = new JTextField(7); 
    private JComboBox<String> convertTo; 


    //constructor 
    UnitConverterView(){ 
    //set up the view and components 

     JPanel unitPanel = new JPanel(); 

     this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     this.setSize(600,300); 

     String[] categories = {"Length","Weight","Speed","Temperature"}; 
     unitCategory = new JComboBox<>(categories); 

     String[] tofromValues = {" "}; 
     convertFrom = new JComboBox<>(tofromValues); 
     convertTo = new JComboBox<>(tofromValues); 


     unitPanel.add(unitCategory); 

     unitPanel.add(fromValue); 
     unitPanel.add(convertFrom); 
     unitPanel.add(equalsLabel); 
     unitPanel.add(toValue); 
     unitPanel.add(convertTo); 

     this.add(unitPanel); 

    } 

    //get value to convert from 
    public int getMeasurement() { 
     return Integer.parseInt(fromValue.getText()); 
    } 

    //listen for unitCategory to be selected 
    void addUnitCategoryListener(ActionListener listenForUnitCategory) { 
     unitCategory.addActionListener(listenForUnitCategory); 
    } 

class UnitCatListener implements ActionListener { 

     public void actionPerformed(ActionEvent e) { 

      /*String unitSelected = (String) unitCategory.getSelectedItem(); 
      if (e.getSource() == unitCategory) { 
       String unitName = (String) unitCategory.getSelectedItem(); 
       System.out.println("UnitName = " + unitName); 
       changeText(unitName); 
      }*/ 

      JComboBox cb = (JComboBox)e.getSource(); 
      String unitName = (String) cb.getSelectedItem(); 
      System.out.println("UnitName = " + unitName); 

     } 

     void changeText(String name) { 
      toValue.setText(name); 
     } 

    } 



} 

Answer 1

你已经声明了一个方法addUnitCategoryListener（）来向侦听器注册侦听器，但是你永远不会调用这个方法。这就是为什么听众永远不会注册。

在构造函数的末尾添加以下行，那么你应该罚款：

addUnitCategoryListener(new UnitCatListener()); 

Answer 2

要简单解决你的问题，叫你创建注册的组件上的监听方法。添加到您的构造函数：

addUnitCategoryListener(new UnitCatListener()); 

不过，也有你想知道的几件事：

• 的ItemListener通常会做比ActionListener为JComboBox更好的工作。如果用户选择已经选择的项目（基本上什么都不做），则前一个不会触发事件。通常在这些情况下你不需要做任何事情。

• 你并不需要一个额外的方法，只是注册的监听器，你可以直接添加到您的构造线

  unitCategory.addActionListener(new UnitCatListener()); 
  

  和删除自定义方法。

• 方法changeText和getMeasurement从不使用。
• 使用参数化类型：而不是JComboBox使用JComboBox<String>。
• 你不需要equalsLabel作为一个字段 - 一个局部变量会做 - 因为你不需要在以后的任何地方引用它（除非你计划在运行时改变标签的属性）。