Recenzje książki «Рецепты чистого кода (PDF + EPUB)», 1 recenzja

Кратко:

Ситуация похожа на:

0. Пытаемся найти материал по теме

1. Натыкаемся на видео

2. Слышим "Хелло эври ван, тудей ай гона шов ю ..." (именно так и с акцентом)

3. Бегло проматываем, видим костыли в коде и посредственный финальный результат

4. Закрываем видео

Полный отзыв:

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

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

Опыта программирования у него 4 года и то везде не полный день,

после этого он то тестировал, то еще что-то, фрилансил.

Про книгу:

Либо перевод корявый, либо противоречия явные.

Тут можно вставить фразу:

"Система, то есть целое, всегда больше суммы частей системы"

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

1. Есть рекомендация

2. Показать пример, где эту рекомендацию не используют

3. Показать как "надо было", но это будет нарушать другие рекомендации из этой же книги

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

5. По итогу нет конечного варианта как правильно в большой системе,

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

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

а на самом деле "ну я не знаю как другие патерны туда вписать"

Еще моменты (самые простые):

1. Удаляйте комментарии, если это не проектное решение (костыль). Используйте тесты вместо них

(Ниже в п.3 проще комментарий указать или имя функции раздуть?

А может быть все в тесты непонятно куда вынесем,

чтобы мы эти тесты нашли и потом начали разбираться что каждый делает? противоречит п.2)

2. "Давайте конкретные имена, чтобы снизить когнетивную нагрузку во время чтения"

(Ниже в п.3 конкретное имя, легче читать? Понимаем что код делает?)

3. В другой главе функцию c regex "CheckIfPasswordIsValid" меняем на "CheckIfStringHas3To7LowercaseCharsFollowedBy3or4Numbers"

4. И в следующей главе тут же "Замена длинных имен"

(А почему выше исключение было?)

Вывод следующий:

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

но рекомендации в сумме противоречат друг другу.

Да, понятно, что всем рекомендациям следовать не получиться,

но экспертность автора вызывает сомнения.

По рекомендациям, очень хорошо показано как "не надо", а как "надо" думайте сами.

"Критикуешь - предлагай"

По кусочкам читать можно, а в сумме - противоречия