C＃IF/ELSEステートメントでMethodの戻り値を使用

Question

今は簡単なプログラムを作成していますが、これは何度も考えてきた問題です。多くの場合、実行する前に戻り値をチェックするためにメソッドを2回実行しますが、これを防ぐ方法があるかどうかを知りたいと思います。説明するのは非常に難しいので、私のプログラムの実際の例がここにあります。

public class SFDBRepository 
{ 
    public static Domain.SF.SFObject GetSFOrder(string WorkOrd) 
    { 
     //As you can see here i'm checking on the output of this method, before trying to return it. 
     if (Domain.SF.SF.GetOrder(WorkOrd) != null) 
     { 
      //If the value is not null (My method returns null if no result), return the object 
      return Domain.SF.SF.GetOrder(WorkOrd); 
     } 
     //Same thing happens here. My method runs twice every time almost. 
     else if(Domain.Building_DeliveryPerformance.Building_DeliveryPerformance.GetObject(WorkOrd) != null) 
     { 
      return Domain.Building_DeliveryPerformance.Building_DeliveryPerformance.GetObject(WorkOrd); 
     } 
     else 
     { 
      return null; 
     } 
    } 

} 

Answer 1

public static Domain.ShopFloor.ShopFloorObject GetShopFloorOrder(string WorkOrd) 
{ 
    //As you can see here i'm checking on the output of this method, before trying to return it. 
    Domain.ShopFloor.ShopFloorObject wo = Domain.ShopFloor.Shopfloor.GetOrder(WorkOrd); 
    if (wo != null) 
    { 
     //If the value is not null (My method returns null if no result), return the object 
     return wo; 
    } 
    //Same thing happens here. My method runs twice every time almost. 
    Domain.ShopFloor.ShopFloorObject yowo = Domain.DG9_DeliveryPerformance.DG9_DeliveryPerformance.GetObject(WorkOrd); 
    if(yowo != null) 
    { 
     return yowo; 
    } 

    /* default return */ 
    return null; 

} 

PS

あなたはちょっと "ファクトリパターン"

、あなたが次のコードにこれをダウン簡素化することができ

http://www.dofactory.com/net/factory-method-design-pattern

Answer 2

を参照してくださいをやっていますこれらのメソッドを一度呼び出すだけで、コードをはるかに読みやすくなります：

public class ShopFloorDBRepository 
{ 
    public static Domain.ShopFloor.ShopFloorObject GetShopFloorOrder(string workOrd) 
    { 
     return Domain.ShopFloor.Shopfloor.GetOrder(workOrd) ?? 
       Domain.DG9_DeliveryPerformance.DG9_DeliveryPerformance.GetObject(workOrd); 
    } 
} 

これがなぜ機能するのかを説明する - ??演算子（nu​​ll-coalescing演算子！）は、基本的に "??の左側の戻り値がNULLの場合は、式の値を右側に戻します"と表示します。

この方法で、関数を一度呼び出すだけで済みます。

Answer 3

一時変数を使用して結果を保持できるように見えますが、これをテストして戻すことができます。

public class ShopFloorDBRepository 
{ 
    public static Domain.ShopFloor.ShopFloorObject GetShopFloorOrder(string WorkOrd) 
    { 
    var result = Domain.ShopFloor.GetOrder(WorkOrd); 

    if (result != null) return result; 
    ... 

これは、呼び出されるメソッドが高価であるおよび/またはあなたが二回負担したくない副作用を持っている場合は特に、一般的なパラダイムです。

ここで、 "var"宣言は、 "result"の型を、呼び出されるメソッドによって返される型に設定します。実際のタイプの名前を使用することもできます。

このように2種類のテストを行う場合は、同じタイプ（この場合は同じように見える）でない限り、2つの異なる変数が必要です。

public static ShopFloorObject GetShopFloorOrder(string WorkOrd) 
{ 
    ShopFloorObject result; 

    if ((result = Domain.ShopFloor.GetOrder(WorkOrd)) != null) 
     return result; 
    if ((result = Domain.DG9_DeliveryPerformance.DG9_DeliveryPerformance.GetObject(WorkOrd)) != null) 
     return result; 
    return null; 

をここでは明示的に示されてきた2つの呼び出しを行った後、戻り値の型を宣言している：あなたも参照してくださいよということ

フルタイプを必要としない代替メカニズム、結果をヌルでテストし、最初の非ヌル値を戻します。