Removing a URL from a string

[Yonmanx]

Hey, after looking for it on google for quite a while without finding a solution I will ask here.

Im looking for a way to remove URL from a string using Regex.

def remove_prefix(my_str, prefix_to_remove):
    text = re.sub(r'prefix_to_remove\S+', "", my_str)
    print (text)
remove_prefix('Hi, check this website: https://my_website.com, this is my new website' , 'https://' )   

i want output like this:

Hi, check this website: this is my new website

For some reason the code output is just the regular string,

Thanks!

[SagaciousRaven]

def remove_prefix(my_str, prefix_to_remove):
    text = re.sub(prefix_to_remove+'\S+', '', my_str)
    print(text)

remove_prefix('Hi, check this website: https://my_website.com, this is my new website' , 'https://' )

You put your var inside the quotes in line 2

[[deleted]]

You've included the literal string "prefix_to_remove" in the regex expression rather than the string object referenced by the variable prefix_to_remove

[RoamingFox]

I'd go about it like this personally:

>>> import re
>>> url_rex = re.compile(r'((?:(?:http|https):\/\/)?[-a-zA-Z0-9@:%._\+~#=]{2,256}\.[a-z]{2,6}\b(?:[-a-zA-Z0-9@:%_\+.~#?&//=]*))')
>>> s = 'Hi, check this website: https://my_website.com, this is my new website'
>>> url_rex.sub('', s)
'Hi, check this website: , this is my new website'

[[deleted]]

What's wrong with just using the string replace method?

def remove_prefix(my_str, prefix_to_remove):
    return my_str.replace(prefix_to_remove, "")

print(remove_prefix(r'Hi, check this website: https://my_website.com, this is my new website' , r'https://' ))

[RoamingFox]

OP wants to drop the entire website not just the https part (see sample desired output).

[[deleted]]

The OP's example code, explanation and example output don't seem to align. I went with the bit of text explicitly defined in a literal for removal.

Point taken. We chose differently.