WIP

Author: tabuna

content/002-communication.md

@@ -46,8 +46,8 @@
 Такое письмо вызывает уже не раздражение, а недоумение.
 
 Теперь представьте: это не кандидат, а **ваш код**.
-Что будет когда два таких специалиста объединятся в одну команду? 
-Скорее всего, что-то такого в итоге будет в кодовой базе:
+Что будет когда два таких специалиста объединятся в одну команду?
+Скорее всего, в кодовой базе появится нечто подобное:
 
 > Здраствуйте господин!
 > Сим доношу, что Я ПРОГРАМИСТ с БОЛЬЩИМ апытом делал сайты и приложения для сети.

content/006-naming.md

@@ -251,7 +251,7 @@ class ExcludeItemStorage
 
 А ведь имя класса — это первый и, зачастую, единственный источник информации о его ответственности.
 
-. Эти имена — дымовая завеса. Они скрывают детали, замыливают смысл, делают код нечитаемым, а архитектуру — расплывчатой. Они подменяют суть интерфейсом, упрощая названия до абсурда. Да, это удобно. Да, так делают все. Но именно поэтому ваш проект через год превращается в груду мусора.
+Эти имена — дымовая завеса. Они скрывают детали, замыливают смысл, делают код нечитаемым, а архитектуру — расплывчатой. Они подменяют суть интерфейсом, упрощая названия до абсурда. Да, это удобно. Да, так делают все. Но именно поэтому ваш проект через год превращается в груду мусора.
 
 ```php
 // Плохо ❌
@@ -271,9 +271,10 @@ class StringTruncatedToLength { /* … */ }
 
 ### Имена тоже должны рифмоваться 
 
-> TODO: Часть симметрия
+Методы в паре работают лучше, если звучат как единое целое.
+Они как диалог: начало должно перекликаться с концом.
 
-Вот пример:
+Вот пример несогласованного именования:
 
 ```php
 // Плохо ❌

content/009-conditions.md

@@ -138,9 +138,6 @@ return $default;
 ```
 
 
-> TODO: сделать и рассказать про if($name = $this->getName()) { ... } и про то, что это не хорошо
-
-
 ### Присваивания в условиях
 
 Иногда хочется быть "умным". Сделать красиво, коротко, выразительно. Одной строкой. Как в старых учебниках:

content/009-early-exit.md

@@ -131,38 +131,21 @@ foreach ($orders as $order) {
 public function hasAccess(User $user): bool {
     if (!$user->isBanned()) {
         if ($user->isAdmin()) {
-            /*
-             * Доступ разрешён:
-             * пользователь не заблокирован и является администратором.
-             */
+            // Доступ разрешён: админ.
             return true;
         } else {
-            if($user->isGranted(GRANT::EDIT)) {
-                /*
-                 * Доступ разрешён:
-                 * пользователь не заблокирован и имеет право на редактирование.
-                 */
+            if ($user->isGranted(GRANT::EDIT)) {
+                // Доступ разрешён: может редактировать.
                 return true;
             } else {
-                /*
-                 * Доступ запрещён:
-                 * пользователь не заблокирован, но не является
-                 * администратором и не имеет прав на редактирование.
-                 */
+                // Доступ запрещён: нет нужных прав.
                 return false;
             }
 
-            /*
-             * Недостижимый код:
-             * предыдущий return уже завершает выполнение.
-             */
             return false;
         }
     } else {
-        /*
-         * Доступ запрещён:
-         * пользователь заблокирован.
-         */
+        // Доступ запрещён: пользователь заблокирован.
         return false;
     }
 }

content/017-copilot.md

@@ -54,7 +54,7 @@ function store()
 
 На сайтах вроде Stack Overflow вопросы задают и решают вручную, люди. Нередко там встречаются ответы, помеченные как "
 решение", но при этом заминусованные — в комментариях поясняют: да, работает, но приведёт к плачевным последствиям в
-долгосрочной перспективе. "Второй пилот" — помощник kkдругого типа. Он не будет указывать на архитектурные проблемы или спорные решения, пока вы
+долгосрочной перспективе. "Второй пилот" — помощник другого типа. Он не будет указывать на архитектурные проблемы или спорные решения, пока вы
 сами его об этом не попросите. Его задача — помогать вам как можно быстрее двигаться к результату, решать
 бизнес-проблемы здесь и сейчас. Но и большинство разработчиков настроены точно так же: задача должна быть закрыта, сроки — вчера, и мало кто будет
 тратить время, чтобы просить помощника подумать над более выразительными именами переменных, архитектурой или стилем.