중복 코드입니까?

Question

시스템을 업그레이드 중이며 다른 개발자 코드 (ASP.NET은 C#)를 사용하고 있습니다.

나는이 건너 온 :

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 객체가 아닌 다른 것으로 속성을 설정하면 코드가 컴파일되지 않습니다. 내가 뭔가를 누락되거나 건가요 만 사용하여 내가 바로이 달성 될 수있다 생각입니다 :

set 
{ 
    Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] = value; 
} 

Answer 1

원래의 코드를 설정하거나 또는 자산에서 얻을되는 ReferralSearchFilterResults의 서브 클래스를 방지 할 수 있습니다. value.GetType()은 value으로 참조되는 개체의 실제 Type을 반환하기 때문입니다. 해당 Type이 ReferralSearchFilterResults의 하위 클래스이면 typeof(ReferralSearchFilterResults)과 같지 않습니다.

본인의 컨텍스트가 확실하지 않으므로 올바른 동작인지 여부를 알 수 없습니다. 그것이 의도 된 행동이라면 서브 클래스의 할당을 자동으로 무시하기 때문에 약간 더러워집니다. 그러나 나는 더 이상의 맥락 없이는 정말로 판단 할 수 없다.

Answer 2

가 나는 당신이 옳은 것 같아요 - 암시 적 캐스트 할 수없는 그 무언가와 함께 제공되는 경우 세터가 컴파일되지해야 ReferralSearchFilterResults.

Answer 3

get bit의 형식 검사를 이해할 수 있지만, setter에서 컴파일시 코드가 실패하므로 ReferralSearchFilterResults가 아닌 것은 전달할 수 없습니다.

는

return Session[SESSION_REFERRAL_SEARCHFILTERRESULTS] as ReferralSearchFilterResults; 

이가 ReferralSearchFilterResults에 캐스트 할 수있는 경우 값을 반환, 그렇지 않으면 null 사용할 수 있으며, get의 경우

Answer 4

(오래된 습관, 다른 개발자가 있었다 수 있을까).

Answer 5

제이미 당신 말이 맞아요. 이 경우에는 Setter의 유형 검사가 필요하지 않습니다. 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

세션 변수는 객체 유형이므로 세션 변수는 그 안에 무엇이든 저장할 수 있습니다. 그러나이 경우 setter 자체는 프로그래머가 ReferralSearchFilterResults 및 파생 된 객체보다 다른 객체 유형을 할당하는 것을 방지합니다. 그래서 당신이 지적한 바와 같이 그 자체가 불필요합니다. 또한 프로그래머가 ReferralSearchFilterResults에서 파생 된 객체를 할당하도록 허용하지 않습니다.

하지만 null로 설정된 경우 세션 변수가 HTTP 컨텍스트에 계속 존재하기 때문에 변수를 null로 설정하는 대신 Session.Remove를 사용합니다. 그래서

는 :

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