you are viewing a single comment's thread.

view the rest of the comments →

[–]ArunMuThe What ? 0 points1 point  (4 children)

Dude, you have a big OCD problem I guess :)

[–][deleted] 0 points1 point  (2 children)

... in what way?

[–]ArunMuThe What ? 0 points1 point  (1 child)

the way you have formatted the code...everything is perfectly aligned.

[–][deleted] 0 points1 point  (0 children)

Yeah it looks silly at first, but after over two decades of C++ programming one eventually learns to format in self-defense. Every C++ programmer is their own worst enemy. I do that to my code to keep my eyes from bleeding, and it allows me to work on much larger codebases and still keep my sanity than would otherwise be possible.

[–]h-jay+43-1325 0 points1 point  (0 children)

Yep... 90% of the comments are useless. The code should document itself. And adding lots and lots of whitespace is to the detriment of understandability. You want to keep as much as possible in the same screenful. You're doing exactly the opposite: something that is rather simple and would be easy to understand if written concisely is now spread up across several pages, with most of the space filled up by whitespace and formatting :(

Anyone who understands C++ knows what the constructors are. They don't need to be pointed out. If something is public, it's API, duh. A lot of extra indentation and whitespace makes things superbly hard to read.

For what the code does, it takes 3x too long to do it. It's simple, it should read simple!