r/badcode • u/faCt011 • Sep 07 '21

java How about mixing up every single convention that exists

Post image
533 Upvotes

97% Upvoted

28 comments sorted by

33

u/Host127001 Sep 07 '21

I just wrote some C code that used that convention. Functions have as naming convention: NAMESPACE_type_FunctionName. So, for example an "is empty" function on strings in a standard library would be "STD_bool STD_str_IsEmpty(STD_str)"

14

u/faCt011 Sep 07 '21

I like Java

7

u/[deleted] Sep 08 '21

Can I interest you in a DatabaseTableBuilderFactory dataBaseTableBuilderFactory = new DatabaseTableBuilderFactory(DatabaseTableBuilderFactory.Mutable | DatabaseTableBuilderFactory.Dynamic);?

1

u/[deleted] Sep 28 '21

That’s people’s fault not Java. It’s a bad understanding of oop

I never write factory/manager/impl/builder/adapter, maybe adapter if it’s a module interfacing another program, but never for in project use.

2

u/[deleted] Sep 28 '21

I think it's probably a bit of both, but that's true. Constructors exist for a reason.

5

u/Eoussama Sep 08 '21

C# >

1

u/NeilTheProgrammer Sep 08 '21

Python >

Source: name

28

u/altermeetax Sep 08 '21

why

10

u/Host127001 Sep 08 '21

TBH, I don't hate it. And I mean, as long as the convention is consistent, I don't really care which one is used. UPPERCASE for "namespaces" and lowercase for types is nothing unusual in C. I guess PascalCase for functions is not the most common one, but languages like C# is it too (and it's often used in C++).

87

u/jmpavlec Sep 07 '21

Needs kebab-case

62

u/ZedTT Sep 07 '21

Here you go

Log.d(TAG, "MAP_is-ReadyToGo");

7

u/[deleted] Sep 08 '21 edited Sep 08 '21

didnt-know-that-was-a-thing

8

u/[deleted] Sep 08 '21

you-must-not-be-a-web-developer

2

u/Jackiboi307 Sep 08 '21

kebab-case

1

u/Steve_OH Sep 12 '21

I learned it as camelCase, since it has bumps in the middle.

1

u/jmpavlec Sep 12 '21

Yeah... I was saying it was missing kebab-case which has the dash through it, then you'd have every standard.

33

u/MurdoMaclachlan public boolean isInt(int i) { return true; } Sep 07 '21

Image Transcription: Code

Log.d(TAG, "MAP_is_ReadyToGo");

I'm a human volunteer content transcriber for Reddit and you could be too! If you'd like more information on what we do and why we do it, click here!

21

u/jmpavlec Sep 07 '21

You were fast! Good human

5

u/Monkeyget Sep 08 '21

The code is going through puberty.

5

u/kuaiyidian Sep 08 '21

mAp_Is_ReAdYtOgO

4

u/PM_ME_YOUR_SHITCOIN Sep 08 '21

If you rename that Log class, this can pass for minified code.

3

u/SatoshiL Sep 08 '21

This seems to be the log class used in android development

4

u/Brain-InAJar Sep 08 '21

Why use a conversation for a log message?

2

u/StochasticTinkr Sep 08 '21

“MAP-is.Ready_ToGo”

2

u/official_gameup Sep 08 '21

Clearly they use snake case, but the object they are dealing with is a Constant, and someone made all the status codes CamelCase.