What's missing?
-
if (_Framework.DuplexPlatform)
{
Synchronize.Visibility = Visibility.Visible;
}
{
Synchronize.Visibility = Visibility.Collapsed;
}*face-palm*
Software Zen:
delete this;What else would be missing? :-\
Keep Clam And Proofread -- √(-1) 23 ∑ π... And it was delicious.
-
if (_Framework.DuplexPlatform)
{
Synchronize.Visibility = Visibility.Visible;
}
{
Synchronize.Visibility = Visibility.Collapsed;
}*face-palm*
Software Zen:
delete this; -
A code review ?
~RaGE();
I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Do not feed the troll ! - Common proverb
Given that this product consists of over 1.2 million lines of code (that was the count two years ago), it's a variant of four products built from the same code base, and they're all maintained by the same group of six overworked developers, who has time for code reviews? The good news is our testers found the problem.
Software Zen:
delete this; -
Given that this product consists of over 1.2 million lines of code (that was the count two years ago), it's a variant of four products built from the same code base, and they're all maintained by the same group of six overworked developers, who has time for code reviews? The good news is our testers found the problem.
Software Zen:
delete this; -
You seem to have time to find bugs, so ...
~RaGE();
I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Do not feed the troll ! - Common proverb
Our test group (a fine bunch of folks, actually) found the problem. I posted my original message here because I thought the missing
elsekeyword was funny. Apparently you agile, oops, pardon me, Agile folks don't find such things humorous. I really would like to do code reviews, given the chance. Tell you what. Here are my shoes. Take a walk in them, and let me know when you find the management support and the extra time to do code reviews. I'll wait.Software Zen:
delete this; -
I'm one of those insecure types who always uses braces, unless the
if()fits on a single line.Software Zen:
delete this;Ditto - also I am becoming a firm believer in white space being good in code as it makes it much easier to read.
“That which can be asserted without evidence, can be dismissed without evidence.”
― Christopher Hitchens
-
Mandatory code fix post - Why you not do inline:
Synchronize.Visibility = _Framework.DuplexPlatform ?
Visibility.Visible : Visibility.Collapsed;:-D
speramus in juniperus
I agree that a ternary should be used when apropriate, but I prefer a more explicit formatting.
Synchronize.Visibility = _Framework.DuplexPlatform
? Visibility.Visible
: Visibility.Collapsed;A ternary operetar is used rarely enough to expose it.
Greetings - Jacek
-
I agree that a ternary should be used when apropriate, but I prefer a more explicit formatting.
Synchronize.Visibility = _Framework.DuplexPlatform
? Visibility.Visible
: Visibility.Collapsed;A ternary operetar is used rarely enough to expose it.
Greetings - Jacek
-
Richard Deeming wrote:
Philistine!
And proud about it!
Richard Deeming wrote:
Allman style[^] is the only way to go.
Eh. In my eyes it's ridiculous. Thousand lines between the code that make it unreadable. X|
Richard Deeming wrote:
And while we're at it, it's tabs every time. Anyone who uses spaces to indent their code shall be shot.
Agree! :thumbsup:
SharePoint Consultant and Developer at acocon Author of Primary ROleplaying SysTem I'm the ninth in a row of seven!
Stop! You're both wrong. Allman style and expand tabs to spaces. Heresy! In my code base only one type of whitespace is allowed (ya ya I don't consider newlines to be whitespace).
Windows 8 is the resurrected version of Microsoft Bob. The only thing missing is the Fisher-Price logo. - Harvey
-
Our test group (a fine bunch of folks, actually) found the problem. I posted my original message here because I thought the missing
elsekeyword was funny. Apparently you agile, oops, pardon me, Agile folks don't find such things humorous. I really would like to do code reviews, given the chance. Tell you what. Here are my shoes. Take a walk in them, and let me know when you find the management support and the extra time to do code reviews. I'll wait.Software Zen:
delete this;Where I worked a few years ago, we did code reviews even when rushed because we found that they reduced the amount of development time, even when rushed... no especially when rushed.
Windows 8 is the resurrected version of Microsoft Bob. The only thing missing is the Fisher-Price logo. - Harvey
-
I'm one of those insecure types who always uses braces, unless the
if()fits on a single line.Software Zen:
delete this;That;s not insecurity - that's being a good developer. I upvoted you just because its nice to find folk coding the same standards as me :)
MVVM # - I did it My Way ___________________________________________ Man, you're a god. - walterhevedeich 26/05/2011 .\\axxx (That's an 'M')