Так есть ли способ упростить это, чтобы в любом случае сделать меньше?

else if(selectedCards.size() == 3
            && cardAt(selectedCards.get(0)).pointValue() + cardAt(selectedCards.get(1)).pointValue() + cardAt(selectedCards.get(2)).pointValue() == 0 
            && !cardAt(selectedCards.get(0)).rank().equals(cardAt(selectedCards.get(1)).rank()) 
            && !cardAt(selectedCards.get(0)).rank().equals(cardAt(selectedCards.get(2)).rank())
            && !cardAt(selectedCards.get(1)).rank().equals(cardAt(selectedCards.get(2)).rank()))
2
Braden 24 Фев 2015 в 01:04

3 ответа

Лучший ответ

Насколько я понимаю, вы пытаетесь проверить, имеют ли 3 карты разные ранги. Более простой способ проверить это - поместить их в набор и посмотреть, совпадает ли размер набора с выбранным набором. Это масштабируется до любого количества выбранных карт ...

public boolean differentRanks(List<Integer> selectedCards) {
    Set<Integer> ranks = new HashSet<Integer>();
    for (int card : selectedCards) {
        ranks.add(cardAt(card).rank());
    }
    return ranks.size() == selectedCards.size();
}

Я бы также создал метод подсчета очков для выбранных карт ...

public int sum(List<Integer> selectedCards) {
    int total;
    for (int card : selectedCards) {
        total += cardAt(card).pointValue();
    }
    return total;
}

Так что условие закончится

} else if (selectedCards.size() == 3 && sum(selectedCards) == 0 && 
           differentRanks(selectedCards) {
3
Adam 23 Фев 2015 в 22:32

На мой взгляд наиболее читаемый:

else if(selectedCards.size() == 3 && checkRanks(selectedCards))
{
    //...
}

//...

private boolean checkRanks(List<Card> cards)
{
    Card zeroCard = cardAt(selectedCards.get(0));
    Card firstCard = cardAt(selectedCards.get(1));
    Card secondCard = cardAt(selectedCards.get(2));

    boolean isZero = zeroCard.pointValue() + firstCard.pointValue() +      secondCard.pointValue() == 0;

    boolean zeroCardRankNotEqualFirst = !zeroCard.rank().equals(firstCard.rank())
    boolean zeroCardRankNotEqualSecond = !zeroCard.rank().equals(secondCard.rank())
    boolean firstCardRankNotEqualsSecond = !firstCard.rank().equals(secondCard.rank()); 

    return isZero && zeroCardRankNotEqualFirst && zeroCardRankNotEqualSecond && firstCardRankNotEqualsSecond;
}
0
Piotr Zych 23 Фев 2015 в 22:45

Это будет один из вариантов:

else if(selectedCards.size() == 3
        && cardAt(selectedCards.get(0)).pointValue() +        cardAt(selectedCards.get(1)).pointValue() + cardAt(selectedCards.get(2)).pointValue() == 0 
        && !(cardAt(selectedCards.get(0)).rank().equals(cardAt(selectedCards.get(1)).rank())).equals(cardAt(selectedCards.get(2)).rank()) )

Чтобы сделать это условие более читабельным, вы можете сделать что-то вроде этого:

//here you extract the values you need only once and use them in your condition block below
        int cardsSize = selectedCards.size();
        int pointValue0 = cardsSize == 3 ? cardAt(selectedCards.get(0)).pointValue() : 0;
        int pointValue1 = cardsSize == 3 ? cardAt(selectedCards.get(1)).pointValue() : 0;
        int pointValue2 = cardsSize == 3 ? cardAt(selectedCards.get(2)).pointValue() : 0;           
        bool rankEquals = CompareRanks(cardAt(selectedCards.get(0)),cardAt(selectedCards.get(1)),cardAt(selectedCards.get(2));

        if (<condition>) {
            //block of sentences
        } else if (cardsSize == 3 && (pointValue0 + pointValue1 + pointValue2) == 0 && !rankEquals )

Я предлагаю создать функцию под названием «CompareRanks», в которой вы получаете 3 разных объекта (результат «cardAt»), и вы получаете там ранг и сравниваете, совпадают ли значения или нет.

Этот вариант приводит к большему количеству строк кода, но он более понятен и удобочитаем для любого человека, кроме вас.

0
GGAle 23 Фев 2015 в 22:25