Getting IndexError: string index out of range in python

Question

Getting IndexError: string index out of range in python, I'm a beginner in python and trying to make a program that will count total uppercase and lowercase letters and I have also tried some solutions as removing spaces of a string as string.replace(" ", "") but again getting error,

My code

string = input('Enter string upto 25 words: ')
if len(string) > 25:
  print('Please enter a string less than 25 words.')
else:
  string.replace(" ", "")
  total_upr = 0
  total_lwr = 0
  i = 0
  while i < 25:
    print(i)
    if string[i].islower():
      total_upr += 1
    elif string[i].isupper():
      total_lwr += 1
    i += 1
  print(f"'{string}' contains {total_lwr} lower-case letters and {total_upr} upper-case letters.")

Vikas Periyadath · Accepted Answer · 2019-09-06 06:56:31Z

3

Try with len(string) instead of 25 , because here you are looping through all 25 integers. But in your case it is not mandatory that you should have exactly 25 characters in your string, it can be 25 or less than 25. So when its less than 25 character and you are trying to take 25th character it will show error:

instead of :

while i < 25 :

try with :

while i < len(string)

edited Sep 6, 2019 at 6:56

answered Dec 26, 2018 at 7:19

Vikas Periyadath

3,1961 gold badge25 silver badges35 bronze badges

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

3 Comments

Developer Over a year ago

Thanks bro, but our teacher has not taught us for loop we have only studied while loop because for loop is for iteratable objects, so sir said will study after list, can you make it with while?

Developer Over a year ago

Yeah I have solved that, @VikasDamodar edit it I will accept it

Developer Over a year ago

@VikasDamodarr u here?

Mehrdad Pedramfar · Accepted Answer · 2018-12-26 07:26:23Z

2

You can also try this:

string = input('Enter string upto 25 words: ')
if len(string) > 25:
    print('Please enter a string less than 25 words.')
else:
    total_lwr = sum(map(str.islower, string))
    total_upr = sum(map(str.isupper, string))

You don't need to use while for that, also no need to replace anything. less code. :)

answered Dec 26, 2018 at 7:26

Mehrdad Pedramfar

11.1k4 gold badges43 silver badges61 bronze badges

Comments

Hasitha Amarathunga · Accepted Answer · 2018-12-26 07:52:55Z

The while should be as below:

while i < len(string):

The code :

string = input('Enter string upto 25 words: ')
if len(string) > 25:
  print('Please enter a string less than 25 words.')
else:
  string.replace(" ", "")
  total_upr = 0
  total_lwr = 0
  i = 0
  while i < len(string):
    print(i)
    if string[i].islower():
      total_lwr += 1
    elif string[i].isupper():
      total_upr += 1
    i += 1
  print(f"'{string}' contains {total_lwr} lower-case letters and {total_upr} upper-case letters.")

Collectives™ on Stack Overflow

Getting IndexError: string index out of range in python

3 Answers 3

3 Comments

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

Comments

Related