无法将按钮图标与另一个图标进行比较

Question

我正在重写 actionListener。除了“播放按钮”之外的所有按钮都有一个作为图标附加的图像（按钮 1、按钮 2、按钮 3）。每次按下按钮时，它应该将其图标与准备好的 ImageIcon“livePicture”进行比较，如果它们相同，则应该使用“Escaped()”方法。否则，程序将运行“Died()”方法。 但是，至少在当前代码中，它只使用“Died()”。我想，这意味着比较图像的 ifs 有问题，但这是我在互联网上找到的唯一比较方法。 另外，请记住，这是我的第一个项目，所以看起来有点混乱。

import javax.imageio.ImageIO;
import javax.swing.*;
import javax.swing.border.Border;
import java.awt.*;
import java.awt.event.ActionListener;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import java.util.Collections;
import java.util.Vector;

public class Frame
{
private final int WIDTH = 1024;
private final int HEIGHT = 768;

private JFrame frame;
private JPanel panel;
private JLabel human;
private JTextArea text;
private JTextArea deathMessage;
private ImageIcon livePicture;
private JButton button1;
private JButton button2;
private JButton button3;
private GridBagConstraints gbc;
private ActionListener actionListener;
private JButton playButton;
private Border border = BorderFactory.createEmptyBorder();
private Font font = new Font(Font.MONOSPACED, Font.PLAIN, 20);

public Frame()
{
    Quest survival = new Quest();

    actionListener = e -> {
        if (e.getSource() == playButton) //playbutton works fine
        {
            if (!survival.IsEmpty())
            {
                AppendQuest(survival.GetQuest());
                survival.RemoveQuest();
            }
            else
            {
                Escaped();
            }
        }
        else if (e.getSource() == button1) //button1 action
        {
            if (button1.getIcon().toString() != livePicture.toString())
            {
                Died();
            }
            else
            {
                if (!survival.IsEmpty())
                {
                    AppendQuest(survival.GetQuest());
                    survival.RemoveQuest();
                }
                else
                {
                    Escaped();
                }
            }
        }
        else if (e.getSource() == button2) //button2 action
        {
            if (button2.getIcon().toString() != livePicture.toString())
            {
                Died();
            }
            else
            {
                if (!survival.IsEmpty())
                {
                    AppendQuest(survival.GetQuest());
                    survival.RemoveQuest();
                }
                else
                {
                    Escaped();
                }
            }
        }
        else if (e.getSource() == button3) //button3 action
        {
            if (button3.getIcon().toString() != livePicture.toString())
            {
                Died();
            }
            else
            {
                if (!survival.IsEmpty())
                {
                    AppendQuest(survival.GetQuest());
                    survival.RemoveQuest();
                }
                else
                {
                    Escaped();
                }
            }
        }
    };
    //I left the rest of the constructor for bonus info
    frame = new JFrame();
    panel = new JPanel();
    gbc = new GridBagConstraints();
    human = new JLabel(ImageSize(200, 200, "res/human.png"));
    text = new JTextArea("You have lost in the forest. Now you have to find " +
            "your way back.");
    FormatText(text);
    deathMessage = new JTextArea();
    frame.setTitle("Shady Path");
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setSize(WIDTH, HEIGHT);
    frame.setLocationRelativeTo(null);
    frame.getContentPane().setBackground(Color.BLACK);
    frame.setResizable(false);

    playButton = new JButton();
    playButton.addActionListener(actionListener);
    playButton.setFont(font);
    playButton.setText("Play");
    playButton.setForeground(Color.WHITE);
    playButton.setBackground(Color.BLACK);
    playButton.setBorder(border);

    panel.setLayout(new GridBagLayout());
    panel.setOpaque(false);
    gbc.anchor = GridBagConstraints.PAGE_START;
    gbc.gridwidth = GridBagConstraints.REMAINDER;
    panel.add(human, gbc);

    gbc.insets = new Insets(30, 0, 0, 0);
    gbc.weightx = 1;
    gbc.fill = GridBagConstraints.HORIZONTAL;
    panel.add(text, gbc);

    gbc.fill = GridBagConstraints.VERTICAL;
    gbc.insets = new Insets(50, 0, 68, 0);
    panel.add(playButton, gbc);
    frame.add(panel);
    frame.setVisible(true);
}

public void AppendQuest(Vector<String> event)
{
    panel.removeAll();

    panel.add(human, gbc);
    gbc.insets = new Insets(0, 0, 30, 0);
    text.setText(event.remove(0));
    FormatText(text);
    panel.add(text, gbc);
    deathMessage.setText(event.remove(0));
    FormatText(deathMessage);
    livePicture = ImageSize(50, 50, event.remove(0));

    Collections.shuffle(event);
    ImageIcon picture1 = ImageSize(50, 50, event.get(0)); //setting button1
    button1 = new JButton();
    button1.addActionListener(actionListener);
    button1.setIcon(picture1);
    button1.setBorder(border);
    ImageIcon picture2 = ImageSize(50, 50, event.get(1)); //setting button2
    button2 = new JButton();
    button2.addActionListener(actionListener);
    button2.setIcon(picture2);
    button2.setBorder(border);
    ImageIcon picture3 = ImageSize(50, 50, event.get(2)); //setting button3
    button3 = new JButton();
    button3.addActionListener(actionListener);
    button3.setIcon(picture3);
    button3.setBorder(border);
    gbc.gridwidth = GridBagConstraints.HORIZONTAL;
    gbc.insets = new Insets(50, 360, 100, 0);
    panel.add(button1, gbc);
    gbc.insets = new Insets(50, 77, 100, 77);
    panel.add(button2, gbc);
    gbc.insets = new Insets(50, 0, 100, 360);
    panel.add(button3, gbc);

    panel.revalidate();
    panel.repaint();
}

private void Escaped()
{
    //Unnecessary info
}

private void Died()
{
    //Unnecessary info
}
//This just resizes the images
private ImageIcon ImageSize(int x, int y, String fileName)
{
    BufferedImage baseImg = null;
    try {
        baseImg = ImageIO.read(new File(fileName));
    } catch (IOException e) {
        e.printStackTrace();
    }
    Image resizedImg = baseImg.getScaledInstance(x, y, Image.SCALE_SMOOTH);
    ImageIcon IconImg = new ImageIcon(resizedImg);
    return IconImg;
}

private void FormatText(JTextArea baseText)
{
    //Unnecessary info
}
}

编辑： 这也是“AppendQuest”中可以作为“事件”的向量的示例

Vector<String> items2 = new Vector<>();
items2.add("You are kind of disoriented. What will you use to find the right way?" +
            " moss, sun or tree barks");
items2.add("Unfortunately you didn't orient yourself well enough. Now, you " +
            "will roam through the forest forever.");
items2.add("res/orientation_sun.png");
items2.add("res/orientation_moss.png");
items2.add("res/orientation_sun.png");
items2.add("res/orientation_tree_bark.png");

Answer 1

您可以使用 .equals(Object) 函数比较对象：

if(!button.getIcon().equals(livePicture))
{
   Died();
}
else
{...}

== 运算符检查对象的身份或本机类型（例如 int）的值。 这意味着：

int nr1 = 1;
int nr2  = 1;
if(nr1 == nr2) {...} //true -> int is a native type

String str1 = "test";
String str2 = "test";
if(str1 == str2) {...} //false -> Same content but not same objects
if(str1.equals(str2)) {...} //true -> Same content, different objects

//编辑： 另一个问题可能是您在创建 livePicture 时从矢量中删除了 image-url：

livePicture = ImageSize(50, 50, event.remove(0));

当您创建按钮时，该 url 不再在列表中。结果是按钮将永远不会具有与您的 livePicture 相同的图像，除非您正在更改它（是吗？）。