The Ultimate Guide to Writing Functions

[@ArjanCodes]

👷 Join the FREE Code Diagnosis Workshop to help you review code more effectively using my 3-Factor Diagnosis Framework: www.arjancodes.com/diagnosis

[@Ziggity]

If only I had this when I first started out. The amount of useful information you were able to so succinctly squeeze into a 25 minute video while keeping things clear and accessible is outstanding. Fantastic Job.

[@Acuzzio]

Agree

[@BenKraal]

Agree x2

[@hovikkevork3069]

Crazy value in less than 30 mins! Not only you know what you're talking about, you also know how to deliver it in simple terms. Thanks.

[@ArjanCodes]

I appreciate that!

[@hcubill]

I have followed you for a couple of years and this is my favourite video. You are getting so clear and concrete with your points. Functions did deserve a video like this!

[@ArjanCodes]

Thanks so much Hector - glad you liked the video!

[@shashankbang10]

Hello Arjan. I have been following your tips for a while now. It’s completely changed the way I write code and I have received accolades at my work for the significant improvement in code style. I want to thank you very much for it. Please make more videos like these.

[@tugrulpinar16]

Same here, thank you Arjan!

[@nitzan3984]

Same

[@gwho]

Another tip: put your 17:00 Put PaymentHandler and StripPaymentHandler, related subclasses/polymorphisms next to each other. PRICES is particular constants (which are akin to arguments) as opposed to PaymenteHandler and StripePaymentHandler class definitions (which are akin to function definitions). So in sum, separate arguments from parameters, or templates from values.

[@StavrosSachtouris]

Guido Van Rossum initially named his language "stroopwafel", but it didn't take off because the name is too long. While he was visiting a zoo and thinking ideas for a shorter name, a snake ate his stroopwafel. Guess what type of snake it was. (I cannot give you any hints, because stroopwafel didn't support type hints).

[@PavelSTL]

Umm Cobra?

[@AnotherAvaibleName]

was it a coralillo ?

[@victor_silva6142]

An Oroshimaru?

[@moustachescarz]

a pythong?

[@ravimaithreyregulagedda2158]

this is a nice story but I hope you aren't implying that's how he named the language because python is named after Monty Python not the snake python

[@mohammednasser2159]

You could've added the cc_validation function as a method for the Card class, and even turn it into property, or validate it upon instantiation in the init, will make even more sense. Great content, keep going ❤️

[@qwerty11111122]

Glad you mentioned there should be a verb in a function name. def verb(direct_object, indirect_object, ...): ... Or for methods, subject.verb(objects). Been trying my best to be consistent with using normal grammar to make my code make sense, like using plurals for collections to make loop variables easy to name.

[@Personal-M.I.S.]

Just to say, I've only dabbled in Python and don't know much more than the fundamentals of the language, but I still get A LOT out of watching your videos. I really appreciate how your presentations are so chill and straight to the point. Even when I can't quite tell what's going on with the language, I can get the idea you're trying to teach. Good stuff!

[@marcioneto3016]

After tulips and cheese, I really thought you were going for windmills, but nope 😂. Great video, as always!

[@kaincenteno3609]

lol i was thinking the same, then BAM! that third one made me giggle 😆

[@heitortremor]

I'm excited for this one! I've always wondered if I was going about functions in the most efficient/optimal way and I'm sure these tips will guide me to be better.

[@angelcaru]

Summary of this video: == TIPS == - Functions should do one thing, and one thing only - Side effects (commands) or computations (queries), not both - Request only what you need - Minimal parameters - Separate creation and usage - No flag parameters (split up functions instead) - Use higher-order functions == NAMING == - Split up functions with `and` in their name - Shorten - Names should be verbs, arguments should be nouns - Always use the same vocabulary - Use the languages preferred_naming_scheme and notTheWrongOne (so never touch C#) - No spelling or grammar mistakes (duh)

[@Kev1305]

You forgot one key bit - Python should be called Prostitute.

[@myce-liam]

You've forgotten one of the most important take-aways: Arjan thinks Python should have been called Prostitute. "Prostitute 3.11 is up to 60% faster than Prostitute 3.10"😄

[@kayakMike1000]

And? The function should do one thing only...

[@paultapping9510]

​@kayakMike1000 a redundant and somewhat idiomatic but nonetheless correct formulation.

[@Personal-M.I.S.]

Nice! You've taken some of the most fundamental clean code practices and distilled them down to a very coherent and concise 25 minutes with no hand waving or fluff. Uncle Bob would be proud =)

[@hukl3945]

Bonus Tip: Don't nest conditionals. While two simple, nested conditionals might be acceptable but I would already try and think of a better way. When you have three levels in your decision tree it might be time for a truth table. It may seem cumbersome to adhere to this rule but in my experience it always leads to better structure in the code.

[@EricT43]

Hi ArjanCodes, thanks for the video! I appreciate the way that you teach these coding concepts - you strike just the right balance, in my opinion, between conceptual and practical explanations.

[@NoobomgWhy]

My favourite Dutch things so far: Frikandeln, Vla and Pasta Choca. Now I can add Python. Thank you!

[@katja_und_mickey]

Another great watch! I am so glad there are so many great videos on your channel! Thank you so much for all the valuable information.

[@ArjanCodes]

I'm glad you're enjoying the content!

[@ARandomOWL]

Python should have been called Stroopwafel.

[@lachlanstanding7386]

That's a funny name, I'd've called 'em Shaswazzas!

[@thunder____]

@@lachlanstanding7386 I see you've played "knifey spoony" before

[@zyklos229]

how in the hell, you have so much tasty food in netherlands :D

[@Einimas]

It's a windmill.

[@leaoaugusto]

why?

[@_baco]

Really gooood video!!!! To @GrammarNazi, at work we happen to have a Django model named “Custom”. Because classes should be singular names in Python... and someone thought Customs was a plural 🤦‍♂. We still can not safely refactor that.

[@ElNachoMacho]

Great video as always. I always learn something new watching your content! Thank you for sharing

[@JZ1917]

The humor quality is increasing with each video. Not only did I learn, I chuckled a few times !

[@ArjanCodes]

Glad you like them!

[@JesusMRamosPerez]

Thanks a lot for these videos! Can't stop watching them!

[@madao4024]

that was a good one at 5:50 🤣

[@SuperDoc3000]

This man's content is a gift, thank you Arjan

[@lekhakaananta5864]

5:48 "There are no Pythons in the netherlands...should have called it Tulip, Cheese, or Prostitutes" Totally, imagine the job ads. "This position requires 5 years of experience in Prostitutes"

[@yahyakhan-rr9ei]

One of the best channels, and for free. Feels amazing.

[@danielkonstantinovsky108]

It's very interesting for me to step into the mainstream Python developer world on KZbin. I'm a scientist and I write tons of "complicated" Python (and Cython and C++) code but the complexity is very different in scientific code - it's in the math and the algorithms rather than the stack of code that manages other code as seems to be common in mainstream codebases. We use abstractions like classes of course but our hierarchy of abstractions is much less branched than here, more like a stack than a web. We like our input files with dozens of parameters (🙂), which flows into some main, which calls some objects and functions, that kind of thing. This comment in itself is not a criticism of common design patterns - I don't feel qualified to criticize them, but I do note how difficult it is often to find the code that actually _does_ something in "professional" codebases. There is so much fluff that is supposed to manage the complexity that it becomes quite complex in itself. That has always frustrated me. Good abstractions are hard to come by, and mainstream code is full of bad abstractions for the sake of abstraction. Sometimes it's better just to _do_ the thing you want to do, rather than erect an entire class of middle managers, usually classes, with abstract-sounding names. If your core algorithm is ten lines or so (as is amazingly often the case by the way), there is no justification for writing 5000 lines across 14 source files to "manage the complexity", but that's often what you see. Bottom line is when you manage complexity you introduce complexity, and that has to be controlled if you're going to make the code friendlier to the developer and user. As you may have guessed, I'm a big fan of Brian Will's diatribe on OOP, not that I think classes are bad (he doesn't either) but that I think most abstractions are bad, and cementing them in with further bad abstraction is a bad idea. Just some thoughts from a scientific programmer - I live in a very different world from all of you, but maybe I have some useful insights because of that. I don't know. You decide.

[@danilkister1650]

This video is heavily underrated!

[@gshan994]

Quicktip: you can select cc and then hit ctrl/cmd + d to select next occurance of cc. With this u can change selected same words.

[@weistrass]

Didn't know those partial functions!!! Sooooo gooooood

[@ArjanCodes]

I'm glad you learned something new!

[@wizardfix]

Nice. Very good advice about separation of concerns. Thank you.

[@ArjanCodes]

Glad it was helpful!

[@danny_p466]

It's great to see Uncle Bob's Clean Code lessons apply still to this day! Thanks for the video :)

[@uwegenosdude]

Thanks for the great video.Especially I liked the trick to force keyword arguments. And the closed open principle.

[@ArjanCodes]

Glad you enjoyed it!

[@drhilm]

Love this kind of videos. Thanks. This is something to return to...

[@ArjanCodes]

Thanks so much Dror, glad the content is helpful!

[@mikelancaster8924]

These videos are very necessary, thanks!

[@ArjanCodes]

Glad you like them!

[@Krazness]

Great video. Have you considered creating a video covering partial applications for functions?

[@ArjanCodes]

Great suggestion Richard, thank you!

[@innstikk]

Excellent video! There is one thing that I didn't understand. In order to not pass all the user information you made a CardInfo protocol and said that it then would only pass on the information in the protocol. I tested this, but I can still access name from the User dataclass in this way. So the protocol doesn't extract only the variables it contains, but I guess only checks that those needed are there? So in effect you send over as much data in this way as with simply passing the User without the protocol. The protocol approach is nicer than sending the User, but having a Card class is even better.

[@abymathew575]

wow I really like this video. What a clarity and clear explanation with real examples. Thanks for this wonderful video. 👏👏👏👏👏👏

[@ArjanCodes]

Thanks so much Aby, glad the content is helpful!

[@gshan994]

Ur explanation is very clear n concise. I learned something new today. Thanks

[@ArjanCodes]

Thanks so much GShan, glad the content is helpful!

[@JackQuarm]

Your channel is brilliant, keep up the great vids!

[@ArjanCodes]

Thanks a ton!

[@fmictsang8874]

In 13:50 you mentioned multiply cards for one person. I would like to ask how should it be store in the Customer object? Would it be like alice.card = [card1, card2] ?

[@NostraDavid2]

Yeah, that would be a decent solution. You'd need to add a few loops here and there, but yeah.

[@InvestLarge]

New to the channel since yesterday's KZbin recommended your video on Classes. Subscribed and liking the tutorials. I am learning and also seeing new things that I don't know what they are for. Decorators perhaps? protocol, hmm Going to look it all up and become better knowledged

[@ArjanCodes]

Thank you so much!

[@skellious]

I learnt so much from this one video

[@Kamil-so1se]

Great as always.

[@CodingIsFun]

*Very helpful, Arjan! Thank you!* 👍

[@ArjanCodes]

Thanks so much, glad the content is helpful!

[@georgH]

Eiffel was used in some classes back in the day at my uni, it really helped me on understanding OO, it was so much nicer than Java (which didn't even have generics back then). Bertrand Meyer named an engineer's language on a great French engineer: Gustave Eiffel. By this rule, Python might have been called "Tideman" or "Lely"

[@aashayamballi]

Thanks for the great video again Arjan! It was really informative and helpful🙏

[@Mike-vj8do]

Enjoyed the video!

[@ArjanCodes]

Glad to hear it!

[@mohammadsharara3170]

Very juicy video, and at the same time funny, especially the joke about PyQt 😂. Thanks!

[@ArjanCodes]

I'm happy you enjoyed the video!

[@nickeldan]

The sound of your keyboard is so soothing.

[@Viertelhund]

13:45 I wished you'd have moved the validate_card method into the Card class as validate method without a parameter.

[@ivannuzhyn4668]

Awsome video, a lot of very usefull info for me. Thanks a lot!

[@ArjanCodes]

Thanks so much Ivan, glad the content is helpful!

[@EAenergy]

The Dutch guy said order salad instead of fries Holly crap haha. The first time I visited Holland my culture shock was how much fries is consumed. I did not connect the Irish historical famine with the current reality. Love your style and had a blast in Amsterdam. Keep up the good work and thank you for putting these together!

[@maortizm99]

What's the advantage of using partials over lambdas? Is it just about readability?

[@Vijay-Yarramsetty]

thanks a lot. great content

[@evgenyocean426]

Serious question: How can you stick to the default vscode theme for so long? It doesn't not even highlight function arguments in the function body. Is it for video recording only or you do actually use it on a daily basis? BTW. Love your content. Thank you!

[@ErikS-]

5:45 - 😂 Arjan is spot on here!

[@philscosta]

Great video as always! I also love partial functions in Python. One issue I have with them though, is that the type checker in Pycharm doesn't really know what to do with them. For e.g. def sum(a: int, b: int) -> int: .... sum_2 = partial(sum, b=2) The type checker shows and error when it checks whether sum_2 is of type Callable[[int], int]. Does anybody else have this issue? What do you all do about it?

[@NostraDavid2]

What error does it show?

[@CoilYsum]

Why do you use validate_card(card) and not validate_card(alice.card) in tip 4 (at around 14:00)?

[@wiktormorela4907]

Hey Arjan / youtube community. every time i watch those videos, i have a feeling that applying this knowledge into my data engineering python tools would be an overkill. Can you recommend something like levels of complexity in python code, where one could judge design quality of code based on those levels? As an example, how long one should stay in single module project, what is the moment when design should consider dataclasses etc.

[@Nellak2011]

So on the first example, the luhn checksum one, it is ok to leave the functions all inline IF AND ONLY IF you know that the requirements wont change and you will just use it in that one place. Other than that it is a bad practice as it leads to less code coherence and more coupling, which makes thing hard to change. The few advantages to inline are efficiency and faster to write, but in a large software in prod, the drawbacks outweight the trivial advantages. In general, nearly always extract out to make your code more modular and easier to read, it is just common sense.

[@zknarc]

For data related code, how do you deal with passing dataframes to functions? Should you select only the colums which are needed or pass the whole thing?

[@NostraDavid2]

I would pass the whole thing, as objects are passed by reference instead of pass by value. If you only pass the bits you need, you're likely to copy a bunch of data unnecessary.

[@doctor6932]

loved this

[@ArjanCodes]

Thanks so much, glad you liked it!

[@tesszheng4586]

at minute 5, for this piece of code: for digit in even_digits: checksum += sum(digits_of(str(digit * 2))) , I feel like we could write: checksum += sum(even_digits) * 2, thus we don't need to enumerate each even_digit, times 2 , convert to string ten convert back to an int list, then sum up.

[@DS-tj2tu]

Thank you!

[@ArjanCodes]

Thanks so much, glad you liked it!

[@irenaronen7497]

Thank for the video! I also seek how to use functions within the dataframes, namely in Polars, and how to incorporate it in DuckDB (using SQL in Polars dataframes). Working with alot of data is a bit frustrating, and I seek ways to use real-case data analyses. Also wonder how to use the Python Class system in the above case... Don't want to overload the system with too many variables.

[@whkoh7619]

Amazing content as always. Thank you Arjan

[@ArjanCodes]

Thanks so much, glad the content is helpful!

[@ervankurniawan9267]

The only thing I just want to say is, since I found your channel and has watched some videos, I never look Python is easy anymore. And every time I look at my current Python code, it's looks like a dumb code. 😂 Thanks for the content. 👏🏻

[@David-124]

Great video! At 23:14 and 23:29 did you mean to say "Parameter" rather than "Argument"? 😱

[@aperturesignaturebandwidth]

@12:05 you seem to indicate we're running validate_card on the CardInfo type but you pass the Customer?

[@mr.wheelers5504]

Hi Arjan! Great video as always, I feel like I've learned more about python programming from you than I have during my astronomy studies. At 16:45 in the video you create a PaymentHandler baseclass using a Protocal. Howcome you don't use an AbstractBaseClass for this? Using a protocal like this seems to ignore the "better explicit than implicit" principle. Is there some advantage to using a protocal like this?

[@gedtoon6451]

At 13:04 you created a "Card" class. If you have a class representing a credit card, would it not be sensible to have a "validate_card" method inside it?

[@leniedor733]

1. SRP from the S of SOLID + some good refactor examples

[@mithunmanoharmithun]

Awesome content. Keep going !

[@ArjanCodes]

Thanks so much Mithun, glad the content is helpful!

[@paxdriver]

I just realized Arjan has 10% the subcribers I just assumed the channel had from watching for so long. I honestly believed for years this was a 1 million+ channel and only just now realized it wasn't. I need to comment and share more, this is my fault.

[@ArjanCodes]

Thank you so much for your support Kristopher!

[@Gukzilla]

As a self taught, these design principles are really that one thing that I felt is missing to pull of brilliant code.

[@ArjanCodes]

Thanks so much Gukzilla, glad the content is helpful!

[@luck3949]

Flags argument seen to be stretched: N flags represent 2^N functions. Flags might be written in configuration file, so using them in some form or another might be necessary. Splitting one function into many just to remove flag might lead to code repetition.

[@tolstoievski4926]

Is the protocol class cardinfo still relevant after creating a separate data class for the card ?

[@confuso1]

Very good video, thanks a lot, instant abo

[@ArjanCodes]

Glad you liked it!

[@teddy930285]

I like how you grumbled random stuffs like the paypal and naming of Python. That just makes the video so much entertaining and fun to watch, while still being extremely informative. Thank you!

[@tadastadux]

Thank you Arjan. I'm just wondering if you swapped ABC with Protocal libraries.

[@kristianhig3283]

thanks it worked

[@ArjanCodes]

Thanks so much Kristian, glad the content is helpful!

[@CFox.7]

The mind of the coder - they think they are creative but they are not they are just master organisers ! and no you dont need to be creative to be organised - its just a matter of finding the optimum for the goal ( within constraints of course ). Choose your focus.

[@nfredrik]

what is the diff btw 'def validate_card(card: CardInfo)' and writing 'def validate_card(card: Card)' Is it syntactic sugar? It do not to scope down number of props passed the validate_card() when executing, property 'valid' still there.

[@marcelgrolain2764]

Thanks for the inspiration

[@MagnusAnand]

Arjan, I have one question not related to this video: working with an ORM (like Django's), how do you use the SOLID principles? If you ever talk about this it would be so cool 😜

[@mikefochtman7164]

One thought that popped up as I watched, couldn't we move the date checking into its own function? I know the date checking is pretty trivial, one line and all, but "do one thing, and do it well". So 'validate-card' would simply be " return validate_number and validate_date;" If later we wanted to do something more, such as 'validate-ccv' that too could be added in a similar manner. Having a separate date validation would help compartmentalize things such that some future changes to 'date checking' are localized to one function. For example, most CC here in the US are valid through the END of the month and don't expire on the first. Looks there like your CC is not valid on the first day of the exp_month?? How many places in all the code have to be searched to fix such a bug if you do date-checking in-line everywhere??

[@AnotherAvaibleName]

thanks Arjan, very nice and based

[@ArjanCodes]

Thanks so much, glad the content is helpful!

[@mikeciul8599]

In other videos, I recall you've advised against including types in variable names. Maybe it doesn't apply here because it's a type variable, but when you called the type HandlePaymentFn I wondered about the naming scheme. When I name function types, I usually put "er" at the end of the name. In this case the function type would have the same name as the class it replaced, "PaymentHandler." I probably would have called the param "handle_payment" rather than "payment_handler" though. e.g.: PaymentHandler = Callable[[int], None] def order_food(items: List[str], handle_payment: PaymentHandler) -> None: ... What are your thoughts? How does everyone here name their Callable type variables?

[@lunkums]

7:48 not a big fan of this tip because of the data clumps code smell. In general, if you're passing in a few members of a class, you really should be passing the whole class because it's clearer than passing individual arguments. It's not immediately apparent to someone reading the code how those arguments are related, unless they're named similarly, and if you end up needing more data members from the Customer class, you need to update the method signature. Also, if you're performing an operation using a customers data, if you pass in the customer then you're ensuring that the operation will only use the customers data

[@pitter6636]

How would you proceed in creating functions for dataframes?

[@kovlabs]

Could you do a deep dive on building your own modules Please 🙏

[@ArjanCodes]

Noted!

[@MrHarleking]

Hi, i love your Videos, but please use a zoom on you screen when u show and work with Code. Thx a lot for your work!

[@pointer333]

I don't understand the card info protocol... Was that left as an incomplete example, or is the class declared completely, for use as a protocol?

[@walis85300]

Thanks for the video. One question: in the protocol class, is it required to mark as properties the class’ attributes or it can be declared as attributes?

[@Shivnaren]

Marking the attributes as properties is the only way to declare them as immutable (useful for NamedTuple instances, or just a `const` like declaration); but they can be declared as attributes too.

[@mikeciul8599]

What is the name of the code smell where an object is created and used in the same place?

[@cgi_arb]

What is the benefit of creating the "digits_of" internal function within "luhn_checksum"? Why not just digits = [int(d) for d in card_number]?

[@grzegorzryznar5101]

The biggest problem with the partial is the fact, that none of the IDE or even mypy cannot support it. So function created via that is totally ambigous without using something like Protocol to annotate return value. Another problem with funcional protocols is the fact, that you are missing annotation if you annoate it with Callable. Better to use Protocol in more advanced scenarios with annotated __call__.

[@ran_red]

But couldn't you just annotate it as functools.partial? like only_partials: functools.partial Pyright complains if I do that and assign it a Literal, as an example. Sorry if I am not understanding your message and suggested something stupid btw

[@MagnusAnand]

One question not related to this video: How to solve the relational impedance mismatch? If I use the solid principles to design a payment system (like you did in a different video), an employee object, contrast object, commission object. You use interfaces, composition and inheritance. But how to reconcile this with the database??? I’m quite confused. Thanks.