मैं एक मूल्य वापस करना चाहता हूं और अपवाद बढ़ा सकता हूं, इसका मतलब है कि मैं कुछ गलत कर रहा हूं?

Question

मेरे पास कई फ़ंक्शन हैं जो फ़ाइलों से डेटा पार्स करते हैं, आमतौर पर परिणामों की एक सूची लौटाते हैं।

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

यह मुझे सफल डेटा वापस करने के लिए छोड़ देता है, लेकिन त्रुटि के कारण अपवाद भी उठाता है, जो स्पष्ट रूप से मैं नहीं कर सकता।

इस कोड पर विचार करें:

try: 
    parseResult = parse(myFile) 
except MyErrorClass, e: 
    HandleErrorsSomehow(str(e)) 

def parse(file): #file is a list of lines from an actual file 
    err = False 
    result = [] 

    for lines in file: 
     processedLine = Process(line) 
     if not processedLine: 
      err = True 
     else 
      result.append(processedLine) 
    return result 
    if err: 
     raise MyErrorClass("Something went wrong") 

जाहिर है पिछले तीन लाइनों कोई अर्थ नहीं है, लेकिन मैं यह करने के लिए एक अच्छा तरीका को समझ नहीं सकता। मुझे लगता है मैं return (err, result) कर सकता है, और यह

parseErr, parseResult = parse(file) 
if parseErr: 
    HandleErrorsSomehow() 

लेकिन लौटने त्रुटि कोड पर्याप्त अन-pythonic लगता है कहते हैं, अकेले त्रुटि कोड और वास्तविक परिणाम मूल्यों की tuples लौटने।

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

Answer 1

कोई भी नहीं कहता है कि केवल "त्रुटि" का इलाज करने का वैध तरीका अपवाद फेंकना है।

आपके डिज़ाइन में कॉलर जानकारी के दो टुकड़े चाहता है: (1) वैध डेटा, (2) क्या कोई त्रुटि हुई है (और शायद कुछ गलत हो गया है, तो यह एक उपयोगी त्रुटि संदेश प्रारूपित कर सकता है)। मूल्यों की एक जोड़ी लौटने के लिए यह एक पूरी तरह से वैध और उपरोक्त जमीन का मामला है।

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

Answer 2

Emit a warning इसके बजाए, और कोड को यह तय करने दें कि इसे कैसे संभालें।

Answer 3

मैं नेट दुनिया से आते हैं, ऐसा नहीं यकीन है कि यह कैसे अजगर में तब्दील हो ...

आपके जैसे मामलों मैं एक MyProcessingResults वापसी चाहते हैं (जहां एक कॉल में कई आइटम पर कार्रवाई करना चाहते हैं) में उद्देश्य यह है कि उदाहरण के लिए दो संग्रह आयोजित:

• MyProcessingResults.ProcessedLines - सभी मान्य डेटा आप पार्स रखती है।
• MyProcessingResults.Errors - सभी त्रुटियों को धारण करता है (इस धारणा पर कि आपके पास एक से अधिक हैं और आप उन सभी के बारे में स्पष्ट रूप से जानना चाहते हैं)।

Answer 4

पैरामीटर के रूप में त्रुटि हैंडलर को पास करके, एक और संभावित डिज़ाइन नियंत्रण को उलटा करना है।

(इसके अलावा, आप की तरह अजगर को बताते हैं कि एक सूची में डेटा जमा करने के लिए किया है महसूस नहीं। यह पहले से ही जानता है। यह यहाँ एक सूची समझ काम करने के लिए मुश्किल नहीं है।)

def sample_handler(): 
    print "OMG, I wasn't expecting that; oh well." 

parseResult = parse(myFile, sample_handler) 

def parse(file, handler): #file is a list of lines from an actual file 
    result = [Process(line) for line in data] 
    if not all(result): handler() # i.e. if there are any false-ish values 
    result = filter(None, result) # remove false-ish values if any 
    return result 

Answer 5

पर निर्भर करता है कॉलर डिजाइन। कॉलबैक का उपयोग करना उचित हो सकता है:

def got_line(line): 
    print 'Got valid line', line 

def got_error(error): 
    print 'got error', error 

def parse(file, line, error): 
    for lines in file: 
     processedLine = Process(line) 
     if not processedLine: 
      error(MyErrorClass("Something went wrong")) 
     else 
      line(processedLine) 


parse(some_file, got_line, got_error) 

Answer 6

मैं एक वैकल्पिक समाधान का प्रस्ताव देना चाहता हूं; एक वर्ग का उपयोग कर।

class MyParser(object): 
    def __init__(self): 
     self.warnings = [] 

    def parse(self, file): 
     ... 

अब पार्स समारोह warnings सूची में चेतावनी सेट कर सकते हैं, और यदि उपयोगकर्ता इस सूची की जांच कर सकते यदि वे ऐसा इच्छा।

जैसे ही मेरे कार्य "प्रक्रिया को संसाधित करें और मेरा मूल्य वापस करें" की तुलना में अधिक उन्नत होने लगते हैं, मुझे इसके बजाय कक्षा का उपयोग करने पर विचार करना पसंद है। यह एक ऑब्जेक्ट में संबंधित कोड की बड़ी क्लस्टरिंग के लिए बनाता है और यह अक्सर जानकारी के tuples लौटने वाले कार्यों की तुलना में क्लीनर कोड और सरल उपयोग के लिए बनाता है।