Tôi muốn trả về một giá trị VÀ nêu ra một ngoại lệ, điều này có nghĩa là tôi đang làm điều gì sai?

Question

Tôi có một số chức năng phân tích cú pháp dữ liệu từ các tệp, thường trả về một danh sách các kết quả.

Nếu tôi gặp phải một đường tinh ranh trong tệp, tôi muốn người lính và xử lý các dòng hợp lệ và trả lại chúng. Nhưng tôi cũng muốn báo cáo lỗi cho chức năng gọi. Lý do tôi muốn báo cáo nó là để các chức năng gọi điện thoại có thể thông báo cho người dùng rằng các tập tin cần xem xét. Tôi không muốn bắt đầu làm những thứ GUI trong chức năng phân tích cú pháp, vì điều đó dường như là một sự vi phạm lớn về việc phân tách các mối quan tâm. Chức năng phân tích cú pháp không có quyền truy cập vào bảng điều khiển Tôi vẫn đang viết thông báo lỗi.

Điều này khiến tôi muốn trả lại dữ liệu thành công, nhưng cũng tăng ngoại lệ do lỗi, điều này rõ ràng là tôi không thể làm được.

xem xét mã này:

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") 

Rõ ràng là ba dòng cuối cùng làm cho không có ý nghĩa, nhưng tôi không thể tìm ra một cách tốt đẹp để làm điều này. Tôi đoán tôi có thể làm return (err, result), và gọi nó là như

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

Nhưng trở về mã lỗi dường như đủ un-pythonic, hãy để một mình trở về tuples của mã lỗi và các giá trị kết quả thực tế.

Thực tế là tôi cảm thấy như tôi muốn làm điều gì đó kỳ lạ trong một ứng dụng không thực sự phức tạp đến mức khủng khiếp, khiến tôi nghĩ rằng có lẽ tôi đang làm điều gì đó sai. Có một giải pháp tốt hơn cho vấn đề này? Hoặc có cách nào để tôi có thể sử dụng finally để trả lại giá trị và tăng ngoại lệ cùng một lúc không?

Answer 1

Không ai nói chỉ chỉ cách hợp lệ để xử lý "lỗi" là ném ngoại lệ. Trong thiết kế của bạn, người gọi muốn hai mẩu thông tin: (1) dữ liệu hợp lệ, (2) có xảy ra lỗi không (và có thể có điều gì đó sai về đâu, vì vậy nó có thể định dạng thông báo lỗi hữu ích). Đó là trường hợp hoàn toàn hợp lệ và trên mặt đất để trả về một cặp giá trị.

Thiết kế thay thế sẽ chuyển bộ sưu tập có thể thay đổi xuống với chức năng làm tham số và để điền vào bất kỳ thông báo lỗi nào muốn phát ra. Điều đó thường sẽ đơn giản hóa hệ thống ống nước trong người gọi, đặc biệt nếu có một số lớp cuộc gọi giữa trình phân tích cú pháp và mã biết cách thực hiện điều gì đó với thông báo lỗi sau đó.

Answer 2

Emit a warning thay vào đó và để mã quyết định cách xử lý.

Answer 3

Tôi đến từ thế giới Net, vì vậy không chắc chắn cách này chuyển thành Python ...

Trong những trường hợp như của bạn (nơi bạn muốn xử lý nhiều mục trong một cuộc gọi duy nhất) Tôi muốn trả lại một MyProcessingResults đối tượng chứa hai bộ sưu tập, ví dụ:

• MyProcessingResults.ProcessedLines - giữ tất cả dữ liệu hợp lệ bạn đã phân tích cú pháp.
• MyProcessingResults.Errors - giữ tất cả các lỗi (với giả thiết rằng bạn có nhiều lỗi và bạn muốn biết rõ ràng về tất cả chúng).

Answer 4

Một thiết kế khác có thể là đảo ngược kiểm soát, bằng cách chuyển trình xử lý lỗi vào làm tham số.

(Ngoài ra, không có cảm giác như bạn phải nói với Python làm thế nào để tích lũy dữ liệu trong danh sách. Nó biết rồi. Nó không khó để thực hiện một công việc danh sách hiểu ở đây.)

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

Tùy thuộc vào thiết kế người gọi. sử dụng gọi lại có thể hợp lý:

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

Tôi muốn đề xuất giải pháp thay thế; sử dụng một lớp học.

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

    def parse(self, file): 
     ... 

Bây giờ chức năng phân tích cú pháp có thể đặt cảnh báo thành danh sách warnings và người dùng có thể kiểm tra danh sách này nếu muốn.

Ngay sau khi chức năng của tôi bắt đầu trở nên tiên tiến hơn là chỉ "xử lý và trả về giá trị của tôi", tôi muốn cân nhắc sử dụng lớp học thay thế. Nó làm cho cụm lớn các mã liên quan thành một đối tượng và nó thường làm cho mã sạch hơn và sử dụng đơn giản hơn các hàm trả về các thông tin.