이 사전 TryGetValue 코드를 더 읽기 쉽게하려면 어떻게해야합니까?

Question

ID가 아직 알려지지 않았는지 테스트하고 싶거나 알려진 값이 있으면 연결된 값이 변경되었는지 테스트하고 싶습니다. 나는 현재 이와 비슷한 코드를 사용하고 있지만 패턴에 익숙하지 않은 사람들은 이해하기 어렵다. LOC에서 짧게 유지하면서 더 읽기 쉽게 만드는 방법을 생각해 볼 수 있습니까?

string id; 
string actual; 
string stored; 

if (!someDictionary.TryGetValue (id, out stored) || stored != actual) { 
    // id not known yet or associated value changed. 
} 

Answer 1

그것은 나에게 잘 보인다는 ... 다른 두 조건 문처럼 쉽게 읽습니다. 나는 아마도 변경하려는 유일한 것은 조기 종료에 대한 부정을 뒤집을 수 있습니다 : 나는 특히 귀찮은 관용구로 생각하지 못했던, 전혀에서 혼란이 표시되지 않는

if (someDictionary.TryGetValue(id, out stored) && stored == actual) { 
    return; 
} 
// store new value 

및 겸손하게 그걸로 혼란스럽게 C#을 devs 제안 익숙해 지. 그것은 공통적으로, 숙신이며, 많은 LOC를 문제만큼 가치있게 제공합니다. 그것을 10 줄의 코드로 바꾸면 은이 너무 중요합니다.

자주 사용하는 경우 ContainsEqualValue과 같은 이름의 확장 메서드를 사용하는 것이 적절합니다. 그러나 확장 메서드에서 사용하는 것과 똑같은 코드를 사용하고 싶습니다.

Answer 2

그래서 나는 아마도 그것을 해독하고 의미있는 이름을 부여 할 것입니다. 이 읽기 더 많은,하지만 당신은 의견 말을 많이 필요하지 않습니다 :

bool isKnown = someDictionary.TryGetValue (id, out stored); 
// can only change when it is known 
bool valueChanged = isKnown && stored != actual; 

// quite self-explanatory, isn't it? 
if (!isKnown || valueChanged) 
{ 

} 

Answer 3

이중성을.

if (!(someDictionary.TryGetValue (id, out stored) && stored == actual)) ... 

더 읽기 쉽지만 잘 모르겠지만 ... 알아두면 좋을 것 같습니다.

Answer 4

각 부분을 || 자신의 메서드 나 속성에, 당신은 내가 새로운 방법을 선호하는 것이

if (IdIsNew() || IdChanged()) 

Answer 5

처럼 쓸 수있는 것보다 : 기존의 방법에 다음

public bool ShouldSetValue(Dictionary someDictionary, object id,object actualValue) 
{ 
    string stored; 

    if (someDictionary.TryGetValue (id, out stored)) 
    { 
     if (stored != actualValue) 
      return true; 
    } 
    else 
    { 
     return true; 
    } 
} 

을 그냥 좋겠 :

if (ShouldSetValue(someDictionary,id,actual)) 
{ 
    someDictionary[id]=actual; 
} 

Answer 6

당신은 좋은 이름으로 확장 방법을 쓸 수 있습니다 :

public static class Utility 
{ 
    public static bool ValueChangedOrUnknown(this Dictionary<string, string> dictionary, string id, string actual) 
    { 
     string stored = null; 
     return (!dictionary.TryGetValue(id, out actual) || stored != actual); 
    } 
} 

그래서 나중에는 매끄러운 것

string id; 
string actual; 

if (someDictionary.ValueChangedOrUnknown(id, actual) { 
    // id not known yet or associated value changed. 
} 

Answer 7

연장 방법을 사용할 수 있습니다 :

public static class DictionaryExtensions 
{ 
    public static bool ShouldAddValue<TKey, TValue>(this Dictionary<TKey, TValue> someDictionary, TKey id, TValue actual) 
    { 
     TValue stored; 
     return (!someDictionary.TryGetValue(id, out stored) || !stored.Equals(actual)); 
    } 
} 

사용법 :

someDictionary.ShouldAddValue("foo", "bar") 

Answer 8

당신이 반복적으로이 작업을 수행해야한다는 것을 의미하는 경우, 그리고 길고 추한, 논리를 다른 클래스로 추상화하고 확장 메소드를 사용하십시오.

public static class DictionaryExtensions 
{ 
    public static DictionaryChecker<TKey,TValue> contains<TKey,TValue>(this IDictionary<TKey,TValue> dictionary, TValue value) 
    { 
     return new DictionaryChecker<TKey,TValue>(value, dictionary); 
    } 
} 

public class DictionaryChecker<TKey,TValue> 
{ 
    TValue value; 
    IDictionary<TKey,TValue> dictionary; 

    internal DictionaryChecker(TValue value, IDictionary<TKey, TValue> dictionary) 
    { 
     this.value = value; 
     this.dictionary = dictionary; 
    } 

    public bool For(TKey key) 
    { 
     TValue result; 
     return dictionary.TryGetValue(key, out result) && result.Equals(value); 
    } 
} 

지금 당신의 코드를 대체 :

if(!someDictionary.contains(actual).For(id)){ 
    // id not known yet or associated value changed. 
} 

Answer 9

public T GetValue(int id, object actual) 
{ 
    object stored; 
if (someDictionary.TryGetValue (id, out stored) || stored == actual) 
    return stored; 
    return new object(); 
} 

Answer 10

"try"패턴이 필요하다는 것을 알고 있지만 "out"매개 변수가 필요한 구현은 싫습니다. 그것은 훨씬 더 유용 TryGetValue 유사한 기능을 보일 수있을 것입니다 : 키가 사전에없는 경우

• TryGetDictValue (사전, 키) null을 반환
• TryGetDictValue 키가 아닌 경우 (사전, 키, DEFAULTVALUE가) DEFAULTVALUE을 반환 사전
• 키가 사전에없는 모든 경우의 결과

을 반환하는 경우 TryGetDictValue이 (사전, 키, valueReturningDelegate)가 공급 대리자를 호출, 결과의 반환 형식이 사전 데이터의 그 것이다.

타임 머신에 몰래 들어가서 그런 것들을 사전의 방법으로 만드는 것은 너무 어렵습니다. 다른 한편으로, 정적 함수로서 사전을 첫 번째 매개 변수로 사용하여이를 구현할 수 있습니다.