如何计算列表中自定义数据类型的值？

Question

我看到了一些关于hashMaps的东西，但我们的工作还没有做得那么远。请保持您的答案和有用的建议，尽可能简单，你可以管理。

我有一个自定义的数据类型，它已经被制作好了，并且可以完美地工作，叫做color。该类型的唯一值是Color.Blue、Color.Red、Color.Yellow和Color.Green。

我的任务是返回颜色。如果列表中的颜色比其他颜色多，则返回颜色。如果列表中的颜色比其他颜色多，则返回颜色。如果列表中的颜色比其他颜色多，则返回颜色。

我进行了研究，并提出了以下代码：

public Color callColor(List<Card> hand) {

    int blueCards = Collections.frequency(hand, Color.BLUE);
    int redCards = Collections.frequency(hand, Color.RED);
    int greenCards = Collections.frequency(hand, Color.GREEN);
    int yellowCards = Collections.frequency(hand, Color.YELLOW);
    Color changeColorTo = Color.NONE;

    if ((blueCards > redCards) || (blueCards > greenCards) || (blueCards > yellowCards)) {
        changeColorTo = Color.BLUE;
    }

    if ((redCards > blueCards) || (redCards > greenCards) || (redCards > yellowCards)) {
        changeColorTo = Color.RED;
    }

    if ((greenCards > redCards) || (greenCards > blueCards) || (greenCards > yellowCards)) {
        changeColorTo = Color.GREEN;
    }

    if ((yellowCards > redCards) || (yellowCards > greenCards) || (yellowCards > blueCards)) {
        changeColorTo = Color.YELLOW;
    }
    return changeColorTo;
}

但是这段代码导致BlueCard、RedCard、greenCards和yellowCards都为0，而它们绝对不应该为零。

因此，在本例中，我的collections实现根本不起作用。救命啊！

Answer 1

输入列表包含卡片而不是颜色。

所以可能的问题解决方法是先将你的卡片列表转换成颜色列表：

hand.stream().map(Card::getColor).collect(Collectors.toList());

结果，您将得到颜色列表，所以您现在可以在它上使用Collection.Frequency了，而不是在初始卡片列表上使用Collection.Frequency。

但是，还有很多其他方法可以解决问题，比如使用另一个集合。

Answer 2

您将一个列表 传递给该方法，但您将在该列表中搜索某个颜色的频率。这就是为什么所有的计数都等于0。