Why is my conditional returning "none?" [duplicate]

Question

Whenever I run this I get the third option when it should be returning the first, since s = 'yes'. What is going wrong here?

def shut_down(s):
    if s is 'yes':
        return 'Shutting down...'
    elif s is 'no':
        return 'Shutdown aborted!'
    else:
        return "Sorry, I didn't understand you"

ans = 'Yes'
s = ans.lower()
shut_down(s)

This is a repeat of stackoverflow.com/questions/2988017/…

Daniel Williams
– Daniel Williams

2013-03-16 22:55:54 +00:00
Commented Mar 16, 2013 at 22:55 — Daniel Williams
– Daniel Williams, Commented Mar 16, 2013 at 22:55

NPE · Accepted Answer · 2013-03-16 22:55:11Z

5

Change

if s is 'yes':

to

if s == 'yes':

and

elif s is 'no':

to

elif s == 'no':

While is is a valid operator, it is not the one to use here (it compares object identity instead of comparing the character sequences).

answered Mar 16, 2013 at 22:55

NPE

503k114 gold badges970 silver badges1k bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Ivaylo Strandjev · Accepted Answer · 2013-03-16 22:55:24Z

4

is tests for identity, not equality. To test if a string is equal to yes use s=='yes'

answered Mar 16, 2013 at 22:55

Ivaylo Strandjev

71.4k18 gold badges130 silver badges183 bronze badges

Collectives™ on Stack Overflow

Why is my conditional returning "none?" [duplicate]

2 Answers 2

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Linked

Related