オブジェクトが所有者の多くを知っている場合は、コードのにおいですか？私たちのデルファイ2007アプリケーションで

Question

は、我々は上向き（例TdmBasicDataに）特定のクラスを検索するには、次の構文

FdmBasic:=TdmBasicData(FindOwnerClass(AOwner,TdmBasicData)); 

FindOwnerClassは、現在のコンポーネントの所有者の階層を移動するの多くを使用しています。結果のオブジェクトは、Field変数FdmBasicに格納されます。これは、主にデータモジュールを渡すために使用します。

例： レポートを生成すると、結果のデータは圧縮され、データモジュールTdmReportBaseDataを介してアクセスされるテーブルのBlobフィールドに格納されます。アプリケーションの別のモジュールには、ReportBuilderを使用してレポートからのデータをPaged形式で表示する機能があります。このモジュール（TdmRbreport）のメインコードは、クラスTRBTempdatabaseを使用して、圧縮BLOBデータをReportbuilderランタイムreportdesignerで使用可能な異なるテーブルに変換します。 TdmRbreportは、あらゆる種類のレポート関連データ（レポートの種類、レポートcalculationsettingsなど）のTdmReportBaseDataにアクセスできます。 TRBTempDatabaseはTdmRbreportで構築されますが、TdmReportBasedataにアクセスできる必要があります。

constructor TRBTempDatabase.Create(aOwner: TComponent); 
begin 
    inherited Create(aOwner); 

    FdmReportBaseData := TdmRBReport(FindOwnerClass(Owner, TdmRBReport)).dmReportBaseData; 
end;{- .Create } 

私の気持ちは、これはTRBTempDatabaseは、その所有者の多くを知っていることを意味していることであり、これはコードのにおいやアンチパターンのいくつかの並べ替えがある場合、私は思っていた：これは今の上に建設を使用して行われます。

これについてのご意見はありますか？これはコードのにおいですか？もしそうなら、もっと良い方法は何ですか？

Answer 1

ここに記載されている説明では、これを軽度に臭いとみなしています。ただし、修正するのは簡単です。

私はdmReportBaseDataオブジェクトを必要とするコンポーネントのコンストラクタに渡す傾向があります。これにより、現時点で実行時に強制的に実行するのではなく、コンパイル時に契約が明確になります。

現在のところ、強制する契約は、必要以上に強くなっています。 TRBTempDatabaseはdmReportBaseDataインスタンスのみを必要としますが、TdmRBReportレポートオブジェクトからそのインスタンスを取得できる場合にのみ機能します。

この変更を行うと、TRBTempDatabaseとTdmRBReportは離婚しても正常に機能します。 @Lievenはコメントの中で指摘しているように、テストを簡単にするでしょう。

Answer 2

基本クラスで行っていることは、親オブジェクトへの参照を維持している場合はいいえ、コード匂いではないことは完全に正当な使用です。 「後で来るかもしれないもの」に関する情報を伝達するために、明示的に基本クラスを設計することができます。

基本クラスがそれ自体に存在しない派生クラスの特性に依存している場合（つまり、一般化されたクラスはその子のうちの1つの特殊化に依存しています）、それはちょっとファンキーです。