Official Everybody Edits Forums

den3107

Member

From: Netherlands

Joined: 2015-04-24

Posts: 1,025

So what's your coding style?

Just a place where people can discuss the title, if they'd like.

Think CammelCase or not for variables and methods, braces on nl or eol, yes or no braces on single line (if-)statements (or just move straight over to a conditional operator).

Please do mind we want to keep this thread friendly, most likely there's no right or wrong

Different55

Forum Admin

Joined: 2015-02-07

Posts: 16,574

Re: So what's your coding style?

if (condition)
{
    multiple();
    lines();
}

if (condition)
    single_line();

---

"Sometimes failing a leap of faith is better than inching forward"
- ShinsukeIto

hummerz5

Member

From: wait I'm not a secret mod huh

Joined: 2015-08-10

Posts: 5,852

Re: So what's your coding style?

I've been recently indoctrinated in the world of Java, so Classes, localVariables, MethodNames... CONSTANT_IDENTIFIERS, I suppose I'd just apply those to other places, regardless as to whether or not the language generally approves of that.

My braces are whatever the editor wants me to do... VS I think has me doing brace gets its own line, IntelliJ sticks it at the end of the conditional line.

how about those one-line conditional "blocks" what are those called? Where you do the if w/o actually following it up with a block? yeah I don't roll with those

edit: yeah, what diff did the second time. no.

---

thanks HG, HG, HG, HG! anatoly, zumza! zoey!

Vinyl Melody

Formerly BananaMilkShake

Joined: 2016-06-19

Posts: 616

Re: So what's your coding style?

if(conditon) {
    multiple();
    functions();
}

if(condition)
    function();

var b1=true, b2=true, b3=true;

And also squeeze everything if possible, so rather than

if(variable == "yes") {
    function();
}

it's

if(variable=="yes")
    function();

Also, slight abuse of #region to make things a bit tidy (but //--\\ , <!--[]--> and --[[]]-- on VS Code when working on JS/HTML/CSS/LUA/LUA+Fake XML)

---

Thanks to: Ernesdo (Current Avatar), Zoey2070 (Signature)

Very inactive, maybe in the future, idk.

John

Member

Joined: 2019-01-11

Posts: 1,979

Re: So what's your coding style?

I like to have { and } on new lines because it is easier for me to read. careful you dont want to trigger HG

---

hummerz5

Member

From: wait I'm not a secret mod huh

Joined: 2015-08-10

Posts: 5,852

Re: So what's your coding style?

And also squeeze everything if possible, so rather than

if (condition){ multiple(); lines(); } var b1=true, b2=true, b3=true; if (condition2){ singleLine(); }

I mean if we use your argument, then we should probably minify.. :b

idk. My biggest qualm with that is when I want to add code inside the condition response. Then I have to add the braces anyway! D:

---

thanks HG, HG, HG, HG! anatoly, zumza! zoey!

Xfrogman43

Member

From: need to find a new home

Joined: 2015-02-15

Posts: 4,174

Re: So what's your coding style?

if()
{
}

---

thanks zoey aaaaaaaaaaaand thanks latif for the avatar

mrjawapa

Corn Man 🌽

From: Ohio, USA

Joined: 2015-02-15

Posts: 5,840

Website

Re: So what's your coding style?

if (condition){ multiple(); lines(); } var b1=true, b2=true, b3=true; if (condition2){ singleLine(); }

this bothers me a lot.

---

Discord: jawp#5123

Zumza

Member

From: root

Joined: 2015-02-17

Posts: 4,645

Re: So what's your coding style?

I usually follow the style guides of the specific language / project I'm using.

---

Everybody edits, but some edit more than others

LukeM

Member

From: England

Joined: 2016-06-03

Posts: 3,009

Website

Re: So what's your coding style?

If in notepad++ then I do this:
if (expression) {
  Thing();
} else if (expression) {
  Thing();
} // Tabs with a size of 2

And if I'm in VS, I do this:
if (expression)
{
    Thing();
}
else if (expression)
{
    Thing();
} // Tabs with a size of 4

I'm not sure why I do it differently...

Also:
functions: DoSomething()
variables: storeSomething
unless it contains something that stands for something:
worldIDthingHere
IDofTheWorld

if (oneLiner) doSomething();
if (oneLiner && longerThan("about half the page"))
  PutOnNewLine("do it again if neccisary", true
    "indent inside brackets", true,
    "more lines here");

Swarth100

Member

Joined: 2015-07-18

Posts: 305

Re: So what's your coding style?

- Avoid { } for oneline conditionals
- Segment code in chunks for different sections
- Inline if when possible
- Header files must contain all exposed function declarations without variable names
- .c file code must be heavily documented
- I always use multiline comments. I use single '//' for commented out code, so it's easy to lookfor and replace when committing some code.

Gosha

Member

From: Russia

Joined: 2015-03-15

Posts: 6,206

Re: So what's your coding style?

if(stuff == true)
{
    return true;
}
else
{
    return false;
}

hypermind code over here

Swarth100

Member

Joined: 2015-07-18

Posts: 305

Re: So what's your coding style?

if(stuff == true)
{
    return true;
}
else
{
    return false;
}

hypermind code over here

kubapolish

Banned

From: ̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍̍

Joined: 2015-02-19

Posts: 1,024

Website

Re: So what's your coding style?

if(!!stuff != !true)
{
    return !!!false;
}
else
{
    return !false;
}

ftfy

---

███████████████████████████████████████████████████████████████████████████████████

Gosha

Member

From: Russia

Joined: 2015-03-15

Posts: 6,206

Re: So what's your coding style?

I think you made it too complicated!!!

i think it should like this:

var logic = 0;
goto checker;
answer1:
return stuff;
answer2:
return !stuff;
answer3:
throw new BoolenEqualsToZeroException();
checker:
if(stuff == true)
{
    logic = logic + Math.Pow(logic,0);
}
else if(stuff == false)
{
   logic = logic + 2 - 4;
}
else 
{
   logic = logic + logic * logic + (logic - logic);
}

if(logic > 0)
{
    goto answer1;
}
else if(logic < 0)
{
   goto answer2;
}
else 
{
   goto answer3;
}

ezpz

den3107

Member

From: Netherlands

Joined: 2015-04-24

Posts: 1,025

Re: So what's your coding style?

I've been recently indoctrinated in the world of Java, so MethodNames...

Methods in Java are actually ALWAYS pascalCase.
In C# it's (most of the time, not sure what's "official") CamelCase for public methods/variables, and pascalCase for private ones.

My classmates generally dislike conditional operators ("1 < 2 ? true : false"). And I do too, except when they help me remove variables or duplication.

▼example

This

String s = "";
if (1 < 2)
{
    s = "correct";
}
else
{
    s = "incorrect";
}

return "1 < 2 = " + s;

or this

if (1 < 2)
{
    return "1 < 2 = correct";
}
// Else obsolete here due to return
return "1 < 2 = incorrect";

would become this

return "1 < 2 = " + (1 < 2 ? "correct" : "incorrect");

I also prefer tabs to be 4 spaces, in whatever language (I noticed PHP, JS, HTML and CSS are commonly written with 2 space indentation).

if(stuff == true)
{
    logic = logic + Math.Pow(logic,0);
}
else if(stuff == false)
{
   logic = logic + 2 - 4;
}
else 
{
   logic = logic + logic * logic + (logic - logic);
}

But... But... That's not a PLSQL/MySQL/whatever procedure or anything, so HOW!? :c

drunkbnu

Formerly HG

Joined: 2017-08-16

Posts: 2,306

Re: So what's your coding style?

• Opening braces at end of line instead of new line

• Lambda expressions instead of delegates

• Statements that run only one method don't use braces

• string.Format instead of string concatenation

• switch instead of if for long statements

• Space before opening parenthesis

• Real tabs instead of spaces

Swarth100

Member

Joined: 2015-07-18

Posts: 305

Re: So what's your coding style?

• Real tabs instead of spaces

You know this will trigger a lot of people?

drunkbnu

Formerly HG

Joined: 2017-08-16

Posts: 2,306

Re: So what's your coding style?

• Real tabs instead of spaces

You know this will trigger a lot of people?

It's your fault for taking the wrong way from the beginning.

Tabs exist for a reason. To indent.

Zumza

Member

From: root

Joined: 2015-02-17

Posts: 4,645

Re: So what's your coding style?

• Real tabs instead of spaces

You know this will trigger a lot of people?

Felling triggered already.

• Opening braces at end of line instead of new line

• Lambda expressions instead of delegates

• Statements that run only one method don't use braces

• string.Format instead of string concatenation

• switch instead of if for long statements

• Space before opening parenthesis(not on functions calls tho) I'd say spaces between any keyword or operator

• Real tabs instead of spaces

This list is quite good and I'd say we should complete it with more good stuff.

I'll add:

• Splitting a line in 2 if it takes too much space

• Nesting avoidance

E.g. I've seen a lot of people that do:

connection.OnMessage += (sender, message) => {
     if (message.Type == "say") {
          <things>
     }
}; 

▼CONTENT WARNING

This is how it should be done:

connection.OnMessage += (sender, message) => {
     if (message.Type != "say") return;
     <things>
};

Nesting avoidance

---

Everybody edits, but some edit more than others

den3107

Member

From: Netherlands

Joined: 2015-04-24

Posts: 1,025

Re: So what's your coding style?

• Real tabs instead of spaces

I actually prefer spaces because they allow me to sub-indent, without having spaces and tabs mixed together everywhere (I think we all can agree that tabs and spaces mixed is the worst kind).

▼Why sub-indent?

For very long method calls/if-statements.
Since I prefer to avoid having to declare 3 extra variables (which contain the results of method calls), just for that one if-statement, I'd generally do something like this:

if (ClassName._instance.VeryLongMethodName(withSome, variables, asParameters) &&
    (someBool || exceptionBool))

Now obviously that if statement it's yet extremely long, but imagine those bools also being method calls with a bunch of parameters etc. etc.
Then, in my opinion, I'd like to sub-indent to make sure everything is indented one space up the if-statement's opening parenthesis.

It's your fault for taking the wrong way from the beginning.

Plenty of IDE's actually default to using spaces instead of tabs. And plenty of the before mentioned plenty, don't even have an option to switch to tabs instead of spaces.
I think the main reason for this is because tab size can differ per system, making some indented stuff a bit strange (namely in complex comments (which, for example, explains how a complex formula works)).

I'll add:

• Nesting avoidance

connection.OnMessage += (sender, message) => {
     if (message.Type != "say") return;
     <things>
};

I'd actually still be quite triggered with that code (not related to nesting avoidance in general).

You generally want to keep your methods single-purpose.
You currently did two things in that method though: Attaching event handlers AND processing those events.
If you'd attach more event handlers, you do even more stuff in that single method.

Besides: By using a method, you can also simulate events easily, by calling the event handling method. Which in turn could help make some specific things easier to make and reduce code duplication (I must admit I haven't had any need for simulation with EE bots, though I have with Unity).

Also: Nesting avoidance wouldn't, theoretically, reduce cyclomatic complexity (by much). Though our brains would commonly find it easier to read since they can eliminate paths more easily.

Does anybody btw use comment documentation (like javadoc in Java) in their personal projects?

Zumza

Member

From: root

Joined: 2015-02-17

Posts: 4,645

Re: So what's your coding style?

• Real tabs instead of spaces

You know this will trigger a lot of people?

It's your fault for taking the wrong way from the beginning.

Tabs exist for a reason. To indent.

Looking at you reminds me of this:

connection.OnMessage += (sender, message) => {
     if (message.Type != "say") return;
     <things>
};

I'd actually still be quite triggered with that code (not related to nesting avoidance in general). It's an example to show what I mean by nesting avoidance. I don't recommend anybody to use such a thing for actual message handling.

You generally want to keep your methods single-purpose. This is a good thing to add on the list.

Also: Nesting avoidance wouldn't, theoretically, reduce cyclomatic complexity (by much). Though our brains would commonly find it easier to read since they can eliminate paths more easily.
It does actually remove a lot of nesting when you intend to put multiple checks in the same method. So check A only if A is true go to step B, only if step B is true etc. I've seen a lot of newbies that end up this scenario in if { if { if { etc }}}

---

Everybody edits, but some edit more than others

Swarth100

Member

Joined: 2015-07-18

Posts: 305

Re: So what's your coding style?

• Real tabs instead of spaces

You know this will trigger a lot of people?

It's your fault for taking the wrong way from the beginning.

Tabs exist for a reason. To indent.

My question is simple: how many spaces to a tab?

Cause indented spaced code will always look the same. Tabbed code might differ given multiple authors.

drunkbnu

Formerly HG

Joined: 2017-08-16

Posts: 2,306

Re: So what's your coding style?

• Real tabs instead of spaces

You know this will trigger a lot of people?

It's your fault for taking the wrong way from the beginning.

Tabs exist for a reason. To indent.

My question is simple: how many spaces to a tab?

Cause indented spaced code will always look the same. Tabbed code might differ given multiple authors.

8 spaces to a tab.

LukeM

Member

From: England

Joined: 2016-06-03

Posts: 3,009

Website

Re: So what's your coding style?

▼HG

• Real tabs instead of spaces

You know this will trigger a lot of people?

It's your fault for taking the wrong way from the beginning.

Tabs exist for a reason. To indent.

My question is simple: how many spaces to a tab?

Cause indented spaced code will always look the same. Tabbed code might differ given multiple authors.

Thats the advantage of tabs, the user can change how they look based on preference, with spaces you would need to use a find and replace if you wanted it to look different, and that could break things if multiple spaces are used in strings, with tabs you can just go into settings and change the size, and then it will automatically change the look of all tabbed text documents to your liking

Also, to answer the question: I usually use 2 spaces to a tab, because I like to program things while having multiple windows open, so like to keep the code as narrow as possible, while still being easy to read