Python loop not iterating through the list

Question

I have this code, where I'm trying to loop through a mailbox with specified email-adresses in a list, but instead I'm getting the output of all email-adresses. When I print the counts, I get like 10 different email adresses, but not the ones I've limited to in the lists. How do I make it print only the ones from my list "emails"

from exchangelib import Credentials, Account
from collections import defaultdict

emails = ['test1@random.com','test2@random.com']

for email in emails:
    credentials = Credentials('hidden@hidden.com', 'hiddenpass')
    account = Account('hidden@hidden.com', credentials=credentials, autodiscover=True)
    counts = defaultdict(int)
    for item in account.inbox.all().order_by('-datetime_received')[:100]:
        counts[item.sender.email_address] += 1
print(counts)

Maroun · Accepted Answer · 2020-09-10T12:27:28.323

1

You don't need to iterate on the emails list. If you want to filter emails that don't appear in the emails list, you can do something like:

for item in account.inbox.all().order_by('-datetime_received')[:100]:
    if item.sender.email_address in emails:
        counts[item.sender.email_address] += 1

I'm not familiar with this library, but maybe you have the option to grab only the relevant results with something similar to:

account.inbox.all().filter(sender=email).order_by('-datetime_received')

As noted by @erikcederstrand, you can get better performance if you fetch only senders:

account.inbox.all().order_by('-datetime_received').only('sender')

edited Sep 10 '20 at 12:27

answered Sep 10 '20 at 11:09

Maroun

94,125
30
188
241

If you want this to go a *lot* faster, consider limiting the number of fields to just the one you need: `account.inbox.all().order_by('-datetime_received').only('sender')` – Erik Cederstrand Sep 10 '20 at 11:27
@ErikCederstrand so what you mean by that is `account.inbox.all().order_by('-datetime_received').only('exampleemail@email.com') ` – Nick Mensen fazzerboy2425 Sep 10 '20 at 11:33
No. I meant what I wrote. – Erik Cederstrand Sep 10 '20 at 11:36
1

@ErikCederstrand Wow tak Erik det var meget hurtigere! Thanks Erik it was very fast now. – Nick Mensen fazzerboy2425 Sep 10 '20 at 11:38

score 0 · Answer 2 · answered Sep 10 '20 at 11:11

Maybe you should try the following (if I understand you correctly). We increase counts variable as before, but print only ones that are in emails:

credentials = Credentials('hidden@hidden.com', 'hiddenpass')
account = Account('hidden@hidden.com', credentials=credentials, autodiscover=True)
counts = defaultdict(int)
for item in account.inbox.all().order_by('-datetime_received')[:100]:
    counts[item.sender.email_address] += 1
    if item.sender.email_address in emails:
        print(item.sender.email_address)
        print(counts[item.sender.email_address])

Python loop not iterating through the list

2 Answers2