I have a txt file with various email addresses and other lines that are not valid emails, I am trying to print only the valid email addresses, when I use the code below, nothing is printed. This is the content of the txt file:
[email protected]
[email protected]
lalalalal
In this case, only both the email addresses should be printed
import re
my_file = open('emails.txt', 'r+')
You may fix your code if you add re.M flag:
re.findall(r"^[a-zA-Z0-9_.+-]+@[a-zA-Z0-9-]+\.[a-zA-Z0-9.-]+$", my_file.read(), re.M)
Since you read in the whole file with my_file.read(), the ^ and $ should match start/end of the line, not string, and the re.M flag does that.
Also, you may read the file line by line and only get those lines that fully match your pattern:
items = []
email_rx = re.compile(r"[a-zA-Z0-9_.+-]+@[a-zA-Z0-9-]+\.[a-zA-Z0-9.-]+$")
with open('emails.txt', 'r+') as my_file:
for line in my_file:
if email_rx.match(line):
items.append(line)
Note that only $ anchor is necessary as re.match only anchors matches at the start of the string.
Note that you may have CRLF endings, then, you might either rstrip each line before testing against regex and appending to items, or add \s* pattern at the end before $ anchor.
import re
my_file = open('emails.txt', 'r+')
items = re.findall(r"([a-zA-Z0-9_.+-]+@[a-zA-Z0-9-]+\.[a-zA-Z0-9-.]+)", my_file.read())
for items in items:
print(items)
Two problems
This should print all emails in the file
import re
reg = '[A-Za-z0-9.]+@[A-Za-z0-9]+[.][a-z]+'
with open('email.txt', 'r') as f1:
for email in f1:
if(len(re.findall(reg,email))!=0):
print(email)
And this should get only whole line emails -
import re
reg = '[A-Za-z0-9.]+@[A-Za-z0-9]+[.][a-z]+'
with open('email.txt', 'r') as f1:
for email in f1:
if(len(re.findall(reg,email))):
if(len(re.findall(reg,email)[0])==len((email.replace("\n","")))):
print(email)
re.Mflag,re.findall(r"^[a-zA-Z0-9_.+-]+@[a-zA-Z0-9-]+\.[a-zA-Z0-9.-]+$", my_file.read(), re.M)forloop is iterating over the file, which has already seeked to the end with.read(). why aren't you iterating overitemsinstead?itemsand in the very next line your are overwritingitems.