java - ColorAction扩展了AbstractAction

我正在尝试使用子类ColorAction扩展AbstractAction类。此类处理ActionPerformed方法，方法是以字符串格式获取用户输入，并将其与colorKey中的Color值，字符串键的HashMap和Color Objects匹配。如果没有找到具有指定键的Color对象，它将抛出异常。

对于我的一生，我不知道该怎么做。也许我的游戏关闭了？我是否应该接受应为颜色名称的String输入，并检查指定的String是否与其中一个Color Objects相匹配？我应该使ActionEvent等于用户输入颜色吗？我有很多想法在脑海中奔波，没有一个确定的赢家。

public class ColorAction extends AbstractAction{
    Map<String, Color> colorMap;
    private String input;

    public ColorAction(String in) {
        input = in;
        colorMap = new HashMap<>(); //how can i enter a value that is a full range of colors?
    }

    @Override
    public void actionPerformed(ActionEvent e){
        if(colorMap.containsKey(input){
            //...
        }
    }
}

最佳答案

如果我正确理解了您的目标，colorMap是您预定义的独立地图，因此应该是静态的。

这是填充静态地图的一种可能方法：

private static final Map<String, Color> colorMap = createColorMap();

private static Map<String, Color> createColorMap() {
    Map<String, Color> colorMap = new HashMap<>();

    //Fill colorMap

    return colorMap;
}

另外，通常可以通过在e.getActionCommand()方法中调用actionPerformed来获取用户输入。我不明白为什么要将用户输入传递给ColorAction的构造函数。

@Override
public void actionPerformed(ActionEvent e){
    //Assumes the map doesn't have any keys containing upper case letters.
    //By calling toLowerCase, if the user inputs "Red", it would still match
    //the key "red".
    String input = e.getActionCommand().toLowerCase();

    if(colorMap.containsKey(input){
        //...
    }
}

也就是说，您可能根本不需要构造函数。