Wondering why this code isn't working? Basic reverse string in python [duplicate]

Question

I took a few coding classes in college and am trying to relearn the skills I have lost. The program I am attempting to write takes a string and returns that string backwards: "Nope" becomes "epoN" for example. I was just wondering if you could help me figure out exactly what is wrong in my logic or syntax!

EDIT: Thanks everyone. I fixed the problem by making the variable lengthOfWord = len(string)- 1

ALSO I'm sorry I didn't post my error message. I forgot that rule and will do it in the future. Thanks again!

def ReverseString(string):
    finalWord = ""
    lengthOfWord = len(string)
    while lengthOfWord >= 0:
       finalWord = finalWord + string[lengthOfWord]
       lengthOfWord = lengthOfWord - 1
    else:
        print(finalWord)
    return

When you have some code that returns an error, you should include the error in your question. Neither your logic or your syntax are wrong, you just have an IndexError because of the indices — Mel
– Mel, Commented Aug 6, 2017 at 16:44
Please take a look at minimal reproducible example. It's important if you ask for debugging help that you include all relevant details (including desired behaviour and traceback). — MSeifert
– MSeifert, Commented Aug 6, 2017 at 16:48

Md. Rezwanul Haque · Accepted Answer · 2017-08-06 16:47:58Z

0

Try this way :

def ReverseString(string):
  return string[::-1]


print(ReverseString("Rezwan"))

Output :

nawzeR

Or you can try :

def ReverseString(string):
  finalWord = ""
  lengthOfWord = len(string) - 1
  while lengthOfWord >= 0:
    finalWord = finalWord + string[lengthOfWord ]
    lengthOfWord = lengthOfWord - 1
  return finalWord

print(ReverseString("Rezwan"))

answered Aug 6, 2017 at 16:47

Md. Rezwanul Haque

2,9487 gold badges35 silver badges49 bronze badges

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

Comments

stamaimer · Accepted Answer · 2017-08-06 16:49:34Z

-1

The index of list-like object in python is zero based. You use length of a string to get the last element of it is out of range. You can get the last element of a list-like object as follows:

list_like_obj[-1]
list_like_obj[length - 1]

And you can reverse a list-like object use slice, method of that object or built-in function:

list_like_ojb[::-1]
list_like.obj.reverse()
reversed(list_like_obj)

edited Aug 6, 2017 at 16:49

answered Aug 6, 2017 at 16:43

stamaimer

6,5456 gold badges39 silver badges57 bronze badges

Collectives™ on Stack Overflow

Wondering why this code isn't working? Basic reverse string in python [duplicate]

2 Answers 2

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Linked

Related