从ActionListener方法访问类变量

Question

我正在使用Java中的gui使用JFrame和JPanel，以及ActionListener来编辑图像，当我点击一个按钮。目前，我有很多麻烦让我的JPanel类ButtonPanel与我的buffereimage图像交互。我试图显示图像的高度，但没有任何工作，我已经尝试了各种解决方案。我的ButtonPanel类的代码是:

class ButtonPanel extends JPanel
   {
        //JButton makeBlue;
      BufferedImage img;
       ButtonPanel(BufferedImage x)
      {
            final JButton makeBlue = new JButton ("Make Blue");
            img = x;
            //int width, height;
         setBackground(Color.RED);
         int height = 0;
         add(makeBlue);
         ActionListener action = 
             new ActionListener()
            {
                public void actionPerformed(ActionEvent e)
               {

                  if (e.getSource()== makeBlue)
                  {
                            //img = x;
                         height = img.getHeight();
//                       System.out.println(rgbvalue);
                             //System.out.println(width + " " + height);
                      setBackground(Color.GREEN);
                     repaint();
                  }
               }
            };
         makeBlue.addActionListener(action);
      }
   }

每当我尝试使用BufferedImage API中的方法来编辑图像时，例如在上面的代码中，我得到这个错误:

ImageEditorDeluxe.java:184: error: local variable height is accessed from within inner class; needs to be declared final
                         height = img.getHeight();

我已经玩了周围的地方，我初始化高度，但没有工作。任何帮助都会很感激。

我有另一个叫做ProgramWindow的类，我将图像编辑器的所有不同的jpanel添加到一个主要的JFrame，我认为这是我的问题可能在哪里，因为BufferedImage是空的。下面是ProgramWindow的代码:

class ProgramWindow extends JFrame  
   {
       ProgramWindow()
      {
         ImagePanel ip = new ImagePanel();
         ChooseFile cf = new ChooseFile();
         ButtonPanel bp = new ButtonPanel(ip.getImg());
         add(ip, BorderLayout.CENTER);
         add(cf, BorderLayout.SOUTH);
         add(bp, BorderLayout.WEST);
      }
   }

我得出的结论是，ProgramWindow中的ButtonPanel被传递了一个空参数，但我不知道为什么会这样。我在ImagePanel类中有一个叫做getImg的方法我把它作为ButtonPanel的参数调用。下面是ImagePanel的代码:

class ImagePanel extends JPanel
   {
      BufferedImage img;
       ImagePanel()
      {
         setBackground(Color.BLUE);  //to test
         final JButton button = new JButton ("Display picture");
         add(button);       
         ActionListener action = 
             new ActionListener()
            {
                public void actionPerformed(ActionEvent e)
               {
                  if (e.getSource()==button)
                  {
                     try
                     {
                        img = ImageIO.read(ChooseFile.getFile());
                     }
                         catch(IOException f)
                        {
                           f.printStackTrace();
                        }
                     repaint();
                  }
               }
            };
         button.addActionListener(action);
      }
       public void paintComponent(Graphics g)
      {
         super.paintComponent(g);
         if (img != null)
            g.drawImage(img, 0, 0, this);
      }
       public void setImage(BufferedImage i)
      {
         img = i;
         repaint();
      }
        public BufferedImage getImg()
        {
            return img;
        }
   }

Answer 1

你在构造函数中声明了高度，所以它是构造函数的局部。也许将其作为类的实例字段会更好。

public class ButtonPanel extends JPanel {
    private BufferedImage img;
    private int height;

话虽如此，既然你可以随时通过调用img.getHeight()来获得它，为什么还要有一个高度变量字段呢?

---

编辑
你的第二个问题，一个我们还不能帮助你，是你的img变量持有一个空引用。给定您的新代码，这表明ip.getImg()返回null，但不要相信我的话，测试它。在你的代码中加入这一行:

System.out.println("is ip.getImg() returning null? " + (ip.getImg()));

---

编辑2
当然你得到的是零。在用户有机会与ImagePanel交互之前，您正在从它提取图像。因为它只在用户按下按钮时获得图像，而且因为您在创建类时提取图像，在用户有机会做蹲下之前，唯一的可能性是，当您在创建类时提取图像时，您将获得null。解决方法:不要这样做