Angular Code Review Best Practices - Refactoring From Junior Level to Senior

Let's do an Angular code review together and refactor it to a senior level. I prepared real junior code that is not written in a best way and we need to improve it.
TIMESTAMPS
0:00 Introduction
0:12 What we have
3:19 Refactoring state
10:02 Refactoring list
14:13 Refactoring state change
18:58 Refactoring totals
20:43 Refactoring toggle all
24:03 Refactoring indeterminate
► CHECK MY COURSES - monsterlessons-academy.com/co...
MOST POPULAR COURSES
► Javascript interview questions - monsterlessons-academy.com/co...
► Typescript interview questions - monsterlessons-academy.com/co...
► React interview questions - monsterlessons-academy.com/co...
► Angular Interview Questions monsterlessons-academy.com/co...
► Building real project with Angular + NgRx - monsterlessons-academy.com/co...
► Building real NestJS API - monsterlessons-academy.com/co...
► Building real fullstack project - monsterlessons-academy.com/co...
► Mastering Git - monsterlessons-academy.com/co...
► Mastering Docker and Docker Compose - monsterlessons-academy.com/co...
FOLLOW ME
► TWITTER - / monster_lessons
► INSTAGRAM - / monsterlessonsacademy
► TIKTOK - / monsterlessonsacademy
REFERENCES
► Source code - github.com/monsterlessonsacad...

[@LunarWaffle]

Absolutely amazing man , been following since 2021 , still the same humble teacher!

[@MonsterlessonsAcademy]

Glad to hear that!

[@luischavez190]

This is pure gold!! Thanks

[@MonsterlessonsAcademy]

Thank you!

[@changcnchang8174]

This is very helpful thank you for sharing!

[@MonsterlessonsAcademy]

Glad it was helpful!

[@izacaqsha3480]

Awesome as always

[@MonsterlessonsAcademy]

Thank you so much 😀

[@harisali104]

Signals are so awesome. Thanks for the this informative video, will definitely be using signals more often now.

[@MonsterlessonsAcademy]

Glad it was helpful!

[@harisali104]

@@MonsterlessonsAcademy Just one thing I wanted to ask. In the video you converted the @Input() to signal. Is it also possible to have the issues property be a signal, eliminating the need for converting from @Input to signal?

[@LuizFMPaiva]

amazing!
U r awesome!
I always learn with u. Thank you

[@MonsterlessonsAcademy]

My pleasure 😊

[@x0z59]

You are my most favorite Angular teacher.

[@MonsterlessonsAcademy]

Wow, thanks!

[@ayushmankishoredash8585]

Great one!

[@MonsterlessonsAcademy]

Thanks!

[@ngathanh8438]

Amazing, thanks

[@MonsterlessonsAcademy]

Thank you too!

[@ngathanh8438]

this video is so helpful

[@MonsterlessonsAcademy]

Glad you like it!

[@robsonxavier5366]

Hello,
When you create a ngclass to control if is selected or not (minute 12), would be better to use ngclass (something like 'issue.id.isSelected: selected') and create a css class to change color?
Great video as always, keep the good work.
Thanks

[@MonsterlessonsAcademy]

It's more a matter of taste. This is just 1 line of code. If we had more styles than sure.

[@adarshsingh3012]

Thanks

[@MonsterlessonsAcademy]

You are welcome!

[@MrKOHKyPEHT]

If our version of angular less than 16 we can initialize input array like empty array in order to avoid non-null assertion

[@MonsterlessonsAcademy]

Yeah, or think about project upgrading -_-

[@sergeypodgaysky8551]

Why do you add an 'interface' postfix in interface? You don't add a 'number' postfix when creating a variable with type number for example, so what's the reason to do this with interfaces and types?

[@MonsterlessonsAcademy]

It is easier to separate a class from the interface or type in a big project. What's the reason to name a component in Angular FooComponent ?

[@VinitNeogi]

Signals make everything so easy :D

[@MonsterlessonsAcademy]

Yeap

[@olehandershevchuk3491]

Why you do not create an IssueClass that implements the IssueInterface and extends it with a "selected" property? Then, in the view, simply change item.selected to true without requiring an additional array for selected elements. To obtain the count of selected elements, you can filter the array of IssueClass elements by the selected field and count the results.

[@MonsterlessonsAcademy]

I don't like to mix view logic and entity. As we don't get isSelected through input we must say that we got IssueInterface and not extended version. Then anyway you need to transform your data type to data type with isSelected.

[@NiGhTm4r389]

You can compute signals from inputs if you provide them as input signals. This is quite the new feature though, is that the reason you're not covering it here?

[@NathanRLaing]

Was also wondering this

[@MonsterlessonsAcademy]

I didn't use them in the video as they are in developer preview. Sure you can do that.

[@NiGhTm4r389]

@@MonsterlessonsAcademy Ah I see. We already use them heavily in our project and they work flawlessly :)

[@ARShindouAR]

Do you have plans for creating a series of Angular CDK tutorial?

[@MonsterlessonsAcademy]

Not really but I will add it to the list of ideas. You mean Angular Material CDK?

[@ARShindouAR]

@@MonsterlessonsAcademy Exactly. Since I have watched your videos about implementing components without libs. So maybe you can create sth about using CDK to create compoents?

[@MonsterlessonsAcademy]

@@ARShindouAR I will add it to the list of ideas

[@alaskaricode989]

Can you Videos zu best structure folder files for big project

[@MonsterlessonsAcademy]

I covered the folder structure for React. It is the same for any framework
th-cam.com/video/IFTqfO_evlc/w-d-xo.htmlsi=1-QaWl1AdMq1fjts

[@D9ID9I]

So junior code turned into "senior junior" one. Now need to call middle guy to upgrade this.

[@ukasza.7836]

When I see suffix *Interface in an interface name I'm thinking that it is a beginner code

[@MonsterlessonsAcademy]

Sounds like a good indicator of junior code.

[@lucasgiunta8874]

Good. But you are still writing angular code with a react background. You refactor can be easly refactor in an half of code by an angular senior developer who realy knows the angular api.

[@MonsterlessonsAcademy]

What do you mean?

[@lucasgiunta8874]

@MonsterlessonsAcademy use a set input to set a signal to cover the ngOnChange. Try to use only computed signal as much as possible to leep one sources of data. Use the formControl api instead of your change event from your checkbox. Utilise the power of rxjs to handle your event logic and convert it to signal in your constructor if you need a signal output (to display the result in your view in a prevision of signal baswd component). Don't use ngClass if you only have one condition. [Class.myClass]="condition" is a nicer way to handle it. Don't use *ngFor/*ngIf which is the "old" syntaxes. If you are in 17.* you should have access to the new template syntaxes.

[@martapfahl940]

When I hear this is Junior level Code I feel offended being considered Junior after now 7 months into the job. I agree that this code is really super bad.

[@MonsterlessonsAcademy]

Yeap

[@Dajuhf]

At GoodTableComponent, won't be better to set the issueSig with a @Input() set instead of doing it inside the OnInit?
@Input({ required: true }) set issues(issues: IssueInterface[]) {
this.issuesSig.set(issues);
}

[@MonsterlessonsAcademy]

Yeap or we could use input signal. I always forget about new Angular features while coding.