What is Better Code in Your Opinion?
-
What if you need to change the representation for "ON" and "OFF"?
Todd Smith
How does it relate to original poster's question, which I believe was about
if...elseconstruct vs ternary operator? Oh well at least I now know why my style got voted down. :doh:
"Throughout human history, we have been dependent on machines to survive. Fate, it seems, is not without a sense of irony. " - Morpheus
-
:P It's style question. People usually get away with it. IMO its equal to "Do you like code red or pitr-cola more?" kind of question.
"Throughout human history, we have been dependent on machines to survive. Fate, it seems, is not without a sense of irony. " - Morpheus
dnh wrote:
Do you like code red or pitr-cola more
Code red! ;P though I think I miss Jolt Cola the most. :)
_________________________ Asu no koto o ieba, tenjo de nezumi ga warau. Talk about things of tomorrow and the mice in the ceiling laugh. (Japanese Proverb)
-
That's pretty much my thoughts on the matter too, just in case anyone actually cared to know that. :laugh:
Jeremy Falcon "It's a good thing to do and a tasty way to do it." - Wilford Brimley[^]
How about a reward system? Once you've submitted X articles of Y quality you can post anything, anywhere you want :D
Todd Smith
-
dnh wrote:
Do you like code red or pitr-cola more
Code red! ;P though I think I miss Jolt Cola the most. :)
_________________________ Asu no koto o ieba, tenjo de nezumi ga warau. Talk about things of tomorrow and the mice in the ceiling laugh. (Japanese Proverb)
http://ars.userfriendly.org/cartoons/?id=20020520[^] :)
"Throughout human history, we have been dependent on machines to survive. Fate, it seems, is not without a sense of irony. " - Morpheus
-
Ooh c'mon, I wanted to ask that for plain mandarin ;P
WM. What about weapons of mass-construction? "You can always try to smash it with a wrench to fix that. It might actually work" - WillemM
WillemM wrote:
Ooh c'mon, I wanted to ask that for plain mandarin
:doh: I had to read that at least three times before I realized you were talking about the language not the fruit! :) Maybe Jeremy can present the 3D Plain Oatmeal implementation as a demo of his API. :)
Matt Gerrans
-
I wonder why a company would pay you by line, because that first construct could really use some more comments if you get payed by the line ;P
WM. What about weapons of mass-construction? "You can always try to smash it with a wrench to fix that. It might actually work" - WillemM
I dunno, there just seems to be an explosion of people who write so many unnecessary lines that it seems they're geting paid by the line count.
:badger:
-
This:
//If the flag is on then if(READBIT(\*pGlobalFlags, uFlag)) { //Value is set to on bstrValue = "ON"; } else { //Value is set to off bstrValue = "OFF"; }Or this:
bstrValue = READBIT(*pGlobalFlags, uFlag) ? "ON" : "OFF";
:confused:
Matt Gerrans
The second, so you can do:
const BSTR bstrValue = READBIT(*pGlobalFlags, uFlag) ? "ON" : "OFF";I tend to write code with lots of consts (kind of an emulation of a functional style), so I use the conditional operator a lot.
-
This:
//If the flag is on then if(READBIT(\*pGlobalFlags, uFlag)) { //Value is set to on bstrValue = "ON"; } else { //Value is set to off bstrValue = "OFF"; }Or this:
bstrValue = READBIT(*pGlobalFlags, uFlag) ? "ON" : "OFF";
:confused:
Matt Gerrans
Matt Gerrans wrote:
bstrValue = READBIT(*pGlobalFlags, uFlag) ? "ON" : "OFF";
I prefer this one, it looks like "advance source code" :-D:-D
"Courage choose who will follow, Fate choose who will lead" - Lord Gunner, Septerra Core "Press any key to continue, where's the ANY key ?" - Homer Simpsons Drinking gives me amazing powers of insight. I can solve all the worlds problems when drunk, but can never remember the solutions in the morning. - Michael P Butler to Paul Watson on 12/08/03
-
Generally the latter, but in this instance I would prefer the former. The comments document the fact the message refers to the state of the 'value'. Without the comments, I may have thought the function simply returned a success flag. I might expect it place the actual value in one of the parameters. I would then need to look more closely at the definition/implementation/comments of the Read function to determine more about how it worked.
-- The Obliterator
-
Generally the latter, but in this instance I would prefer the former. The comments document the fact the message refers to the state of the 'value'. Without the comments, I may have thought the function simply returned a success flag. I might expect it place the actual value in one of the parameters. I would then need to look more closely at the definition/implementation/comments of the Read function to determine more about how it worked.
-- The Obliterator
Obliterator wrote:
I would then need to look more closely at the definition/implementation/comments of the Read function to determine more about how it worked.
If there were side-effects, this is something i would expect to be indicated, either in code or in comments. Neither block indicated such a thing, the comments in the first merely describing what was implicit in the code.