Composition Is Better Than Inheritance in Python

[@ArjanCodes]

I'm curious if it helps the click rate if I add a background image with money to the thumbnail ;). Several times in the video, I relate back to concepts like cohesion and coupling. I did a separate video about that as well a while ago: kzbin.info/www/bejne/m5qnqn6VpMuDhq8

[@AndresTalks]

Nice inheritance pun! I actually didn't saw that nice detail, but could also be due to the fact that whenever I see arjanCodes new video upload I insta-click it and smash the like button! have a good weekend Arjan!

[@ArjanCodes]

Thank you Andrés, glad to be of service and wish you a great weekend too.

[@iluvsyphonfilter]

To be honest I didn't notice the money in the thumbnail before reading this comment

[@python360]

Clickbait-ey thumbnails suck, I tried using them but on balance, it's probably better not to drop to the level of the creators who do?

[@v0id_d3m0n]

Did it work?

[@AzureFlash]

Counter-argument: I'd rather get an inheritance from a deceased relative than a composition, no matter how good at poetry they were.

[@anhnhvn]

Why not both? You can inherit a bunch of compositions :)?

[@nomadvagabond1263]

@@anhnhvn that sucks haha

[@Azurath100]

Depending on if they were popular, the compositions could be worth a great fortune.

[@aarondewindt]

@@Azurath100 Not from my relatives.

[@izzikora6751]

That's cute

[@MichaelFJ1969]

I really like how you make small incremental changes, and run/test the code after each iteration.

[@ArjanCodes]

Glad you like the video, Michael!

[@RyuhoKudo]

@@ArjanCodes It would have been better if it was a unit test :)

[@dansplain2393]

That is basically the best way to code. Small change. Review. Commit.

[@ShlomoGottlieb]

I love it that your videos are somewhat advanced, there are not a lot of these on KZbin. Keep up the great work!

[@cristobaljvp]

Man your videos are so high quality! Sadly I don't think you're gonna grow fast 'cause advanced topics have a smaller audience. I really hope you keep the videos coming anyway. Thanks for the content!

[@ArjanCodes]

Thanks - happy you like them! Fast growth is not my goal, to be honest. This is a topic I really like talking about, and already with my current subscriber number, I’m able to help way more people than when teaching a programming course at my local university. So that’s a win-win in my book.

[@cabanford]

@@ArjanCodes What a great answer and attitude.

[@NateROCKS112]

To be honest, slightly more niche videos actually make channels blow up quicker.

[@VivekYadav-ds8oz]

This is not that advanced. This is something every programmer is supposed to research and know about.

[@RoamingAdhocrat]

@@VivekYadav-ds8oz So, going forward, every new programmer who reaches this stage and searches "composition vs inheritance" is likely to find this video.

[@michaelyost9820]

Been listening to the audio book of a Pragmatic Programmer and in the book they emphasize that "has a" is a lot better than "is a". Actually doing it in code is a different story. Seeing it here it makes a ton of sense! Thanks for the vid!

[@ArjanCodes]

Thank you Michael, glad it was helpful!

[@freacas]

Would love to see some creational patterns as the continuation of this video when that main() gets crowded because there are many parts to create before instantiating The Employee or when you need to decide which Employee to create during runtime. P.S. Please keep up this great work! In python you have either hello_world() tutorials or data science ML. Your videos are like a fresh air in Python world! ;]

[@ArjanCodes]

Thanks, glad you enjoy the videos! It seems you have been reading my mind. I have just finished recording two videos about object creation, including a video about the factory pattern.

[@MrCmon113]

Lesson 1: Hello world. Lesson 2: Using a three billion parameter transformer to optimize the hyperparameters in our novel PDE solver that's going to help us predict the temperature of my left butt cheek on the Sunday three weeks in the future.

[@69k_gold]

It's important to understand the tradeoff between code deduplication and the speed of incremental code deduplication. Inheritance is useful when there is a clear vertical heirarchial pattern, where the top level parent appears in a lot of levels below Composition is useful in a horizontal hierarchy, where there are a lot of subclasses at a single level and the odds of one of them changing in the future is high So the best practice is a combination of both

[@WthyrBendragon]

I generally find it helpful to approach my Object model from an RDBMS mindset. The Employee class might be better treated as a Personnel record containing only that information needed to identify a person. That person's relationship to the company, pay structure, and other factors are defined by other related data structures.

[@Graham_Wideman]

Yes, exactly this. Most of the problem with this example is that the original Employee class was used to contain pay records, (and presumably just for a particular time period, which is doubly mad.) Combining two data structures where the individual data structures have their own variants of course results in a combinatorial explosion. That's not the fault of inheritance per se, it's incorrectly modeling the problem domain.

[@mauisam1]

Thank you! Finally a more advance level of Python programming and specifically dealing with classes. A lot of good information, I'll need to watch this again! I just subscribed!

[@ArjanCodes]

Thank you Sam, glad you liked it!

[@reddit7372]

OOP composition has not been covered enough, so thank you for creating this video!

[@ArjanCodes]

You’re most welcome!

[@laurentverge5512]

Very nice content ! One more thing you could add to this codebase is a NoCommission class that returns 0 when get_payment is called. With this class there is no need for "Optional Commission" in the Employee class and the concept of "there is no commission" has now a concrete representation.

[@esra_erimez]

The importance of this video simply cannot be overstated. Well done. Programmers embraced inheritance with reckless abandon.

[@ArjanCodes]

Thank you!

[@utube999ify]

I am a tester by experience and looked at coding with that mindset. I am also old school procedural methods (think Basic with sub-routines or PL/SQL). When object oriented programming first hit I tried to follow the approach for inheritance in the book to mimic the complex applications I tested on. We went too deep with inheritance (as the examples in the early books used multiple layers), so it tainted my experience and I left it behind as too difficult for me to understand when dealing with large data volumes in structured databases. Nowadays, I am more open minded. I have more time on my hands to research and experience the design methods and changes in coding practice. I find your videos has taken the hard learning and presents the knowledge in a straight forward practical manner.

[@amitsinghsansoya4038]

Man! you are great with your videos, most of the people really don't include the incremental changes while making the code and sometimes it adds a lot of complexity when you are studying way complex things but in your video things start to make much sense as we do see the changes happening and it helps in understanding the stuff very well. Really awesome videos!!

[@SaurabhSingh1987]

After appearing in so many coding interviews, now I understand the market expectation is highly improved for real hands-on even for lesser experience. great to have someone doing hands-on on screen and keeping us motivated!!

[@nianeddavid7985]

the only video I really understood. It is important to know the real problem before thinking how to write the code.

[@sachinjogi1995]

Very well explained 😀! It'll be great if in future you also consider building complex application following the design principles. Something like Twitter clone etc, where you can understand how to think about software architecture, oop design etc in complex applications.

[@ArjanCodes]

Thank you very much for the suggestion!

[@Wahlnetwork]

Fantastic content, clear explanations, and great editing. Subscribed!

[@ArjanCodes]

Thank you so much! And congrats on your own channel, those are really nice topics you cover.

[@izvarzone]

why do ppl always write that?

[@noimnotnice]

I discovered this principle without knowing its name in my testing of automated trading strategies. Every building block naturally lends itself to be derived from a base strategy (e.g. setting stop losses). All strategies are then assembled as compositions by various building blocks. The key is that both inheritance and composition have their utility for different purposes.

[@drooten]

Thank you for yet another quality topic while also getting to the point in ways that make sense and is easy to follow. I particularly like the code zooming you do, as it means I can watch these while on the bus to/from work, on my small phone.

[@Elite7555]

3:05 That's a perfect example for mixins: shared behaviour without class hirachy.

[@magenertech9412]

Great video. For those who want a simple sentence to sum up composition - Composition is dividing different class responsibilities to abstract interfaces stored inside the class as opposed to defining these responsibilities through inheritance.

[@pope-pylinux-vi]

Thank you for such an excellent content. Wish I could give you multiple likes. A special thank you for the timestamps in your videos, it really makes things easier. You are great!

[@ArjanCodes]

Thank you! Glad you enjoy the videos!

[@AdobadoFantastico]

Thanks I've been programming for a while and kind of understood but still wondered a lot about this. Your vid really elucidated things for me. Best explanation I've seen, 10/10 will share any time the topic comes up.

[@maroben225]

Watching you is intellectually stimulating....what a relief to find such an amazing and ADVANCED content

[@henrikolsen5]

Great content, presentation and personality. Solid content. Your students are lucky to have you, and we here are too - thanks again for making this. Much appreciated. Feels like actually studying again instead of more or less random quality Python content.

[@ArjanCodes]

I appreciate that!

[@jackbotman]

The composition approach effectively forces you to simplify your objects into more reusable units Which is nice

[@moizmansur]

Arjan, I love your videos. These are things I've always wanted to read from the books but they seem too intimidating. The way you explain them make them very understandable in a language I'm familiar with so thank you so much :)

[@ArjanCodes]

Thanks! Happy that you like the videos!

[@jonathankirkegaard2784]

Good example. It's refreshing to see a little more detail than the usual "Shape" examples

[@ArjanCodes]

Thanks Jonathan, glad you enjoyed it.

[@GrimChu69]

I THINK the Composition style is the way I'm currently using. It feels like the most logical design pattern, inheritance feels like you need to complicate stuff unnecessary often times.

[@AbhirupMishra]

I'm not sure that it would be a very bad design or not, but it would it be interesting to see if Multiple Inheritance (Mixins) were used to solve this problem. What are your thoughts on it?

[@dimatrushin8516]

An inheritance without abstract methods (or virtual functions if you wish) is a different form of a composition it is a "horizontal"composition. The main difference is the following: 1) In case A is a member of B you create a hierarchy in the interface of B: x = B() x.method_of_B() x.a.method_of_A() 2) In case A is a base class of B you merge the two interfaces together: x = B() x.method_of_B() x.method_of_A() So you can do something like that def EmployeeType(PersonalData, Contract, Commission): class Employee(PersonalData, Contract, Commission): ## implementation of the class return Employee to create a specific type of Employee and then use type alias Employee1 = EmployeeType(PersonalData1, Contract1, Commission1) to use it in a convenient way like that dude = Employee1() However, all the methods of PersonalData, Contract, and Commission will be available in Employee interface. If this is meaningful and what you want, this is a right solution. If the interface of Employee has excess methods, then you should not do this, because "horizontal" composition means that you really do want to merge the interfaces together. This technique is called "Policy based design". It is useful when you want to construct a class with an interface merged from several independent ones and you want to control these peaces.

[@flam1ngicecream]

Really cool. As a C++ nerd, my brain was shouting "multiple inheritance!" but now I can use this method instead when I'm in a language like Java that forbids it.

[@paul_devos]

This is the best video I've seen on the tradeoffs of Inheritance vs Composition. One question I'd have is could you manage the explosion of subclasses better with multiple inheritance? Perhaps that's just a nomenclature thing as you essentially have an explosion of parent (super) classes. Note: Most of my experience is for Data Engineering and web scraping. Where I might have a "data container" of attributes (10-12) I want as defaults for say, an http request, where each instance or new request adds or updates 5 new ones (e.g. API endpoint, persistence location, database destination, filename, etc) and then I add a few different methods if the user wants to add a 2nd API call to another endpoint to modify/enrich the current "state" and then persist. In function this would be similar to querying two database tables (via http requests), then joining/mapping the data, then persisting to a destination. For this scenario, I hadn't really considered composition altho the way you presented it I was using some of that "design" in that I use multiple parents with ABCs to get a subclass that is ultimately used. I had mostly hemmed and hawed on whether to use functions (passing in a large dictionary of 10-15 attributes or having 10-15 defaults) instead of classes. I'm still not sure I have done the design correctly. And I think the main problem I have is my final package seems where I am scraping a website and have 80+ API endpoints where 60 or so are more "functional" in design -- while 20 API endpoints need two different API calls for that sort of "join" to another data set (using a class here vs two function calls, persist in memory, join via say Pandas merge, then persist). So wanting to make it one package as it's for consumption of a single package so should it be composed of all classes? functions without classes? And the big thing is helping users with the default of all those default URL parameters (8-15 per HTTP request) and how to best manage that as each API call does have that many (or more) parameters needed to get a data request back. I will be watching more of your videos to see if you already have the right answer to this question. Thanks again for this content. Very helpful.

[@ArjanCodes]

Hi Paul, glad you enjoyed the video! I would strongly advise against using multiple inheritance, as it leads to all kinds of issues, the most well-known one is the Diamond of Death (see en.wikipedia.org/wiki/Multiple_inheritance#The_diamond_problem). I found I can solve most of my design problems with composition and careful use of inheritance, mainly in combination with abstract classes.

[@paul_devos]

Thank you@@ArjanCodes will definitely work on becoming better at implementing composition instead.

[@yd_]

Thanks for these videos, I think these are very important resources, especially in academia, where graduate students from non-computational background usually self-teach programming for data analysis, modelling, etc. As such, it's often the case that they write code that performs well, but isn't readable or extensible.

[@ArjanCodes]

You're very welcome!

[@noblessetech]

I love your typing speed and sound of it. Amazing content. Subscribed!!

[@rbettsx]

I think a fairer title would have been 'Composition is better than the incorrect use of inheritance'. Which is hardly a shocking conclusion 🙂 The components of their pay packages are not attributes of employees.

[@ruszkait]

Very nice explanation indeed. Two remarks: for testing you can use a unit test instead of just looking at the output - this would set a good example. The other comment: maybe you can show how you can use the decorator pattern to build up class hierarchies in runtime and override behavior. That would be simpler than the composition, because you do not have to know in your base class that you have a contract and an optional comission, but it would be much more flexible than the inheritance, because you can add up the layers of decoration in runtime (similar as you have done the dependency injection in runtime)

[@ArjanCodes]

Thank you, glad you liked it! I try to strike a balance between using best practices in my examples while keeping things simple. That’s why I generally leave unit tests out of the examples if that’s not the focus. The decorator pattern is indeed a good suggestion for a video (I think it’s already somewhere on my list).

[@floodu17]

Definetely in love with your video. I am a beginner and I am new to OOP, I was trying to figure out how organise my classes in the programm I am currently so I clicked on your video by curiosity. You really well- explained it (timing, choice of words). Thank you very much for your content.

[@ArjanCodes]

Thank you too!

[@edwardkent1503]

Great explanation! I would say the "favour composition over inheritance" concept is applicable to many other languages too. In my opinion, you're the stackoverflow hero of the youtube world! 😄

[@ArjanCodes]

Wow, thank you!

[@kennethgee2004]

I am revisiting this as this is an important concept. I notice though that the inheritance you set up is not how it is done. The compute pay should be its own class. You would then inherit from multiple base classes. This is true OOP under C/C++. Unfortunately, most languages today only allow for inheriting from one base class. This is because so many people cannot get the inheritance setup up correctly. Composition makes it a lot easier when it comes to data classes like you have been using. When it comes to behavioral things like building a window system, then inheritance is the way to go.

[@evadeflow]

It really caught my eye when you inherited from `ABC`, but also used the `dataclass` decorator. What sorcery is this?? I've clearly missed some recent developments in Python. Any chance you have vids/links that explain why one would do this? Also... your videos absolutely rock, man. I dunno if they hit other people's sweet spot for relevance to their own place within the IT ecosystem, but... your content is super-relevant to my own little corner of this crazy space we work in. Thank you!

[@mystisification]

Would you mind explaining why we couldn't use a Mixin here rather than composition?

[@jampk24]

This channel has been a game changer for my coding skills.

[@ArjanCodes]

Thank you!

[@mario7501]

I think it would be even neater to use lambdas to define contracts and commissions. Then pass them as parameters which get called by the Base class

[@TechandCodeInGreek]

This channel is a treasure for advanced python! Thanks!

[@Operaatoors]

I know 100% for sure that this was not scripted, but I really love to see you struggle to find the correct place on 8:02 :D Very good "bad" example :) I had a feeling that this might come up but I didn't know if you really wanted to force your self in that stupid situation. I also did similar thing displaying strugle with WET code to my students, and I was suffering a lot :D

[@wlcrutch]

I have been having my first dabblings in OO with java, coming from a mathematical/functional background. While I understand what inheritance IS, I haven’t ever been a fan of using it, mostly for the reasons you mention. It just isn’t how my brain works, so it’s nice to see this and feel validated 😃

[@izvarzone]

but you still need it sometimes. Use both inheritance and composition, not only composition, where it make sense.

[@izvarzone]

For example LinkGun would be Actor -> Inventory -> Weapon - > LinkGun with inheritance but then LinkGun would have many components of which it's made. In case you'd want make hybrid of Minigun and Link gun later, using components of both. Actor, Inventory and Weapon are abstract classes.

[@fexofenadinaGenerica]

This is so great. Thank you for the explantation! As I aspiring software developer this kinds of videos are very important.

[@michaelhoffman8468]

This is probably the best and most important concept video regardless of the language is so critical for new programmers to learn.

[@ArjanCodes]

Thanks, glad you liked it!

[@robertsjoblom9279]

I would perhaps add a NullCommission class instead of making the commission optional with None; it would clean up the code a little bit, since you wouldn't need to check for nulls

[@ArjanCodes]

Good suggestion!

[@DonGioification]

Very happy I stumbled on your videos. I’m just finishing up my three year University course in software engineering and your videos are very interesting for me. Keep up the awesome work !

[@ArjanCodes]

Congrats on finishing your university degree, and thanks - will definitely keep it up!

[@rodi4850]

the second option is basically combining both composition and inheritance. depending on how you conceptualize your classes you can reduce the combinatorial number of cases.

[@andyanderson222]

Great explanation! I hope one day my code will become as clean as every example at this channel) Thanks, Arjan! Your videos helps a lot!

[@ArjanCodes]

Thank you so much, Andy!

[@MateHegyhati]

I know, this is just an example, but the redundancy issue you mentioned for yearly bonus, etc. can be solved easily by multiple inheritance I think. I'm still not saying, that this should be a way to go for this situation. I feel, that inheritance has more reason for statically typed languages. In a duck typed language that story is different. Also, it probably matters, if inheritance means "extension" or "specification". Not arguing, just sharing thoughts. It is actually really interesting to see, how different languages address these different issues. I'm more of a C++ guy, and when people ask, why C++ is difficult, I usually answer that it is not because of memory and stuff. It is because you have (multiple) inheritance, composition, templates, concepts, crazy things like CRTP. And the difficult thing to know is, which is better used for a certain problem. As always, thanks for the vid.

[@MateHegyhati]

Another thought. If you have n of these features, and that gives 2^n different "types", then an important question is if you are going to use all of them, or you know, that only 3 specific types will be ever used. Maybe 2 more added later. The best applied design probably depends on this. Also, it is rather important if you "depend" on the parent functions (like in your example) or not (like personal data mentioned in the end). Inheritance is probably not such a problem in the second case. The advantage of inheritance in my eyes is that it also "copies" the interface of a class. If I do composition and want to make that functionality public, I have to write the function that calls the function of the component. If no additional logic is added, it is just a boilerplate code. My python background is not the best, maybe there is a @delegate decorator or something like traits in PHP. But I'm curious on your opinion.

[@ishtiaquehussain]

This is super helpful! Learned a lot, thank you! I'm going to recommend your channel to my network. Keep up the good work!

[@ArjanCodes]

Awesome, thank you!

[@keepitsimple01]

It's good simple explanation, though, I have a question when you said there are two issues with the design. 1. It has duplication like commission, contracts landed which is correct and we are violating DRY. 2. The class has lot of responsibility, like storing personal data, but isn't the class has only one responsibility which is to compute pay. It has no other behavior.

[@Christina__V]

Found your video as i was looking for the basics. Bit too advanced for me but this is solid content. Keep up the good work

[@learn9475]

Thanks for the video, i have watched many videos on this topic but didnt get the clarity as to why use composition but this video has a clean and neat explanation

[@ArjanCodes]

Thank you - glad it was helpful to you!

[@nyashachiroro2531]

One of the best videos on inheritance vs composition. I'm definitely going to try and port this knowledge to Golang since it's big on composition and I can see why now. It's now very very cleary.

[@ArjanCodes]

Thanks!

[@ddctechinstitute6861]

The statement is pretty clear, FAVOUR COMPOSITION OVER INHERITANCE, it doesn't say do away with inheritance completely as they're cases where inheritance is most suitable. Also don't forget that most inheritance cases can be modeled as composition. 💕

[@kaninchengaming-inactive-6529]

I was skeptical at first but I think you convinced me

[@JohnDoe-wq9pr]

These videos are great reminders of concepts that were learned then long forgotten, due to various circumstances. I actually learned about design patterns in university using the same Design Patterns book by Grady Booch (so glad I decided to keep it). I just had a couple of thoughts for this type of comparison video for your consideration, as follows: - instead of changing the code that is already there, maybe have a side by side view that will allow the new code to be seen against the previous version, for improved clarity on the differences between the two - it would be nice to see a diagram that visually shows the structural difference between them Take these suggestions as you see fit. I look forward to seeing more of your videos either way.

[@ArjanCodes]

Hi John, glad that you like the videos and thank you for your suggestions!

[@engcisco]

Thank you, your videos are direct, simple and to the point. great explanation, I really enjoy your videos

[@ArjanCodes]

Thanks Baher, happy you’re enjoying the content!

[@dennissmith6867]

Crystal clear examples and explanations. Love your videos Arjan!

[@ArjanCodes]

Glad you like ‘em Dennis!

[@TimWackrow]

Would be great to include an example of aggregation vs composition

[@ravirockz7958]

You are just awesome. It really helpful to know more about the advanced topics on python. Please Keep sharing

[@OMGclueless]

I'm curious, is there a good reason to have separate abstract base classes for Commissions and for Contracts? They are both defined identically by the end of the refactoring, and you even call the concrete commissions class ContractCommission which is a mighty confusing name given that its implementation is independent of Contract. It seems to me like there should be only one type of abstract class, a Contract. And Employee, rather than taking a single Contract + optional Commission, can just take an arbitrary collection of Contracts and sum up their pays.

[@imadetheuniverse4fun]

But even though they have similar interfaces, they are fundamentally different things. For example, it wouldn't make any sense for an Employee to have an arbitrary collections of Contracts (what does it mean for an Employee to have both an HourlyContract and a SalariedContract?), but it may make sense to have an Employee with an HourlyContract and an arbitrary collection of Commissions. I would say that's enough reason to keep them as separate classes. But definitely adding a collection of Commissions to Employee could make sense, and maybe improve naming between Contract and Commission to avoid confusion.

[@AWriterWandering]

The Go language is sometimes criticized over its lack of support for inheritance. However after watching this video, I believe the language's developers were wise to focus on composition. (with interfaces to handle abstract methods)

[@leeseoWestport]

I’m having a hard time understanding how coupling is reduced because the compositions are now your coupling. I can only think that this example reduces duplication of logic between the multiple branches of inheritance, which is very cool in itself. Again, I believe coupling complexity hasn’t been reduced. In fact, the components are shared between more classes; thus, I feel the coupling is stronger and the code duplication is diminished.

[@izvarzone]

but if they're soft references, components only loaded when class instanciated. Same soft reference wont load 2 times.

[@filippobuonco95]

Thanks a lot for this amazing work! You are literally helping me reaching the next level of Python code!

[@ArjanCodes]

I'm happy to hear that my content has been helpful, Filipo!

[@matrixtoogood5601]

Instead of inheriting broad classes from a single ABC create multiple smaller ABCs encapsulating some data and use a broader class to encapsulate specific implementations of these smaller ABCs

[@smann43231816]

The best Python videos I've seen so far. Thank you!

[@ArjanCodes]

Glad you like them, Steve!

[@nato.musica]

Arjan your channel is just great! Thanks for this material :)

[@ArjanCodes]

Thanks so much, glad it was helpful!

[@CounterBarry]

Thank you for this video. In a simple and understandable way, you have explained the concept of Composition.

[@ArjanCodes]

Glad it was helpful!

[@edgeeffect]

It's a rare opportunity to say "we've got a good thing in PHP", so I'll go for it... We've got a good thing in PHP, we have `interfaces` which are basically the same as abstract base classes - but like "he" says in PEP 20: "Explicit is better than implicit".

[@cfossto]

Here I was thinking I was good at Python. Thank you for this! Python have become fun again thanks to this video.

[@ArjanCodes]

Great to hear!

[@cetilly]

Another brilliant video. Thanks Arjan! I just love your content.

[@ArjanCodes]

Thanks Chuck, always nice to hear from you and glad that you like the content!

[@oncedidactic]

@ArjanCodes clicked because of the cute thumbnail, and the great content ;) Halfway through, just wanted to comment- watching the inheritance demo gave me an increasingly nauseated feeling. Looking forward to cleanliness of composition. I find often, with a modicum of intuition in the domain, you can really follow your gut about whether you're using a good pattern for a given objective.

[@ArjanCodes]

Thanks, glad you like it! You're right - the intuition is something you develop over time as you gain more experience. Often, I find I don't explicitly think about design patterns or principles anymore, they just "emerge" when I write code.

[@oncedidactic]

@@ArjanCodes That's really cool to hear. Like great musicians or athletes who make it look natural. It is definitely a skill and an art to give clear and concise explanations of the concepts though! Satisfying and elucidating to watch, even when you sort of know where it's going already. Big ups for that! :)

[@shadhumydee9730]

Hey, Arjan! Isn't this an example of Aggregation instead of Composition since the object creation isn't a responsibility of the classes that are using it rather is used by passing references? Just asking, I am learning Aggregation vs Composition very recently. I can be wrong. BTW, Huge Fan!

[@gremblexyz]

This is also a very useful use case for the structural subtyping stuff (Protocols).

[@ArjanCodes]

Thanks so much, glad the content is helpful!

[@finnjensen9002]

You also could have created a Payment Interface which the Employee could retrieve an Array of implementations of. GetPayment would then iterate over all payment implementions and add up the payment. This would remove the necessity of Optional and open up for more payment options.

[@lbb2rfarangkiinok]

Really good content, thnx so much for your hard work on the videos.

[@baloothebeardogretreiver8419]

Best video on this topic I've seen.

[@ArjanCodes]

Glad you think so!

[@radeksmola3422]

Hi Arjan, great video by the way. One question at you: What is the difference between composition in this video and dependency injection?

[@Protocycle]

from what I understood, inheritance is relatively impossible to get rid of if you want to write better, scalable code... BUT it should only be used for abstract classes/interfaces and then combine it with composition whenever possible to avoid creating long inheritance hierarchies (as well as coupling)?

[@bartosak]

We could even add more flexibility creating a Protocol "GetPaymentProtocol" and employee could have list of that protocol so in compute_pay we could use simple sum of all protocol based classes

[@pablonapan4698]

Question: What was the reasoning for separating salaried employee class into with-commission and without-commission? The original class doesn't show that there are employees with and without commission

[@jhonshephard921]

it looks like the reason it was better in this case was because it turned out you had to make several derived classes with duplicated code, when you updated it you still used inheritance but just over a different class. The example and video are still great however this looks like something you will find out when refactoring and seeing duplicate code not when originally writing the code.

[@f4bglv]

Excellent tutorial, very well done and directed. Thanks a lot!

[@ArjanCodes]

Thank you Fabio, glad you liked the video!

[@reefhound9902]

I will get flamed by the purists but while the original code had duplication and separation issues, it was also extremely simple for anyone to follow and understand exactly what was happening with each employee without bouncing around references, which are most likely going to be in various files.

[@ethanlee9633]

Composition combined with inheritance = mind blown

[@yomajo]

In slight defense of inheritance: if Arjan had not used abstract classes, but Commision class (with 3 different methods for example), code duplicaion would have been minimal, no?

[@dimas-]

You don't need ‘is not None’ check. Without it it simply becomes ‘if commission:’.

[@ArjanCodes]

Correct, that’s a nice, short way to do this as well. I’m always careful with truthy/falsy values - been bitten by that a couple of times in the past.

[@schoolmonkey13]

"Explicit is better than implicit." Relying on just "if commission" could cause errors if for some reason commission might evaluate as false.

[@AJMansfield1]

The point of inheritance is to simplify implementing polymorphism. Using it for anything else is usually a code smell.

[@ArjanCodes]

Well said!

[@LucianoPinheiro78]

In 13:57, I would usually verify "if self.comission", not "if self.commision is not None". Is there a advantage in the longer version that I'm not aware? The diference is that in the shorter comission can be something, but also representing False.

[@jondahl3161]

Since self.commission is set to None by default, and since self.commission only affects the salary if the default is overwritten, then it is cleaner to check exactly for 'not the default value'. Hence 'if self.commission is not None'. Though I'd argue that it is even better to check 'if isinstance(self.commission, Commission)' since, strictly speaking, a commission can only be paid if self.commission is an instance of Commission. A commission cannot be guaranteed to be paid just because self.commission is not None. Then it could in principle be anything else than None.

[@danielj.rodriguez8621]

Thank you for your detailed example. The tip about thinking in terms of "is-a" vs. "has-a" really helped me grasp the advantage of composition vs. inheritance. Also, the link for the "Design Principles and Design Patterns" article by Robert Martin appears to be broken. Is it possible to access the article elsewhere? If so it will be helpful to update the link.

[@kanchanfriend123]

Great content. What was the keyboard you were using? Liked the sound of it.

[@natel2891]

thank you for many interesting videos. You got my sub! The videos on youtube about Python are either too basic or just blah blah import...ML...data science. Your content really stands out and I have learned so much about software dev standards that can be applied to other languages not necessarily only Python.

[@ArjanCodes]

Thank you Nate, glad the content is helpful to you,

[@fred.flintstone4099]

I know the difference between composition and inheritance and that I should favor composition. But should I *always* favor composition over inheritance? If not, when if ever should I use inheritance?