Чи безпечно покладатися на порядок оцінки стану у твердженнях if?

Чи погана практика використовувати наступний формат, коли my_varможе бути None?

if my_var and 'something' in my_var:
    #do something

Проблема в тому, що 'something' in my_varвикине помилку TypeError, якщо для my_var встановлено значення None.

Або я повинен використовувати:

if my_var:
    if 'something' in my_var:
        #do something

або

try:
    if 'something' in my_var:
        #do something
except TypeError:
    pass

Переформулювавши запитання, що з наведеного є найкращою практикою в Python (якщо така є)?

Ласкаво просимо!

Безпечно залежати від порядку умовних умов ( посилання на Python тут ), зокрема через проблему, на яку ви вказуєте, - дуже корисно мати можливість короткого замикання, яке може спричинити проблеми в ряді умовних умов.

Цей тип коду з’являється на більшості мов:

IF exists(variable) AND variable.doSomething()
    THEN ...

Так, це безпечно, це чітко і дуже чітко визначено в мовному посиланні:

Вираз x and yспочатку оцінює x; якщо xє false, повертається його значення; в іншому випадку yобчислюється і отримане значення повертається.

Вираз x or yспочатку оцінює x; якщо xtrue, повертається його значення; в іншому випадку yобчислюється і отримане значення повертається.

Можливо, я тут трохи педантичний, але я б сказав, що найкраща відповідь

if my_var is not None and 'something' in my_var:
    #do something

Різниця полягає в явній перевірці None, а не в неявному перетворенні my_varна Trueабо False.

Хоча я впевнений, що у вашому випадку відмінність не є важливою, у більш загальному випадку цілком можливо, щоб змінної не було, Noneале все одно обчислювалося False, наприклад, цілим числом 0чи порожнім списком.

Отже, всупереч твердженню більшості інших плакатів, що це безпечно, я б сказав, що це безпечно, якщо ви явно заявляєте. Якщо ви не впевнені, тоді розгляньте цей дуже надуманий клас:

class Contrived(object):
    def __contains__(self, s):
        return True
    def __nonzero__(self):
        return False

my_var = Contrived()
if 'something' in my_var:
    print "Yes the condition is true"
if my_var and 'something' in my_var:
    print "But this statement won't get reached."
if my_var is not None and 'something' in my_var:
    print "Whereas this one will."

Так, я знаю, що це не реалістичний приклад, але варіації трапляються в реальному коді, особливо коли Noneвикористовується для позначення аргументу функції за замовчуванням.

Це не так просто. Як чувак C #, я дуже звик робити щось на зразок:

if(x != null && ! string.isnullorempty(x.Name))
{
   //do something
}

Вищевказане працює чудово і оцінюється як очікувалось. Однак у VB.Net наступне призведе до результату, якого ви НЕ очікували:

If Not x Is Nothing **And** Not String.IsNullOrEmpty(x.Name) Then

   'do something

End If

Вищевикладене виняток. Повинен бути правильний синтаксис

If Not x Is Nothing **AndAlso** Not String.IsNullOrEmpty(x.Name) Then

   'do something

End If

Зверніть увагу на дуже тонку різницю. Це мене збентежило близько 10 хвилин (занадто довго), і тому чуваки з C # (та інших) повинні бути дуже обережними при кодуванні іншими мовами.

Я хотів би спробувати / виключити, але це залежить від того, що ви знаєте про змінну.

Якщо ви очікуєте, що змінна буде існувати більшу частину часу, тоді спроба / крім менше операцій. Якщо ви очікуєте, що змінна більшу частину часу матиме значення None, тоді оператор IF матиме менше операцій.

Це абсолютно безпечно, і я роблю це постійно.