Comments by jfriedman (Top 30 by date)

terinary operations such as (true ? 0 : 1) are basically nested if statements and some can ever short circuit as pointed out below in the alternates. What's insane is not using a TimeSpan when it is a Framework construct.

Reason for my vote of 3
I would have used Enum.HasFlag and I would have used a single switch statement with goto for each case not a if else with nested if statements. Additionally you could just use the result of the time - anotherTime and use TotalSeconds or similar... e.g. someTime - anotherTime >= TimeSpan.FromSeconds(1.5)

may want to fix your formatting though :)

Reason for my vote of 3
Cleaner than last time...

It seems you are not familiar with GoF Principles. Google 'Chain of Responsibility' and 'Gang of Four'... It will greatly help you in the future to understand these principles so that you do not back yourself into these types of corners.

Reason for my vote of 1
Because you are trying to appear as if you know what you are doing. This maybe would have been valid about 20 years ago in c and still might hold true for a unoptomized C compiler however for .Net you clearly display you lack of understanding of the CLR.

Dr. Abell,

Please stop.

http://www.codeproject.com/KB/recipes/CRP.aspx

You are just causing stack overflows and wasting cycles with your code

http://www.codeproject.com/KB/recipes/Iterative_vs_Recursive.aspx

We appreciate your post, your comments have been summarily noted and dually overruled.

Thank you for yout time.

Dr.Abell, int is a value type which can never be null. The bit work you perform with AND and OR and the multiple branches of the code result is a less optimzed function call which utilizes more stack space. These are the hard facts of the matter.

Thank you for your article.

-Reguards.

1.) You are claiming that if both numbers are equal the Euclid part does not need to be run however I am arguing that rather than performing a binary AND followed by a jump if equal or just the jump if equal call you just perform the modulo which will result in a single operation and get the required result which will be much easier for the JITTER to optimize due to less possible branches of code paths to evaluate.
2.) I did not write alternate 1 so I do not care to make points about it.
3.) I have also made this discovery when working on a brain teaser at my work.
4.) Due to recursive calling there is more stack space utilized in your function call.
Sincerely,
Julius

I am not sure how correct your assertation is... I dont' use var once. I accept the point about the two numbers being the same but I would like to see how that realtes to the problem mathematically. E.g. so what if I pass 1,1? is this not the point of the algorithm...

How this classifies as a "Article" is beyond me.... Totally beyond me.

Fortunately I watched Bambi as a child and I took away a valuable lesson, "If you don't have anything nice to say don't say anything at all".

I guess for people who are new to programming this is very helpful... Good Job!

Should we not be using ++i (prefix) to prevent the overhead of the temporary copy of the variable which occurs when you use the postfix increment operator?

Even without unsafe there is no denying that the methods I have shown here and have been shown by others are more efficient then the posted methods... Unsafe code will only marginally increase the performance anyhow. Thank you for the readability edits.

Dooper Super

Your code wont compile... You return false without assigning to the result variable. You probably meant:

else return result = false;

Thank you for clarifying however...

Whitespace variants can be matched with a RegEx and are outside the scope of the name of the functions which implies Whitespace not WhiteSpace Variants...

http://www.csharp411.com/remove-whitespace-from-c-strings/

The name of the function is not IsNullOrWhitespaceOrNewLineorTabOrSomeOtherCharacterWhichTakes0Space()

Reason for my vote of 5
Because its just a fast and does the same thing! (Obviously Sarcastic)