Tôi làm cách nào để làm cho mã từ điển TryGetValue này dễ đọc hơn?

Question

Tôi muốn kiểm tra nếu một id chưa được biết hoặc, nếu nó được biết, nếu giá trị liên quan đã thay đổi. Tôi hiện đang sử dụng mã tương tự như thế này, nhưng khó hiểu đối với những người không quen với mẫu. Bạn có thể nghĩ ra một cách để làm cho nó dễ đọc hơn trong khi vẫn giữ nó ngắn trong 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

Có vẻ tốt với tôi ... đọc dễ dàng như bất kỳ điều kiện khác 2 câu lệnh if. Về điều duy nhất tôi có thể thay đổi là lật các phủ định cho một lối ra sớm:

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

Tôi không thấy bất kỳ sự nhầm lẫn nào, chưa bao giờ nghĩ nó là một thành ngữ đặc biệt phiền hà, và humbly cho rằng những người phát triển C# bị nhầm lẫn bởi nó quen với nó. Đó là phổ biến, succint, và cho càng nhiều LOC cho vấn đề như nó xứng đáng. Biến nó thành 10 dòng mã làm cho nó cách quá quan trọng.

Nếu tôi thường xuyên sử dụng, phương pháp mở rộng có tên là ContainsEqualValue sẽ phù hợp - nhưng tôi sẽ sử dụng cùng một mã chính xác trong phương pháp tiện ích như bạn có.

Answer 2

Vì vậy, tôi có thể sẽ phá vỡ nó và đặt tên có ý nghĩa. Đây là cách đọc nhiều hơn, nhưng bạn không cần nói nhiều trong nhận xét:

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

Tính hai mặt.

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

Không chắc chắn nếu nó dễ đọc hơn ... nhưng bạn nên biết.

Answer 4

quấn từng phần của || vào phương pháp hoặc tài sản riêng của mình, hơn là bạn có thể viết nó như thế này

if (IdIsNew() || IdChanged()) 

Answer 5

Tôi muốn một phương pháp mới:

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; 
    } 
} 

sau đó trong phương pháp hiện tại tôi muốn chỉ:

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

Answer 6

Bạn có thể viết một phương thức tiện ích mở rộng có tên tốt:

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); 
    } 
} 

để sau này bạn có thể sử dụng

string id; 
string actual; 

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

Answer 7

Một phương pháp mở rộng sẽ được slick:

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)); 
    } 
} 

Cách sử dụng:

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

Answer 8

Nếu bạn có nghĩa là bạn phải làm điều này nhiều lần, và nó dài và xấu, trừu tượng logic sang một lớp khác và sử dụng một phương thức mở rộng.

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); 
    } 
} 

Bây giờ thay thế mã của bạn với:

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

Mặc dù tôi nhận ra rằng mẫu "thử" là cần thiết, tôi không thích triển khai yêu cầu thông số "không tham gia". Nó sẽ có vẻ nhiều hơn nữa hữu ích có chức năng tương tự như TryGetValue:

• TryGetDictValue (từ điển, key) trả về null nếu chìa khóa không có trong từ điển
• TryGetDictValue (từ điển, chìa khóa, defaultValue) trả defaultValue nếu chìa khóa không có trong từ điển
• TryGetDictValue (từ điển, chìa khóa, valueReturningDelegate) gọi các đại biểu được cung cấp nếu chìa khóa không có trong từ điển và trả về kết quả của nó

trong mọi trường hợp, các kiểu trả về của kết quả sẽ là dữ liệu của từ điển.

Thật tệ khi không có cách nào lẻn vào một cỗ máy thời gian và làm những thứ như vậy là các phương pháp của Từ điển. Mặt khác, người ta có thể thực hiện chúng như các hàm tĩnh lấy từ điển làm tham số đầu tiên.