この冗長コードですか？

Question

システムをアップグレードしていて、別の開発者コード（C＃のASP.NET）を使用しています。

私はこれに出くわした：

private ReferralSearchFilterResults ReferralsMatched 
{ 
    get 
    { 
     if (Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] == null || Session[SESSION_REFERRAL_SEARCHFILTERRESULTS].GetType() != typeof(ReferralSearchFilterResults)) 
      return null; 
     else 
      return (ReferralSearchFilterResults)Session[SESSION_REFERRAL_SEARCHFILTERRESULTS]; 
    } 
    set 
    { 
     if (value == null) 
     { 
      Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] = value; 
     } 
     else if (value.GetType() == typeof(ReferralSearchFilterResults)) 
     { 
      Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] = value; 
     } 
    } 

} 

は不要セッターのタイプをチェックしていますか？確かに、プロパティをReferralSearchFilterResultsオブジェクト以外に設定すると、コードはコンパイルされません。私は何かが欠けまたはAMだけ使用することによって、私は右これを達成することができると思うしています：

set 
{ 
    Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] = value; 
} 

Answer 1

元のコードは、設定またはプロパティにまたはから取得されることからReferralSearchFilterResultsのいずれかのサブクラスを防ぐことができます。 value.GetType()は、valueで参照されるオブジェクトの実際のTypeを返します。そのTypeがReferralSearchFilterResultsのサブクラスである場合、それはtypeof(ReferralSearchFilterResults)と等しくありません。

私はここにあなたの文脈がわからないので、正しい動作であるかどうかはわかりません。それが意図された振る舞いであれば、サブクラスの割り当てを黙って無視するので少し汚れてしまいます。しかし、私は本当にそれ以上の文脈で判断することはできません。

Answer 2

は、私はあなたが正しいと思う - 暗黙的にキャストできないの何かを提供する場合はセッターがコンパイルべきではありませんReferralSearchFilterResults。

Answer 3

私はgetビットの型チェックを理解できますが、あなたが言うように、setterでは、コンパイル時にコードが失敗するため、ReferralSearchFilterResults以外のものを渡すことはできません。

あなたは

return Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] as ReferralSearchFilterResults; 

これは、それがReferralSearchFilterResultsにキャストできるかどうかの値を返し、そうでない場合はnull使用することができ、get部分について

Answer 4

（いくつかの古い習慣、他の開発者が持っていたのだろう）。

Answer 5

Jamieあなたは正しいですか？この場合、セッターのタイプチェックは不要です。valueはである必要があります。ReferralSearchFilterResultsです。

Typeのオブジェクトを比較する代わりに、isとasというキーワードを使用することも考えられます。

private ReferralSearchFilterResults ReferralsMatched 
{ 
    get 
    { 
     if (Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] == null || !(Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] is ReferralSearchFilterResults)) 
      return null; 
     else 
      return Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] as ReferralSearchFilterResults; 
    } 
    set 
    { 
     Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] = value; 
    } 

} 

Answer 6

セッション変数はオブジェクト型であるため、その中には何も格納できません。しかし、この場合、セッター自体は、プログラマがReferralSearchFilterResultsおよび派生オブジェクト以外のオブジェクトタイプを割り当てるのを防止します。 あなたが指摘したように、そのチェック自体は不必要です。さらに、プログラマがReferralSearchFilterResultsから派生したオブジェクトを割り当てることはできません。

しかし変数をnullに設定するのではなくSession.Removeを使用します。これは、nullに設定されている場合にのみセッション変数がHTTPコンテキストに存在するためです。だから、

：

set 
{ 
     if (value == null) 
      Session.Remove(SESSION_REFERRAL_SEARCHFILTERRESULTS); 
     else 
      Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] = value; 
}