फिक्सिंग सी ++ एकाधिक विरासत अस्पष्ट कॉल

Question

मैं तीन वर्गों इस तरह संरचित है

In function ‘int main()’:
error: request for member ‘GetValue’ is ambiguous
error: candidates are: virtual float Keyword::GetValue()
error: virtual float MeasurementKeyword::GetValue()
error: virtual float CharacterKeyword::GetValue()

:

#include <iostream> 
using namespace std; 

class Keyword 
{ 
    public: 
     virtual float GetValue() = 0; 
}; 

class CharacterKeyword : public Keyword 
{ 
    public: 
     virtual float GetValue(){return _value;} 
    private: 
     float _value; 
}; 

class MeasurementKeyword : public Keyword 
{ 
    public: 
     virtual float GetValue(){return _value;} 
    private: 
     float _value; 
}; 

class AddressType : public CharacterKeyword, public MeasurementKeyword 
{ 

    private: 
     float address; 
     float addresExt; 
}; 

int main() 
{ 
    AddressType *a = new AddressType(); 
    a->GetValue(); 
    return 0; 
} 

मैं निम्नलिखित हो रही है 0

मैंने कई विरासत में कुछ पढ़ा है और मुझे पता है कि इसमें बहुत सारे नुकसान हैं - यह उनमें से एक है। मुझे अपनी कक्षा संरचना इस तरह होने की ज़रूरत है, इसलिए मैं सोच रहा था कि क्या कोई तरीका था कि मैं टेम्पलेट का उपयोग करके इसे ठीक कर सकता हूं?

अद्यतन
अपनी टिप्पणी पढ़ने के बाद, मेरा मूल विचार है कि शायद मैं सिर्फ एक AddressType कि एक CharacterKeyword है और एक AddressType कि AddressType templating द्वारा एक MeasurementKeyword है के बीच चित्रित कर सकते थे। और इसे अद्यतन कोड में उपयोग कर रहा है। या मैं केवल उस सदस्य का नामस्थान निर्दिष्ट कर सकता हूं जिसे मैं चाहूंगा। चूंकि टेम्पलेटेड तरीके का अभी तक एक उत्तर के रूप में उल्लेख नहीं किया गया है, क्या यह एक खराब फिक्स है? क्या मुझे बस उस सदस्य का नामस्थान निर्दिष्ट करना चाहिए जिसे मैं चाहता हूं?

template <class T> 
class AddressType : public T 
{ 

    private: 
     float address; 
     float addresExt; 
}; 

int main() 
{ 
    AddressType<MeasurementKeyword> *a = new AddressType<MeasurementKeyword>(); 
    a->GetValue(); 
    return 0; 
} 

Answer 1

यह एक diamond inheritance pattern की वजह से है, त्रुटि को हल करने के लिए आप विशिष्ट नाम स्थान आप की तरह से सदस्य चाहते हैं निर्दिष्ट कर सकते हैं।

paddressType->MeasurementKeyword::GetValue() 

या

paddressType->CharacterKeyword::GetValue() 

---

1. मूल रूप से अपने AddressType वर्ग यह से विरासत और नहीं चुन सकते हैं एक (कॉल अस्पष्ट है) दोनों वर्गों से GetValue सदस्यों के लिए पहुँच नहीं है।
2. scope resolution operator (::) निर्दिष्ट करता है कि आप वास्तव में कौन सा चाहते हैं।
3. आपने यह नहीं कहा है कि आप वास्तव में यह कोड क्या करना चाहते हैं, इसलिए मैं कहूंगा कि आम तौर पर जटिल विरासत पैटर्न पठनीय कोड बनाने के लिए अनुकूल नहीं होते हैं, जो वास्तव में आप चाहते हैं पर पुनर्विचार करते हैं।

Answer 2

आम तौर पर, जब आप deadly diamond of death में भागते हैं तो यह एक संकेत है कि आपको अपने डिज़ाइन पर पुनर्विचार करना चाहिए। हालांकि, अगर आप पूरी तरह से इस स्थिति से नहीं बच सकते हैं, तो सी ++ virtual inheritance के रूप में एक समाधान प्रदान करता है। आभासी विरासत "हीरा अस्पष्टताओं" में से कुछ को हल करती है, लेकिन यह भी घबराहट है। उदाहरण के लिए, आपको व्युत्पन्न वर्ग 'कन्स्ट्रक्टर में माता-पिता के गैर-डिफ़ॉल्ट कन्स्ट्रक्टर को स्पष्ट रूप से कॉल करना होगा।

एक बार फिर, सबसे अच्छा तरीका हीरा से बचने का सबसे अच्छा तरीका है। मैं कई वर्षों से सी ++ में प्रोग्रामिंग कर रहा हूं, और अब तक मुझे अपने कोड में यह समस्या कभी नहीं मिली है।

Answer 3

इस तरह AddressType को परिभाषित:

class AddressType : public CharacterKeyword, public MeasurementKeyword 
{ 
public: 
    using MeasurementKeyword::GetValue; 
private: 
    float address; 
    float addresExt; 
};