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作爲一個字段 - 一個局部變量會做 - 因爲你不需要在以後的任何地方引用它（除非你計劃在運行時改變標籤的屬性）。