ใน Python ฉันควรทดสอบว่าตัวแปรเป็น None, True หรือ False

ฉันมีฟังก์ชั่นที่สามารถคืนหนึ่งในสามสิ่ง:

• สำเร็จ ( True)
• ความล้มเหลว ( False)
• ข้อผิดพลาดในการอ่าน / การแยกสตรีม ( None)

คำถามของฉันคือถ้าฉันไม่ควรทดสอบTrueหรือFalseฉันจะดูได้อย่างไรว่าผลลัพธ์คืออะไร ด้านล่างเป็นวิธีที่ฉันทำอยู่ในปัจจุบัน:

result = simulate(open("myfile"))
if result == None:
    print "error parsing stream"
elif result == True: # shouldn't do this
    print "result pass"
else:
    print "result fail"

มันง่ายพอ ๆ กับการถอดชิ้น== Trueส่วนออกหรือว่าฉันควรจะเพิ่ม tri-bool data-type ฉันไม่ต้องการให้simulateฟังก์ชันโยนข้อยกเว้นเนื่องจากฉันต้องการให้โปรแกรมภายนอกทำโดยมีข้อผิดพลาดคือบันทึกและดำเนินการต่อ

อย่ากลัวข้อยกเว้น! มีโปรแกรมของคุณเพียงเข้าสู่ระบบและดำเนินการต่อเป็นเรื่องง่ายเหมือน:

try:
    result = simulate(open("myfile"))
except SimulationException as sim_exc:
    print "error parsing stream", sim_exc
else:
    if result:
        print "result pass"
    else:
        print "result fail"

# execution continues from here, regardless of exception or not

และตอนนี้คุณสามารถมีการแจ้งเตือนที่หลากหลายยิ่งขึ้นจากวิธีการจำลองเป็นสิ่งที่ผิดพลาดอย่างแน่นอนในกรณีที่คุณพบข้อผิดพลาด / ไม่มีข้อผิดพลาดไม่ให้ข้อมูลเพียงพอ

if result is None:
    print "error parsing stream"
elif result:
    print "result pass"
else:
    print "result fail"

ทำให้มันง่ายและชัดเจน แน่นอนคุณสามารถกำหนดพจนานุกรมล่วงหน้าได้

messages = {None: 'error', True: 'pass', False: 'fail'}
print messages[result]

หากคุณวางแผนที่จะแก้ไขsimulateฟังก์ชั่นของคุณให้ใส่รหัสที่ส่งคืนมากขึ้นการรักษารหัสนี้อาจกลายเป็นปัญหาเล็กน้อย

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

ไม่เคยไม่เคยไม่เคยพูด

if something == True:

ไม่เคย มันบ้าเพราะคุณซ้ำซ้อนซ้ำสิ่งที่ระบุซ้ำซ้อนเป็นกฎเงื่อนไขซ้ำซ้อนสำหรับคำสั่ง if

แย่ลงยังไม่เคยไม่เคยไม่เคยพูด

if something == False:

notคุณมี รู้สึกอิสระที่จะใช้มัน

ในที่สุดการทำa == Noneไม่มีประสิทธิภาพ a is NoneDo Noneเป็นวัตถุแบบซิงเกิลพิเศษสามารถมีได้เพียงวัตถุเดียว เพียงตรวจสอบเพื่อดูว่าคุณมีวัตถุนั้น

ผมอยากจะเน้นว่าแม้ว่ามีสถานการณ์ที่if expr :ไม่เพียงพอเพราะใครอยากที่จะให้แน่ใจว่าexprเป็นTrueและไม่เพียง แต่แตกต่างจาก0/ None/ สิ่งที่isจะได้รับการที่ต้องการจาก== เหตุผลเดียวกัน S.Lott mentionned == Noneสำหรับการหลีกเลี่ยง

มันมีประสิทธิภาพมากขึ้นเล็กน้อยและเชอร์รี่บนเค้กอ่านง่ายขึ้น

In [1]: %timeit (1 == 1) == True
38.1 ns ± 0.116 ns per loop (mean ± std. dev. of 7 runs, 10000000 loops each)

In [2]: %timeit (1 == 1) is True
33.7 ns ± 0.141 ns per loop (mean ± std. dev. of 7 runs, 10000000 loops each)

ฉันเชื่อว่าการโยนข้อยกเว้นเป็นความคิดที่ดีกว่าสำหรับสถานการณ์ของคุณ ทางเลือกอื่นจะเป็นวิธีการจำลองเพื่อคืนค่า tuple รายการแรกจะเป็นสถานะและผลลัพธ์ที่สอง:

result = simulate(open("myfile"))
if not result[0]:
  print "error parsing stream"
else:
  ret= result[1]