сейчас 504 ошибка, Gateway Timeout

Логи?

походу sql

Парни, это ненормально ведь держать в контроллере функцию, которая достает последний добавленный сайт в базу (Имеется сущность Site)? Подозреваю, что в файл Repository/SiteRepository.php ? Направьте меня, пожалуйста

метод*

а для чего используется?

просто для UI?

короч простое правило - все методы репозитория должны либо ничего не возвращать (add, remove например) либо возвращать не null и не массив)

в целом это только те штуки которые используются в бизнес логике

"поиск" что бы отобращить список чего-нибудь - это UI логика

У меня контроллер для генерации сайдбара, в который достаются из разных таблиц данные. В контроллере реализована пара методов типа getLastSite(), которые достают нужную инфу из БД

ну вынеси в отдельные сервисы

но не в репозиторий

и покури про Single Reponsibility

Спасибо?

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

(и нарушать их там где лень уже)

То есть в SiteRepository.php стоит реализовать метод для удаления записи по id, но никак не для получения списка последних 10 сайтов?

ну как.... зависит от уровня лени и как много у тебя выборок

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

слишком много власти для одного объекта

в сферическом вакууме зависимости объектв должны предоставлять ровно те методы, которые юзаются и не методом больше

но настолько загоняться частенько бывает лениво

Это ж сколько сервисов должно быть

много много много

есть еще хитрости

например - нотификации

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

пилить 20 сервисов как-то не прикольно, учитывая что они будут копией друг дружки

так?

Угу

что можно сделать - один сервис нотификатор, который имеет метод public function send(Notification $notification) { $swiftMailerNotification = $notification->prepare($this->twig, $this->someOtherStuff); $this->mailer->send($swiftMailerNotification); }

теперь у нас по объекту на каждую нотификацию и они прекрасно знают как себя "отрендрить".

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

точно так же можно делать и с выборками

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

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

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

Но придёт момент и ты сойдёшь с ума?

да нет, ты скорее сойдешь с ума если у тебя система будет состоять из классов по 1000 строк

с наследованием, статикой и кучей ифов

когда непонятно что кто юзает и зачем

вопрос масштабов

unix way

Нет... мы пока не такие большие

Но логика ясна

Осталось заставать себя следовать ей

Не лениться то есть

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

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

главное что бы тупой бойлерплейт за тебя IDE генерила

главное что бы тупой бойлерплейт за тебя IDE генерила

Я не знаю что это)

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

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

да если руку набить недельку то как бы и в начале не сильно задерживает

Я просто люблю подход. хуяк в продакшен, а если взлетит, все по умному =)

Я просто люблю подход. хуяк в продакшен, а если взлетит, все по умному =)

Dmitriy, Нельзя ругаться! [Предупреждений 2/5]

Я не знаю что это)

ну тип... ты создаешь класс как? Создаешь пустой файл и фигачишь руками или просишь IDE все создать и оно само правильный нэймспейс сделает и все такое?

Dmitriy, Нельзя ругаться! [Предупреждений 2/5]

сорян, забыл

Я просто люблю подход. хуяк в продакшен, а если взлетит, все по умному =)

тип сначала прототип а потом реализация?)

Да, все так

нормальный подход особенно когда ты не знаешь что делаешь

или заказчик не знает чего хочет

но как бы... в этом случае разделение на маленькие объектики только профит дает

ибо.... выкатывать прототипы тебе надо будет и сильно после продакшена если ты работаешь над каким-то более-менее непонятным приложением

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

ERROR: S client not available

но как бы... в этом случае разделение на маленькие объектики только профит дает

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

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

Dmitriy, Нельзя ругаться! [Предупреждений 3/5]

Dmitriy, Нельзя ругаться! [Предупреждений 3/5]

Да блин. это не плохое слово. это же IT чат.

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

потому повторюсь, если у тебя нормально настроена IDE и ты ей пользуешься не как блокнотом, то как бы время особо не бернится

более того, даже если ты пишешь прототип, хотя бы 5 минут линших потратить на именование вещей сэкономит тебе потом минут 15 на чтение кода.

Именование то не проблема. Про г*окод я имею ввиду. Какую-ту фигню в модель засунуть(хотя она там не нужна). Конечно IDE помогает здорого с шаблонами.

> Какую-ту фигню в модель засунуть(хотя она там не нужна). не ну у меня например в текущем проекте безумные сроки

и потому говнишка тоже хватает

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

буквально недели две назад сажал нового чувака, он вкурил что где за 2 дня

Напимер была задача отправлять данные в 3х местах на сервер. Сделать нужно было прям сейчас. Я тупо копипастом вставил и не парился. Но я не чисто разрабатываю продукт, у меня еще много административной работы. Мне проще было копипастнуть 3 раза, а уже дома спокойно вынести это в норм классы.

не норм

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

Sergey был заблокирован за мат [5/5]…не хотят писать [говно]код как преждевременая оптимизация

Жестковатый бот )

Почистите регекспы

Пойду-ка лучше в конфу типа @symfony_php, где нет упоротого бота. Пользы от него никакой, один вред. Всем успехов.

Привет, Yet Another Stats! Пожалуйста соблюдай правила и не ругайся /help@AngryModerBot

ну тип... ты создаешь класс как? Создаешь пустой файл и фигачишь руками или просишь IDE все создать и оно само правильный нэймспейс сделает и все такое?

IDE само все делает ) сеттеры / геттеры, а конструктор - в зависимости от случая. Неймспейсы само все.

Ну вот и кому я ответил.

Мы тебя слушаем, не переживай

Да я к тому, что бот банит не тех

Да я к тому, что бот банит не тех

Dmitriy Начал голосование за бан Salavat Причина: Да я к тому, что бот банит не тех Чтобы выгнать нужно больше 70%, сейчас 67

))) я увидел твое сообщение )

ну вот =(

Хэльго проголосовал Да

Nick проголосовал Нет

f4rt~ проголосовал Да

Привет, f4rt~! Пожалуйста соблюдай правила и не ругайся /help@AngryModerBot

говнокод

говнокод

f4rt~, Нельзя ругаться! [Предупреждений 1/5]