फ़ंक्शन से स्थानीय ऑब्जेक्ट लौटाना

Question

क्या यह किसी फ़ंक्शन से किसी ऑब्जेक्ट को वापस करने का सही तरीका है?

Car getCar(string model, int year) { 
    Car c(model, year); 
    return c; 
} 

void displayCar(Car &car) { 
    cout << car.getModel() << ", " << car.getYear() << endl; 
} 

displayCar(getCar("Honda", 1999)); 

मुझे "अस्थायी पता लेना" त्रुटि हो रही है। क्या मुझे इस तरह का उपयोग करना चाहिए:

Car &getCar(string model, int year) { 
    Car c(model, year); 
    return c; 
} 

Answer 1

getCar रिटर्न मूल्य है, जो सही है द्वारा एक Car।

आप उस वापसी मूल्य को पार नहीं कर सकते हैं, जो एक अस्थायी वस्तु है, displayCar में, क्योंकि displayCarCar& लेता है। आप गैर-कॉन्स्ट संदर्भ के लिए एक अस्थायी बाध्य नहीं कर सकते हैं। आप displayCar बदलना चाहिए एक स्थिरांक संदर्भ लेने के लिए:

void displayCar(const Car& car) { } 

या, आप एक स्थानीय चर में अस्थायी स्टोर कर सकते हैं:

Car c = getCar("Honda", 1999); 
displayCar(c); 

लेकिन, यह displayCar एक स्थिरांक संदर्भ लेने के लिए के बाद से यह नहीं करता है बेहतर है ऑब्जेक्ट को संशोधित नहीं करें।

स्थानीय Car चर के संदर्भ को वापस न करें।

Answer 2

किसी फ़ंक्शन से स्थानीय चर के संदर्भ को वापस सुरक्षित करना सुरक्षित नहीं है।

तो हाँ यह सही है:

Car getCar(string model, int year) { 
    Car c(model, year); 
    return c; 
} 

Answer 3

आपका समस्या है:

void displayCar(Car &car) { 
    cout << car.getModel() << ", " << car.getYear() << endl; 
} 

आप एक स्थिरांक संदर्भ का उपयोग करना चाहिए:

void displayCar(const Car & car) { 
    cout << car.getModel() << ", " << car.getYear() << endl; 
} 

यह समारोह:

Car getCar(string model, int year) { 
    Car c(model, year); 
    return c; 
} 

ठीक के रूप में यह खड़ा है, लेकिन सभी यह क्या कर रहा है कन्स्ट्रक्टर क्या करता है, इसलिए यह अनावश्यक है।

Car getCar(const string & model, int year) { 

सामान्य तौर पर, जैसे वर्ग प्रकार के लिए: एक मूल्य वापस पासिंग, बल्कि एक संदर्भ से, इस प्रकार समारोह के के लिए ऐसा करना सही है, तथापि, मॉडल पैरामीटर स्थिरांक संदर्भ द्वारा पारित किया जाना चाहिए स्ट्रिंग या कार, पैरामीटर के लिए आपकी डिफ़ॉल्ट पसंद हमेशा एक कॉन्स्ट संदर्भ होना चाहिए।

Answer 4

और भी बेहतर:

Car getCar(string model, int year) { 
     return Car(model, year); 
} 

Answer 5

हाँ, यह निश्चित रूप से सुरक्षित नहीं एक अस्थायी ऑब्जेक्ट के संदर्भ या एक सूचक वापस जाने के लिए है। जब यह समाप्त हो जाता है (यानी, जब फ़ंक्शन getCar बाहर निकलता है), तो आप "लटकते सूचक" के रूप में जाने जाने वाले तकनीकी के साथ छोड़ देंगे।

हालांकि, यदि आप किसी ऑब्जेक्ट पर कॉपी ऑपरेशंस को कम करने के इच्छुक हैं, तो आपको सी ++ 0x की "मूवमेंट्स ले जाएं" देखना चाहिए। यह एक अपेक्षाकृत नई अवधारणा है, लेकिन मुझे यकीन है कि यह जल्द ही मुख्य धारा बन जाएगा। जीसीसी 4.4 ऊपर सी ++ 0x का समर्थन करता है (सक्षम करने के लिए कंपाइलर विकल्प -std=c++0x का उपयोग करें)।