У 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частину чи слід додати тип даних трибулевого типу. Я не хочу, щоб 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-else.

Ніколи, ніколи, ніколи не кажи

if something == True:

Ніколи. Це божевільно, оскільки ви надмірно повторюєте те, що надмірно вказано як правило надлишкової умови для заяви-if.

Гірше, все-таки, ніколи, ніколи, ніколи не кажіть

if something == False:

У вас є not. Сміливо користуйтеся ним.

Нарешті, робити a == Noneце неефективно. Зробіть a is None. Noneє спеціальним однотонним об'єктом, може бути лише один. Просто перевірте, чи є у вас цей об’єкт.

Я хотів би підкреслити, що, навіть якщо існують ситуації, коли if expr :недостатньо, оскільки хочеться переконатися, що exprце, Trueа не просто відрізняється від 0/ None/ будь-якого, isслід віддати перевагу з == тієї ж причини, про яку С.Лотт згадував, щоб уникнути== 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)

Я вважаю, що кидати виняток - це краща ідея для вашої ситуації. Альтернативою буде метод моделювання повернення кортежу. Перший елемент буде статусом, а другий - результатом:

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