Коля Митинhttps://teletype.in/atom/nickmitin2026-06-17T16:21:45.186Znickmitin:save-you-weekend2020-12-25T05:34:27.939Z2020-12-25T05:53:19.872Z

Допустим есть заказчик, который обращается к вам раз в полгода. Он пришёл в очередной раз и хочет, чтобы вы сделали на его сайте корзину покупок. Вы можете приступить на следующей неделе. Так как это большая задача, вы написали план работы:

<p>Допустим есть заказчик, который обращается к вам раз в полгода. Он пришёл в очередной раз и хочет, чтобы вы сделали на его сайте корзину покупок. Вы можете приступить на следующей неделе. Так как это большая задача, вы написали <strong>план работы</strong>:</p> <p>— пн: создание моделей и контроллеров и предствалений для корзины: кнопка «добавить», список товаров, оплата, список заказов с их состояниями.</p> <p>— вт-ср: написание тестов и логики работы корзины.</p> <p>— чт: сборка на тестовом сервере, показ заказчику, согласование</p> <p>— пт: деплой</p> <blockquote>(Это, безусловно, нереальный план, но для нашего обсуждения пойдёт)</blockquote> <p>Объявили о своих намерениях заказчику, он согласился. Вы продолжаете работать над текущими задачами, размышляя на фоне о корзине. Внезапно вы осознаёте, что уже не очень хорошо помните как в том проекте что устроено, полгода ведь прошло и решаете выкачать последнюю версию кода из репозитория. Выкачиваете, а там 46 изменённых файлов и 4237 новых строк кода! </p> <p>Теперь придётся это всё просматривать и разбираться, кто и зачем это всё написал и как это повлияет на вашу задачу. Нужно выделять на это время, значит посиделки с друзьями сегодня вечером в пролёте.</p> <p>Разбирая код, вы обнаруживаете, что кто-то как раз пытался сделать корзину. Узнаёте у заказчика, он вспоминает, что три месяца назад он попытался поработать с другим подрядчиком, но у них не сложились отношения.</p> <p>Вам теперь совсем непонятно, что конкретно там было сделано, можно ли это полностью откатить или попытаться это доделать. </p> <p>В этот момент вы осознаёте, что у вас нарисовались адские выходные, потому что вы уже занимаетесь задачей, а планировали с понедельника.</p> <p>Давайте теперь отмотаем время на начало истории, когда вы сели писать план и перепишем его:</p> <p>— пн: приём текущего состояния проекта, обновление кода, сборка, анализ изменений.</p> <p>— вт: выбор оптимальной тактики: написание собственной корзины, использование сторонней библиотеки, внешнего сервиса.</p> <p>— ср: согласование тактики с заказчиком, написание плана работ, согласование плана работ по реализации согласованной тактики.</p> <p>Вуаля, теперь вам не нужно думать о задаче до понедельника. <strong>Посиделки с друзьями и выходные спасены!</strong></p> nickmitin:productive-conversations2020-12-23T03:36:45.933Z2020-12-23T03:50:56.558Z

<img src="https://teletype.in/files/51/21/51216440-b6ee-41c8-84e3-a1a503daa9cf.png">Каждый день в районе 6 утра, я завариваю кофе и сажусь проверять пул-реквесты разработчиков в Гитхабе:

<p>Каждый день в районе 6 утра, я завариваю кофе и сажусь проверять пул-реквесты разработчиков в Гитхабе:</p> <figure class="m_column"> <img src="https://teletype.in/files/51/21/51216440-b6ee-41c8-84e3-a1a503daa9cf.png" width="1618" /> </figure> <p>На следующий день я вижу ответ от разработчика:</p> <figure class="m_column"> <img src="https://teletype.in/files/6a/50/6a508286-a441-4a87-81f3-80e9790213a8.png" width="2360" /> </figure> <p>Сначала я такой:</p> <figure class="m_column"> <img src="https://teletype.in/files/c0/9e/c09eb256-2e78-42b9-97dd-c6d1f0afbc58.png" width="693" /> </figure> <p>А потом думаю: «Хм... А это тема для поста.»</p> <p>Итак. Проблема ответа разработчика в том, что мне теперь непонятно, что с этим ответом делать. Во-первых, я не понимаю, что он собирается по этому поводу предпринять. Во-вторых, я не знаю, почему ему кажется, что это костыль. </p> <p>Получается, я могу написать только «Почему? И что с этим тогда делать?» и отправить целый пул-реквест на полку ещё на 24 часа.</p> <p>То есть бесполезный ответ на моё замечание увеличил время до релиза фичи на 24 часа.</p> <p>Уберите эмоциональную окраску своих сообщений. Собеседник вложит в ваш текст эмоции на свой вкус, обидится и дальнейшая дискуссия будет идти на эмоциях. </p> <p>Дополняйте оценочные суждения развёрнутыми объяснениями, примерами и ссылками на источники. Если ваш собеседник не телепат, он ничего не сможет сделать с неаргументированным мнением.</p> <p><strong>Опасно</strong>:</p> <ul> <li>Это полная хуйня.</li> <li>Как ты считаешь, эту поебень кто-то через год сможет понять?</li> <li>Блять! Мы уже полгода говорим о том, что модели нельзя из контроллеров вызывать, хули ты так сделал?</li> <li>Кажется, это костыль.</li> </ul> <p><strong>Продуктивно</strong>:</p> <ul> <li>Я не могу принять этот код, потому что он нарушает наши принципы: оформлен не по PSR и не покрыт тестами.</li> <li>Тут восемь строчек вызовов библиотечных функций PHP. Ты работаешь со строками, массивами и файлами. Это не имеет никакого отношения к бизнес-логике приложения. Разбей код на функции и назови их согласно предметной области задачи. Ну и чтобы два раза не вставать, этот код сейчас не тестируемый.</li> <li>Зачем здесь модель создаётся в контроллере?</li> <li>Кажется, это костыль. Я посмотрел в документации по Ларавелю, это решается через <a href="https://laravel.com/docs/8.x/eloquent#default-attribute-values" target="_blank">default attribute values</a></li> </ul>