Tiny Refactors Make a Big Difference - Laracasts

In this Larabit, we’ll review some deconstructed, but typical real-life messy code that you’ll often find in the programming wild. Specifically, I want to talk about how, often, it’s the tiny refactors that can add up to make a massive difference in your applications. Watch thousands of videos, track your progress, and participate in a massive Laravel community at Laracasts.com.
Laracasts: laracasts.com
Laracasts Twitter: / laracasts
Jeffrey Way Twitter: / jeffrey_way

[@renwar]

I am usually having a feeling of "over" refactoring.
But seeing you doing the same as mine, is a relieving...
Thanks for making such videos..

[@hollengrhen1]

Tiny bits of refactoring **Proceeds to make an 18m long video** KEKW

[@TimothySmithDev]

Great video, I'm a senior developer and I still enjoyed watching.

[@lucasj.pereira4912]

I feel like I would stop at 16:59, code seems more readable for me, but it's just personal preference. I don't like to inline arrays that much and 'mess' with de indentation. Not saying its a bad video, it is awesome, just my personal preference. Thanks Jeffrey!

[@Laracastsofficial]

It's whatever looks prettier. Sometimes, that's inlining the array. Other times, I'd rather keep the variable.

[@FilipePetter]

At 14:30 he referes to a method "Auth::user()->isOnATeam"
English is not my first language, so I could miss something
My question is, this "isOnATeam" is a built-in method or I have to create this method on the User's model ?

[@Laracastsofficial]

You would manually add that method to your User model. And call it anything you want. $user->onTeam(), $user->isTeamMember(), etc.

[@Bykv78]

I liked everything you did, but I would only do the last part differently. I mean I prefer creating a variable rather than repeating the query.

[@jsjunior]

Which PHPStorm theme are you using now? @Laracasts

[@aissa.bouguern]

I really enjoyed to see how you was writing code that was "good enough" to be shipped to production! That's the mindset I hope one day I will have. I shall ship as long as my code is readable enough and does the job.
The perfectionist will never deploy something.

[@mohammadashrafuddinferdous9347]

Nice one! To my very personal opinion, i would rather use one codition and then return the result. I would omit else part and return if it reaches there! But I get your main point. We just need to recheck and refactor wneneer possible.

[@smtkumar007]

Mostly in my team after refactoring , it always ends with silly bugs

[@udofiagodwin8194]

Nice! Very nice! Personally, I would choose to keep the $isOnATeam variable so that I can change the value at a single point if ever there is a need to do so.

[@TDarkCoder]

Totally agree! Moreover it is always a better practice to choose resource over readability, however here even with assigning variable, readability did get worse.

[@Laracastsofficial]

If you ever need to do that (with emphasis on ever), *then* you extract the variable.

[@nonamechoosen]

The message could also easily be inlined imo!

[@hassamulhaq7762]

such kind of tutorials are really helpful.

[@jamesrogers2680]

Very helpful video; the thought process of your refactor was great to hear. Often coding videos are missing the "why" of decisions. Much appreciated!

It seems to me it would be nice to use a technique inspired by early return - to set the "default" value outside of any conditionals. Then have one conditional block, that would set the more complicated values with query string, etc. It feels weird and prone to errors down the road to have the ternary duplicated.

[@VadimBesedin]

Nice refactoring. I like this gradual refactoring style - very clear. Thank you.

[@proksalevente]

At 7:45, what's the keyboard shortcut for selecting the whole thing?

[@TristanBailey]

You refactor so nicely when you talk through it. I sometimes imagine you talking when I do the same 😂

[@KgfLikia]

Doing so to all files i visit... And asking devs to follow like stop declaring variables that are not variables or they are used only once.

[@DougLopes]

Great tutorial of refactoring, not just for beginners, may i say.

[@jeffreyway520]

Thanks!

[@adityakunhare]

Of course not for beginners, they should write code at first...

[@afsofatehabad1901]

Very helpful..thanks a lot.

[@clashoftroopers4545]

Love your lesson, 'n love listening to you.

[@khayamkhan2702]

amazing refactoring ...

[@wasidhossain3848]

wow 😲 it was really cool

[@farazsalehi9034]

even with these little refactors .. i think its better to run unit tests .. because who would grantee that code still works ?

[@Laracastsofficial]

Of course. You do both.

[@ycombinator765]

Thank You!!!!!

[@abrahamghaemi8895]

good job

[@piaIy]

What do you think about destructuring assignment? You could rewrite it so that you have to check the condition only once:
[$message, $url] = $isOnATeam ? ['message1', 'url1'] : ['message2', 'url2'];

[@Laracastsofficial]

Yeah, if you want. I probably wouldn't go that route, but it's fine.