2016-11-28 2 views
2

Мне нужно создать прослушиватель, который изменяет размер шрифта, когда я нажимаю на JMenuItem. Тем не менее, у меня разные метки для разных частей, поэтому, когда я меняю размер шрифта, это вызывает у меня ошибку, когда метки для других частей программы вызывают, но неактивны. В принципе, я не могу сразу изменить все шрифты.Изменить размер шрифта с помощью ActionListener

Я пробовал использовать if-утверждения, но по какой-то причине работает только первый! Остальные дают мне nullPointerException.

Вот мой слушатель код до сих пор ..

import java.awt.BorderLayout; 
import java.awt.Color; 
import java.awt.Font; 
import java.awt.event.ActionEvent; 
import java.awt.event.ActionListener; 
import java.awt.event.KeyEvent; 
import java.awt.event.MouseEvent; 
import java.awt.event.MouseListener; 
import java.text.DecimalFormat; 

import javax.swing.ImageIcon; 
import javax.swing.JButton; 
import javax.swing.JComboBox; 
import javax.swing.JFrame; 
import javax.swing.JLabel; 
import javax.swing.JMenu; 
import javax.swing.JMenuBar; 
import javax.swing.JMenuItem; 
import javax.swing.JOptionPane; 
import javax.swing.JPanel; 
import javax.swing.JTextField; 
import javax.swing.UIManager; 










public class financeFormula extends JFrame { 



    private JMenu fileMenu; 
     private JMenu textMenu; 
     private JMenuItem exitItem; 
     private JMenuItem fontSize; 
     private JMenuItem help; 
     private JPanel presentValuePanel; 
     private JPanel financeFinderPanel;// A panel container 
     private JLabel principalMessage; 
     private JLabel yearlyRateMessage; 
     private JLabel termYearMessage; 
     private JPanel simpleInterestPanel; 
     private JPanel doublingTimePanel; 
     private JPanel compoundInterestPanel; 
     private JLabel NONEMessage; 
     private JLabel imageLabel; 
     private JLabel label;// A message to display 
     private JMenuBar menuBar; 
     private JTextField principalText; // To hold user input 
     private JButton calcButton;  // Performs calculation 

     private final int WINDOW_WIDTH = 600; // Window width 
     private final int WINDOW_HEIGHT = 600; // Window height 
     private JTextField yearlyRateText; 
     private JTextField termYearText; 
     DecimalFormat dc =new DecimalFormat("####0.00"); //formater 
     private JComboBox financeBox; 
     double principal = 400.0; 
     double yearlyRate = .04; 
     double termYears = 4.0; 


     private String[] financeFormulas = { "NONE", "Present value", "Simple interest", 
      "Doubling time", "Compound interest", "Decaf"}; 


     //financeFormulaClass financeFormula = new financeFormulaClass(principal, yearlyRate, termYears); 

     /** 
     * Constructor 
     */ 



     public financeFormula() 
     { 
      // Call the JFrame constructor. 
      super("Finance Class"); 

      // Set the size of the window. 
      setSize(WINDOW_WIDTH, WINDOW_HEIGHT); 

      // Specify what happens when the close 
      // button is clicked. 
      setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 

      // Build the panel and add it to the frame. 
      financeFinderPanel(); 
      setLayout(new BorderLayout()); 

      //buildPanel(); 

      menuBar = new JMenuBar(); 
      //buildFileMenu(); 
      buildTextMenu(); 
      //menuBar.add(fileMenu); 
      menuBar.add(textMenu); 
      setJMenuBar(menuBar); 

      // Add the panel to the frame's content pane. 
      add(financeFinderPanel); 
      //add(panel); 

      // Display the window. 
      setVisible(true); 
     } 

     private void financeFinderPanel() 
     { 
      // Create a panel to hold the combo box. 


      financeFinderPanel = new JPanel(); 

      //create label 
      label = new JLabel("Pick your formula"); 
      //ImageIcon funnyImage = new ImageIcon("funny.gif"); 
      //imageLabel = new JLabel(); 
      //imageLabel.setLocation(50, 55); 

      label.setForeground(Color.BLUE); 
      // Create the combo box 
      financeBox = new JComboBox(financeFormulas); 
      //imageLabel.setIcon(funnyImage); 
      // Register an action listener. 
      financeBox.addActionListener(new financeBoxListener()); 
      exitItem = new JMenuItem("Exit"); 
      exitItem.setMnemonic(KeyEvent.VK_X); 

      // Add the combo box to the panel. 
      financeFinderPanel.add(financeBox); 
      financeFinderPanel.add(label); 
     // financeFinderPanel.add(imageLabel); 


     } 


     private void buildMenuBar() 
     { 
      // Create the menu bar. 
      menuBar = new JMenuBar(); 

      // Create the file and text menus. 
      //buildFileMenu(); 
      buildTextMenu(); 

      // Add the file and text menus to the menu bar. 
      //menuBar.add(fileMenu); 
      menuBar.add(textMenu); 

      // Set the window's menu bar. 
      setJMenuBar(menuBar); 
     } 



     public void buildTextMenu(){ 

      fontSize = new JMenuItem("Large font"); 
      fontSize.setMnemonic(KeyEvent.VK_2); 
      fontSize.addActionListener(new FontListener()); 
      textMenu = new JMenu("Text"); 
      textMenu.add(fontSize); 



     } 

     private void presentValuePanel() 
     { 
      // Create the label, text field, and button components. 
      principalMessage = new JLabel("principal"); 
      principalText = new JTextField(10); 
      yearlyRateMessage = new JLabel("Yearly Rate"); 
      yearlyRateText = new JTextField(10); 
      termYearMessage = new JLabel("Term Year"); 
      termYearText = new JTextField(10); 
      calcButton = new JButton("Calc Present Value"); 
      //fontButton = new JButton("Enlarge Font"); 

      // Add an action listener to the button. 
      //calcButton.addActionListener(new CalcButtonListener()); 

      //calcButton.addActionListener(new financeFormListener()); 
      //fontButton.addActionListener(new FontListener()); 
      // Create a panel to hold the components. 
      presentValuePanel = new JPanel(); 

      // Add the label, text field, and button to the panel. 
      presentValuePanel.add(principalMessage); 
      presentValuePanel.add(principalText); 
      presentValuePanel.add(yearlyRateMessage); 
      presentValuePanel.add(yearlyRateText); 
      presentValuePanel.add(termYearMessage); 
      presentValuePanel.add(termYearText); 
      presentValuePanel.add(calcButton); 
      //presentValuePanel.add(fontButton); 
     } 

      private void simpleInterestPanel(){ 

        principalMessage = new JLabel("principal"); 
        principalText = new JTextField(10); 
        yearlyRateMessage = new JLabel("Yearly Rate"); 
        yearlyRateText = new JTextField(10); 
        termYearMessage = new JLabel("Term Year"); 
        termYearText = new JTextField(10); 
        calcButton = new JButton("Calc Simple Interest"); 

        simpleInterestPanel = new JPanel(); 
       // calcButton.addActionListener(new financeFormListener()); 

        simpleInterestPanel.add(principalMessage); 
        simpleInterestPanel.add(principalText); 
        simpleInterestPanel.add(yearlyRateMessage); 
        simpleInterestPanel.add(yearlyRateText); 
        simpleInterestPanel.add(termYearMessage); 
        simpleInterestPanel.add(termYearText); 
        simpleInterestPanel.add(calcButton); 



      } 




     private class financeBoxListener implements ActionListener 
     { 
      public void actionPerformed(ActionEvent e) 
      { 

       String selection = (String) financeBox.getSelectedItem(); 
       if(selection.equals("Present value")){ 
        //financeFinderPanel.removeAll(); 
        presentValuePanel(); 
        add(presentValuePanel, BorderLayout.NORTH); 
        financeFinderPanel.removeAll(); 
        pack(); 

       } 



      else if(selection.equals("Simple interest")){ 

       simpleInterestPanel(); 
       add(simpleInterestPanel, BorderLayout.NORTH); 
       financeFinderPanel.removeAll(); 
       pack();      
       } 

      } 
     } 


       private class FontListener implements ActionListener{ 

        @Override 
        public void actionPerformed(ActionEvent e) { 
         // TODO Auto-generated method stub 


         String actionCommand = e.getActionCommand(); 

         if(presentValuePanel.isEnabled() && actionCommand.equals("Large font")){ 
          principalMessage.setFont(new Font("sans-serif", Font.PLAIN, 22)); 
         } 
         else if(simpleInterestPanel.isEnabled() && actionCommand.equals("Large font")){ 
          principalMessage.setFont(new Font("sans-serif", Font.PLAIN, 22)); 
         } 


        } 
       } 


     static void main(String[] args) { 
     // TODO Auto-generated method stub 


     new financeFormula(); 

    } 

} 

ПРИМЕЧАНИЕ: По некоторым причинам кнопки увеличения шрифт еще showing..That не то, что я прошу, я прошу, когда я меняю разные формулы и нажмите JMenuItem Large font, он работает только для текущей стоимости, а не простой интерес.

+1

Ваш вопрос связан с проблемами с взаимосвязанности вашего кода, вызывающего ошибки, даже NPE. Я не знаю, как мы сможем догадаться, что может быть неправильным в вашей структуре программы на основе вашего описания и небольшого фрагмента кода, а это значит, что вы захотите опубликовать больше кода и рассказать нам больше , Нет, мы не хотим видеть вашу всю программу, но это помогло бы, если бы вы могли создать ** небольшую ** (достаточно небольшую, чтобы опубликовать как отредактировать свой вопрос, а не ссылку), компилируемую и исполняемую программу, которая демонстрирует ваша ошибка непосредственно для нас, [mcve]. –

+0

Хорошо, я сделал небольшую программу, которая делает то, что я прошу ... Когда вы нажимаете текущее значение, а затем меняете шрифт с помощью JMenuItem, он работает, но когда вы делаете это с простым интересом, это не ... Также, пожалуйста, игнорируйте JButton «Увеличить шрифт» Idk, почему он все еще появляется. –

+0

У вас есть более одного объекта JLabel, которые помещаются в переменную mainMessage, и что важно для JVM - это используемый объект, а не переменная. Таким образом, ваш ActionListener меняет шрифт на объекте, на котором он ссылается только. Вы должны сделать вашу программу более простой и не использовать одну и ту же переменную для нескольких объектов. –

ответ

0

Вы не говорите нам о самой важной части вашей проблемы - вы получаете исключение NullPointerException. Опять же, это важная информация, которая является ключом к пониманию того, что не так, и ее решение, и не должно быть исключено из ваших текущих или будущих вопросов.

Ключом к решению NPE является поиск линии, которая его выбрасывает, а затем тщательно проверяйте для переменных, которые являются нулевыми. Затем загляните в свой код, чтобы узнать, почему и исправить его. Ваш NPE выбрасывает здесь:

if(presentValuePanel.isEnabled() && actionCommand.equals("Large font")){ 

и связанно с presentValuePanel переменного является нулем (поскольку при выборе простого интереса, то simpleInterestPanel создается, но presentValuePanel нет), а затем вы пытаетесь вызвать метод выключения этой нулевой переменной - не разрешено.

Решение состоит в том, чтобы не допустить этого, чтобы не допускать вызовы методов для нулевых переменных. Один из вариантов заключается в создании всех ваших JPanels при инициализации программы, и это будет допустимый вариант, если ваша программа подключена, чтобы иметь возможность изменять другую формулу, что я думаю, что будущая программа будет делать. Так что все в порядке. В противном случае другой вариант должен проверить значение null перед вызовом кода. Например, увидеть изменения в программу:

public class financeFormula extends JFrame { 
    public static final String LARGE_FONT = "Large Font"; // !! constant 

    // .... 

    public void buildTextMenu() { 
     fontSize = new JMenuItem(LARGE_FONT); // !! Use constant 
     fontSize.setMnemonic(KeyEvent.VK_2); 
     fontSize.addActionListener(new FontListener()); 
     textMenu = new JMenu("Text"); 
     textMenu.add(fontSize); 
    } 

    // .... 

    private class FontListener implements ActionListener { 
     @Override 
     public void actionPerformed(ActionEvent e) { 

      // !! block change 
      String actionCommand = e.getActionCommand(); 
      if (actionCommand.equals(LARGE_FONT)) { // !! check for Large Font, and if pressed set the Fonts 
       if (presentValuePanel != null && presentValuePanel.isEnabled()) { // Check for null! 
        principalMessage.setFont(new Font("sans-serif", Font.PLAIN, 22)); 
       } else if (simpleInterestPanel != null && simpleInterestPanel.isEnabled()) { // Check for null! 
        principalMessage.setFont(new Font("sans-serif", Font.PLAIN, 22)); 
       } 
      } 
      // !! block change 
     } 
    } 

Примечания при использовании строк для проверки равенства, всегда использовать строковую константу, чтобы избежать тривиальных, но трудно отлаживать орфографические или капитализацию ошибок.

Более подробную информацию о NullPointerExceptions, пожалуйста, прочитайте: What is a NullPointerException, and how do I fix it?

Смежные вопросы