Determining whether a number is a palindrome

Question

Can someone improve this for me? It seems ugly.

The method should test whether an incoming number is a palindrome and return a Boolean result.

private static bool IsPalindrome(int value)
{
    var inverseValue = string.Concat(value.ToString().Reverse());
    return inverseValue == value.ToString();
}

Answer 1

Not much there to be ugly to be honest. I guess all I might consider would be the reverse of MrSmith42 and adding functionality to avoid the duplication of ToString().

private static bool IsPalindrome(int value)
{
    var valueStr = value.ToString();    
    return valueStr == string.Concat(valueStr.Reverse());
}

Answer 2

Looks o.k. and clear to me. But you can reduce it to one line, if you want.

private static bool IsPalindrome(int value){
    return string.Concat(value.ToString().Reverse()) == value.ToString();
}

Answer 3

Well it totally depends on what you want as a "clean" result.

If the conversion to string for the reverse bothers you, you can make an extension method to hide it :

public static class Extensions
{
    public static int Reverse(this int input)
    {
        return Convert.ToInt32(string.Concat(input.ToString().Reverse()));
    }
}

so the original method is reduced to :

private static bool IsPalindrome(int value)
{
    return value == value.Reverse();
}

---

If you prefer Fluent expressions, you can use .Equals and make it a one liner :

private static bool IsPalindrome(int value)
{
    return string.Concat(value.ToString().Reverse()).Equals(value.ToString());
}

---

If the duplicate ToString() calls bothers you, you can put it in a temporary variable :

private static bool IsPalindrome(int value)
{
    string valueString = value.ToString();
    string inverse = string.Concat(valueString.Reverse());

    return inverse == valueString;
}

---

And of course if you want to leave it like that (and maybe change the var to string) it would be correct too, as the original method is truly not horrible.

Answer 4

Yes, you can do much better! You don't need to use string. I don't know C#, so feel free to edit my answer.

Here is an idea for a function:

function int reverse(int myNumber)
{
    int lastDigit = myNumber % 10;
    if (lastDigit == myNumber)
    {
        return myNumber;
    }
    else
    {
        int myNumberWithLastDigitDeleted = (myNumber - lastDigit) / 10;
        return lastDigit + 10 * reverse(myNumberWithLastDigitDeleted);
    }
}