вот эти проверки тоже доставляют https://github.com/alfredleo/shelter/blob/master/local/shelter/Shelter.php#L94

ну это все имхо

Это если удалять животных в конце будет ошибка

Защита

тут немного речь о другом

я понимаю зачем но почему за всем этим следит приют ?

Логика вся на приюте

Это типа журнала приюта

Нельзя кидать ошибку если у приюта пустой журнал

зачем столько проверок ?

Pet - интерфейс это да, но вот нужен ли AbstractPet...

Ну если одинаковая реализация всюду, то почемубы и нет. Все и так на интерфейс завязано. Хоть трейтом можешь сделать, его в любой момент можно выбросить

тебе достаточно одной проверки

В данном случае при удалении другие массивы ругались

Будет одна проверка когда будет один массив

суть твое проблемы что ты слишком много хранишь

избыточно

и проверяешь все свои вспомогательные структуры, это опять же имхо

тебе никто не запрещает внутри приюта юзать коллекцию

которая умет делать группировку и сортировку

через композицию

или весь приют можно сделать простой коллекцией даже имутабельной

Будет одна проверка когда будет один массив

тестовое задание случаем не для питерской компании?)

да

мне просто не пояснили что не так. Решил разобраться, что думает общественность

мне просто не пояснили что не так. Решил разобраться, что думает общественность

Я так понимаю, тебя туда не взяли?

не devjs случаем?

Я так понимаю, тебя туда не взяли?

да, я попросил дать пояснение

имхо для прохождения тестового задания взял бы либу реализующую простую коллекцию (чтобы не хранить в массиве потому что там все кишки наружу) в приюте сделал бы 1 фильтр функцию (по типу) и одну для сортировки (по имени) и просто бы композицию

и тогда получение бы выглядело бы примерно вот так

по задаче должно быть голое пхп

имхо для прохождения тестового задания взял бы либу реализующую простую коллекцию (чтобы не хранить в массиве потому что там все кишки наружу) в приюте сделал бы 1 фильтр функцию (по типу) и одну для сортировки (по имени) и просто бы композицию

А чем массив + array_filter+usort не угодил? Зачем в такую простую задачу тянуть зависимость

getByType return $this->collection->filter(self::filterByType)->order(self::orderByName);

getFirstByType return $this->collection->filter(self::filterByType)->firstOrNull();

getFirst return $this->collection->firstOrNull();

задача от компании фотосклад.ру. это задание для junior developer

ну написал бы отдельный объект с двумя методами filter и order

и first

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

ну написал бы отдельный объект с двумя методами filter и order

И потом еще обвешать спецификациями ;))) И получить https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition

И потом еще обвешать спецификациями ;))) И получить https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition

коллекция это просто возможность писать более красиво в данном случае, любишь хардкод пишешь на array_* функциях они все есть

и те кто сортирует и фильтруют

просто выглядеть будет так себе

поэтому чтобы не работать с ними имутабельная коллекция была бы плюсом имхо

коллекция это просто возможность писать более красиво в данном случае, любишь хардкод пишешь на array_* функциях они все есть

В данном случае имхо коллекция излишня, тем более это все внутри, никаких кишков наружу. В любой момент можно подменить реализацию, если будет нужно, без проблем для клиентского кода.

В данном случае имхо коллекция излишня, тем более это все внутри, никаких кишков наружу. В любой момент можно подменить реализацию, если будет нужно, без проблем для клиентского кода.

проблема в том что приют реализует в себе коллекцию

она может быть вынесена с помощью композиции

проблема в том что приют реализует в себе коллекцию

А чем массив не коллекция?

в отдельный объект и там можно как курица лапой что угодно написать

А чем массив не коллекция?

отсутствием методов по работе с коллекцией

и работа через функции

и там их довольно много и некоторые мутабельные

отсутствием методов по работе с коллекцией

Ф-и есть

и работа через функции

А что плохого в функциях?

я написал выше про функции и про то что некоторые из них мутабельные

А что плохого в функциях?

ничего

А чем массив не коллекция?

и еще смотря какие термины использовать для коллекций где то есть такое понятие что там filter, map и тд должно быть

ERROR: S client not available

В пыхе низя полиморфизм делать с функциями

еще нельзя записать тип для функции

в пыхе мало что можно делать (

А еще сраный порядок аргументов

стрелять в ногу

В пыхе низя полиморфизм делать с функциями

Зачем он в этой задаче?

Зачем он в этой задаче?

Замени коллекцию in-memory на персистентную

Замени коллекцию in-memory на персистентную

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

Замени коллекцию in-memory на персистентную

В данной задаче этого нет. Потому я бы сделал через массив, появилась бы задача сделать возможность сохранять стейт, внедрил бы коллекции

подумаешь

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

Только доктрине нужна коллекция как объект))

В данной задаче этого нет. Потому я бы сделал через массив, появилась бы задача сделать возможность сохранять стейт, внедрил бы коллекции

никто не говорит что через массив плохо, просто если ты понял что условия задачи сводятся к коллекции

не плохо бы это в коде показать

В данной задаче этого нет. Потому я бы сделал через массив, появилась бы задача сделать возможность сохранять стейт, внедрил бы коллекции

Вроде есть

никто не говорит что через массив плохо, просто если ты понял что условия задачи сводятся к коллекции

Ну массив та же коллекция ;) я к этому ;)

Вроде есть

Покажи

Ну массив та же коллекция ;) я к этому ;)

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

никто не говорит что через массив плохо, просто если ты понял что условия задачи сводятся к коллекции

И если можно задачу не усложнять, то и не нужно ее усложнять ;)

Вроде есть

там только о solid была речь((

И если можно задачу не усложнять, то и не нужно ее усложнять ;)

никто не говорит что нужно усложнять просто мне было бы проще показать на коллекциях чем вспоминать какая из функций должна юзаться для сортировки

там только о solid была речь((

Ну думать про солид изначально гиблое дело для таких задач, потому что нет понимания потока изменений

Ну думать про солид изначально гиблое дело для таких задач, потому что нет понимания потока изменений

это обычное тестовое задание

помыслить об абстрактных конях

а потом на работе гавно кодить на вордпрессе)

Ну вот, обычная пыховская задача - пробрасывать эти объектики в базу из реквеста и обратно

не вордпреса там нет.)

не вордпреса там нет.)

зато есть yii :D И потом, там может быть, скажем, битрикс