（なぜ）拡張メソッドとして優先されている方法を呼び出すのですか？

Question

私はReSharperののリリースされたばかりのバージョン（2016年2月2日）

このフラグのコード行使用して、私の解決策をresharpeningの過程で午前：私がすべきことをintimating ...

ReportRunnerConstsAndUtils.ConvertValueToAppropriateTypeAndAssign(totalPackagesCell, packages); 

を「拡張メソッドとして起動」私は黙認した場合

、それはこれにその行を変更します。

totalPackagesCell.ConvertValueToAppropriateTypeAndAssign(packages); 

これは良いですか？もしそうなら、どうですか？どうして？

ここ「ConstsAndUtils」クラスであると呼ばれている方法であり、コメントの一部が示されているよう

// Adapted from https://stackoverflow.com/questions/26483496/is-it-possible-to-ignore-excel-warnings-when-generating-spreadsheets-using-epplu 
public static void ConvertValueToAppropriateTypeAndAssign(this ExcelRangeBase range, object value) 
{ 
    string strVal = value.ToString(); 
    if (!String.IsNullOrEmpty(strVal)) 
    { 
     decimal decVal; 
     double dVal; 
     int iVal; 

     if (decimal.TryParse(strVal, out decVal)) 
      range.Value = decVal; 
     if (double.TryParse(strVal, out dVal)) 
      range.Value = dVal; 
     else if (Int32.TryParse(strVal, out iVal)) 
      range.Value = iVal; 
     else 
      range.Value = strVal; 
    } 
    else 
     range.Value = null; 
} 

Answer 1

、これは、少なくとも部分的に好みの問題です。個人的には、ここでは拡張メソッドを使用するのが「よりクリーン」で明確になっていると思いますが、一部の人々はこれに同意しないかもしれません。

もちろん、拡張メソッドは静的メソッド（実際のインスタンスメソッドではありません）です。コンパイラがここに構文的な砂糖を与えているだけです。

Answer 2

これは意見に基づいた質問であり、本当に答えることはできませんが、次の点を指摘したいと思います。

この特定のケースでは、メソッドがvoidを返すため、拡張メソッドは使用しません。何かを返さず単に副作用を起こさないメソッドは良い拡張メソッドの候補ではなく、標準の静的メソッド呼び出しとして読みやすくなっています。

私は拡張メソッドを「純粋な」ポーズで保つようにしていますが、私が言ったように、これは私の個人的意見です。

これについて考えるならば、C＃の非常に機能的な側面であるLINQを可能にする拡張メソッドが実装されました。私は、私が実装している拡張メソッドで同じ "感じ"を保つ傾向があります。

Answer 3

あなた（または誰か）が拡張メソッドとして作成したため、拡張メソッドとして呼び出すことをお勧めします。構文this ExcelRangeBase rangeはそのメソッドを拡張メソッドにします。したがって、一貫性のためには、呼び出されるときに拡張メソッドとして使用する必要があります。それ以外の場合は、ReportRunnerConstAndUtils.ConvertValueToAppropriateTypeAndAssign（range）と、range.ConvertValueToAppropriateTypeAndAssign（）と同じことをする行を読み取る行があります。

C＃6ではいくつかの新しい構文が導入されました。ファイルの先頭にusing ReportRunnerConstAndUtils、次にコールサイトでConvertValueToAppropriateTypeAndAssign（範囲）を設定できます。