Logic code in python giving the wrong answer

Question

Given a day of the week encoded as 0=Sun, 1=Mon, 2=Tue, ...6=Sat, and a boolean indicating if we are on vacation, return a string of the form "7:00" indicating when the alarm clock should ring. Weekdays, the alarm should be "7:00" and on the weekend it should be "10:00". Unless we are on vacation -- then on weekdays it should be "10:00" and weekends it should be "off".

My code:

  def alarm_clock(day,vacation):
  if(vacation):
      if(day == 0 | day == 6):
          return "off"
      return "10:00"
  else:
      if(day == 0 | day == 6):
          return "10:00"
      return "7:00"

With input:

print(alarm_clock(0,True))

My code returns '10:00' when it should be 'off'

With input:

print(alarm_clock(0,False))

My code returns '7:00', it should be '10:00'

Where is the bug in my code?

`if(day == 0 or day == 6):` ? in python use `or` – Rakesh Jul 25 '18 at 13:23 — Rakesh, Jul 25 '18 at 13:23

score 1 · Accepted Answer · answered Jul 25 '18 at 13:25

Change it like this:

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

print(alarm_clock(0,True))

results:

off

The pipe isn't doing what you'd expect: Pipe character in Python it is in fact a bitwise operator. :)

score 0 · Answer 2 · answered Jul 25 '18 at 13:26

0

As you can see in the official documentation, the logical "or" operator in Python is or, not | which is a bitwise operator

answered Jul 25 '18 at 13:26

Hearner

2,711
3
17
34

score 0 · Answer 3 · answered Jul 25 '18 at 13:26

You are using a bitwise or "|". You need to use a logical or "or"

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

Logic code in python giving the wrong answer

3 Answers3