maro512 / Loop2

Apache License 2.0
0 stars 0 forks source link

Board - poprawka #14

Closed cieplinski-tobiasz closed 7 years ago

cieplinski-tobiasz commented 7 years ago

byłoby miło Piotrze (jako, że pisałeś te klasę), gdybyś mógł przerobić winningLine aby zwracało Collection< Cell >, jeśli to możliwe, sporo by to ułatwiło przy renderowaniu

pzjp commented 7 years ago

Obecne List<Tile> to w szczególności Collection<Tile> a nawet Collection<? extends Cell>. Czy użycie ogólnego typu Cell jest aż tak ważne?

cieplinski-tobiasz commented 7 years ago

Właśnie cell jest tutaj kluczowy, pal sześć listę. Zmienna Collection< Cell> nie akceptuje typu collection< Tile>

pzjp commented 7 years ago

Może można zastąpić typ docelowy tym z "wildcardem"? Oczywiście wpisanie kilku Cell zamiast Tile to nie problem.

cieplinski-tobiasz commented 7 years ago

ok, nie znałem tego typu z wildcardem, teraz będzie ok