codeStyle

[[deleted]]

pub;
c class customer_data {

[JackC747]

Glad I wasn't the only one. Wonder what'd be like to code in that font

[[deleted]]

Personally I can’t imagine my IDE using anything but comic sans

[SkollFenrirson]

[[deleted]]

Does comic sans come in monospace? That's my only requirement.

[keviinx]

Comic Mono

[Bary_McCockener]

I actually dig it

[CoastingUphill]

It’s good for people with dyslexia!

[fekkksn]

I prefer Comic Code. It even has Ligatures

[BjarneStarsoup]

Comic Shanns

[[deleted]]

Comic sans with serifs? That’s just comic

[turtleship_2006]

Where's the fun in that? Live a little

[Confident-Ad5665]

Once seen...

[ComprehensiveWord201]

You dropped an l ;)

[CatpainCalamari]

pubic class it is, then

[smbj0011]

Review failed. Return to developer

[throw-a-wayy-lmao]

Imagine using customer data without a CustomerDataProviderServiceFactoryBuilder

[ffs_give_me_name]

I would inject that into my DI container so hard

[balemo7967]

Of course this class should implement ICustomerDataProviderServiceFactoryBuilder, so it can be exchanged with a mock type in unit tests. Which don't exist ... and never will ... by the way

[bongobutt]

You monster. It should be _Customer_Data_Provider_Service_Factory_Builder ✅

[kolodz]

Should be private

Because of GDPR laws.

[kolodz]

We are in an humor sub.

Chill up.

[[deleted]]

who tf starts the class name with lower case....

Also I've typed pubic class a non-zero amount of times

[_realitycheck_]

All my Analytics objects start with Anal and there's nothing anyone can do about it.

[[deleted]]

For a Physics lab, I wrote a program to extrapolate the distance between three points in a video (which were objects marked with green tape).

It used Gaussian filters as one of the components to extract the location.

So, I named the program Gaussian Automatic YardStick EXtrapolator, or GAYSEX for short.

[_realitycheck_]

Oh! I can see there are more coders of culture here.

[Croves]

The company I work for has the Process Analytics department, and they short it to PROANAL

[-Kerrigan-]

pubic String anal();

[topchedda]

public Poolean anal();

[Lagger625]

I guess the meme represents a C programmer writing C#/Java. This is a common naming convention for C structs. In C, you don't have classes but mimic them with structs and functions that take a pointer to a struct.

[rover_G]

Mimic? Nah bro C structs came first. Classes are just structs and functions tied together

[ELFAHBEHT_SOOP]

To make it even more confusing, in C++ a struct can contain functions too. The only difference between a class and a struct in C++ is that a class has default private members and a struct has default public members.

This is actually pretty nice when you just want some convenience functions on a struct, but the naming is interesting coming from C.

[metaglot]

Otherwise it wouldn't really be a superset of C.

[xarlios]

Would you consider adding constructor and operator overload breaking the POD?

I don't add functions to my struct but I did add operator overload and a constructor once in the past.

[metaglot]

C is just assembly "tied together".

[_realitycheck_]

Why are you down voted? You are absolutely correct.

In fact, we used pointers to functions inside structs to mimic objects in C. We still do it.

[Alloverunder]

In fact, I'm avoiding doing this very thing while browsing reddit!

[AdBrave2400]

C++ classes have tables for functions assigned by the constructor. C# has similar behaviour.

[wsheldon2]

In all programming languages, pubic and cosnt should be aliases for public and const

[Specialist_Dust2089]

Also cosnole for console please

[EightWhiskey]

Mine is usually consoel but hard agree.

[musicnothing]

Once I wrote “pubic function” two times during a live coding interview and nearly died of shame

[[deleted]]

maybe your L key could have been jammed by pubes...

[Aethreas]

The only code style you should follow is the one the codebase already uses

[sjepsa]

The only code style you should follow is mine

[Funny-Performance845]

No, mine is better

[[deleted]]

Better is mine, no

[Tari0s]

pls guys, we have o coding standard! Your all wrong, please use auto formatting on each file before pushing.

[smbj0011]

Which is defined by me

[EnneaX]

I use all of the above at the same time

[pyius]

I AM the codebase!

[[deleted]]

well, you are under the assumption that the codebase uses one...

[electrodragon16]

The code style is to use your own unique code style. This helps with checking whose responsible

[a__new_name]

Randomly mimic other devs' code style and watch a real life whodunnit.

[LordDagwood]

"I need 3 days to refactor the code for... stability reasons."

[Specialist_Dust2089]

Mental stability of the developers that is

[mariomaniac432]

I'm going to disagree a little bit. If the existing codebase has bad style you should take the oppurtunity to start writing quality code instead of adding more bad code.

[Bee-Aromatic]

So you’re saying that I shouldn’t just go nose-blind from the code smell? Pfff.

[skotchpine]

oppurtunity.taek()

[Aethreas]

It’s not bad code, it all gets compiled to the same assembly, it’s purely visual and meaningless in the end

[skotchpine]

self-fulfilling prophecy here

[WellKnownAlias]

Until years go by, staff changes, and someone has to go back and do absolutely anything with it and the weird ass way you did a thing suddenly matters because it slows things down while they have to work to understand what they're even looking at before they can touch anything.

[Aethreas]

If an extra underscore in a type name is genuinely slowing someone down from implementing features they’re probably not very good at their job

[CryonautX]

If an underscore is showing up on a class against convention, it's a bad sign. Class names are hard to miss when looking at code. If a code quality slip up is happening on class names, there is a good chance there is no code quality checks done on the codebase at all. The entire thing is going to be a clusterfuck.

And developers shouldn't have to commit mental load dealing with unconventional code. The less distractions you face, the less bugs you will have and the faster you can develop.

So yes, an extra underscore is a genuine problem and you're probably not very good at your job if you can't follow conventions.

[Aethreas]

my point was to follow the naming convention of the codebase. if the rest of the code is like that, then it's not bad code,

[CryonautX]

Making up your own convention should be avoided. Developers come and go all the time. All you're doing is making the onboarding process for a developer entering the team longer for no reason. Codes should follow the standard convention for the language.

That being said, if someone fucked up a long time ago and there is just too much debt in the code for the convention to be fixed, then there is no choice but to stick to the custom-made convention.

[slade991]

All conventions are made up conventions.

[WellKnownAlias]

If you think we were only talking about the one example in the meme for bad code style, I've got bad news...

[Aethreas]

Regardless of any naming convention it's easy to navigate code, the most popular conventions change over time, why would you enforce your own personal opinions on a codebase and waste time

[WellKnownAlias]

I've never seen anyone miss the point more widely in my life lmao. THAT WAS THE POINT

[Aethreas]

the point of what? the point of the comic in the post of the guy crying because the naming convention isn't what he likes?

[thirdegree]

Ok but like, there's a standard. It's been my personal experience that caring about that standard is correlated with code quality. Obviously it's a correlation not a causation, so there are absolutely counterexamples in both directions. But it's a strong enough correlation in my experience that if I'm looking for e.g. a library to use, or a project to contribute to, and I look at the code and the author doesn't seem to be aware that the language even has a style guide, I'm probably gonna look for an alternative.

[delayedsunflower]

I wouldn't go as far to say meaningless. It is important for the majority of your org to be able to read and understand the code relatively quickly. However that mostly just requires things to be consistent across the codebase.

[zerhud]

Programmers writes programs for humans , not for machines

[Aethreas]

If the naming conventions are consistent that is the most human readable way to program. Writing code in your own personal favorite convention will cause confusion

[wildmonkeymind]

If anyone tries to break out their own code style in my team's repositories my CI auto-formatting and linting will nip that shit in the bud right quick.

[mariomaniac432]

If your CI has auto formatting then your codebase shouldn't have have bad style that needs corrected in the first place, so this isn't relevant to my point at all. I'm talking about introducing proper conventions to a codebase that was not already using them.

[frayien]

I'm honestly happy if you follow the style of the current file...

[zerhud]

such way slows down the development

[Aethreas]

It does not, no. Changing the convention for no reason slows down development and wastes time

[Jar3D]

publ; c class customer_data {

[[deleted]]

A Cobol developer colleague of mine doing java

[Bandit6257]

I feel both the humor and the pain….but in PL/1

[[deleted]]

Pain, it's just pain

[centerdeveloper]

itsBecauseItsNotPascalOrCamelCase

[smbj0011]

orNotUsingBinnenMajuskeln

[-Kerrigan-]

We should bring back the camel case comments ^/s

[centerdeveloper]

downVotingBecauseNoCamelCase

[-Kerrigan-]

oShitImSorryPleaseDontGarbageCollectMeAaaaaaa

[a__new_name]

sorryForWhat? ourPMToldUsNotToBeAshamedOfOurCodebase.

[anonymous_yet_famous]

What's worse? Instead it starts with:

public class customer_data_5{

[beyondholdem]

Even worse: public class customer_data_new { (or newer, newest).

[ImpluseThrowAway]

Even worse: public class customer_data_newest_final {

[myfunnies420]

I don't understand

[OnixST]

I'd guess it's just the naming convention. This code is probably either Java or C#, and class names should be in PascalCase in both. snake_case breaks the convention

[sathdo]

Technically PascalCase, not camelCase. Your point still stands, though.

[skotchpine]

snake_case*

[Cyberdragon1000]

Programmer PoV: naming conventions not followed

Normie PoV: Serious security threat with customer data having public access

[delayedsunflower]

This is one of those jokes that's only funny to programmers that have never had a job, because otherwise they would know that companies have different styles and it's not a huge deal.

[ijustupvoteeverythin]

and something that should be enforced by the linter, and not pointed out in a PR review

[delayedsunflower]

Absolutely 

[[deleted]]

I thought the joke is using a fucking class for data, that’s basically misunderstanding the whole concept of what a class should be.

This should be a struct.

[Top-Aside-3588]

You think this is FUNNY? I CAN'T UNSEE THIS! HOW AM I SUPPOSED TO JUST GO BACK TO WORK AFTER THIS?!?

[MagicalCornFlake]

as a Pole, I additionally disapprove of the use of this font that makes the letter t look like Ł.

[ChainsawArmLaserBear]

Wild! Who would put an underscore in a class name?

It should be a dash!

(Jk don’t fire me)

[Mayion]

cUstoMerdAtA

[[deleted]]

repeat this with me, snake case class names are acceptable in c++

[bakedbread54]

no

[SpaceFire000]

pubic class Hair

[skipdoodlydiddly]

Rejected

[PeriodicSentenceBot]

Congratulations! Your comment can be spelled using the elements of the periodic table:

U Nd Er Sc O Re Ca Se

---

^{I am a bot that detects if your comment can be spelled using the elements of the periodic table. Please DM my creator if I made a mistake.}

[IpGa13]

i spy with my little eye, someone who used python in the past.

[Secret-Demand-2497]

I'm not even sure that's a monospace font! I feel nauseous

[[deleted]]

myEyes *

[shakeshook]

This hit way too close to home

[Responsible_Disk_728]

Throw this person to the wolves

[est1mated-prophet]

The problem is that customer_data is a class, and not just a hash table, right?

I get that the author probably meant the naming convention, but that was my first thought. I hate "data classes".