You can't have code coverage серьезно?

I can't use a data provider

:O

из коробки - не сможешь, но можно поставить экстеншен

дата провайдеров тоже нет

I can't mock abstract methods - а интерфейсы можешь?

НО

вопервых возможно что-то такое сейчас должно появиться в новых релизах ну и опять же есть экстеншены

а несколько интерфейсов одновременно?)

I can't mock abstract methods - а интерфейсы можешь?

могу

а несколько интерфейсов одновременно?)

а тебе надо несколько интерфейсов одновременно?)

да, были такие кейсы

на одной зависимости

да, были такие кейсы

а ты считаешь что это нормально?)

а почему нет?)

ну как тебе сказать... как минимум звучит странно

быть может это должен быть не 2 интерфейса а один?

может ты не так разделил интерфейсы?

может твой класс делает что-то слишком много?

не, именно 2 надо было, один как маркер вроде был

щас по коду поищу где это было

не, именно 2 надо было, один как маркер вроде был

если это два разных объекта (пусть один и тот же просто две зависимости) то норм, но тут тебе phpspec норм

иначе что-то не так с сегрегацией интерфейсов

а да вот нашел. там коллекция, в нее добавляются элементы интерфейса Filter, но у некоторых элементов может быть еще интерфейс StatefulFilter и вот надо было проверить метод который делает getStatefulFilters

а да вот нашел. там коллекция, в нее добавляются элементы интерфейса Filter, но у некоторых элементов может быть еще интерфейс StatefulFilter и вот надо было проверить метод который делает getStatefulFilters

то есть у тебя inscenceof там7

да

ну... чувствуешь запашек легкий?

та не, норм

ну это не вонь, это так... как если ты открываешь холодильник и тебе не комфортно

ок, твой вариант)

я ж задачу не знаю)

чем отличается поведение StetefulFilter от Filter и что ты делаешь внутри коллекции?

class Filters { /** * @var FilterInterface[] */ private $filters = []; /** * @return FilterInterface[] */ public function getFilters() { return $this->filters; } /** * @param FilterInterface $filter */ public function addFilter(FilterInterface $filter) { $this->filters[get_class($filter)] = $filter; } /** * @param string[] $names * @return FilterInterface[] */ public function getFiltersExcept(...$names) { $filters = $this->filters; foreach ($names as $name) { if (!isset($filters[$name])) { continue; } unset($filters[$name]); } return $filters; } /** * @param $name * @return FilterInterface */ public function getFilter($name) { return $this->filters[$name]; } /** * @return StatefulFilterInterface[] */ public function getStatefulFilters() { $filters = []; foreach ($this->filters as $key => $filter) { if ($filter instanceof StatefulFilterInterface) { $filters[$key] = $filter; } } return $filters; } }

вообще можно было сделать расширение интерфейса, но это слишком просто

getStatefulFilters

только там вижу inscenceof

и не понимаю зачем этот метод

ну вот надо)

ну вот запах

короч... пишка ограничений спека в том что отговорки в духе "ну вот надо" там не сработают. Ты тупо не сможешь это протестить и тебе придется подумать получше

а поскольку мы со спеком должны работать в контексте tdd

то проблемы быстрее находятся

вот где он юзается public function createSearchRequest($searchParams) { $searchRequest = new SearchRequest(); $searchRequest->setSearchParams($searchParams); foreach ($this->filtersProvider->getStatefulFilters() as $filter) { $searchRequest->addFilterContext(get_class($filter), $filter->createContext($searchRequest)); } return $searchRequest; }

так почему бы тебе для stateful фильтров не завести тупо отдельную коллекцию?

зачем?)

избавился бы от ифов

ну это ж разные интерфейсы

и разные штуки

если бы у тебя были дженерики

то тебе нужно было бы что-то в духе

Filters<Filter|StatefulFilter>

не. просто StatefulFilter не может сам существовать

только как маркер для Filter

поэтому List<Filter>

...хз стремная хрень

короч я хз... сча попробую переписать свои тест кейсы под phpunit и посмотреть насколько тошнотворно

если это стремная хрень, зачем ее добавляют в core lib котлина?

https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.collections/filter-is-instance.html

filters.filterIsInstance<StatefulFilter>() вот так бы этот метод выглядел

нуууу... хз)

компромисы?)

? да нет, у меня подобные кейсы есть еще в других местах

когда есть пачка провайдеров, но все умеют разное. мне нужно выбрать по маркеру их

точнее там не маркеры, а полноценные интерфейсы

Про котлин был кстати годный доклад, о том, что любой язык — сполшные компромисы, ну и примеры компромисов, которые они приняли.

на спринге это проще... захуярил себе List<OloloInterface> services и не ебешь себе мозг

Все, кроме хаскеля, он прекрасен.

котлин практичен)

практичность = компромисы

это как инженеры vs ученые

ну экстеншены мало кто назовет здравой идеей

инженеры используют допущения, неточности, рандом

ну как рандом.. скорее реальные условия

а не лабораторные или какие-то теории

ну экстеншены мало кто назовет здравой идеей

ты о экстеншен методах?

да

я считаю что это прекрасно

если не злоупотреблять

а то код быстро превращается в непонятные вещи)

нодоело писать StringUtils, BlaBlaUtils

а выглядят они лучше, чем теже имлиситы в скале

я вот сегодня запилил себе fun Document.createElement(name: String, value: String = "", attributes: Map<String, String> = emptyMap()): Node { val element = this.createElement(name) element.textContent = value for ((attrName, attrValue) in attributes) { element.setAttribute(attrName, attrValue) } return element }

и теперь можно хреначить appendChild(createElement("RelCode", attributes = mapOf("value" to "Regular")))

вместо унылых записей в 3 строки

val relCode = doc.createElement("RelCode") relCode.setAttribute("value", "Regular") requestItem.appendChild(relCode)

короч я хз... сча попробую переписать свои тест кейсы под phpunit и посмотреть насколько тошнотворно

попробуй junit4 + mockito2 + kotlin-mockito

а вот на оборот хочу как-то производную spec попробовать

зачем?)

spock какой-то?

не знаю, spoсk выглядит довольно вкусно

его ж вроде с груви только юзают?

Нет, судя по тому что я видел, вполне удачно тестируют java код

https://github.com/JetBrains/spek/blob/master/spek-samples/src/test/kotlin/org/jetbrains/spek/samples/CalculatorSpec.kt

можно spek взять еще)

https://www.youtube.com/watch?v=ZsHMHukIlJY

оч годный видос

можно собрать список видосов которые говорят "не юзайте геттеры и сеттеры")

массивы в php в качестве DTO - это ок?

массивы в php в качестве DTO - это ок?

ну ты теряешь контроль над типами... а так ок