कार्यों में अपवाद बनाम वापसी को बढ़ाएं?

Question 1

पायथन में उपयोगकर्ता-परिभाषित फ़ंक्शन में बेहतर अभ्यास क्या है: raiseएक अपवाद या return None? उदाहरण के लिए, मेरे पास एक फ़ंक्शन है जो एक फ़ोल्डर में सबसे हाल की फ़ाइल पाता है।

def latestpdf(folder):
    # list the files and sort them
    try:
        latest = files[-1]
    except IndexError:
        # Folder is empty.
        return None  # One possibility
        raise FileNotFoundError()  # Alternative
    else:
        return somefunc(latest)  # In my case, somefunc parses the filename

एक अन्य विकल्प अपवाद छोड़ रहा है और इसे कॉलर कोड में संभाल सकता है, लेकिन मुझे लगता है कि यह एक FileNotFoundErrorसे अधिक से निपटने के लिए अधिक स्पष्ट है IndexError। या एक अलग नाम के साथ एक अपवाद को फिर से बढ़ाने के लिए यह बुरा रूप है?

Question 2

यह वास्तव में शब्दार्थ की बात है। क्या foo = latestpdf(d) मतलब है ?

क्या यह पूरी तरह से उचित है कि कोई नवीनतम फ़ाइल नहीं है? फिर यकीन है, बस कोई नहीं लौटा।

क्या आप हमेशा एक नवीनतम फ़ाइल खोजने की उम्मीद कर रहे हैं? एक अपवाद उठाएं। और हाँ, अधिक उपयुक्त अपवाद को फिर से उठाना ठीक है।

यदि यह केवल एक सामान्य कार्य है जिसे किसी भी निर्देशिका में लागू किया जाना है, तो मैं पूर्व करूंगा और कोई नहीं लौटाऊंगा। यदि निर्देशिका है, उदाहरण के लिए, एक विशिष्ट डेटा निर्देशिका होने का मतलब है जिसमें किसी एप्लिकेशन के ज्ञात सेट शामिल हैं, तो मैं एक अपवाद बढ़ाऊंगा।

Question 3

मैं आपके प्रश्न का उत्तर देने से पहले कुछ सुझाव दूंगा क्योंकि यह आपके लिए प्रश्न का उत्तर दे सकता है।

हमेशा अपने कार्यों को वर्णनात्मक नाम दें। latestpdfकिसी के लिए बहुत कम मतलब है, लेकिन अपने कार्य latestpdf()को देखने से नवीनतम पीडीएफ मिलता है। मेरा सुझाव है कि आप इसे नाम दें getLatestPdfFromFolder(folder)।

जैसे ही मैंने ऐसा किया, यह स्पष्ट हो गया कि इसे क्या लौटना चाहिए .. अगर कोई pdf अपवाद नहीं है। लेकिन वहां और इंतजार करें ..

कार्यों को स्पष्ट रूप से परिभाषित रखें। चूंकि यह स्पष्ट नहीं है कि कोई व्यक्ति क्या करना चाहता है और यह (जाहिरा तौर पर) स्पष्ट नहीं है कि यह नवीनतम पीडीएफ प्राप्त करने से संबंधित है जो मैं आपको इसे स्थानांतरित करने का सुझाव दूंगा। यह कोड को और अधिक पठनीय बनाता है।

for folder in folders:
   try:
       latest = getLatestPdfFromFolder(folder)
       results = somefuc(latest)
   except IOError: pass

उम्मीद है की यह मदद करेगा!

Question 4

मैं आमतौर पर अपवादों को आंतरिक रूप से संभालना पसंद करता हूं (अर्थात कोशिश की गई फ़ंक्शन के अंदर को छोड़कर, संभवतः कोई नहीं लौटाता) क्योंकि अजगर गतिशील रूप से टाइप किया गया है। सामान्य तौर पर, मैं इसे एक तरह से या दूसरे तरीके से कॉल करने वाला निर्णय मानता हूं, लेकिन गतिशील रूप से टाइप की गई भाषा में, ऐसे छोटे कारक होते हैं जो कॉल करने वाले को अपवाद नहीं पारित करने के पक्ष में तराजू को टिप देते हैं:

आपके फ़ंक्शन को कॉल करने वाले किसी भी व्यक्ति को उन अपवादों के बारे में सूचित नहीं किया जाता है जिन्हें फेंक दिया जा सकता है। यह जानने के लिए कि आप किस तरह के अपवाद का शिकार हो रहे हैं (और सामान्य से हटकर किए जाने वाले ब्लॉकों को छोड़कर) किस तरह का एक कला रूप हो जाता है।
if val is Noneकी तुलना में थोड़ा आसान है except ComplicatedCustomExceptionThatHadToBeImportedFromSomeNameSpace। सच में, मैं from django.core.exceptions import ObjectDoesNotExistअपने सभी django फ़ाइलों के शीर्ष पर टाइप करने के लिए याद रखने से नफरत करता हूँ बस एक बहुत ही सामान्य उपयोग के मामले को संभालने के लिए। वैधानिक रूप से टाइप की गई दुनिया में, संपादक को इसे आपके लिए करने दें।

ईमानदारी से, हालांकि, यह हमेशा एक निर्णय कॉल है, और जिस स्थिति का आप वर्णन कर रहे हैं, जहां एक फ़ंक्शन को एक त्रुटि प्राप्त होती है जो इसे मदद नहीं कर सकती है, यह एक अपवाद को फिर से बढ़ाने का एक उत्कृष्ट कारण है जो सार्थक है। आपके पास सटीक सही विचार है, लेकिन जब तक आप अपवाद नहीं हैं, तब तक स्टैक ट्रेस में अधिक सार्थक जानकारी प्रदान करने जा रहे हैं

AttributeError: 'NoneType' object has no attribute 'foo'

जो, दस में से नौ बार है, अगर आप किसी को न लौटाने वाले को वापस बुलाते हैं, तो फोन करने वाला आपको परेशान नहीं करेगा।

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

Question 5

अजगर 3.5 टाइपिंग के साथ :

उदाहरण समारोह जब कोई नहीं लौटेगा:

def latestpdf(folder: str) -> Union[str, None]

और जब एक अपवाद होगा:

def latestpdf(folder: str) -> str

विकल्प 2 अधिक पठनीय और पाइथोनिक लगता है

(जैसा कि पहले कहा गया था, अपवाद को टिप्पणी जोड़ने का विकल्प।)

Question 6

सामान्य तौर पर, मैं कहूंगा कि अगर कोई विपत्तिपूर्ण घटना घट गई है, तो उसे अपवाद नहीं कहा जाना चाहिए (यानी आपका फ़ंक्शन कुछ इंटरनेट संसाधन के साथ संबंधित है, जो जुड़ा नहीं हो सकता है), और आपको कोई भी नहीं लौटना चाहिए यदि आपका फ़ंक्शन वास्तव में कुछ वापस करना चाहिए लेकिन कुछ भी वापस करने के लिए उपयुक्त नहीं होगा (यानी "कोई नहीं" यदि आपका फ़ंक्शन उदाहरण के लिए स्ट्रिंग में एक विकल्प से मेल खाने की कोशिश करता है)।