重複コードのまとめ方

Question

[開発環境]Visual Studio 2015, C#, WindowsFormアプリケーション, .NetFramework4.5
オブジェクト指向勉強中の初心者です。
コードを書いていて重複する処理がある別々の関数ができてしまいました。
その為、これらの関数をまとめたいと思っております。
内容は、DataGridViewに対し行を追加していく処理で一方は抽出条件があり、一方はありません。
・関数1（抽出条件あり）

private void SetGrid1(List<Data> listData)
{
    // Data=独自クラス
    foreach(Data data in listData)
    {
        if(data.value1 != "条件1")
            continue;
        
         AddRow(data);    // 行追加処理
    }
}

・関数2（抽出条件なし）

private void SetGrid2(List<Data> listData)
{
    // Data=独自クラス
    foreach(Data data in listData)
    {
         AddRow(data);    // 行追加処理
    }
}

いつもならフラグで解決してしまうのですが、
半端にオブジェクト指向の知識を入れた為、どうしたら綺麗に重複をなくせるのかと考えるようになってしまいました。
こんなちょっとしたことに抽象クラスを作った方が良いのでしょうか？
初歩的な質問でお恥ずかしいのですが、ご教示よろしくお願いします

Answer 1

Zuishinが示してくださったように、BindingSourceやFilter といった仕組みを利用するのがもっとも良いと思いますが、「重複コードのまとめ方」という観点で回答させていただくと、値を追加するのかどうかの判断を外から渡してあげるとより汎用的になるかと思います。

private  void SetGrid(List<string> listData, Func<string,bool> judgeFunc = null)
{
    // Data=独自クラス
    foreach (var data in listData)
    {
    	//追加対象のデータか？
        if(judgeFunc == null || judgeFunc(data) == true )
            AddRow(data);    // 行追加処理
    }
           

}

呼び出し例:

//"条件1" じゃないものは除外
SetGrid(lst, (v)=> v != "条件1");

//無条件に追加
SetGrid(lst );

//nullまたは空文字は除外
SetGrid(lst, (v) => string.IsNullOrEmpty( v ) == false );

//一文字目がAで始まるものが対象
SetGrid(lst, (v) => 
{
    if (string.IsNullOrEmpty(v) == true)
        return false;
    return v.StartsWith("A");
});

Comments

[f_5_b_a]

ebiryoさん、ありがとうございます。
これなら条件が増えた場合も関数内が大きくならず、様々なパターンに対応できてすごいです。
まだデリゲートやラムダ式は使い慣れていない為、勉強して自然に使える様頑張ります。
ありがとうございます。

Answer 2

メソッド名に 1 とか 2 とか使い出したら「まとめろ」のサインです。条件が文字列であるなら引数として文字列を取り、null の場合に条件無しと見なすのが常套手段の一つだろうと思います。

しかしこの場合に限っては、もっと推奨される手段があります。それはデータバインディングです。

データバインディングについての情報はすぐみつかりますのでここでは割愛しますが、方針としては全てのデータを BindingSource を通して DataGridView にバインドし、Filter プロパティを使ってデータを絞り込むようになります。

追記

データバインディングを検索する上で注意すべきは、WPF というキーワードが出てきたら飛ばすことです。Google で検索ワードに「-WPF」を含めると良いでしょう。ここでは Windows Forms のデータバインディングの話をしており、WPF はまた別の話です。

Comments

[f_5_b_a]

Zuishinさん、ありがとうございます。
教えていただきましたデータバインディングを勉強しています。
カラムにDataGridViewComboBoxColumnがあるのでその部分のバインドに躓いていますが、
この方法で出来る様頑張ります。
「-WPF」の検索方法知りませんでした。ありがとうございます。

[Zuishin]

http://www.itlab51.com/?p=3608
この辺りが参考になるのではないでしょうか？

[f_5_b_a]

教えていただきましたURLも含め色々調べてみたのですが、DataGridViewComboBoxColumnの一行一行に別々のアイテムをバインドしたい場合にどのように書けばいいのかが分からなくなってしました。
List<Data> listData = SetData();
BindingSource bs = new BindingSource();
bs.DataSource = listData;
とした場合にDataクラスのDataGridViewComboBoxColumn列にバインドさせるプロパティは何クラスにするべきなのか？
public class Data{
public string value1{get;set;}
public ????? combo{get;set;} //DataGridViewComboBoxColumnにバインドするプロパティ
}
もう少し調べてみます。

Answer 3

抽出条件を引数として追加し、抽出条件に関してはデフォルト値を設定しておいて引数を省略すればそのデフォルト値が渡されたものになるので、その時は抽出しないように処理すれば1つにまとめる事ができます。

private void SetGrid(List<Data> listData, string condition = "")
{
    if (condition != "") {
        if(data.value1 != condition) {
           continue;
        }
    }
    AddRow(data);    // 行追加処理
}

// 呼び出し
    SetGrid(list, "条件1");  // 抽出条件付きで呼び出し
    SetGrid(list);          // 無条件に追加

Comments

[f_5_b_a]

PineMatsuさん、ありがとうございます。
サンプルまで書いていただきありがとうございます。
こんな簡単に出来るなんて頭が固かったです。
ありがとうございます。

[ebiryo]

PineMatsuさんが提示して下さった方法ですと、""　を除外することができないと思うのですが。

[h_matsumura]

違うアカウントで入ってしまったww

[PineMatsu]

まあそうですね。
から文字列も抽出条件に含めるのならありえない文字列をデフォにするしかありません。