Если заявления Заказа

Question

Я выполнял некоторые упражнения CodingBat с использованием Python, пока не наткнулся на проблему с моим предлагаемым решением.

Ссылка для упражнения here

Я пытался решить ее, используя этот код:

def alarm_clock(day, vacation): 
    if vacation and day == 0 or day == 6: 
     return "off" 
    elif vacation and day != 0 and day != 6: 
     return "10:00" 
    if vacation == False and day == 0 or day == 6: 
     return "10:00" 
    elif vacation == False and day != 0 or day != 6: 
     return "7:00" 

Что происходит, что когда день 6 и отдых в False, то он возвращает «выключено» вместо "10:00". Можете ли вы поделиться светом здесь? Является ли одно из первых утверждений If оценочным значением «10:00»?

Answer 1

Приоритет логического or ниже, чем приоритет and в Python

Таким образом, в строке здесь

if vacation and day == 0 or day == 6: 

до тех пор, как day == 6, все утверждение будет верным.

Так правильным было бы

if vacation and (day == 0 or day == 6): 

Answer 2

vacation and day == 0 or day == 6 

оценивает как

(vacation and day == 0) or day == 6 

Так что, если ваш day == 6 затем пункт оценивает.

Вы можете вместо

vacation and (day == 0 or day == 6) 

Answer 3

and принимает precedense к or. Меняем:

if vacation and day == 0 or day == 6: 

с

if vacation and (day == 0 or day == 6): 

или лучше и английский, как:

if vacation and day in [0, 6]: 

Так что я бы сделать его похожим на это, чтобы быть более читаемым

weekend = [0, 6] 
def alarm_clock(day, vacation): 
    if vacation: 
    return day in weekend and "off" or "10:00" 
    else: 
    return day in weekend and "10:00" or "7:00" 

Answer 4

Другие уже отбросил приоритет логических операторов.Может быть, вы могли бы рассмотреть рефакторинга ваши условия несколько, как это (что ИМХО отражает более красиво дерево решений):

if vacation: 
    if day == 0 or day == 6: #or if day in (0, 6) 
     return "off" 
    else: 
     return "10:00" 
else: 
    if day == 0 or day == 6: #or if day in (0, 6) 
     return "10:00" 
    else: 
     return "7:00" 

развивавших dimo414 Комментарий:

weekend = day in (0, 6) 
if vacation: 
    if weekend: 
     return "off" 
    else: 
     return "10:00" 
else: 
    if weekend: 
     return "10:00" 
    else: 
     return "7:00" 

Или даже:

weekend = day in (0, 6) 
if vacation: 
    return "off" if weekend else "10:00" 
else: 
    return "10:00" if weekend else "7:00" 

Answer 5

I подумайте, что самое ясное решение здесь - просто назвать ваши промежуточные значения:

weekend = day == 0 or day == 6 

Теперь ли вы оставить вещи структурированы как есть:

def alarm_clock(day, vacation): 
    if vacation and weekend: 
     return "off" 
    elif vacation and weekday: 
     return "10:00" 
    elif not vacation and weekend: 
     return "10:00" 
    elif not vacation and not weekend: 
     return "7:00" 

... или реорганизовать его в вложенными if заявления:

def alarm_clock(day, vacation): 
    if vacation: 
     if weekend: 
      return "off" 
     else: 
      return "10:00" 
    else: 
     if weekend: 
      return "10:00" 
     else: 
      return "7:00" 

... или реорганизовать его на более высоком логическом уровне:

def alarm_clock(day, vacation): 
    if vacation and weekend: 
     return "off" 
    elif vacation or weekend: 
     return "10:00" 
    else: 
     return "7:00" 

... сразу видно, что происходит, и эта проблема не может быть даже ари в первую очередь.

Это также позволяет избежать еще очень распространенная ошибка людей, где ДЭ-Моргану сочетание not с and или or только на полпути, и вы в конечном итоге с вещами, как not a and b, когда вы имели в виду not a and not b.

И это делает ваш код более надежным для расширения - если позже вы решите, например, запустить свой код в стране, которая начинает нумерацию недель с понедельника вместо воскресенья, вам нужно изменить только одну строку кода вместо четырех разные строки (и четыре строки почти достаточно, чтобы гарантировать, что вы получите один из них неправильным ...).