java - 在 Java 中使用访问器方法的条件语句

Question

my_holder.getColor()应该默认返回颜色 RED 并且这些条件语句不起作用。我认为我的语法错误。

基本上我希望能够在单击它时突出显示一个paintswatch，如果样本颜色与 中的相同ColorHolder，我希望它取消选择样本。

import wheels.users.*;
import java.awt.Color;
import java.awt.event.MouseEvent;

public class PaintSwatch extends Ellipse
{
    private ColorHolder _holder; 
    public int _i;

    public PaintSwatch(Color c,ColorHolder holder){
        super(c);
        _holder = holder;
        _i = 1;
    }

    public void mouseClicked(MouseEvent e){
        if (_holder.getColor() != super.getColor()){
            super.setFrameColor(Color.BLACK);
            super.setFrameThickness(3);
            _holder.setColor(super.getColor());
        }

        if (_holder.getColor() == super.getColor()){
            super.setFrameThickness(10);
            super.setFrameColor(Color.WHITE);
            _holder.setColor(Color.RED);
        }
    }
}

这是我的 ColorHolder 类：

import java.awt.Color;

public class ColorHolder implements Colorable
{
    private Color _currentColor;

    public ColorHolder()
    {
        _currentColor = Color.RED;
    }

    public Color getColor(){
        return _currentColor;
    }

    public void setColor(Color c){
        _currentColor = c;
    }
}

Answer 1

除了使用 equals 之外，您还应该使用 if else 而不是两个不同的 ifs ：

if (_holder.getColor() != super.getColor()) {
   ...
   _holder.setColor(super.getColor());
}

if (_holder.getColor() == super.getColor()) { 
   ...
}

请注意，第二个 if 条件将始终评估为 true - 如果持有者的颜色不等于 super 的颜色，它将在第一个 if 条件中设置。

if (_holder.getColor().equals(super.getColor())) {
   ... do sth
} else {
   ... do sth else
}

还要确保 holder 和 super 不返回 null 作为颜色或明确检查它。

Answer 2

类内PaintSwatch替换

if (_holder.getColor() != super.getColor())

至

if (!_holder.getColor().equals(super.getColor()))