.NET/C# デザインパターンレビュー

${selection} 内の C#/.NET コードをデザインパターン実装についてレビューし、ソリューション/プロジェクトの改善提案を行います。コードは変更せず、レビューのみを提供します。

必須デザインパターン

• Command Pattern: ジェネリック基本クラス (CommandHandler<TOptions>)、ICommandHandler<TOptions> インターフェース、CommandHandlerOptions 継承、静的 SetupCommand(IHost host) メソッド
• Factory Pattern: 複雑なオブジェクト生成サービスプロバイダー統合
• Dependency Injection: プライマリコンストラクタ構文、ArgumentNullException null チェック、インターフェース抽象化、適切なサービスライフタイム
• Repository Pattern: 非同期データアクセスインターフェースプロバイダー接続抽象化
• Provider Pattern: 外部サービス抽象化 (データベース、AI)、明確なコントラクト、設定処理
• Resource Pattern: ローカライズされたメッセージ用 ResourceManager、別ファイル .resx (LogMessages、ErrorMessages)

レビューチェックリスト

• デザインパターン: 使用されているパターンを特定してください。Command Handler、Factory、Provider、Repository パターンが正しく実装されていますか？有益なパターンが不足していますか？
• アーキテクチャ: 名前空間規約 ({Core|Console|App|Service}.{Feature}) に従っていますか？Core/Console プロジェクト間の適切な分離がありますか？モジュール化されており読みやすいですか？
• .NET ベストプラクティス: プライマリコンストラクタ、async/await と Task 戻り値、ResourceManager 使用、構造化ログ、強い型付けされた設定？
• GoF パターン: Command、Factory、Template Method、Strategy パターンが正しく実装されていますか？
• SOLID 原則: 単一責任、オープン/クローズ、リスコフ置換、インターフェース分離、依存性逆転の違反がありますか？
• パフォーマンス: 適切な async/await、リソース破棄、ConfigureAwait(false)、並列処理の機会がありますか？
• 保守性: 関心の明確な分離、一貫したエラーハンドリング、適切な設定使用がありますか？
• テスト容易性: インターフェースを通じた依存性の抽象化、モック可能なコンポーネント、非同期テスト容易性、AAA パターン互換性がありますか？
• セキュリティ: 入力検証、安全な認証情報処理、パラメータ化クエリ、安全な例外処理がありますか？
• ドキュメント: パブリック API の XML ドキュメント、パラメータ/戻り値の説明、リソースファイル組織がありますか？
• コード明確性: ドメイン概念を反映した意味のある名前、パターンを通じた明確な意図、自己説明的な構造がありますか？
• クリーンコード: 一貫したスタイル、適切なメソッド/クラスサイズ、最小限の複雑性、重複排除がありますか？

改善の重点領域

• Command Handlers: 基本クラスでの検証、一貫したエラーハンドリング、適切なリソース管理
• Factories: 依存性設定、サービスプロバイダー統合、破棄パターン
• Providers: 接続管理、非同期パターン、例外処理とログ
• 設定: データ注釈、検証属性、機密値の安全な処理
• AI/ML 統合: Semantic Kernel パターン、構造化出力処理、モデル設定

プロジェクトのアーキテクチャと .NET ベストプラクティスに基づいた、具体的で実行可能な改善推奨事項を提供してください。