4

これは、効果的なGOがgolangへの埋め込みについて言わなければならなかったことです

型を埋め込むと、その型のメソッドは外側の型のメソッドになりますが、それらが呼び出されると、メソッドのレシーバーは外側の型ではなく内側の型になります。

Struct User次のように定義したコードスニペットがありました

type User struct {
    Name     string
    Password string
    *sql.Tx
}

そして、などを呼び出します。明らかにデメテルの法則に違反するu.Query("some query here")ような呼び出しを回避できるように、これを特に行いました。ドキュメントと効果的なgoを読んだ後、最初のアプローチのメリットについても疑問があります。デメテルの法則に違反していますか? はいの場合、どうすれば回避できますか?u.Transaction.Query("query")

4

1 に答える 1

6

埋め込みの概念は、型自体がエクスポートされた場合に型が埋め込まれたという事実を隠さないため、デメテルの法則に多少違反します。エクスポートされていない型を埋め込んでもLoD に違反しないことに注意してください (エクスポートされていないフィールドとメソッドを参照することはできません)。

しかし、これは、プロモートされたフィールドやメソッドを LoD にも違反する方法で参照することを強制するものではありません。埋め込み自体は、一般的な共有コードを他のタイプに「アウトソーシング」できるようにするための単なる手法です。または別の観点から、新しいタイプを作成するときに他のタイプを利用します。昇格された埋め込み型のフィールドまたはメソッドを参照する方法は、法律に違反する可能性があります。

あなたが言ったように、それを と呼ぶと、それはデメテルu.Tx.Query()の法則の明らかな違反です: を埋め込む実装の詳細を使用しています。User*sql.Tx

しかし、次のように呼び出すと、問題ありませんu.Query()*sql.Txこのフォームは、埋め込まれているという事実を公開したり利用したりしません。このフォームは、実装が変更され、もはや埋め込まれなくても機能し続け*sql.Txます (たとえば、「通常の」フィールドに変更されるか、完全に削除され、User.Query()メソッドが追加されます)。

エクスポートされた埋め込み型のフィールド値へのアクセスを許可したくない場合は、エクスポートされていない通常のフィールドにUser.Query()して、フィールドに委任できる「明示的な」メソッドを追加します。

type User struct {
    Name     string
    Password string
    tx       *sql.Tx // regular field, not embedded; and not exported
}

func (u *User) Query(query string, args ...interface{}) (*sql.Rows, error) {
    return u.tx.Query(query, args...)
}

その他の注意事項:

例では、u.Query()が使用されている場合、これを使用しているクライアントは、 の内部Userが変更されても影響を受けません (u.Query()昇格されたメソッドを示しているか、 のメソッドUser、つまり: を示しているかは問題ではありませんUser.Query())。

変更した場合sql.Tx、はい、u.Query()もう有効ではない可能性があります。しかし、互換性sql.Txがないことはほとんどありません。また、あなたが変更されたパッケージの開発者であり、互換性のない変更を行っている場合、互換性のない変更に依存する他のコードを変更するのはあなたの責任です。そうする(適切に更新するu.Query())ことで、呼び出したクライアントはu.Query()影響を受けず、クライアントは内部で何かが変更されたことを知る必要はありません。

これはまさに LoD が保証することu.Query()です。LoD は悪いことではありません。落としてはいけません。従う原則を選択することはできますが、選択した原則が指示するすべてのことに常に従うのではなく、考えるべきです。u.Tx.Query()Useru.Query()

もう 1 つクリアする必要があるのは、LoD には API 非互換の変更は含まれていないということです。それが提供するものは、従う場合、エンティティの「公開」面を使用するエンティティの内部変更が他のエンティティに影響を与えないことです。sql.Txが大幅に変更されて利用できなくなった場合Tx.Query()、それは LoD で「カバー」されません。

于 2015-10-20T10:46:36.833 に答える