Python オブジェクト指向 クラス設計 デメテルの法則

Question

###前提・実現したいこと
オブジェクト指向のクラス設計で困っています。以下のコードについて説明すると、
Comment1,2,3はそれぞれCommentクラスを継承しており、ラッパークラスです。
ReportクラスはComment1,2,3以外にもDate,Nameというラッパークラスを所持しています。
まず、何故Comment1,2,3と分けているかというと、1,2,3それぞれで異なる処理を行うからです。同じ処理を行うこともあるので継承という形にしています。

※各クラスのメソッドは省略しています。

ここでなのですが、デメテルの法則に従うと、Comment1,2,3共通の処理をラッパーメソッドとしてReportクラスに３回同じものを実装しなくてはなりません。これじゃ継承した意味もなくとてつもなく汚く無駄なコードになってしまいます。ですが、ここでデメテルの法則を無視すると、Reportクラスにプロパティを実装するだけで済みます（例：Report.Comment1.length()）

そこでなのですが、一般的にデメテルの法則は必ず守った方が良いものなのでしょうか？
また、このケースにおいて、デメテルの法則を守ってかつコードに無駄がないような実装にするにはどのような解決策があるでしょうか？

元々はcomment1,2,3をCommentという１つのクラスで管理しており、引数に数字を与えることで処理するコメントを選んでいました。しかし、commentによって違った処理をすることがあるので、それぞれ変数をラッピングした方が良いのではないかという考えにいたり、今回のような問題が発生したために質問させていただきました。

###該当のソースコード

class Comment:
    def __init(self,comment):
        self._comment = comment

class Comment1(Comment):
    def __init(self,comment1):
        super().__initi(comment1)
        self._comment1 = comment1

class Comment2(Comment):
    def __init(self,comment2):
        super().__initi(comment2)
        self._comment1 = comment2

class Comment3(Comment):
    def __init(self,comment3):
        super().__initi(comment3)
        self._comment1 = comment3

class Report:
    def __init__(self,comment1,comment2,comment3,date,name)
        self._comment1 = Comment1(comment1)
        self._comment2 = Comment2(comment2)
        self._comment3 = Comment3(comment3)
        self._name = Name(name)
        self._date = Date(date)

Answer 1

Reportクラスにプロパティを実装するだけで済みます（例：Report.Comment1.length()）

おっしゃるように、今回ですとそのように書くほうがスマートです。

オブジェクト指向の設計・原則はあちらを立てばこちらが立たず...のようなトレードオフの関係になることがあったり、場合によっては従わない方がいい、というのはご存知かと思います。
また、言語自体の設計思想であったり、そのコミュニティで生まれた考え方がこれらの原則やお約束と衝突することもあります。

例えば、Pythonにおける属性のへのアクセスは基本的にオープンにする方がいい、というものがあります。
他のいくつかの言語と違い、クラスの変数に対してとりあえずゲッター、セッターを付ける、ということはあまりしないですし、今回の例でいえばReport.Comment1.length()という書き方でそのままアクセスするのも別におかしくありません。

一般的にデメテルの法則は必ず守った方が良いものなのでしょうか？

場合によるとは思いますが、Pythonではそこまで固執しなくても良いのかなぁと思います。
Pythonの禅に
「特殊条件だからといって原則を破っていいわけではないが、実用性は純粋性に勝る」
という旨の格言があるように、あるルールや原則を守るために冗長になったり、複雑になる実装は避けたほうが良いように感じます。

Answer 2

オブジェクト指向とは、そもそも読みやすくするためにあるはずなので、オブジェクト指向を実践したいからと、読みにくいコードを作ったら、本末転倒だと思います。デメテルの法則があると、設計が綺麗にならないのならば、それはデメテルの法則を使うべきときではないと思います。

私の肌感覚では、あらゆる設計について完璧に正しい「法則」があるとは思えません。全ては設計のヒント、ぐらいだと捉えておくほうが良いと思います。

Comments

[_Victorique__]

質問文にも書いていますが、法則に全て従わないといけないなんて思っていません。
この質問は設計を見直すことでデメテルの法則を無視しないで済むかという問いかけです。具体的なクラス設計について答えていただけると嬉しいのですが。

[sannye]

なぜデメテルの法則を使いたがるのかがわかりません。ご自身で書かれているように、無視したほうが綺麗になる可能性が高いです。YAGNIです。

[_Victorique__]

良いとされているものを使うのは普通だと思いますが何がおかしいんですか？
ちょっと勘違いされすぎではないでしょうか？質問をよく読んでくださると助かります。