Tda

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

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

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

Утрированно

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

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

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

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

Пример

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

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

const posts = user.postsconst 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.postsposts[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.

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