check palindrome in java code

Question

I'm making a really simple app in java code, but for some reason it doesnt work. its a palindrome checker.

here is the code.

MAIN:

public class main { public static void main(String[] args) { Palindroom.palindroomChecker("RACECAR"); } }

`Palindroom class:

public class Palindroom {

public static void palindroomChecker(String input) {
    String omgekeerd = "";
    boolean isPalindroom = false;
    int length = input.length();
    for(int i = 0; i < length; i++ ) {
        String hulp = "" + input.charAt(i);
        omgekeerd = omgekeerd + hulp;
    }
    System.out.println(omgekeerd);
    System.out.println(input);
    if(omgekeerd.equals(input)){
        System.out.println("DIT IS EEN PALINDROOM!");
    }
    else {
        System.out.println("HELAAS, DIT IS GEEN PALINDROOM!");
    }
}

}`

For some reason the check in the if-statement doesnt go as it has to go. As you can see i checked omgekeerd and input and i also checked earlier the length of omgekeerd to see if there were clear spaces.

Can someone help me out!

thanks in advance

greetings Mauro Palsgraaf

Check http://stackoverflow.com/questions/4138827/check-string-for-palindrome?rq=1 — Petteri H, Feb 21 '14 at 22:07

score 2 · Answer 1 · answered Feb 21 '14 at 22:10

Your logic is flawed. You're reconstructing a new string by appending every char of the input, in the same order, and then check that both strings are equal. So your method always says that the input is a palindrome.

You should construct a new string by appending the chars in the reverse order.

Or you could make it faster by checking that the nth character is the same as the character at the length - 1 - n position, for each n between 0 and length / 2.

Petteri H · Accepted Answer · 2014-02-21T22:15:06.370

1

You are not actually reversing the string, looks like omgekeerd will be in the same order as input.

Replace for with for(int i = length-1; i >= 0; i--) {

edited Feb 21 '14 at 22:15

answered Feb 21 '14 at 22:09

Petteri H

11,779
12
64
94

score 0 · Answer 3 · answered Feb 22 '14 at 01:45

0

This can be simplified a lot

 boolean isPalindrome = new StringBuilder(input).reverse().equals(input);

answered Feb 22 '14 at 01:45

renke

1,180
2
12
27

In NETBEANS i get a yellow line under equals saying its on incompatible types. Would you happen to know why? i fixed it with this boolean isPalindroom = input.equals(new StringBuilder(input).reverse().toString()); – Vahx Oct 19 '14 at 09:24

score 0 · Answer 4 · answered Aug 25 '14 at 18:07

Maybe this would work for you?

String str = "madam i'm adam."; // String to compare
str = str.replaceAll("[//\\s,',,,.]",""); // Remove special characters
int len = str.length();
boolean isSame = false;

for(int i =0; i<len;i++){
    if(str.charAt(i) == str.charAt(len-1-i)){
        isSame = true;
    }
    else{
        isSame = false;
        break;
    }
}
if(isSame){
    System.out.print("Equal");
}
else{
    System.out.print("Not equal");
}

Chintan · Answer 5 · 2019-07-04T03:23:13.870

0

i=0;
j=str.length()-1; //length of given string
String str; // your input string
while((i<j)||(i!=j)){
    if(str.charAt(i)!=str.charAt(j)){
               System.out.println("not palindrome");
                     break;
                   }
    i++;
    j--;
}
System.out.print("palindrome");
//this can used for checking without the need of generating and storing a reverse string

edited Jul 04 '19 at 03:23

answered Jul 04 '19 at 02:59

Chintan

1
1

please try to justify your answer, not just code snippet. – BSeitkazin Jul 04 '19 at 03:10

check palindrome in java code

5 Answers5