1

我不知道如何正确使用 Comparator 界面。

注意：我不想使用我在大多数代码上看到的“单行”比较器实现，这意味着：

Collections.sort(deck, new Comparator<Card>() {
  @Override
  public int compare(Card o1, Card o2) {
   return WHATEVER;
  });

同样，我不想使用它，因为我要对许多不同的 ArrayList 进行排序，每次都这样做似乎是浪费空间。我想以某种方式使其工作，将我的 compareTo 方法写在一个地方一次。

我尝试了很多不同的东西，但我是实现接口的新手，所以我可能遗漏了一些东西。

这是我的 Card 类和它的比较方法。（注意：让我为您节省一些时间，并说我对使用枚举来简化代码和比较不感兴趣，我只是想学习如何正确使用 Comparator。我知道它看起来很糟糕但我必须使用一些已经写好了代码。）

import java.util.*;
public class Card implements Comparator<Card>
{
    private String number;
    private int value;
    private int finalValue;
    private String suit;
    private Player owner;

    public Card(String number, String suit)
    {
        this.number=number;
        switch (number){
            case "2": this.value = 2;
                break;
            case "3": this.value = 3;
                break;
            case "4": this.value = 4;
                break;
            case "5": this.value = 5;
                break;    
            case "6": this.value = 6;
                break;
            case "7": this.value = 7;
                break;
            case "8": this.value = 8;
                break;    
            case "9": this.value = 9;
                break;
            case "10": this.value = 10;
                break;
            case "J": this.value = 11;
                break;
            case "Q": this.value = 12;
                break;
            case "K": this.value = 13;
                break;
            case "A": this.value = 14;
                break;
            }

        this.suit=suit;
        switch (suit){
            case "Spades": this.finalValue = this.value*4;
                break;
            case "Hearts": this.finalValue = this.value*5;
                break;
            case "Clubs": this.finalValue = this.value*2;
                break;
            case "Diamonds": this.finalValue = this.value*3;
                break;
            }
    }

    public int compare(Card card1, Card card2)
    {
        Integer suitValue1;
        if (card1.getSuit() == "Hearts")
        suitValue1=5;
        else if (card1.getSuit() == "Spades")
        suitValue1=4;
        else if (card1.getSuit() == "Diamonds")
        suitValue1=3;
        else
        suitValue1=2;

        Integer suitValue2;
        if (card2.getSuit() == "Hearts")
        suitValue2=5;
        else if (card2.getSuit() == "Spades")
        suitValue2=4;
        else if (card2.getSuit() == "Diamonds")
        suitValue2=3;
        else
        suitValue2=2;

        Integer value1, value2;
        value1 = card1.getValue();
        value2 = card2.getValue();

        if (suitValue1 != suitValue2)
        return suitValue1.compareTo(suitValue2);
        else
        return value1.compareTo(value2);

    }

    //get methods
    public int getValue()
    {return this.value;}

    public int getFinalValue()
    {return this.finalValue;}

    public String getNumber()
    {return this.number;}

    public String getSuit()
    {return this.suit;}

    public Player getOwner()
    {return this.owner;}

    //set methods
    public void setValue(int value)
    {this.value = value;}

    public void setFinalValue(int finalValue)
    {this.finalValue = finalValue;}

    public void setNumber(String number)
    {this.number = number;}

    public void setSuit(String suit)
    {this.suit = suit;}

    public void setOwner(Player player)
    {this.owner = player;}

}

这就是我的比较方法。我应该把它放在另一个班级还是什么？我试着用这个来称呼它：

Collections.sort(deck, new Comparator<Card>());

但这显然是错误的。

deck 是一个 ArrayList ：private ArrayList<Card> deck = new ArrayList<Card>();填充了各种 Card 对象。

4

6 回答 6

4

你Comparable对Comparator. 它们当然是相似的，但实际上根本不同。

Comparable定义一个类的自然顺序——你的类的实例应该如何默认排序。这实际上就是你想要的。

Comparator当您想要使用与自然顺序不同的排序顺序时，可以帮助您对类的实例进行排序。这似乎不是你的目标。

所以你真正想要的是：

public class Card implements Comparable<Card> {
  public int compareTo(Card otherCard) {
    //return 1 when this instance is greater than the parameter, -1 if less; 0 if equal
  } 

  //Other stuff
  //And make sure compareTo is consistent with equals
}

请注意，您必须实现的方法是compareTo，这与您实现的方法不同Comparator。

然后你只需这样排序：

Collections.sort(deck)

卡片将知道如何自行分类。

如果您需要更多指导，我们实际上做了一个关于Github上可用文件的教程。如果您喜欢，请查看。ComparableComparator

于 2013-11-13T01:07:31.587 回答
1

由于您似乎只有一种比较 Card 对象的方法，因此您实际上不必使用 Comparator。让 Card 实现 Comparable，然后按 1 参数方法排序：

Collections.sort(deck);
于 2013-11-13T00:57:09.787 回答
1

总结一下：

  • 要对数组进行排序，请使用Collections.sort(deck);
  • 您的 Card 类需要实现Comparable
  • 比较字符串时最好使用等于
  • 最好先使用常量字符串，即“Hearts”.equals(this.getSuit())，这样可以防止空指针
于 2013-11-13T01:07:50.910 回答
0

您想实现 Comparable，然后您可以对数组进行排序。

于 2013-11-13T00:57:51.253 回答
0

我同意 Juned 的观点，使用 value1.equals(value2) 这将返回一个布尔值。从外观上看，您只进行了相当直接的字符串比较。

祝你好运。

于 2013-11-13T01:11:09.583 回答
-1

我想这就是你的意图。我不相信你可以在字符串上使用 switch 语句，除非你是 1.7+

public class Card implements Comparable<Card>
{
    private String number;
    private int value;
    private int finalValue;
    private String suit;
    private Player owner;

    public Card(String number, String suit)
    {
        this.number = number;
        this.value = 0;
        this.finalValue = 0;


        //card value
        if ("2".equalsIgnoreCase(number))
        {
            this.value = 2;
        }
        else if ("3".equalsIgnoreCase(number))
        {
            this.value = 3;

        }
        else if ("4".equalsIgnoreCase(number))
        {
            this.value = 4;

        }
        else if ("5".equalsIgnoreCase(number))
        {
            this.value = 5;

        }
        else if ("6".equalsIgnoreCase(number))
        {
            this.value = 6;

        }
        else if ("7".equalsIgnoreCase(number))
        {
            this.value = 7;

        }
        else if ("8".equalsIgnoreCase(number))
        {
            this.value = 8;

        }
        else if ("9".equalsIgnoreCase(number))
        {
            this.value = 9;
        }
        else if ("10".equalsIgnoreCase(number))
        {
            this.value = 10;
        }
        else if ("J".equalsIgnoreCase(number))
        {
            this.value = 11;

        }
        else if ("Q".equalsIgnoreCase(number))
        {
            this.value = 12;
        }
        else if ("K".equalsIgnoreCase(number))
        {
            this.value = 13;

        }
        else if ("A".equalsIgnoreCase(number))
        {
            this.value = 14;
        }


        //suits
        if ("Spades".equalsIgnoreCase(number))
        {
            this.finalValue = this.value * 4;
        }
        else if ("Hearts".equalsIgnoreCase(number))
        {
            this.finalValue = this.value * 5;

        }
        else if ("Clubs".equalsIgnoreCase(number))
        {
            this.finalValue = this.value * 2;

        }
        else if ("Diamonds".equalsIgnoreCase(number))
        {
            this.finalValue = this.value * 3;

        }

    }


    @Override
    public int compareTo(Card o)
    {
        final int EQUAL = 0;
        final int LESS_THAN = -1;
        final int GREATER_THAN = 1;

        if (this == o || this.getFinalValue() == o.getFinalValue())
        {
            return EQUAL;
        }
        else if (this.getFinalValue() < o.getFinalValue())
        {
            return LESS_THAN;
        }
        else
        {
            return GREATER_THAN;
        }
    }

    // get methods
    public int getValue()
    {
        return this.value;
    }

    public int getFinalValue()
    {
        return this.finalValue;
    }

    public String getNumber()
    {
        return this.number;
    }

    public String getSuit()
    {
        return this.suit;
    }

    public Player getOwner()
    {
        return this.owner;
    }

    // set methods
    public void setValue(int value)
    {
        this.value = value;
    }

    public void setFinalValue(int finalValue)
    {
        this.finalValue = finalValue;
    }

    public void setNumber(String number)
    {
        this.number = number;
    }

    public void setSuit(String suit)
    {
        this.suit = suit;
    }

    public void setOwner(Player player)
    {
        this.owner = player;
    }
}
于 2013-11-13T04:15:58.830 回答