原則検証中DRY？

Question

ツールボックスメニューの標準のDataGridViewを使用します。

イベントcellEditEndによって、DataGridView内の各セルの検証を行います。それは次のようになります。だから、

private void dataGrid_CellEditEnding(object sender,DataGridCellEditEndingEventArgs e) 
{ 
    // Below I demonstrate pseudo code for brevity 

    if(cell.name.value is not Number){ 
     print "Wrong cell value"; 
    } 
} 

、別のフォーム（リサイズ）で、私は機能を持っている、というのDataGridView内のExcelファイルと表示され、それをからの輸入データ。

各セルに挿入する前にExcelデータを検証する必要があります。そのためには、イベントCellValidationを使用することができます。しかし、私はdataGrid_CellEditEndingメソッドで使用したのと同じコードを繰り返すことは望ましくありません。

このコードの再発行を避けるにはどうすればよいですか？

Answer 1

静的ValidationHelperクラスを作成し、その中にすべての検証メソッドを入れ、必要なときにそれらを呼び出す方法について説明します。

あなたが呼び出すことができます

public static class ValidationHelper 
{ 
    public static void ValidateIsNumeric(string value) 
    { 
    if(value is not Number){ 
     print "Wrong cell value"; 
    } 
    } 
} 

：

private void dataGrid_CellEditEnding(object sender,DataGridCellEditEndingEventArgs e) 
{ 
    ValidationHelper.ValidateIsNumeric(cell.name.value); 
} 

Answer 2

をあなたはresponsabilityデザインパターンの連鎖に見てみることができ、あなたのコードが設定可能で、よりオブジェクト指向、このようになりますされます：

public interface IHandler 
    { 
     void Handle(string value); 
    } 

    public abstract class CellRequestTemplate : IHandler 
    { 
     protected readonly IHandler _next; 
     protected CellRequestTemplate(IHandler next) 
     { 
      _next = next ?? throw new ArgumentNullException(nameof(next)); 
     } 

     public abstract void Handle(string value); 
    } 

    public sealed class FirtsRuleForCell : CellRequestTemplate 
    { 
     public FirtsRuleForCell(IHandler next) : base(value, next) { } 
     public override void Handle(string value) 
     { 
      if(value is number) 
      { 
       _next.Handle(value); 
      } 
      else 
      { 
       //print "Wrong cell value"; 
      } 
     } 
    } 

    public sealed class SecondRuleForCell : CellRequestTemplate 
    { 
     public SecondRuleForCell(IHandler next) : base(value, next) { } 
     public override void Handle(string value) 
     { 
      //if some validation 
      //do something 
      //else 
      // 
      _next.Handle(value); 

     } 
    } 

    public sealed class EndOfChain : IHandler 
    { 
     public void Handle(string value) 
     { 
      throw new InvalidOperationException("End of Chaing, cant handle"); 
     } 
    } 

    public interface IHandleCellFactory 
    { 
     IHandler CreateHandler(); 
    } 

    public sealed class Form1GridHandler : IHandleCellFactory 
    { 
     public IHandler CreateHandler() 
     { 
      return new FirtsRuleForCell(new SecondRuleForCell(new EndOfChain())); 
     } 
    } 

    public sealed class Form2GridHandler : IHandleCellFactory 
    { 
     public IHandler CreateHandler() 
     { 
      return new SecondRuleForCell(new EndOfChain()); 
     } 
    } 


    public abstract class ClientCode 
    { 
     private void dataGrid_CellEditEnding(object sender, DataGridCellEditEndingEventArgs e) 
     { 
      // Below I demonstrate pseudo code for brevity 
      var valueToHandle = string.Empty; //cell.name.value; 
      var handler = new Form1GridHandler().CreateHandler(); 
      handler.Handle(valueToHandle); 
      //continue with Execution code 
     } 
    } 

デザインパターンの詳細については、look at this link as referenceをご覧ください。 これが役立つことを願っています。