Coding Horrors

[GodFather]

Originally Posted by Admin

Today I encountered this piece of code in our Application and i was like WTF..This has been developed by our self proclaimed .Net Expert

This method is supposed to return true if passed string is a number. However, the developer has used interesting logic
If lower case and upper case of the string are equal then it is a number

public static bool isNumber(string strToCompare)
        {
            try
            {
                if (String.IsNullOrEmpty(strToCompare) == false && strToCompare.ToLower() == strToCompare.ToUpper())
return true;
                else
                    return false;
            }
            catch (Exception ex)
            {
                
            }
                return false;
            }
        }

You know, I too have one self proclaimed .NET expert here. Who hates Javascript and one day I told him, do you know how the .NET framework operates. How the postback happens....