— пн: создание моделей и контроллеров и предствалений для корзины: кнопка «добавить», список товаров, оплата, список заказов с их состояниями.

— вт-ср: написание тестов и логики работы корзины.

— чт: сборка на тестовом сервере, показ заказчику, согласование

— пт: деплой

(Это, безусловно, нереальный план, но для нашего обсуждения пойдёт)

Объявили о своих намерениях заказчику, он согласился. Вы продолжаете работать над текущими задачами, размышляя на фоне о корзине. Внезапно вы осознаёте, что уже не очень хорошо помните как в том проекте что устроено, полгода ведь прошло и решаете выкачать последнюю версию кода из репозитория. Выкачиваете, а там 46 изменённых файлов и 4237 новых строк кода!

Теперь придётся это всё просматривать и разбираться, кто и зачем это всё написал и как это повлияет на вашу задачу. Нужно выделять на это время, значит посиделки с друзьями сегодня вечером в пролёте.

Разбирая код, вы обнаруживаете, что кто-то как раз пытался сделать корзину. Узнаёте у заказчика, он вспоминает, что три месяца назад он попытался поработать с другим подрядчиком, но у них не сложились отношения.

Вам теперь совсем непонятно, что конкретно там было сделано, можно ли это полностью откатить или попытаться это доделать.

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

Давайте теперь отмотаем время на начало истории, когда вы сели писать план и перепишем его:

— пн: приём текущего состояния проекта, обновление кода, сборка, анализ изменений.

— вт: выбор оптимальной тактики: написание собственной корзины, использование сторонней библиотеки, внешнего сервиса.

— ср: согласование тактики с заказчиком, написание плана работ, согласование плана работ по реализации согласованной тактики.

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

На следующий день я вижу ответ от разработчика:

Сначала я такой:

А потом думаю: «Хм... А это тема для поста.»

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

Получается, я могу написать только «Почему? И что с этим тогда делать?» и отправить целый пул-реквест на полку ещё на 24 часа.

То есть бесполезный ответ на моё замечание увеличил время до релиза фичи на 24 часа.

Уберите эмоциональную окраску своих сообщений. Собеседник вложит в ваш текст эмоции на свой вкус, обидится и дальнейшая дискуссия будет идти на эмоциях.

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

Опасно:

• Это полная хуйня.
• Как ты считаешь, эту поебень кто-то через год сможет понять?
• Блять! Мы уже полгода говорим о том, что модели нельзя из контроллеров вызывать, хули ты так сделал?
• Кажется, это костыль.

Продуктивно:

• Я не могу принять этот код, потому что он нарушает наши принципы: оформлен не по PSR и не покрыт тестами.
• Тут восемь строчек вызовов библиотечных функций PHP. Ты работаешь со строками, массивами и файлами. Это не имеет никакого отношения к бизнес-логике приложения. Разбей код на функции и назови их согласно предметной области задачи. Ну и чтобы два раза не вставать, этот код сейчас не тестируемый.
• Зачем здесь модель создаётся в контроллере?
• Кажется, это костыль. Я посмотрел в документации по Ларавелю, это решается через default attribute values