質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.53%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

2回答

813閲覧

C#のクラス定義を配列にして、そこからnewしたい

Narua

総合スコア3

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

0クリップ

投稿2023/01/20 11:39

前提

C#で処理の分岐を各クラス別に記述する事で処理をおこなっています。
ですが、クラスの生成が上手く書く事ができず、この方法がよいのか悩んでおります。

実現したいこと

識別子をwitch文で分岐し、そこからクラスを生成することで処理を分岐されております。
しかし、タスクの種類が多くなるとSwitch文が長くなってしまい、コードが見難くなってしまいます。
また、識別子とクラスの関係が1:1となっておりますので、もっとスマートに実装できそうな気がしておりますが、方法がわかりません。

例えば、

ITaskBase[] TaskTypeArray = { TaskA, TaskB, TaskC, ... };
return new TaskTypeArray[task_type];

みたいな書き方や、

var taskname = "Task" + task_type;
return new taskname;

といった記述で作成する方法はないのでしょうか?

該当のソースコード

C#

1■ 定義 2 3// 抽象クラス 4abstract class ITaskBase 5{ 6 public abstract bool Process(); 7} 8 9// 処理タイプ(A) 10class TaskA : ITaskBase 11{ 12 public override bool Process() 13 { 14 // ...処理... 15 } 16} 17 18// 処理タイプ(B) 19class TaskB : ITaskBase 20{ 21 public override bool Process() 22 { 23 // ...処理... 24 } 25} 26 27// 処理タイプ(C) 28class TaskC : ITaskBase 29{ 30 public override bool Process() 31 { 32 // ...処理... 33 } 34}

C#

1■ 作成する際は、 2 3enum task_type; 4 5switch(task_type) 6{ 7 case A: 8 return new TaskA(); 9 10 case B: 11 return new TaskB(); 12 13 case C: 14 return new TaskC(); 15 16 // ...タスクの種類だけ続く... 17}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2023/01/20 12:35

> タスクの種類が多くなるとSwitch文が長くなってしまい、コードが見難くなってしまいます。 質問のコードの「// ...処理...」というのはどういうものですか? task_type によって全く処理が異なるコードを書かなければならないのであれば、switch 文を使おうが何をしようが、処理が異なるコードを全部書かざるを得ず、その分コードが長くなるのは避けようがないと思うのですが?
Zuishin

2023/01/20 13:04

今日退会した例の人かな?
dodox86

2023/01/20 13:26

古くからあるデザインパターンのひとつ、Factoryパターンやその類型が参考になりそうに思います。今はもっとモダンなやり方があるのかも知れませんが。
Narua

2023/01/23 01:01

> 「// ...処理...」というのはどういうものですか? DB操作だったり、XML解析だったり、出力だったり、まったく別の処理内容となっています。 簡易スクリプトのような文字列を読み込んで、それに沿って処理を行おうとしています。 > 今日退会した例の人かな? はじめまして 初めて登録しましたので、別の方かと… > Factoryパターンやその類型が参考 ですね、Factoryで言うところ(?)のCreateの実装になります。
退会済みユーザー

退会済みユーザー

2023/01/23 11:51 編集

>> 「// ...処理...」というのはどういうものですか? > DB操作だったり、XML解析だったり、出力だったり、まったく別の処理内容となっています。 そういうことだとすると、そもそもの考え方を根本から見直した方が良さそうな気がしますけど。 クラスの中の Process メソッドは「まったく別の処理内容」を行うのに、共通の abstract クラスを継承したクラスを処理に必要な数作って、処理によってクラスを選択してインスタンスを生成し、処理を実行するなんてことは自分的には(たぶん一般的にも)あり得ません。 Dependency injection を考えた方が良さそうな気がします。
退会済みユーザー

退会済みユーザー

2023/01/24 00:45

質問者さん、その後無言ですが、回答が出ているのでそれらに対するフィードバックを返してください。役に立たなかったならどこがダメかを書くとより期待に近い回答が出てくるかも。解決したなら解決に役立った回答にベストアンサーをつけてクローズしてください。
Narua

2023/01/24 05:29

ご指摘ありがとうございます。 クローズする必要があるんですね、了解です。
退会済みユーザー

退会済みユーザー

2023/01/24 05:40

> クローズする必要があるんですね、了解です。 そうです。 「ベストアンサーをつける」=「スレッドをクローズする」ということになります。
guest

回答2

2

ベストアンサー

Type.GetType メソッドを使って生成したいクラスの Type を文字列から取得することが出来ます。

Type.GetType メソッド
https://learn.microsoft.com/ja-jp/dotnet/api/system.type.gettype?view=netframework-4.8

取得した Type から Activator.CreateInstance(Type) でインスタンスを作ることが出来ます。

Activator.CreateInstance(Type)
https://learn.microsoft.com/ja-jp/dotnet/api/system.activator.createinstance?view=netframework-4.8#system-activator-createinstance(system-type)

csharp

1var o = (ITaskBase)Activator.CreateInstance(Type.GetType("ConsoleApp1.TaskA")); 2 3public enum task_type 4{ 5 A,B,C 6} 7 8static class Factory 9{ 10 public static ITaskBase CreateInstance(task_type taskType) { 11 return (ITaskBase)Activator.CreateInstance(Type.GetType("ConsoleApp1.Task" + taskType.ToString())); 12 } 13}

Type[] TaskTypeArray = { typeof(TaskA), typeof(TaskB), typeof(TaskC), ... };

のように配列を作るのもいいでしょうね。

投稿2023/01/20 13:46

編集2023/01/20 13:58
KOZ6.0

総合スコア2597

dodox86, Narua👍を押しています

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Narua

2023/01/23 01:22

助言ありがとうございます! まったく知らなかった機能ですので試してみます。
guest

1

質問に対する直接の答えにはなってませんが・・・

「// ...処理...」というのはどういうものですか?

DB操作だったり、XML解析だったり、出力だったり、まったく別の処理内容となっています。

そういうことだとすると、そもそもの考え方を根本から見直した方が良さそうな気がします。

クラスの中の Process メソッドは「まったく別の処理内容」を行うのに、共通の abstract クラスを継承したクラスを処理に必要な数作って、処理によってクラスを選択してインスタンスを生成し、処理を実行するなんてことは自分的には(たぶん一般的にも)あり得ません。

Dependency injection (DI) を考えた方が良さそうな気がします。

何を作っているのか分かりませんが、.NET Core 3.1 とか .NET 6.0 などのアプリであれば、Microsoft の Microsoft.Extensions.DependencyInjection 名前空間にあるクラス類を使って DI 機能を実装できます。

TaskA などの各クラスを DI コンテナに登録しておいて、TaskA などに依存する(すなわち「// ...処理...」が必要な)クラスを初期化する際、自動的にコンストラクタ経由で TaskA などを初期化して渡してやることができます。

コンソールアプリでも DI は利用できます。具体例は以下の記事を見てください。

.NET Core での Dependency Injection
http://surferonwww.info/BlogEngine/post/2021/01/01/dependency-injection-for-dotnet-core-application.aspx

Windows Forms アプリの具体例は以下の記事を見てください。

WinForms で構成情報とコンテキストの DI (CORE)
http://surferonwww.info/BlogEngine/post/2021/03/30/how-to-inject-configuration-and-dbcontext-in-windows-forms-application.aspx

投稿2023/01/23 12:16

退会済みユーザー

退会済みユーザー

総合スコア0

Narua😄を押しています

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Narua

2023/01/24 06:24

DIを知らなかったので調べてみました。 知識不足で申し訳ないのですが、理解が追い付いてきていません… 処理内容とクラス定義の依存性(abstractクラスが不要になる)が解決できるのはわかってきたのですが、識別子から各オブジェクトをインタンス化する方法がわかりませんでした。 GetRequiredServiceや、GetServiceを行う際は、Typeとかで何かしらに変換して指定する必要がある形でしょうか? 文字列を登録して生成する方法がありそうに見えるのですが、実際には方法を見つけきれませんでした…汗
退会済みユーザー

退会済みユーザー

2023/01/24 06:53 編集

そもそも、文字列を組み立ててそれを変数とかメソッドとかクラスとして使うということは、普通は(その「普通」の考え方次第ではありますが、少なくとも自分的には)、やらないことなのです。今回の質問のケースのような場合は特に。 なので、そのあたりから考え直して、今回のケースでは DI が利用できそうですのでそれをお勧めしている次第です。「文字列を登録して生成する方法」はないです。
Narua

2023/01/24 08:47

なるほど…ありがとうございます。 何となくですが、理解できました。 今回はすべてがC#で構成されておらず、外部からの文字列や識別子などから処理を起こす必要があり、かつ、その処理順番が動的に変わっていくような仕様があります。 外部の部分は変えられないので厳しいのですが、読み込みから流れの部分で設計を検討しなおしてみます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.53%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

同じタグがついた質問を見る

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。