ฉันควรเพิ่มข้อยกเว้นใดในการรวมอาร์กิวเมนต์ที่ไม่ถูกต้อง / ผิดกฎหมายใน Python

ฉันสงสัยเกี่ยวกับวิธีปฏิบัติที่ดีที่สุดในการระบุการรวมอาร์กิวเมนต์ที่ไม่ถูกต้องใน Python ฉันเจอสถานการณ์บางอย่างที่คุณมีฟังก์ชั่นดังนี้

def import_to_orm(name, save=False, recurse=False):
    """
    :param name: Name of some external entity to import.
    :param save: Save the ORM object before returning.
    :param recurse: Attempt to import associated objects as well. Because you
        need the original object to have a key to relate to, save must be
        `True` for recurse to be `True`.
    :raise BadValueError: If `recurse and not save`.
    :return: The ORM object.
    """
    pass

BadValueErrorแกล้งเฉพาะกับเรื่องนี้ก็คือแพคเกจทุกคนมีของตัวเองมักจะแตกต่างกันเล็กน้อย ฉันรู้ว่าใน Java มีอยู่java.lang.IllegalArgumentException- เป็นที่เข้าใจกันหรือไม่ว่าทุกคนจะสร้างของตัวเองBadValueErrorใน Python หรือมีวิธีอื่นที่ต้องการหรือไม่

ฉันแค่จะเพิ่มValueErrorเว้นแต่คุณจะต้องการข้อยกเว้นที่เฉพาะเจาะจงมากขึ้น ..

def import_to_orm(name, save=False, recurse=False):
    if recurse and not save:
        raise ValueError("save must be True if recurse is True")

ไม่มีจุดทำจริง ๆclass BadValueError(ValueError):pass- คลาสที่กำหนดเองของคุณเหมือนกันกับการใช้งานกับValueErrorดังนั้นทำไมไม่ใช้มัน

ฉันจะสืบทอดจาก ValueError

class IllegalArgumentError(ValueError):
    pass

บางครั้งเป็นการดีกว่าที่จะสร้างข้อยกเว้นของคุณเอง แต่สืบทอดจากสิ่งที่มีอยู่ภายในซึ่งใกล้เคียงกับสิ่งที่คุณต้องการมากที่สุด

หากคุณต้องการตรวจสอบข้อผิดพลาดนั้นจะมีประโยชน์หากมีชื่อ

ฉันคิดว่าวิธีที่ดีที่สุดในการจัดการสิ่งนี้คือวิธีที่ไพ ธ อนจัดการกับมัน Python เพิ่ม TypeError ตัวอย่างเช่น:

$ python -c 'print(sum())'
Traceback (most recent call last):
File "<string>", line 1, in <module>
TypeError: sum expected at least 1 arguments, got 0

นักพัฒนาซอฟต์แวร์ระดับสูงของเราเพิ่งพบหน้านี้ในการค้นหา google สำหรับ "ข้อผิดพลาดข้อผิดพลาดของ python" และฉันประหลาดใจที่คำตอบที่ชัดเจน (สำหรับฉัน) ไม่เคยแนะนำในทศวรรษนับตั้งแต่คำถามนี้ถูกถาม

ฉันเพิ่งเห็นตัวบิวอินที่ValueErrorใช้ในสถานการณ์นี้

ขึ้นอยู่กับปัญหาที่เกิดขึ้นกับข้อโต้แย้ง

ถ้าอาร์กิวเมนต์มีชนิดไม่ถูกต้องยก TypeError ตัวอย่างเช่นเมื่อคุณได้รับสตริงแทนหนึ่งใน Booleans เหล่านั้น

if not isinstance(save, bool):
    raise TypeError(f"Argument save must be of type bool, not {type(save)}")

อย่างไรก็ตามโปรดทราบว่าใน Python เราไม่ค่อยทำการตรวจสอบเช่นนี้ หากการโต้แย้งนั้นไม่ถูกต้องจริง ๆ ฟังก์ชั่นที่ลึกกว่าอาจจะทำให้เราบ่น และถ้าเราแค่ตรวจสอบค่าบูลีนบางทีผู้ใช้รหัสบางคนอาจจะดึงมันมาเป็นสตริงโดยรู้ว่าสตริงที่ไม่ว่างนั้นเป็น True เสมอ มันอาจช่วยเขาหล่อ

หากอาร์กิวเมนต์มีค่าที่ไม่ถูกต้องให้เพิ่ม ValueError ดูเหมือนว่าจะเหมาะสมกว่าในกรณีของคุณ:

if recurse and not save:
    raise ValueError("If recurse is True, save should be True too")

หรือในกรณีนี้มีค่าที่แท้จริงของการเรียกคืนซึ่งแสดงถึงมูลค่าที่แท้จริงของการบันทึก เนื่องจากฉันจะพิจารณาการกู้คืนนี้จากข้อผิดพลาดคุณอาจต้องบ่นในบันทึก

if recurse and not save:
    logging.warning("Bad arguments in import_to_orm() - if recurse is True, so should save be")
    save = True

ผมไม่แน่ใจว่าผมเห็นด้วยกับการรับมรดกจากValueError- การตีความของฉันของเอกสารคือว่าValueErrorเป็นเพียงควรจะได้รับการเลี้ยงดูจาก builtins ... สืบทอดจากมันหรือเพิ่มมันเองดูเหมือนว่าไม่ถูกต้อง

ยกระดับเมื่อการดำเนินการในตัวหรือฟังก์ชั่นได้รับการโต้แย้งที่มีประเภทที่ถูกต้อง แต่เป็นค่าที่ไม่เหมาะสมและสถานการณ์ไม่ได้ถูกอธิบายโดยข้อยกเว้นที่แม่นยำมากขึ้นเช่น IndexError

- เอกสารประกอบ ValueError

เห็นด้วยกับข้อเสนอแนะของ Markus ในการม้วนข้อยกเว้นของคุณเอง แต่ข้อความของข้อยกเว้นควรชี้แจงว่าปัญหาอยู่ในรายการอาร์กิวเมนต์ไม่ใช่ค่าอาร์กิวเมนต์แต่ละรายการ ฉันขอเสนอ:

class BadCallError(ValueError):
    pass

ใช้เมื่อไม่มีข้อโต้แย้งคำหลักที่จำเป็นสำหรับการโทรที่เฉพาะเจาะจงหรือค่าอาร์กิวเมนต์มีความถูกต้องเป็นรายบุคคล แต่ไม่สอดคล้องกัน ValueErrorจะยังคงถูกต้องเมื่ออาร์กิวเมนต์ที่ระบุเป็นประเภทที่ถูกต้อง แต่อยู่นอกช่วง

นี่ไม่ควรเป็นข้อยกเว้นมาตรฐานใน Python ใช่ไหม

โดยทั่วไปแล้วฉันต้องการสไตล์ Python ให้คมชัดขึ้นเล็กน้อยในการแยกอินพุตที่ไม่ดีเข้ากับฟังก์ชัน (ความผิดของผู้เรียก) จากผลลัพธ์ที่ไม่ดีภายในฟังก์ชัน (ความผิดของฉัน) ดังนั้นอาจมี BadArgumentError เพื่อแยกข้อผิดพลาดของค่าในอาร์กิวเมนต์จากข้อผิดพลาดของค่าในท้องถิ่น