Palindrome

[mp3909]

def isStringPalindrome(string):
  
  rString=''
  def reverse(string):
    index=len(string)
    while index>0:
      rString=rString+string[index-1]
      index=index-1


  if string==rString:
    print('String is Palindrome')
  else:
    print('string is not palindrome')
    

isStringPalindrome('reviver')

Why is this not working as it is printing back string is not palindrome when it is?

[kilthar]

Try he below code.
You were not calling the reverse function anywhere and it had no return statement.

def isStringPalindrome(string):
   
  
    def reverse(string):
      
        rString=''
        index=len(string)
        while index>0:
            rString=rString+string[index-1]
            index=index-1      
      
        return rString
 
 
    if string==reverse(string):
        print('String is Palindrome')
    else:
        print('string is not palindrome')


print(isStringPalindrome('abba'))

[mp3909]

that works!
thank you

[sparkz_alot]

You could also use slicing:

a_string = "Anna".lower()
rev_string = a_string[::-1]

if rev_string == a_string:
    print("True")
else:
    print("False")

Notice we change 'a_string' to lowercase, otherwise 'Anna' would reverse as 'annA', resulting in a "False'