Нерушимые законы крутого кода: Law of Demeter (с примерами на TypeScript)

от автора

Когда я узнал об этих принципах, качество моего кода выросла х2, а скорость принятия решения х5.

Если SOLID – это набор принципов написания качественного кода, то Law of Demeter (LoD) и Tell Don’t Ask (TDA) – это конкретные приемы как добиться SOLID.

Сегодня поговорим про Law of Demeter («Закон Деметры»).

Утрированно

Это принцип помогает определиться: «Как я буду получать / изменять вложенные объекты» – применим в языках, где можно определить «классы» со свойствами и методами.

Часто складывается ситуация, когда мы откуда-то (например, из HTTP запроса) получили id сущности `a`, пошли за ней в БД и из сущности `a` нам надо получить / изменить сущность `b` вызвав метод `Method`.

Так вот Википедия гласит:

Код `a.b.Method()` нарушает Закон Деметры, а код `a.Method()` является корректным.

Пример

У Пользователя есть Посты, в которых есть Комментарии. Вы хотите получить «Комментарии последнего Поста».

Можно запилить такую ересь:

const posts = user.posts const lastPostComments = posts[posts.length-1].comments

Или вот такую (одного толку):

const userLastPostComments = user.getPosts().getLast().getComments()

Проблема: код, который вызывает это должен знать о всей иерархии вложенных данных.

Соответственно, если эта иерархия меняется / расширяется, везде, где был вызвана вот эта цепочка, придется вносить изменения (рефакторить код + тесты).

Чтобы решить проблему, применяем LOD:

const userLastPostComments = user.getLastPostComments()

А вот уже в `User` пишем:

class User {   // ...   getLastPostComments(): Comments {     const lastPost = this.posts.getLastPost()     return lastPost.getComments()   }   // ... }

Та же самая история с добавлением комментария. Мы из:

const newComment = new Comment(req.body.postid, req.body.content) user.getPosts().addComment(newComment)

Или, если вы хотите блеснуть 3-ей хромосомой:

const newComment = new Comment(req.body.postid, req.body.content) const posts = user.posts posts[posts.length-1].comments.push(newComment)

Превращаем вот в это:

const posts = user.addComment(req.body.postid, req.body.content)

А у `User`:

class User {   // ...   addComment(postId: string, content: string): void {     // The cleanest     const post = this.posts.getById(postId)     return post.addComment(content)   }   // ... }

Уточнение: `new Comment` можно создать и вне `user` или `post`, все зависит от того, как устроена логика вашего приложения, но чем ближе к владельцу сущности (в данном случае, `post`), тем лучше.

Что это дает?

Мы скрываем детали реализации, и если когда-нибудь произойдет изменение / расширение иерархии (например, посты начнут лежать не только в свойстве `posts`) вам придется только отрефакторить метод `getLastPostComments` и переписать юнит-тест только этого метода.

В чем минусы

Много доп. кода.

В небольших проектах большая часть методов окажутся просто `getter` / `setter`.

Когда использовать

(1) Вы видите, что цепочки запросов внутрь сущностей становятся слишком длинными (более 3-х `.`)

(2) Код требует понятия: «Получить комментарии последнего поста» – а посты у вас не в 1-ом свойстве, а в 2-х и более, тогда, точно нужно делать метод `getLastPostComments` и уже там мерджить несколько свойств с разными постами.

(3) Я стараюсь максимально часто использовать этот принцип, когда речь идет о трансформации (изменение, создание, удаление) данных. И реже, когда речь о получении данных.

(4) Основываясь на здравом смысле.

Что для этого нужно

LOD хорошо работает в том случае, если у вас правильно сделано дерево / иерархия зависимостей сущностей.

Что почитать

(1) https://qna.habr.com/q/44822
Обязательно почитать все комментарии и комментарии комментариев (до коммента Вячеслав Голованов SLY_G), помня, что там и правильные и неправильные примеры
(2) https://ru.wikipedia.org/wiki/Закон_Деметры
(3) Статья

P.S.

Я мог напортачить с некоторыми деталями / примерами или объяснить недостаточно понятно, поэтому отпишитесь в комментарии что заметили, я внесу изменения. Всем добра.

ссылка на оригинал статьи https://habr.com/ru/post/512716/


Комментарии

Добавить комментарий

Ваш адрес email не будет опубликован. Обязательные поля помечены *