search

bespoyasov / solidbook

Book about the SOLID principles and object-oriented software design.
https://solidbook.vercel.app
MIT License
609 stars 58 forks source link

SRP: Исправление опечатки и сокращение примера #129

Closed tenorok closed 5 years ago

tenorok commented 5 years ago

Привет!

Нашёл опечатку: метод format() должен принимать параметр с данными.

Заодно сократил пример класса ReportExporter, чтобы убрать шум, акцентируя внимание на сути. Хотя поле name вроде как тоже избыточно, но ладно, это можно считать неким именем отчета.

vercel[bot] commented 5 years ago

This pull request is automatically deployed with Now. To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://ota-solid-git-fork-tenorok-fix-book-typo.ota.now.sh

dex157 commented 5 years ago

@tenorok, @bespoyasov даже тем кто знаком, такая нотация не всегда разрешена по code guide, так как слишком большая перегрузка аргументов конструктора идёт. Мне привычнее выносить аннотацию переменных класса на первый уровень класса. Если аргументов будет много, конструктор раздует и читать его уже будет не так комфортно. Ну и, у нас тут скорее формат учебника, мы стараемся писать как можно более простым языком )

tenorok commented 5 years ago

Согласен, ребят, явное лучше неявного, особенно в учебных целях. Вернул обратно.

Продолжил читать сегодня книгу и нашёл ещё ошибки, добавил их в первый коммит. Например, декларация class Player implements Player работает очень странно и вообще выглядит запутанно.

Вторым коммитом привёл к единому стилю код примеров, а то режет глаз. Функции в поле класса — это вообще что-то очень страшное и странное, ну а точки с запятыми я так понял отсутствуют в подавляющем большинстве примеров.

tenorok commented 5 years ago

Исправил ещё одну ошибку (см. последний коммит) в главе "OCP: Примеры из идеального мира" — там в первоначальном примере класса Circle лишний метод упоминается. Если всё окей, то апрувьте пул, я сосквошу коммиты :-)

tenorok commented 5 years ago

Готово. Методы Саша отдельно уже сделал, так что здесь из кодстайла остались только точки с запятыми.

tenorok commented 5 years ago

@dex157 давай вольём?