Как избавиться от лишних приведений к реализованному интерфейсу? - PullRequest
Новая
       21

Как избавиться от лишних приведений к реализованному интерфейсу?

2 голосов
/ 11 марта 2019

Допустим, у меня есть интерфейс 

public interface ICardSuit {
    /**short name*/
    public String getName();

    /** the colour of this card*/
    public ICardColour getColour();
}

, который я решил реализовать с помощью перечисления: 

public enum  CardSuit implements ICardSuit {
    HEART{
        @Override
        public ICardColour getColour() {
            return CardColour.RED;
        }
    },
    SPADE{
        @Override
        public ICardColour getColour() {
            return CardColour.BLACK;
        }
    },
    DIAMOND{
        @Override
        public ICardColour getColour() {
            return CardColour.RED;
        }
    },
    CLUBS {
        @Override
        public ICardColour getColour() {
            return CardColour.BLACK;
        }
    }
    ;

    @Override
    public String getName() {
        return this.name();
    }
}

Теперь я хочу протестировать его (используя kotlintest, потому что я к нему привязан): 

class CardSuitTest : FunSpec(){
    init {
        test("there are exactly four suits"){CardSuit.values().size shouldBe 4}
        test("suits implement interface"){CardSuit.values().forEach { it shouldBe instanceOf(ICardSuit::class) }}
        test("suits have correct names"){
            val suits = CardSuit.values() as Array<out ICardSuit>
            suits.forEach { when(it.name){
                "HEART" -> it should beTheSameInstanceAs(CardSuit.HEART as ICardSuit)
                "SPADE" -> it should beTheSameInstanceAs(CardSuit.SPADE as ICardSuit)
                "DIAMOND" -> it should beTheSameInstanceAs(CardSuit.DIAMOND as ICardSuit)
                "CLUBS" -> it should beTheSameInstanceAs(CardSuit.CLUBS as ICardSuit)
            } }
        }
        test("suits have correct colours"){
            CardSuit.values().forEach { when(it){
                CardSuit.HEART,CardSuit.DIAMOND -> it.colour shouldBe CardColour.RED
                CardSuit.CLUBS, CardSuit.SPADE -> it.colour shouldBe CardColour.BLACK
            } }
        }
    }
}

, где мне нужно привести к ICardSuit, потому что если я этого не сделаю, компилятор жалуется, что 

None of the following functions can be called with the arguments supplied.

* T.should(Matcher<T>)   where T cannot be inferred for    infix fun <T> T.should(matcher: Matcher<T>): Unit defined in io.kotlintest.matchers

* ICardSuit.should((ICardSuit) → Unit)   where T = ICardSuit for    infix fun <T> T.should(matcher: (T) → Unit): Unit defined in io.kotlintest.matchers

Я хочу оставить as Array<out ICardSuit>, потому что это самый простой способ убедиться, что я получаю доступ только к свойствам интерфейса,

но я действительно не люблю бросать экземпляры, для которых проверяю.

Что я могу с этим поделать?

Ответы [ 2 ]

3 голосов
/ 13 марта 2019

Есть ли конкретная причина, по которой вам нужно использовать устройство для сопоставления beSameInstanceAs?

Вы можете сделать что-то вроде: 

val suits = CardSuit.values() as Array<out ICardSuite>

suits.forEach {
    when (it.name) {
        "HEART" -> it shouldBe CardSuit.HEART
        "SPADE" -> it shouldBe CardSuit.SPADE
   }
}

Однако, если вы действительно хотите использовать beSameInstanceAs, вы можете: 

suits.forEach {
    when(it.name) {
       "HEART" -> it shouldBeSameInstanceAs CardSuit.HEART
       "SPADE" -> it shouldBeSameInstanceAs CardSuit.SPADE
    }
}

Я не получаю никаких жалоб от компилятора здесь

2 голосов
/ 11 марта 2019

beTheSameInstanceAs(CardSuit.HEART) возвращает Matcher<CardSuit>, поэтому он не может совпадать с произвольным ICardSuit. Это имеет смысл (хотя Matcher может быть против -вариант, здесь вам понадобится ковариация). Но вы можете:

  1. Звоните явно beTheSameInstanceAs<ICardSuit>(CardSuit.HEART).

  2. Создание вспомогательной функции 

    inline fun <T1, reified T2 : T1> Matcher<T2>.widen() = object : Matcher<T1>() {
    override fun test(value: T1) = 
        if (value is T2) 
            this.test(value) 
        else 
            Result(false, "$value is not a ${T2::class.name}", "$value is a ${T2::class.name}")
}

    и звоните 

    it should beTheSameInstanceAs(CardSuit.HEART).widen()

    (я думаю, здесь должен работать вывод типов).

  3. Поскольку beTheSameInstanceAs(x) действительно может соответствовать чему-либо, объявите эквивалентную функцию, которая возвращает Matcher<Any>: 

    fun beTheSameInstanceAsAny(x: Any) = beTheSameInstanceAs(x)

// usage
"HEART" -> it should beTheSameInstanceAsAny(CardSuit.HEART)
...