Removing a group of specific characters from text (in python)

Question

I wrote this code in python to remove a group of specific characters from texts. But the code works only for the first type of characters, for instance:

"I live in street 223, Los Angeles and this isn't true :)"

should be:

"i live in street los angeles and this isnt true"

but as the number 2 is before the rest of the characters that should be removed -in the set v- the result that I get instead:

"i live in street 3, los angeles and this isn't true :)"

My code:

v = ["1","2","3","4","5","6","7","8","9","0",".","(",")",',',':',";",'"',"'","?","!","+","=","-","$","#","@","<",">"]

def p(r):
   for e in r:
       if e in v:
           return r.replace(e,"")
       else:
           pass
       
print(p("I live in street 223, Los Angeles and this isn't true :)".lower()))

How to modify this code to remove the rest characters? Or is there a better different code? thanks in advance.

you are returning the value of r before it is done looping through array v. instead of return, set r = r.replace(e, "") then after you are through the for loop, return the value of r — Chris Reed
– Chris Reed, Commented May 4, 2021 at 13:11

VisioN · Accepted Answer · 2021-05-04 13:11:39Z

2

Just create a new string without the invalid characters:

chars = {"1","2","3","4","5","6","7","8","9","0",".","(",")",',',':',";",'"',"'","?","!","+","=","-","$","#","@","<",">"}

def replace(s):
    buf = ""
    for c in s:
        if c not in chars:
            buf += c
    return buf

print(replace("I live in street 223, Los Angeles and this isn't true :)"))

answered May 4, 2021 at 13:11

VisioN

146k35 gold badges287 silver badges291 bronze badges

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

5 Comments

George Nabil Over a year ago

Thanks this also worked and added some ideas for me but I think it's a little bit complicated.

VisioN Over a year ago

@GeorgeNabil This is by far much better than calling replace function for every character in a string. What you are doing is O(n^2) operations instead of O(n), which I suggest.

George Nabil Over a year ago

Oh, thanks a lot after understanding your code and analyze mine I found that your code is really better in all directions. And I used your idea in the project.

VisioN Over a year ago

@GeorgeNabil If you want to improve it even more, use io.StringIO instead of a string: buf = io.StringIO(); for c in s: if c not in chars: buf.write(c); return buf.getvalue(). This will work even faster due to better memory allocation in the Python internals.

George Nabil Over a year ago

Thanks again for this helpful information. If you can suggest to me good resources for GUI in python (for beginners) please attach the links.

Collectives™ on Stack Overflow

Removing a group of specific characters from text (in python)

1 Answer 1

5 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related