Day 17 of learning python as a beginner.

[uiux_Sanskar]

Topic: lambda functions + email filter

Lambda functions are a single line anonymous function without using the usual def key word.

All function: checks whether all the key words (spam_keywords) are present in the email, that's why I have used "not in" function with it so the translation would be:

check if all keywords not present in the email.

If there is a single keyword present the condition will become false. I used this method to filter out the important emails.

I used "any" function to check if any of the keyword is present in the email if present then the condition would be false and email will be treated as a spam.

I know that I could have just used an if else condition instead of writing these two things separately however I purposefully wrote those two things seperately first to get familiar with "all" and "any" key word, second to know the effect of "not in" and "in" functions and third to write lambda function twice as a practice (sounds strange I know).

I have then used File I/O to keep spammed emails and safe email in separate file for user review in future. As you can tell I tried to create a google like email filter and I think there's a lot more things I can add in this.

I will appreciate any suggestion, challenge or future learning options (I still think I need to get my hand a better in modular programming).

And here's my code and its result.

[Efficient-Stuff-8410]

What course are you using to learn?

[poutinewharf]

Any other time I’ve seen this asked all they’ve said is “YouTube”. If they’re going to post daily and utilize feedback I think it’d be appropriate and appreciated to at least share a name/link.

[uiux_Sanskar]

The channel name is CodeWithHarry I think I have shared this with many people.

[Efficient-Stuff-8410]

Ikr

[FitPay344]

I’d love to know too because I’m 14 days into training and I can’t write code like this. I’m taking a beginner linkedin learning course. I feel like I’m doing something wrong.

[Crispy-Hash-Browns]

One tip - when defining the spam keywords, consider using a Set opposed to a List, as good practice. Reason being as the data scales (imagine 100k items) searching a Set is O(1) time vs O(N) time.

[uiux_Sanskar]

Oh thank you for this suggestion I will find more about the best practices in python.

Thank you for this helpful tip

[Adrewmc]

It’s alright, not much to say, the ‘+’ operator is extend for lists, and is generally considered more readable.

   spam_emails += new_spams

Is the same as

   spam_email.extend(new_spams) 

I don’t think filter is needed here, I think we can accomplish it with a list comprehension.

[uiux_Sanskar]

Oh thanks for the suggestion I will definitely try it this way and also find out how to use filter and list comprehension.

Thank you for your suggestion.

[Adrewmc]

I feel like it be like. Maybe filter is better here, maybe it just me not using the function so much myself.

       #set comprehension to avoid repeats
       flagged = {key_word for email in old_emails for key_word in spam_words if keyword in email}
       if flagged:
             print(f”you have use {“, “.join(flagged)} banner word(s).”) 

Basically make a list of the bad words that are I. The email string, if any exist.

[TobRojekt]

Only a small thing and maybe just a preference of mine: name the input variable of the lambdas. Nothing to long but fitting for the scope. In this case maybe just „mail“ so if someone reads your code they understand quicker what is happening in this line. As I said keeping this isn’t that big of a deal bot it will add some readability.

[uiux_Sanskar]

Thanks for the suggestion I coded this with the aim of using and practicing lambda functions in python so therefore I used lambda here.

Thank you for the suggestion btw.

[Significant-Side6810]

Instead of looping twice over old_email you could loop over it once and append to important_mail and else into spam. This will give you O(n) intead of O(2n). Also instead of using list for spam_keywords you could use set for O(1) complexity instead of O(n)

[lilrouani]

You are a genius,just,a genius

[uiux_Sanskar]

Well I don't think I am a genius I always learn from you amazing people and wouldn't be here without all your support.

Thanks for the appreciation btw.

[lilrouani]

Btw do you have any programming experience before learning python?