このExpressionエバリュエータークラスのプライベートメソッドの単体テストを記述してもよろしいですか？

Question

私はコード

public interface IInterpreter 
{ 
    decimal Evaluate(string expression); 
} 

public class Interpreter : IInterpreter 
{ 
    public decimal Evaluate(string expression) 
    { 
     if (String.IsNullOrWhiteSpace(expression)) 
      throw new ArgumentException("Parameter " + nameof(expression) + " cannot be empty"); 

     var rpnExpression = ConvertToReversePolishNotation(expression); 
     return EvaluateReversePolishExpression(rpnExpression); 
    } 
... 
} 

このクラスは、 "+ 5 * 6" のような式を評価したりします "（3-5）*（2 + 2）+5"
を以下のカントー今私が欲しいいますユニットテストを書く。ここで公開されている唯一の関数はEvaluateであり、すべての勧告によれば、このメソッドのみをテストする必要があります。
問題は、私は両方のConvertToReversePolishNotation(expression)EvaluateReversePolishExpression(rpnExpression)関数が単体テストでカバーされなければならないという強い感情を持っています... Evaluateメソッドのためにいくつかの単体テストが失敗する場合は、バグがどこにあるのかを指摘しません（ConvertToRPNExpressionメソッドEvaluateReversePolishExpressionファンクション）。

質問があります - この場合、プライベート関数の単体テストを書くのは大丈夫ですか？

Answer 1

号

リファクタリング別々のクラスにConvertToReversePolishNotation方法。その関心事は変換です。

インタープリタの懸念は評価することです。あなたが変換の出力を模倣している場合は、公開のEvaluateメソッドを使ってテストするのは簡単です。

Answer 2

はい、問題ありません。。

Evaluateのユニットテストでは、ConvertToReversePolishNotationとEvaluateReversePolishExpressionが実際に実装されています。

Aphelionは自分自身の責任のためにクラスを分けることができますが、ConvertToReversePolishNotationの場合にのみ、EvaluateReversePolishExpressionのプライベートメソッドを作成する必要はありません。これはそれが既に行われているためです。 evaluateを呼び出す前に、呼び出し元はConvertToReversePolishNotationのクラスを呼び出し、それをパラメータとして渡す必要があります。

privateメソッドのユニットテストではないと心配している場合は、internalメソッドを使用できます。

トピック：この機能を拡張および拡大する方法も念頭に置く必要があります。通訳者がdecimal以外のデータ型を返すことを希望する場合はどうなりますか？ジェネリックも最大化したいとは思わないでしょうか？

public decimal Evaluate(string expression) 
{ 
    if (String.IsNullOrWhiteSpace(expression)) 
     throw new ArgumentException("Parameter " + nameof(expression) + " cannot be empty"); 

    return EvaluateReversePolishExpression(expression); 
} 

Answer 3

私は3つのクラスを作成します.1つは検証の責任者、1つは変換のためのもの、もう1つは評価のためのクラスです。

実際の実装から切り離すコンストラクタ依存性注入を使用するよりも。 これを手動でインスタンス化するか、コンテナを委譲してコンテナを初期化してください。インタプリタに変更を加えることなく動作を変更することができます。

このルートを使用すると、依存関係の呼び出し順序を簡単にテストできるため、インタープリターの動作をテストできます。 妥当性検査、変換、評価と同様です。

これとは別に、単一のクラスの各実装をテストできます。

クラスはまた、検証する責任があり、外部クラスを呼び出して操作を実行するので、イベントの流れを記述する責任があるため、単一責任の原則を制動しています。

また、変換と評価の実装クラスを密接に結合すると、Open Closeの原則が破られます。