Skip to main content
removeu 3 caracteres do conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Além deste código não ser idiomático C# (parece que foi escrito em outra linguagem), ele faz muitas coisas sem necessidade, inclusive alocação de memória.

A principal mudança é não alocar um novo objeto que é algo que pode criar um problema de mudança, só fiz comparações porque o problema só pede istoisso. E comparar um por um é a melhor forma. Apesar de não ser visível esses métodos que está usando são compostos por laços, eles varrem os dados inteiros todas as vezes, é muito ruim.

Outra mudança é sair tão logo saiba que não é palíndromo, e istoisso deve ocorrer muito, é fácil não ser. Quando você já sabe que não é não tem porque continuar o algoritmo.

Por último verifico apenas até a metade, porque se eu continuar o resultado já é sabido. O que diferiria eu analisar o primeiro com o último e depois o último com o primeiro? Dá no mesmo.

Não melhorei a validação porque não sei os requisitos.

using static System.Console;
                    
public class Program {
    public static void Main() {
        WriteLine(checkPalindrome("ana"));
        WriteLine(checkPalindrome("abba"));
        WriteLine(checkPalindrome("oki"));
    }
    static bool checkPalindrome(string inputString) {
        if(inputString.Length >= 1 && inputString.Length <= 100_000) {
            for (var i = 0; i < inputString.Length / 2; i++) if (char.ToLower(inputString[i]) != char.ToLower(inputString[inputString.Length - i - 1])) return false;
            return true;
        }
        return false;
    }
}

Veja funcionando no .NET Fiddlefuncionando no ideone. E no Coding Groundno .NET Fiddle. Também coloquei no GitHub para referência futura.

Além deste código não ser idiomático C# (parece que foi escrito em outra linguagem), ele faz muitas coisas sem necessidade, inclusive alocação de memória.

A principal mudança é não alocar um novo objeto que é algo que pode criar um problema de mudança, só fiz comparações porque o problema só pede isto. E comparar um por um é a melhor forma. Apesar de não ser visível esses métodos que está usando são compostos por laços, eles varrem os dados inteiros todas as vezes, é muito ruim.

Outra mudança é sair tão logo saiba que não é palíndromo, e isto deve ocorrer muito, é fácil não ser. Quando você já sabe que não é não tem porque continuar o algoritmo.

Por último verifico apenas até a metade, porque se eu continuar o resultado já é sabido. O que diferiria eu analisar o primeiro com o último e depois o último com o primeiro? Dá no mesmo.

Não melhorei a validação porque não sei os requisitos.

using static System.Console;
                    
public class Program {
    public static void Main() {
        WriteLine(checkPalindrome("ana"));
        WriteLine(checkPalindrome("abba"));
        WriteLine(checkPalindrome("oki"));
    }
    static bool checkPalindrome(string inputString) {
        if(inputString.Length >= 1 && inputString.Length <= 100_000) {
            for (var i = 0; i < inputString.Length / 2; i++) if (char.ToLower(inputString[i]) != char.ToLower(inputString[inputString.Length - i - 1])) return false;
            return true;
        }
        return false;
    }
}

Veja funcionando no .NET Fiddle. E no Coding Ground. Também coloquei no GitHub para referência futura.

Além deste código não ser idiomático C# (parece que foi escrito em outra linguagem), ele faz muitas coisas sem necessidade, inclusive alocação de memória.

A principal mudança é não alocar um novo objeto que é algo que pode criar um problema de mudança, só fiz comparações porque o problema só pede isso. E comparar um por um é a melhor forma. Apesar de não ser visível esses métodos que está usando são compostos por laços, eles varrem os dados inteiros todas as vezes, é muito ruim.

Outra mudança é sair tão logo saiba que não é palíndromo, e isso deve ocorrer muito, é fácil não ser. Quando você já sabe que não é não tem porque continuar o algoritmo.

Por último verifico apenas até a metade, porque se eu continuar o resultado já é sabido. O que diferiria eu analisar o primeiro com o último e depois o último com o primeiro? Dá no mesmo.

Não melhorei a validação porque não sei os requisitos.

using static System.Console;
                    
public class Program {
    public static void Main() {
        WriteLine(checkPalindrome("ana"));
        WriteLine(checkPalindrome("abba"));
        WriteLine(checkPalindrome("oki"));
    }
    static bool checkPalindrome(string inputString) {
        if(inputString.Length >= 1 && inputString.Length <= 100_000) {
            for (var i = 0; i < inputString.Length / 2; i++) if (char.ToLower(inputString[i]) != char.ToLower(inputString[inputString.Length - i - 1])) return false;
            return true;
        }
        return false;
    }
}

Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.

Reversão para edição 2
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
            using static System.Console;
                                
            public class Program {
                public static void Main() {
                    WriteLine(checkPalindrome("ana"));
                    WriteLine(checkPalindrome("abba"));
                    WriteLine(checkPalindrome("oki"));
                }
                static bool checkPalindrome(string inputString) {
                    if(inputString.Length >= 1 && inputString.Length <= 100_000) {
                        for (var i = 0; i < inputString.Length / 2; i++) 
                              if (char.ToLower(inputString[i]) != char
                                      .ToLower(inputString[inputString.Length - i - 1])) 
                                  return false;
                        return true;
                    }
                    return false;
                }
            }
            using static System.Console;
                                
            public class Program {
                public static void Main() {
                    WriteLine(checkPalindrome("ana"));
                    WriteLine(checkPalindrome("abba"));
                    WriteLine(checkPalindrome("oki"));
                }
                static bool checkPalindrome(string inputString) {
                    if(inputString.Length >= 1 && inputString.Length <= 100_000) {
                        for (var i = 0; i < inputString.Length / 2; i++) 
                              if (char.ToLower(inputString[i]) != char
                                      .ToLower(inputString[inputString.Length - i - 1])) 
                                  return false;
                        return true;
                    }
                    return false;
                }
            }
using static System.Console;
                    
public class Program {
    public static void Main() {
        WriteLine(checkPalindrome("ana"));
        WriteLine(checkPalindrome("abba"));
        WriteLine(checkPalindrome("oki"));
    }
    static bool checkPalindrome(string inputString) {
        if(inputString.Length >= 1 && inputString.Length <= 100_000) {
            for (var i = 0; i < inputString.Length / 2; i++) if (char.ToLower(inputString[i]) != char.ToLower(inputString[inputString.Length - i - 1])) return false;
            return true;
        }
        return false;
    }
}
Só mesmo para ficar sem o scrool cá em baixo e o código ficar todo visível
Fonte Link
Amadeu Antunes
  • 3,4mil
  • 3
  • 31
  • 66
            using static System.Console;
                                
            public class Program {
                public static void Main() {
                    WriteLine(checkPalindrome("ana"));
                    WriteLine(checkPalindrome("abba"));
                    WriteLine(checkPalindrome("oki"));
                }
                static bool checkPalindrome(string inputString) {
                    if(inputString.Length >= 1 && inputString.Length <= 100_000) {
                        for (var i = 0; i < inputString.Length / 2; i++) 
                              if (char.ToLower(inputString[i]) != char
                                      .ToLower(inputString[inputString.Length - i - 1])) 
                                  return false;
                        return true;
                    }
                    return false;
                }
            }
using static System.Console;
                    
public class Program {
    public static void Main() {
        WriteLine(checkPalindrome("ana"));
        WriteLine(checkPalindrome("abba"));
        WriteLine(checkPalindrome("oki"));
    }
    static bool checkPalindrome(string inputString) {
        if(inputString.Length >= 1 && inputString.Length <= 100_000) {
            for (var i = 0; i < inputString.Length / 2; i++) if (char.ToLower(inputString[i]) != char.ToLower(inputString[inputString.Length - i - 1])) return false;
            return true;
        }
        return false;
    }
}
            using static System.Console;
                                
            public class Program {
                public static void Main() {
                    WriteLine(checkPalindrome("ana"));
                    WriteLine(checkPalindrome("abba"));
                    WriteLine(checkPalindrome("oki"));
                }
                static bool checkPalindrome(string inputString) {
                    if(inputString.Length >= 1 && inputString.Length <= 100_000) {
                        for (var i = 0; i < inputString.Length / 2; i++) 
                              if (char.ToLower(inputString[i]) != char
                                      .ToLower(inputString[inputString.Length - i - 1])) 
                                  return false;
                        return true;
                    }
                    return false;
                }
            }
adicionou 290 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando