多くのusing / importステートメントがあるのは、デザインやコードの悪臭を示していますか？

通常、クラスファイルの先頭にいくつかのusingステートメントが表示されます。例：

using System.Collections.Generic;
using System.IO;
using System.Xml.Linq;

しかし、私が取り組んでいるプロジェクトでは、いくつかの機会に、1つのクラスファイルで20以上の使用/インポートが見られます。これは悪いデザインですか？1つのことを行うために設計されたクラスは、少数のコンポーネントにのみ依存する必要があるようです。

悪いデザインを示している可能性があります。あなたが見ているクラスがあまりにも多くのことをしているかもしれませんが、インポートしている名前空間は、名前空間の分離が意味するよりも本当に結合していることを意味するかもしれません。それは過剰なエンジニアや過剰な抽象化によるものかもしれませんが、使用とうまく調和しない設計であるかもしれません。

とはいえ、それは臭いです-時にはそれはあなたを悪い何かに導き、時にはそれは単なる誤報です。

クラスは、目的の機能を提供するために必要なものより大きくも小さくもないはずです。

1つのアプリケーションには、データを暗号化するという1つのジョブを持つ暗号化クラスがあります。コア機能、ロギング、および例外処理の間で、この1つのジョブを実行するには、環境に11の異なるネームスペースインポートが必要です。

「輸入を最小限に抑える」と自分で言うのは難しいです。または、インポートされた名前空間の数を数えるだけでクラスの成功を判断します。クラスの成功を示すものとして、これを単独で正当に使用することはできません。私にとって、「輸入」は授業の最後の手段です。クラスのジョブが明確に定義されている場合、インポートを含むサポートシステム内の他のすべてが処理されます。