The Insider News

kalberts11-May-20 6:27

11-May-20 6:27

Eddy Vluggen wrote:
Because they had to read the entire thing and build a model in their head

OK, if you think that understanding how the program works is undesirable (or impossible), and believe that something gets better by tripling the line count in the source file by adding lots of blank lines, putting braces and parentheses and maybe even semicolons on separate lines... If that is your taste, feel free too do so. Even if I dive into some huge system where I don't have a chance in a million to understand the entire code, the very least you could expect is that I understand a single method/function, and hopefully an entire module or even subsystem. If a more "prosaic" layout helps me grasp the logic, I am happy with that, even when I do not understand the entire system.

Eddy Vluggen11-May-20 7:06

Eddy Vluggen

11-May-20 7:06

Member 7989122 wrote:
OK, if you think that understanding how the program works is undesirable (or impossible),

I didn't say that.

You don't need to read the entire book, if the compiler points to a certain chapter. It is ineffective.

Member 7989122 wrote:
If that is your taste

That's not taste, but science. People are good at spotting patterns and seeing deviations. SOLID also means we do not have to read the entire codebase to explain why the logger blew up.

Member 7989122 wrote:
If a more "prosaic" layout helps me grasp the logic, I am happy with that, even when I do not understand the entire system.

"More prosaic" means using more chars to convey the same information, which is what you were advising against.

Yes, that was a set up, and an easy one. Call back if you know what you want; but if I see a blot of spaghetti in SourceSafe I will simply delete it.

..and yes, I'm being "friendly", in a Unix-way.

Bastard Programmer from Hell Suspicious | :suss:

If you can't read my code, try converting it here[^]
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.

kalberts11-May-20 7:14

11-May-20 7:14

Eddy Vluggen wrote:
but if I see a blot of spaghetti in SourceSafe I will simply delete it.

If you "see a bowl of spaghetti" just because a programmer doesn't spend one line on an opening brace (alone) and another line on the closing brace (alone), and maybe a completely blank line before and after the if-statement, then it sounds as if you are really hungry for spaghetti!

Maybe I will start using the term "homeopathic code" for that kind where only a single token carrying any information specific to the problem solution, per screenful (with the rest being blank lines, opening and closing braces and all sorts of decorations) Smile | :)

Eddy Vluggen11-May-20 7:23

Eddy Vluggen

11-May-20 7:23

Member 7989122 wrote:
If you "see a bowl of spaghetti" just because a programmer doesn't spend one line on an opening brace (alone) and another line on the closing brace (alone), and maybe a completely blank line before and after the if-statement, then it sounds as if you are really hungry for spaghetti!

Takes a bit more than that, like "proza".

Member 7989122 wrote:
Maybe I will start using the term "homeopathic code" for that kind where only a single token carrying any information specific to the problem solution, per screenful (with the rest being blank lines, opening and closing braces and all sorts of decorations)

Kinda anal on the bracket-whitespace for a single-line statement there. I don't care about that particalur example; it is being enforced because someone said it was a best-practice to have a codeblock around single-line statements.

"if (bar)
{
foo
}"

vs

"if (bar) foo)" is not worth the money, nor my time.

I once worked for someone who demanded no-one formatting their SQL, and to put the entire statement in a single line (without CR/LF). He read the SQL statement aloud while scrolling to the right. Some bugs are very visible if you apply a minimum of formatting.

Keep it up, I'm reloading.

Bastard Programmer from Hell Suspicious | :suss:

If you can't read my code, try converting it here[^]
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.

David O'Neil11-May-20 5:10

David O'Neil

11-May-20 5:10

Member 7989122 said:
I never understood how putting braces on a separate line "improves readability".

Then you should like my articles! Laugh | :laugh:

It took a few years to come to the same conclusion.

The forgotten roots of science | C++ Programming | DWinLib

Nelek11-May-20 7:13

Nelek

11-May-20 7:13

About readability... I can give you that.
But I have spent a lot of time in my life trying to find errors of execution that were caused by a line of code that was supposed to be in an inlined if without brackets.

I came to the conclusion, than putting the brackets in the very first time, it might spare you a lot of time searching for it.

Member 7989122 wrote:
All of this is explicitly expressed when code goes like (C style):
C++
[blank line]
if (age < 12)
{
    charge = childfare;
    some = other;
}
[blank line]
rather than (Pascal style):

C++
if age < 1 then charge = childfare; some = other;

or even better:

Member 7989122 wrote:
All of this is explicitly expressed when code goes like (C style):
C++
[blank line]
if (age < 12)
{
    some = other;
    charge = childfare;
}
[blank line]
rather than (Pascal style):

C++
if age < 1 then some = other; charge = childfare;

Have fun searching in a big program.

C++

if (age < 12)
    some = other;
    charge = childfare;

You eyes would go over this several times without realizing it is incorrect.

M.D.V.

If something has a solution... Why do we have to worry about?. If it has no solution... For what reason do we have to worry about?
Help me to understand what I'm saying, and I'll explain it better to you
Rating helpful answers is nice, but saying thanks can be even nicer.

kalberts11-May-20 7:33

11-May-20 7:33

E.g. the Coverity analyzer pinpoints such problem as those you point out. (But you have to do the reformatting yourself)

In any case: If you have a code formatter that enforces code layout according to a complete set of rigid rules, then it doesn't matter if the coder writes the code in the way you indicate - the formatter will fix it.
If the formatter does not enforce a rigid set of rules, or if the rule set is incomplete, then there will always be ways in which the coder could mess up things.

So we are talking about either which rules should a formatter enforce, or which rules should the coders write theuir code by. When I am not under the control of some other coding style, among the rules I follow are

* You never have an indent without a bracing construct (which one is language dependent)
* You never have a bracing construct without an indentation.
* You never have a statement separator (such as a semicolon) anywhere but at the end of a line.
* If the body of a structured statement (such as an if) is a single statement (and need no bracketing) and it fits within a line lenght of 80, they it might be accepted on the same line. Otherwise, and it is moved to the next line, it must be indented and braced (according to the first rule).

Having programmed by these rules for thirty years, it would take me a lot of pain to break them. I would never accept it in code that I am responsible for.

#realJSOP11-May-20 1:11

#realJSOP

11-May-20 1:11

Another guy that can't write code, yet he feels qualified to tell everybody else how it should be done.

I put that as succinctly as I could.

".45 ACP - because shooting twice is just silly" - JSOP, 2010
-----
You can never have too much ammo - unless you're swimming, or on fire. - JSOP, 2010
-----
When you pry the gun from my cold dead hands, be careful - the barrel will be very hot. - JSOP, 2013

kalberts11-May-20 1:32

11-May-20 1:32

You make it sound as if "writing code" is dominated by blank lines, possibly with a single brace, and statements of at most 40 char length.

If he "feels qualified to tell everybody else how it should be done", how is that with you? Maybe the difference is that you are qualified to do so?

#realJSOP11-May-20 2:05

#realJSOP

11-May-20 2:05

Member 7989122 wrote:
You make it sound as if "writing code" is dominated by blank lines, possibly with a single brace, and statements of at most 40 char length.

You can put all the comments and blank lines in the code that you want, braces on their own lines, and limit lines to any length you like. The compiler optimizes out anything it can't use, and even optimizes "bloated code" to a point. If you didn't already know that, then maybe you should be writing articles to tell us how to code, as well.

kalberts11-May-20 6:19

11-May-20 6:19

#realJSOP wrote:
The compiler optimizes out anything it can't use, and even optimizes "bloated code" to a point.

I am certainly not worried about how readable the code is to the compiler. I am worried about the humans. Arguing that "the compiler can handle bloated code, so let me write bloated code" doesn't fit well with my ideas about good coding practice.

I actually agree a lot with the author of the referenced article. I have one major point of disagreement: A code indent is triggered by a brace, or similar construct in other languages (e.g. begin-end keywords). A brace (etc.) triggers an indent. The author suggests newline/indent for single statement condition/loop bodies without using braces, that causes me to stall, just as much as I would have if people were using braces without indenting (but that never happens).

Aside from that, I essentially agree with him.

Scientists obtain 'lucky' image of Jupiter

Kevin McFarlane11-May-20 6:45

Kevin McFarlane

11-May-20 6:45

Other things being equal, readability is a desirable end.

A lot of the time LINQ and other functional approaches can make code more readable. But sometimes you're better off doing it the long way. I've often gone for a LINQ solution and then decided that the readability suffers and then backed out.

But sometimes you reach a sweet spot where your succinct code is simultaneously more concise, more expressive, more readable and more robust.

Kevin

Kent Sharkey10-May-20 10:00

Re: Scientists obtain 'lucky' image of Jupiter

10-May-20 10:00

BBC[^]:
Astronomers have produced a remarkable new image of Jupiter, tracing the glowing regions of warmth that lurk beneath the gas giant's cloud tops.

"In place of a Dark Lord, you would have a Queen! Not dark, but beautiful and terrible as the Dawn!"

Nelek11-May-20 7:16

Nelek

11-May-20 7:16

Kent Sharkey wrote:
"In place of a Dark Lord, you would have a Queen! Not dark, but beautiful and terrible as the Dawn!"

My treassssure... Thumbs Up | :thumbsup:

M.D.V.

Microsoft may have tipped the Windows 10 May 2020 Update release date

Kent Sharkey10-May-20 9:45

Microsoft publishes the agenda for its Build 2020 virtual event

10-May-20 9:45

Windows Central[^]:
In an update to its driver shiproom schedule, Microsoft has blocked out May 26-28 as a Feature Update timeframe.

Just in case you want to (temporarily) disable Windows Update

Kent Sharkey10-May-20 9:30

Work-from-home mandates may be DevOps' shining moment

10-May-20 9:30

Neowin[^]:
As developers come together to help the world solve new challenges, sharing knowledge and staying connected is more important than ever. Join your community to learn, connect, and code—to expand your skillset today, and innovate for tomorrow.

Find out what they want you to build in the future

I'm assuming, "Azure, Azure, Azure". Oh, and they're due to have a new Windows coding framework - XAML/Silverlight/UWP/WinUI are getting old. Maybe a retro-"back to Winforms" movement?

Kent Sharkey10-May-20 9:15

Re: Work-from-home mandates may be DevOps' shining moment

10-May-20 9:15

ZDNet[^]:
DevOps -- which syncs, organizes and automates the pace of software releases -- 'is by default built for remote operations' such as that required for the COVID-19 crisis. However, DevOps skills have been hard to find.

When your best moment is when no one is at work to appreciate it?

#realJSOP11-May-20 1:13

#realJSOP

11-May-20 1:13

Kent Sharkey wrote:
However, DevOps skills have been hard to find.

I think the author meant to say "hard to justify". I don't know any developer that thinks we (developers) have any business doing anything other than writing code.

Microsoft adds protection against Reply-All email storms in Office 365

Kent Sharkey10-May-20 9:15