複数のメソッドを数値だけ変えて実行出来るようにしたい

Question

前提・実現したいこと

現在VB.netでプログラム中のシステムに、
複数メソッドを数値だけ変えて実行する工程があります。

複数のメソッドで同じ数値を使いまわすので、
クラスのフィールドで数値を保持したいと考えておりますが、
中々いい方法が思いつきません。

使用する数値は固定値で、20個程度の数値が2パターンです。

「処理を行うクラスのコンストラクターで分岐させて
数値をフィールドに設定する」
等の処理を考えたのですがいまいちしっくりきません。

C#も分からなくはないので、
VB.net又はC#で教えていただけると助かります。

なにか良い方法があればご教授ください。
よろしくおねがいいたします。

追記(5/29/17:45頃)
現状での設計は以下の通りです。
分かり辛くてすいません。

Class 呼び出し元
    Sub 制御メソッド()
        Dim 処理インスタンスA = New 処理クラス("パターンA")
        処理インスタンスA.処理メソッド１()
        処理インスタンスA.処理メソッド２()

        Dim 処理インスタンスB = New 処理クラス("パターンB")
        処理インスタンスB.処理メソッド１()
        処理インスタンスB.処理メソッド２()

    End Sub
End Class


Class 処理クラス

    ReadOnly Property 数値１ As Integer
    ReadOnly Property 数値２０ As Integer

    Sub New(パターン As String)
        If パターン = "パターンA" Then
            数値１ = 数値を記載するクラス.数値A.数値１
            数値２０ = 数値を記載するクラス.数値A.数値２０
        Else
            数値１ = 数値を記載するクラス.数値B.数値１
            数値２０ = 数値を記載するクラス.数値B.数値２０
        End If
    End Sub

    Sub 処理メソッド１()
        '処理１
    End Sub

    Sub 処理メソッド２()
        '処理２
    End Sub
End Class

Class 数値を記載するクラス
    Enum 数値A
        数値１ = 1
        数値２０ = 2
    End Enum

    Enum 数値B
        数値１ = 10
        数値２０ = 11
    End Enum

End Class

Comments

[退会済みユーザー]

質問者さんがやりたいことを実現するために質問者さんが考えたコードをアップしてください。いまいちしっくりこないものでも良いので。

[Phonesuca]

大まかな処理の流れを質問の下部に追加いたしました。
よろしくおねがいします。

[len_souko]

処理クラスのコンストラクタのelseは書き間違いですよね？

[Phonesuca]

変数名の間違いでしょうか？
気付きませんでした。
Else内で「数値１」を二度セットする処理になっていたので片方を「数値２０」に修正いたしました。
ありがとうございます。

[YAmaGNZ]

数値１や数値２０は処理クラスでしか使用しないのですか？呼び出し元など他のところでも利用するのですか？

[Q71]

> 複数メソッドを数値だけ変えて実行する工程があります。

> 複数のメソッドで同じ数値を使いまわすので、

数値を変えて繰り返し実行？同じ数値で異なるメソッドを実行？それらが入り混じってる？

[Phonesuca]

呼び出し元では使用しません。
処理クラス内でしか使用いたしません。

Answer 1

enum を使う必要があるかないかによってもコードが変わってきます。

public class 呼び出し元
{
	public void 制御メソッド()
	{
		var 処理インスタンスA = new 子クラスA();
		処理インスタンスA.処理メソッド1();
		処理インスタンスA.処理メソッド2();

		var 処理インスタンスB = new 子クラスB();
		処理インスタンスB.処理メソッド1();
		処理インスタンスB.処理メソッド2();
	}
}

public abstract class 親クラス
{
	protected abstract int 数値1 { get; set; }
	protected abstract int 数値2 { get; set; }
	protected abstract int 数値3 { get; set; }

	public void 処理メソッド1()
	{
	}

	public void 処理メソッド2()
	{
	}
}

public class 子クラスA : 親クラス
{
	protected override int 数値1 { get; set; } = 1;
	protected override int 数値2 { get; set; } = 2;
	protected override int 数値3 { get; set; } = 3;
}

public class 子クラスB : 親クラス
{
	protected override int 数値1 { get; set; } = 10;
	protected override int 数値2 { get; set; } = 11;
	protected override int 数値3 { get; set; } = 12;
}

追記

制御メソッドはループを使いたいですね。

public class 呼び出し元
{
	public void 制御メソッド()
	{
		var instances = new 親クラス[]
		{
			new 子クラスA(),
			new 子クラスB()
		};
		foreach (var instance in instances)
		{
			instance.処理メソッド1();
			instance.処理メソッド2();
		}
	}
}

Comments

[Zuishin]

場合によっては配列でいいんじゃないかと思います。

[Phonesuca]

ありがとうございます！
ちょっとC#どころかvb.netも怪しいペーペーなので理解に時間が掛かっていました。

継承ってこういう風に使うんですね…
ForEach文で回せるのはとても便利ですね！
ちょっと今まで書いてきたコードを思い出して恥ずかしくなってまいりました。
早速明日から活用させていただきたいと思います。

Answer 2

そもそも、処理クラスの数値を

    ReadOnly Property 数値 As 数値を記載するクラス

に変更して、コンストラクタを

    Sub New(パターン As String)
        If パターン = "パターンA" Then
            数値 = New 数値を記載するクラス(数値を記載するクラス.数値タイプ.数値A)
        Else
            数値 = New 数値を記載するクラス(数値を記載するクラス.数値タイプ.数値B)
        End If
    End Sub

とし、数値を記載するクラスの方を

Class 数値を記載するクラス
    Enum 数値タイプ
        数値A
        数値B
    End Enum

    ReadOnly Property 数値１ As Integer
    ReadOnly Property 数値２０ As Integer

    Sub New(タイプ As 数値タイプ)
        Select Case タイプ
            Case 数値タイプ.数値A
                数値１ = 1
                数値２０ = 2
            Case 数値タイプ.数値B
                数値１ = 10
                数値２０ = 11
            Case Else
                数値１ = 0
                数値２０ = 0
        End Select
    End Sub
End Class

こんな感じにしてみたらどうですか？
値そのものを考えるのは数値を記載するクラスに任せてパターン自体もenumで持っておき、
コンストラクタへパターンを渡すようにします
処理クラスの方では自身の引数からどのパターンなのかを判断するだけにしておきます

質問者さんの提示したソースだとenumの使い方が間違ってますよねってところで意味不明な感じになってるんじゃないかと思います

追記
そもそも、enumの値を数値として使用するってのが本来ではないと思うんですよ
というわけで解凍に書いたenumに値を設定している個所を修正しています
コメントで書いたDictionaryを使用する場合は以下のような感じ
なお、コメントを書いた後にVBだとどうおやるんだっけ？と調べてるのでコメントはC#な表現になっています、VB.NETは10年ぶりくらいなので
でもって本回答のソースはすべてVS2019で実際にVBプロジェクトを作成して実行確認しています

Class 処理クラス
    ReadOnly Property 数値 As 数値を記載するクラス

    Enum 数値タイプ
        数値A
        数値B
    End Enum

    Private Shared ReadOnly Property 数値パターン As New Dictionary(Of 数値タイプ, 数値を記載するクラス) From
        {{数値タイプ.数値A, New 数値を記載するクラス(1, 2)},
        {数値タイプ.数値B, New 数値を記載するクラス(10, 11)}}

    Sub New(パターン As String)
        Me.パターン = パターン
        If パターン = "パターンA" Then
            数値 = 数値パターン(数値タイプ.数値A)
        Else
            数値 = 数値パターン(数値タイプ.数値B)
        End If
    End Sub

End Class

Class 数値を記載するクラス
    ReadOnly Property 数値１ As Integer
    ReadOnly Property 数値２０ As Integer

    Sub New(数値１ As Integer, 数値２０ As Integer)
        Me.数値１ = 数値１
        Me.数値２０ = 数値２０
    End Sub
End Class

Comments

[Phonesuca]

ありがとうございます。
確かに処理クラス内で数値のセット処理を長々と書いていくのは間違ってる気がしますね。

＞質問者さんの提示したソースだとenumの使い方が間違ってますよねってところで
これまで数字の定数をグループ単位で扱いたい時に、Enumを使用してましたね…
そうか…邪道な使い方だったのか…
本とかウェブサイトとか見ても、列挙体を私が書いたコードみたいな扱い方してないですね…
初歩に立ち返ってみます。

[len_souko]

何が問題かっていうと、数値１と数値２０に同じ値を使うパターンが出てきたらどうします？ってところです
enumはパターンを区別するものであって、1パターンで設定する各値を定義するものではありません
ちなみに上記ソースコードは質問で提示されたソースをベースとしていますのでこの形になっていますが、例えばenumの定義はそのままだけど、数値を記載するクラスをnewする際に引数の数値タイプで値を初期化するつくりではなくコンストラクタの引数に数値１と数値２０を渡すようにしてDictionary<数値タイプ, 数値を記載するクラス>を処理クラスのフィールドに持つ方が良いと思います

[Phonesuca]

専用クラスに数値のセットを持たせて、処理するクラスのフィールドにセットするのは良いですね
ですが今回は持たせたい数値の数が多く、
コンストラクタの引数で全ての数値を渡すのは管理ができなくなる気がするので、
今回のプログラムでは上のコードの方を参考にさせていただきたいと思います。
ありがとうございました。

＞何が問題かっていうと、数値１と数値２０に同じ値を使うパターンが出てきたらどうします？ってところです
おお…
やばい…同じ値があるときでも常習的に列挙体使ってた…
今後はそのような記述はせずに記載してまいります。

Answer 3

質問がよく理解できなかったので多分ですが、
呼び出すメソッドは共通にしてメソッドの引数で数値を記載するクラスを利用するのが
まっとうな設計だと思います。
こんなクラスを作っておいて、ParametersクラスのインスタンスはParameters.NumbersAなどで呼びます。

/// <summary>
/// 数値を記載するクラス
/// </summary>
class Parameters
{
    /// <summary>
    /// 数値１
    /// </summary>
    public int Number1 { get; set; }

    /// <summary>
    /// 数値２０
    /// </summary>
    public int Number20 { get; set; }

    /// <summary>
    /// 数値A
    /// </summary>
    public static Parameters NumbersA { get; } =
        new Parameters
        {
            Number1 = 1,
            Number20 = 20
        };

    /// <summary>
    /// 数値B
    /// </summary>
    public static Parameters NumbersB { get; } =
        new Parameters
        {
            Number1 = 10,
            Number20 = 11
        };
}

余談ですが　日本語のクラスやメソッドはとても嫌われますので、避けることをお勧めします。

Comments

[hihijiji]

実は余談が一番言いたいことだったり^^;

[hihijiji]

ちなみに私は学生時代に英語は赤点しか取ったことはありません。
社会人になってからも英語を習ったことがありません。
そんな私でも今では立派なエセ英語使いです。

[Phonesuca]

コードありがとうございます！
こういう設計があるなんて知らなかったですし、
C#のプロパティでこういう書き方が出来ること自体知らなかったです。
二重に勉強させていただきました。

普段のコーディングでは英語を使用しているのですが、
今回は実際のプログラムではないので、説明のためにも日本語でいいかなと無精してしまいました。
変数名は英語で書いて、コメントで付記するのが真っ当ですよね。
恐縮です。

Answer 4

単にメソッドの引数を変えて呼び出せばいいだけの話なんでは。

＃なにをやりたいのかいまいちピンとこない

Comments

[Phonesuca]

・同じ数値を同クラス内の異なるメソッドで使いまわす
・一つのメソッドで最大１０個程度数値を使用するので、全て引数にするのは避けたい
以上の二つの点からクラス内に使用する数値のセットを保持したいなと考えております。

[y_waiwai]

ならクラス内変数にそれらの数値を保存しといて、メソッド多種からそれらを参照、ということになろうかと。
クラス内変数は別のアクセス用メソッドで値を設定、あるいは動作切り替え？するということで。

Answer 5

ぜんぜんピンとこないんですが、同じ数値を使い回す、というのは、最初のインプット１に対して「処理１、処理２...」というデータ処理のフローがあるだけだと思うので、単純に「処理フロー」と「データ」が適切に整理されてれば何も迷うことく、データ件数分処理を回すだけだと思うんですけど。

なんか簡単な話をわざわざ難しく考えてるような…？

Comments

[Phonesuca]

他の処理では数値は１パターンしかないので、
数値はクラス内に列挙体で持っているんですよね。

他のクラスではクラス内の列挙体を参照して処理を行うのに、
この処理ではフィールドの変数を参照して処理を行う。
だからしっくりこなかったんだと思います。