कास्टिंग और इंटरफ़ेस विरासत

Question

प्रत्येक आइटम में एक इंटरफ़ेस है, IItem। साथ ही, एक इंटरफेस IDrawableItem के रूप में जाना जाता है जो आइटम से प्राप्त होता है।

नीचे दिया गया कोड, एक ड्रॉइंग आइटम खींचने की कोशिश कर रहा है, लेकिन संग्रह के रूप में इस श्रेणी के स्टोर केवल IItem स्वीकार नहीं कर सकते हैं। आप इस वर्ग में IItem से प्राप्त कुछ भी जोड़ सकते हैं, लेकिन अन्य विधियों का उपयोग करके केवल कास्टिंग करके हासिल किया जा सकता है।

foreach (var item in Items) { 
    item.Draw();    // The casting would go here. 
} 

मैं कैसे कास्ट करने के लिए, as आदि ... लेकिन यह स्वीकार्य है पता है? क्या यह सबसे अच्छा अभ्यास है?

बस सोच रहा है कि ऐसे परिदृश्यों को संभालने के अन्य तरीके हैं या नहीं।

Answer 1

उपयोग Enumerable.OfType केवल Items के उन तत्वों को लागू करने को निकालने के लिए IDrawableItem:

:

foreach(var item in Items.OfType<IDrawableItem>()) { 
    item.Draw(); 
} 

सवाल है कि नानुक टिप्पणी में पूछा उपरोक्त कोड शायद निम्न के कोड बराबर में अनुवाद किया जाएगा पता करने के लिए

:

foreach(var item in Items) { 
    if(item is IDrawableItem) { 
     ((IDrawable)item).Draw(); 
    } 
} 

बेशक

, वास्तव में वहाँ पर्दे के पीछे एक इटरेटर कि कुछ इस तरह दिखता है

public static IEnumerable<T> OfType<T>(this IEnumerable<TSource> source) { 
    if(source == null) { 
     throw new ArgumentNullException("source"); 
    } 
    foreach(TSource item in source) { 
     if(item is T) { 
      yield return (T)item; 
     } 
    } 
} 

तो, यह दिखाता है कि हम शायद Items के माध्यम से केवल एक बार फिर से सक्रिय हो जाते हैं। बेशक, कोई आवश्यकता नहीं है कि OfType उपरोक्त की तरह कार्यान्वित किया गया है लेकिन यह समझदारी की बात है।

Answer 2

दो वैकल्पिक समाधान:

• एक अलग संग्रह में ड्रॉएबल रखें।
• DrawIfPossible()IItem पर विधि जोड़ें। IDrawableItem को इसे Draw() पर कॉल करने के लिए ओवरराइड करना चाहिए, अन्य IItem() कार्यान्वयनकर्ताओं के पास खाली कार्यान्वयन होना चाहिए।

स्पष्ट प्रकार पूछताछ को एक संकेत माना जाता है कि डिजाइन में कुछ गड़बड़ हो सकती है।