Надо учиться. Тут думаю сидят умные люди). да я из йии

Ну тогда зачем классу животного знать айди и дату приема в приют? у животного есть только имя и возраст, а дата приема и идентификатор это уже ответственность приюта

да, кстати интересно зачем

+ dateCreate у Pet это что? Дата рождения?

+ dateCreate у Pet это что? Дата рождения?

Это когда животное занесли в приют

+ dateCreate у Pet это что? Дата рождения?

Дата приема, кмк

Дата приема, кмк

ну я продолжаю намекать на странность решения)

listPetsByType -> array_filter

listPetsByType -> array_filter

именно)

ну я продолжаю намекать на странность решения)

ActiveRecord ;)))

listPetsByType -> array_filter

ну или $collection->filter()

ActiveRecord ;)))

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

ну т.е. это еще хотя бы удобно)

Я по замечания создам issues. Вы не против если ваши ники впишу?

пожалуйста)

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

Дату создания обьекта, но если прочитать задачу, то это другая дата ;)

Ну и чуть позже попробую правильно решить

Надо учиться. Тут думаю сидят умные люди). да я из йии

Отвечает Александр Др... Ой, то есть пхп-кнопкодав, который уже год не может найти работу даже за копейки (это о достоверности того, что написано ниже). Думаю у тебя в модели не хватает человека, который забирает животное из приюта, поэтому появляются всякие гет-ы странные И вообще странно, если ты забираешь из приюта, то как там может быть doRemove? Животное может наполовину остаться в приюте если его забирают? Не любишь ты животных!

По ремуву согласен надо гет и ремув методы

точнее оно клонируется. Ну тогда ладно! Это вроде не жестоко

Ну и чуть позже попробую правильно решить

И не нужно тебе несколько копий массива, хватит и одного

Array filter?

а хотя не, не клонируется же :D ну короче не важно

Array filter?

Угу

Shelter->addPet(new Cat('Tom', 3));

Животному точно не нужен ид. Я по типу базы планировал

Животному точно не нужен ид. Я по типу базы планировал

Мы же разбираем в контексте классов, а не непонятно какой ORM

Хорошо принято. Насчёт времени поступления.

Животному точно не нужен ид. Я по типу базы планировал

Ну и Pet лучше интерфейсом, и AbstractPet то что у тебя сейчас в Pet

Ид, время в Шелтер. Один массив. Правильно?

Можешь отдельный класс сделать для внутреннего массива new ShelterPet(Pet $pet, DateTime $entryDate)

Ид, у тебя ключ массива

Ид, у тебя ключ массива

Но можешь ид и в ShelterPet писать ;)

Спасибо всем. Не всегда видишь свои ошибки.

Пацаны, как стать пиздатым софтвэар инжэниар? С чего начать обучение? Ознакомился с принципами программирования, покодил на скриптовых языках веб-приложия. С партнерами проектирования также ознакомился. Как дальше развиваться?

Пацаны, как стать пиздатым софтвэар инжэниар? С чего начать обучение? Ознакомился с принципами программирования, покодил на скриптовых языках веб-приложия. С партнерами проектирования также ознакомился. Как дальше развиваться?

Можно на работу пойти.

Можно на работу пойти.

Но ведь без опыта возьмут на какую-то обезьянничью должность

Но ведь без опыта возьмут на какую-то обезьянничью должность

А ты хочешь чтобы тебя взяли на обезьянничью работу с опытом обезьянничей работы?)

А ты хочешь чтобы тебя взяли на обезьянничью работу с опытом обезьянничей работы?)

Нет, я хочу научиться

Ну вот и иди.

Та я уже 10 лет так делаю, вот решил разнообразить

Ну вот и иди.

Ты предлагаешь не самый эффективный, хоть и рабочий способ

Я согласен

Я немного не то спрашивал. Хотел узнать, какие технологии и принципы стоить изучать

Само собой, что практика и только практика прокачает, но ведь не лучше ли быть к ней готовым?

Попробуй ТК

что это?

Пацаны, как стать пиздатым софтвэар инжэниар? С чего начать обучение? Ознакомился с принципами программирования, покодил на скриптовых языках веб-приложия. С партнерами проектирования также ознакомился. Как дальше развиваться?

в вузе факультет есть такой)0

Пацаны, как стать пиздатым софтвэар инжэниар? С чего начать обучение? Ознакомился с принципами программирования, покодил на скриптовых языках веб-приложия. С партнерами проектирования также ознакомился. Как дальше развиваться?

Зарядочка по утрам, контрастный душ, отжимания и обтирание холодным полотенцем

https://github.com/alfredleo/shelter

Пока нет никаких отличий у животных нет смысла делать им отдельные классы. Преждевременный оверинжиниринг получается. Хватит и string поля в Pet. Это как у всех подряд классов интерфейс делать и никак его не использовать, кроме как указания типа. Дата помещения в приют должна быть опциональной в конструкторе, т.к. при создании объекта из данных из бд уже будет дата, как и id. Из Shelter можно вынести класс, который занимается работой с коллекцией, потому что такой код более низкоуровневый и замусоривает этот объект, ухудшает понимание того, что он на самом деле делает при чтении. Этот вынесенный объект потом переиспользовать можно будет. Это тоже может оказаться оверинжинирингом в данный момент, но этот класс как-бе будет имитировать работу с бд, понадобятся в дальнейшем более низкоуровневые вещи, которые будут внутри использоваться. Хранение нескольких отсортированных коллекций и работа с ними неоптимальна. При работе с БД на каждое действие будет производиться запрос, потому что делать это в пхп дорого. Бд делает это быстро, тем более при использовании кэширования. Поэтому можно смело возвращать новую отсортированную копию массива при каждом вызове метода.

+ PSR - наименование классов и директорий

+ composer

Но ведь без опыта возьмут на какую-то обезьянничью должность

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

Типичный путь развития "софтвэар инжэниа" - начинать с самой малой должности в крупной компании с кучей сервисов. Постепенно получать опыт, брать в работу более высокоуровневые задачи -> рост должности. И в определенный момент дойти до задач типа распила большого сервиса на сервисы поменьше, либо написания своих. Естественно, что все это должно происходить в среде с постоянным ревью кода, наличием опытных коллег, которые готовы помогать и делиться опытом. Серьезные дела в одиночку не делаются. Это - легенда.

С архитектором мог спутать

Единицы людей, которые дома могут "подготовиться" лучше, чем на работе к работе

ERROR: S client not available

Я тоже так хотел, но все свелось к тому, чтобы либо нанимать ментора за большие деньги, которых нет, либо переходить в серьезное место и там получать это бесплатно

С архитектором мог спутать

Кстати, а что архитектор делает?

В основном умный вид

Кто нибудь знает, что это за люди с вип каналом?

Это люди которые идут в бан

Привет всем, есть тут кто хорошо разбирается в классах, PSR, SOLID на php? Хочу спросить посмотреть на репу, есть ли, на ваш взгляд, недочеты в плане реализации.

почему тут не объект животного передается? зачем ты их тогда создавал ? https://github.com/alfredleo/shelter/blob/master/local/shelter/Shelter.php#L34

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

поэтому кличку животному в рамках приюта надо давать

во вторых зачем тебе столько вспомогательных массивов по типам и тд животных ? https://github.com/alfredleo/shelter/blob/master/local/shelter/Shelter.php#L24 просто сделай один массив с живтоными разных типов

Дату создания обьекта, но если прочитать задачу, то это другая дата ;)

Я же написал "если" :)

Ну и Pet лучше интерфейсом, и AbstractPet то что у тебя сейчас в Pet

-____-

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

Как может одно животное быть сразу в двух приютах?

Животному точно не нужен ид. Я по типу базы планировал

Для персистентности ты не можешь полагаться на сравнение по ссылке доступное в языке, хотя тем не менее в системе должна существовать только один объект для одной сущности

ну если предложить что в городе несколько приютов и оно убегает

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

во вторых зачем тебе столько вспомогательных массивов по типам и тд животных ? https://github.com/alfredleo/shelter/blob/master/local/shelter/Shelter.php#L24 просто сделай один массив с живтоными разных типов

Это уже обсудили. Будет один массив

первый пункт указывает на то что твой приют отвечает за создание объектов животных (инстанцирование) в себе и за их хранение что как бы класс делает кое что не свое

Ну и Pet лучше интерфейсом, и AbstractPet то что у тебя сейчас в Pet

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

ну гоу каждый свою видимость кода напишет )

чтобы сраться знатно)

еще не совсем понимаю зачем такие phpdoc методам ? https://github.com/alfredleo/shelter/blob/master/local/shelter/Shelter.php#L60 в методе указано все тоже самое (не относится к solid)

Сраться не надо, можно решить несколько путями. Главное что бы основные принципы не нарушались.

Пхпдок нарушает что то?

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

вот еще https://github.com/alfredleo/shelter/blob/master/local/shelter/Shelter.php#L48

public function addPet(Pet $pet) вот так бы я сделал инстанцировать животное за пределами

и тогда весь приют сводится к коллекции элементов