.csファイル内に複数のクラスを作成することをお勧めしますか、それとも各.csファイルに個別のクラスが必要ですか？
例えば：
public class Items
{
public class Animal
{
}
public class Person
{
}
public class Object
{
}
}これが良いアーキテクチャの貧弱な例であるという事実を少し避けて、.csファイルに複数のクラスがあることはコード臭いですか？
回答:
私の意見では、あなたが与えた例は実際には問題ありません。あなたは内部クラスを宣言しているので、それらを同じファイルに保管するのは完全に賢明です。これを回避する唯一の方法は、Itemsクラスを部分クラスにし、それを複数のファイルに分割することです。私はこの悪い習慣を検討します。ネストされたクラスの私の一般的なポリシーは、それらが小さくプライベートであることです。これには2つの例外があります。
質問を少し違った言葉で言い、「各名前空間レベルのクラスを独自のファイルに入れるべきか」と尋ねると、私の答えは「はい」になります。
クラスを設計するときは、単一責任の原則を尊重します。コードの形がセマンティクスに従うとコードが読みやすくなります。したがって、クラスごとにファイルを分割するのが賢明です。
機械的な観点からは、クラスごとにファイルを持つことにはいくつかの利点があります。異なるウィンドウで複数のクラスを同時に開くことができます。真面目な開発者は2つ未満の画面で作業しないため、これは特に重要です。頭の前でより多くのコンテキストを持つことができるということは、頭の中でより多くのコンテキストを維持できることを意味します。（ほとんどのIDEでは同じファイルを2回開くことができますが、これは厄介です）。
次の重要な側面は、ソース管理とマージです。クラスを分離しておくことで、同じファイルに変更が加えられたときに、別のクラスを変更する必要があるため、多くの面倒を回避できます。
正直に言うと、ファイルに複数のルートクラスを追加しないでください。私の最後の仕事では、複数のクラスだけでなく、複数の名前空間を持つファイルがあり、これは数千行のコードに広がっていました。 フォローするのは非常に難しい。
密接に関連するクラスがある場合は、それらのファイルに同様の名前を付けるか、サブフォルダーに入れます。
クラスファイルを物理的に分離すると、すべての問題が解決するわけではなく、懸念の分離と疎結合の発生に役立ちます。
一方、この例では複数のルートクラスを示していません。ネストされたクラスがいくつかあります（注：ネストしたクラスを作成しないでください。ただしprivate、これを設計できる場合のみ）。これは、1つのファイルに含めるのにまったく問題ありません。
ファイルが非常にまとまりがある場合、たとえば、いくつかの非常に短い、同様の名前のファイルを持つ代わりにファイルを使用する場合は、良いアイデアです。
たとえば、Fluent NHibernateを使用する場合、ツールがそれについて何と言っていても、1つのファイルに保存しておくEntityと簡単ですEntityMap。
ほとんどの場合、クラスを見つけるのが難しくなります。慎重に使用してください。
単純に「はい」、これを行うには良い形式ではありません。その理由を説明します。ソリューションが大きくなると、ファイル名が内容やファイル名を表さないため、クラスの場所を忘れてしまいます。 AnimalPersonObject.csは実行不可能です。
もちろん、resharperなどのツールの機能を使用して型にジャンプすることでこれを回避できますが、ファイルごとに1つのクラス（インターフェイスを含む）は、.netだけでなくJavaで見たコード標準のバックボーンです。 c ++や他の多くの言語、メンテナンスの問題が頻繁にない言語、そしてジュニア開発者がコードを把握するのが難しいことに気付くでしょう。
ほとんどすべてのコード最適化ツールは、クラスを別のファイルに移動するよう指示するため、私にとってはこれはコードの匂いであり、それを中和するためにいくつかの追い出しが必要です:)
別の問題は、同じファイルに非常に大きく類似したクラスがある場合- クラス宣言が常に表示されない場合-間違ったクラスにブレークポイントを配置し、なぜヒットしないのか疑問に思うことです...うん！:)