从字符串中删除大写字母

Question

I have to remove the capitalized letters from the string given

When I try to remove the capital letters from the given string, the return function exists at the end of the function which I wrote to print the output causes to the last character to get printed. Is there any better way to solve it?

def remove_capitals(a):    
    for i in a:    
          if ord(i)>64 and ord(i)<91:   
               pass  
          else:  
               print(i,end='')  
     return i

print(remove_capitals("A1H5J3ETD"))

expected : 153 actual : 153D

Answer 1

You print inside and print the return - you return the last i without test .. which is the last letter of your input.

Modifications:

• Instead of checking ordinals and magic numbers, use built in fuction isupper() to decide.

• do not use pass - simply do only something if it is ok to do so

• collect all parts into a list and join() them back into a string

Fix:

def remove_capitals(a):    
    non_upper = []
    for i in a:    
        if not i.isupper():  
            non_upper.append(i)

    return ''.join(non_upper)


print(remove_capitals("A1H5J3ETD"))

Output:

153

---

You can also append to strings - but strings are immutable so it destroyes the old one and creates a new longer version - which is wasteful:

def remove_capitals(a):    
    non_upper = "" 
    for i in a:    
        if not i.isupper():  
            non_upper += i

    return non_upper

The only benefit this has, is that it does not use lists - it is slower and more memory intensive.

Answer 2

Use list comprehension

def remove_capitals(a):
    return ''.join([x for x in a if ord(x)<64 or ord(x)>91]))

Answer 3

The last unwanted character is returned because of the return i statement at the end of your function. If you just want to print out the non-capitalised letters putting return None and just calling the function (not doing print(remove_capitals("A1H5J3ETD")) : remove_capitals("A1H5J3ETD") should do the trick.