Clean code challenge - Silicon Valley Season 5, Ep6 🚀

🚀🚀🚀 Become ELON MUSK playing most HARDCORE iOS mobile GAME: saviorx.com ! 🚀🚀🚀
#saviorx #mobilegame #unity3d
Nothing puts the fear of God into a programmer like the inevitable code review. This is when a group of people analyze your code for errors and shortcuts and violations of standard operating procedure like using spaces instead of tabs. I once worked at a Wall Street joint where our code review was like American Idol auditions. We had a panel of reviewers made up of my boss, the COO, and the CFO. They were ruthless. I would have to fill out paperwork describing my feature/fix, project my code on the conference room screen, and execute it while explaining my workflow. The slightest violation sent me back to the drawing board while my team members snickered nervously. I still have nightmares about this particular panel. My soul review at the Pearly Gates will be less terrifying.
Thankfully, Dinesh and Gilfoyle’s code will be observed by a special prosecutor - I mean reviewer - because they’re senior management, thereby saving them the embarrassment of a public flogging. However, Dinesh wants a public hearing, if only to prove that his code is superior to Gilfoyle’s. Jared is against this for a valid reason. “A little cocksmanship was okay when we were a small company,” he tells them, “but the coders need to see you guys as equally competent leaders.” Dinesh can’t handle not knowing if he’s superior, so he hounds Danny the Code Review Guy relentlessly until he reveals Dinesh was better! Danny asks Dinesh not to tell anyone. Of course, Dinesh tells the entire office.

[@JSmellerM]

The second someone else enters into Dinesh and Gilfoye's relationship they unite. I love it.

[@enkiimuto1041]

I do wish the engineer from season one was there more often, she was the perfect balance.

[@VietVuHunzter]

With respect to Dinesh, Gilfoyle is so arrogant that I doubt he allows bugs live in his code.

[@lilyliao9521]

What

[@MangoMotors]

But that's the thing about coding. When you write a very complex code, even small tiny things can cause bugs. That's why even companies like apple Facebook and Google who literally have billions of dollars still end up with bugs.

[@saqlainalvi5485]

This comment makes no sense

[@osalbaro]

Yea, that's not possible

[@freeshavaacadooo1095]

@@MangoMotors I mean you are somewhat correct in the sense that bugs will occur, but they literally have hour-long debug sessions or just use something called checkpoint testing where all valuable data is relayed after a certain section of code to the console for analysis. If there is an error they know where it is and if there isn't the code is ready to be peer-reviewed again. Most "bugs" come from human logic error more so than syntactic.

[@perfectionbox]

Nothing more devastating than Gilfoyle's monotone delivery of soul-crushing news 🤣

[@thecollegecoder7356]

Dinesh's pull requests must be brutal

[@aheathcliff7227]

What does that mean

[@ticallionz]

@@aheathcliff7227 A "pull request" is when you submit a change to the existing codebase. One or more of your colleagues will look at your changes and either approve them or ask for more changes before the code is "merged" into the main codebase. The more changes they ask for, and the harsher their comments are, the more "brutal" the pull request is.

[@UnicycleSoul]

@@ticallionz I never understood why code review comments would need to be harsh, it merely shows the insecurity and egotism of the reviewer, much more than the competency of the submitter.

[@victorscherman1437]

Also there'd be so many new changes to review since I'd imagine Dinesh doesn't push his code in small chunks.

[@victorscherman1437]

@@ccriztoff what the fuck is your problem

[@AdityaDendukuri]

as a programmer they must've been really good friends for them to meme each other like that in a professional workspace

[@georgebrantley776]

They've been coworkers, housemates, and rivals from before this company was even a company. Then they both joined as founding members, and now they've got a team of 46 engineers reporting to them. So yeah, good friends is pretty accurate.

[@buildnothingoutofsomething2513]

Clean Code does not just mean having less Errors by the way

[@ettiennepieterse371]

It means having fewer errors.

[@saqlainalvi5485]

@@ettiennepieterse371 no it means writing a code using the latest frameworks and technologies with best coding practices in mind. A tight code without any unnecessary extra coding is a clean code

[@Aspectalice]

@@saqlainalvi5485 It doesn't have to be the latest frameworks but everything else you said is right.

[@saqlainalvi5485]

@@Aspectalice Yes brother. But a programmer should atleast know the basics of every new framework that comes out for his/her favorite language. But yeah you are right it's not necessary. Note: i forgot to mention. Clean code also involves managing complexity through oop.

[@LitCactus]

@@saqlainalvi5485 what are you talking about? Not everything is object oriented

[@synsyko]

Dinesh is using spaces, Richard is going to be pissed.

[@Megalomaniakaal]

pep80 biach

[@mishikookropiridze]

He is not

[@ThunderAppeal]

Tabs are for amatuers

[@Isaic02]

ThunderAppeal ha!

[@madkinggaming3744]

ThunderAppeal tabs are for people who like to save time. I will admit it did screw me once.

[@GouchuInc]

The real winner here is Jared (and Richard) who told them to layoff each other, because they're senior management now. He's proven right in the end when the engineers band together to make fun of 'senior management'.

[@francisconicolau8528]

You mean Donald

[@kossend1]

@@francisconicolau8528 not Donald, Other Jared. OJ for short

[@tanned_cosines_]

pfp 💀

[@lmtx45]

-How's the code been going? -T minus Dinesh.

[@vicentcarro]

every office has one Dinesh and Dany

[@fixafix69]

Wait is that a racist joke or not

[@drodriguez3293]

i wish i made enough money to relate to this joke

[@jjhassy]

Fix a Fix yep its rly racist don’t read it

[@kapils2862]

But what every office needs is a Gilfoyle and a hairy Arab who can Chortle his balls

[@Jarasi98]

Silicon Valley is my comfort show, I wish it was still on

[@nandinisingh3317]

Same :(

[@diegoelcool]

It finish at the right time... after 6 or 7 season shows start to decay

[@lavelafetsha]

Just rewatch it

[@kunraiyan]

I think it ended correctly

[@xerp7626]

I had it when "Gabe" says , "Guys, Nononono, remember, we talked about this." Seemed like he was giving advice to Jim and Dwight lol 😂😂

[@jaywjr2456]

I love Dinesh and Gilfoyle. I've said it 1,000 times but the writing on this show complemented by their delivery is the most under rated. Of all time possibly.

[@r3wturb0x51]

Mike Judge is a fucking genius

[@orionh5535]

Danny was the human, who ate the tortoise and the hair after the race.

[@jjhassy]

hare

[@TooDamnFishy]

No, he actually ate the hair

@@TooDamnFishy Tortoises don't have hair.

[@night00glider]

@ or...do they ? Dun dun dunnnn

[@pranav5498]

@@night00glider no they don't, they are not mammals

[@fahimminhaz7071]

Dinesh: Less errors.. Stannis 'Gilfoyle' Baratheon : Fewer..

[@justgame5508]

This has nothing to do with “clean code”, clean code refers to a codebases readability, you can have clean code with a lot of bugs and ugly code with no bugs

[@kaushalrjain6326]

Correct

[@kylel8258]

I think they’re just trying to make it easier to understand. “Error” is one word and easier to write into this scene than “code with readability”.

[@iamdseth]

Yessirr

[@rishikakkar1528]

properly refactored and debugged code.

[@apidas]

eh

[@kjyothirganesh]

"I have to fire Danny" 😂😂

[@quinndirks5653]

Do you know what dinesh says after that? I can't quite make it out over the music.

[@kjyothirganesh]

@@quinndirks5653 "I'll ask Richard" 😅😅

[@zissler1]

Jyothir ganesh thought it was “afterwards”

[@bigsassyster]

OH, I thought he said "I have to fight Danny" haha

[@marlom7882]

“Nothing puts the fear of god into a programmer like the inevitable code review” Jesus Christ well I guess that’s one thing to look forward to when I get a job in coding

[@oferyrd]

if you approche it right, you can grow a great deal from code and design reviews.

[@JohnDemetriou]

Reviews are actually the best way to learn and improve. Both reviewing and being reviewed

[@БейбутТукибаев-п3у]

I actually like code review

[@wombocombo6878]

... its really fucking easy just follow the rules. The best reviews are the ones where you learn a new way to look at things.

[@michaelmano7261]

Its also not just about bugs, you learn to do things more efficiently from people who have years experience.

[@WhistleFantasy]

"did his code have less errors than mine?" - you can hear non-programmers wrote this script.

[@cdjwmusic]

These two guys are now marvel actors

[@Twoswordsdrizzt]

That MisSteak line was actually pretty good though

[@eravar2831]

Gilfoyle plays magic lmao

[@5hort5ubject]

Gilfoyle is a key here and in whole

[@vectorhacker-r2]

Private review is stupid. Senior staff should be subject to the same standards as everyone else publicly

[@Wanooknox]

Better yet, hanging people out to dry when they make mistakes is counter productive. You don't get more productive by being shamed, you get more productive by being taught.

[@MagikFrogs]

Wanooknox also we’re all humans we all make mistakes.

[@antonyjr.devlogs5957]

@@Wanooknox Yes. And this is actually used in top tech companies like Google, Facebook and etc...

[@flashcat6665]

In the end it doesn't matter, all code is committed to the same repository, and all commits are accessible by everyone who has access to the code. Easy to see how made mistakes, how they got fixed, etc. And ideally anyway, your branch would not get merged until every PR is reviewed and approved. And all PRs are public as well. So this whole "private review" is basically useless and bullshit.

[@OgeeDogee]

@@Wanooknox It is productive and you're not supposed to be shamed it's supposed to set an example so other devs won't encounter the same mistakes. While you can get tips from your fellow colleagues and not just the ones reviewing your codes.

[@jrock2720]

haha we're all laughing at you, gilfoyle - makes me laugh every time

[@zackzhu9865]

"I see, fun at my expense" lol

[@JordanBakke]

3:30 he should have made A Bug's Life joke

[@Angel-yq8jd]

that would make his joke good

[@beowulf555]

“Laugh it up GILF”. Hahaha

[@nandan580i]

These two are like a married couple with 20+ or so years of marriage behind them...they are comfortable enough to abuse, tease, make derogatory comments to each other and yet they are the best duo in tge series.

[@brothersum1359]

Lmao how jarad walks over and stops the loud gong

[@chukwumakelechi2006]

“You just have to swear not to tell everyone” Dinesh: “I swear” Everyone, everyone... my code is better than......

[@jaredbitz]

I relate at an uncomfortable level to Dinesh repeatedly practicing the delivery and phrasing of his joke at 3:23

[@nik-ys8ki]

" i m not gonna beg " "PLEASE" "PLEASE" "PLEASE"

[@joseville]

I'm just impressed someone can find all the errors and bugs in a piece of course

[@aiden7279]

Never knew The Hound is a coder. What a cool guy.

[@MonteLogic]

Dude this show is my life.

[@derrick20]

I don’t really see how gilfoyle saw that as a “victory”, he just predicted the obvious basically

[@katsuutheunker]

2:37 Lmao that coffee mug

[@kintondie]

Ohhh. The guy is literally like one of my old colleagues. So noisy in that high pitch voice. And taking credits for everything

[@saqlainalvi5485]

My supervisor was that high-pitched guy at my old company. Guy made me want to cut my ears off every time he called my name

[@trym2121]

Three way fight. The only way to coexist 'peacefully'

[@ROJTHEGOAT]

The Type O joke is low key 🤣 funny

[@mustangrt8866]

I love how Gilfoyle can troll Dinesh into a scene

[@erons_xo]

Nobody: Dinesh: I'm the fucking Tortoise!

[@basti3944]

It took me 7 minutes to realize the video isn't about clean code at all

[@NatureFreak1127]

I have so much sympathy for Dinesh, he tries so hard. 😂

[@timturner7609]

Not sure if I'd quit or give that one dude swirlies every time he opened his goddamned mouth.

[@kshyr811]

2:59 "I hope the book you're reading is for work" Book: "Coders at work"

[@kshyr811]

@@user-dp5fq4vc8m thanks

[@npip99]

4:42 That pentagram lol

[@medicalbar]

lol

[@rigoblues]

What if San Francisco people didn't saw this as a comedy series and actually think this is the way they should behave at work.

[@TheLionManOnline]

00:54 if you saw it give yourself a punch.

[@samrodshenassa978]

Wonder what those who neither work in tech nor are familiar with it thin of this. I must admit, while I used to be intimidated by PRs early on, now they either reinforce my good habits or address my bad ones. So they're win-wins all around. And yes, especially with bigger PRs reviewed by more people, there's a nice sense of satisfaction when it's approved on the first pass. Like, BOOM! Nailed it!

[@Enzorgullochapin]

The Jackrabbit Always Wins

[@tarifadib]

gilfoyle watches GOT. uses FEWER from Baratheon🤣

[@BastardOfTheNorth]

Actually Stannis used it because it's correct. Same reason Gil does.

[@tarifadib]

@@BastardOfTheNorth Baratheon is Stannis' last name😂 if you watched the show, I dont have to clarify which Baratheon.

[@BastardOfTheNorth]

@@tarifadib I am aware of that.

[@denisblack9897]

MISTAKE HOUSE! haha

[@alberts.3631]

haha they actually put "made in china" song lol.

[@mirsahib596]

dinesh and gilfoye are like two sibling always fighting each other but the moment someone hurt one of them their blood gets thicker than water

[@menzimabuza1746]

Wait thats Gabe..what is Gabe dokng there.. Probably telling people to shut up about the sun😊

[@ohWowYoutube]

"Everyone is laughing" 😂

[@amansharma4358]

One of the best series I have ever watched

[@johnhinn448]

Same here 🤣

[@A_A_K_123]

I'm the f***in' tortoise.

[@arabianprince7508]

I liked his jokes

[@theencryptedpartition4633]

Damn. Gilfoyle is just a Rick from Rick and Morty. But a 32bit

[@brucetrunks01]

So danny got them both ??

[@themorbidhero2987]

Pretty much

[@rayo146-x8x]

2:36 He's flippin him off too 😂

[@lfrit0]

Dinesh my man... Type O is the type you go to when you need blood : (

[@Kncperseus]

Cute how Jared stills the gong

[@giovannigonzalez6349]

Danny, the hr lady,and John are the only 3 people who won against Gilfoyle(HR lady managed to actually annoy Gilfoyle by being right,to an extent).Dinesh never beat Gilfoyle but he was the only person who could keep up with him.

[@redemolisher]

They are the definition of a married couple

[@besllu8116]

Man, I would live in their world, as Russ Hanneman. Tres commas!!! lol.

[@mindful_clip]

ChatGPT enters room.

[@jeroddd]

HIGHER BROTHERS

[@Gambit_LeBeau]

I loved how this show always wrote Gilfoyle & Dinesh to have a in work feud as who is the better coder. Yet like the episode where they go to Double A warehouse & they see the calculation of Blaine's pre skecthes & they start SWATTING (code)aka insults like fuck Gina, let Blaine die etc.

[@_nfl_nerd_]

Private review, I'm a senior engineer and no one reviews my code except me :D

[@dougsam6341]

I can't wait to be working in this field. Got 3 class to go

[@fofopads4450]

You are up for a massive disappointment if you expect this kind of environment.

[@dougsam6341]

@@fofopads4450 what do you mean?

[@danielstory2761]

@@dougsam6341 how is it

[@hunterstommygun5716]

Consider the tortoise- Dinesh

[@Ravikumar-gj6qw]

Nice video

[@emptylanternmedia2248]

He really is the cool cousin

[@jeroddd]

Higher Brothers 👌🏾👌🏾👌🏾👌🏾

[@atulkumar-sg2ik]

1:08 Gabe is back.

[@nomoredarts8918]

It's not about clean code, it's about clear code

[@adamslodowy-gi7vx]

I mean, I need to write it: there is no way someone found bugs/errors in review (unless code was super bad). Review is done to improve style. If you want identify bugs you need testing. This is the only way. Yeah, I don't have friends

[@thomaspotterdotexe]

So this is where Gabe after Sabre.

[@vicentcarro]

Ixalan is a Magic: The Gathering expansion block

[@oakleyorbit]

I hope I never have to work with a Danny.

[@rach101]

George Error Martin 😂😂😂

[@StrayCatInTheStreets]

Wait, would they not just run the code in Jenkins? Doing the build and integration tests thus finding the bugs anyway? It's the inaccuracies like this that take away from the show

[@thomasip9938]

Unit/integration tests cannot prove the absence of bugs, that is, 100% pass rate and coverage do not mean your code is bug-free.

[@StrayCatInTheStreets]

@@thomasip9938 yes but also code reviewing using eyeballs doesnt either. They should be code reviewing then running it through automated builds. Simply getting one person to reviewing all their code is bullshit.

[@shinobi-no-bueno]

@1:57 Gilfoyle slips a secret smile and coyly stows it away

[@AutoFirePad]

Gilfoyle == JonathanCoulton + RonSwanson

[@dmr0718able]

i am currently learning how to be a programmer. and everytime i see these guys code that fast intimidates me.

[@iii-ei5cv]

dmr0718able it's a show. No one writes meaningful code like that. You should be spending a majority of your time designing and analyzing your program, not much time actually coding and then quite a bit of time debugging

Cut and paste.

[@MetalMutant]

Just copy n paste n adapt, no one write clean code these day lol

[@EM-sz8ft]

@iii i have seen people code that fast but they ended up being great. All became hackers and got caught

[@morgangalvin8893]

@Colten DeYoung git add seppuku.gif commit -m "RIP"

[@largolegato]

What kind of senior managers are these?

[@abraofrancisco7198]

I have no idea

[@reinersitanggang]

Senior management that can function is still code.

[@nccamsc]

Dinesh didn't run the test suite, assuming they have one.

[@medilies]

Going to the comments section is a must

[@andreiribeiro2904]

an entire tv series about reddittors

[@pyew9038]

Love the outro

[@CrispyWater4Sale]

Damn the Higher Brothers got a song used in silicon valley

[@deville295]

George eRRor Martin

[@Mrslykid1992]

I like the man in China song!

[@ninadlohar103]

the office looks more like coderHQ, Texas

[@justins7796]

man didn't use protected but used private, what's he making - nuclear warheads?

[@marco-n3z8g]

They should have used Object Calisthenics... They wouldn't have finished

[@forgottenmma3694]

Lmao Hair-y. 😂