Palindrome (flawed)

JavaScript
1 
// My code can detect whether a str is not a palindrome but cant decipher palindromes. Any hints would be appreciated.
function palindrome(str) {
 let adjStr = str.toLowerCase().replace(/[^a-z0-9]/g, "");
 for(let i=0;i<adjStr.length; i++) {
    if(adjStr[i] != adjStr.length-[i]){
      return false;
  }else{
    return true;
  }
}
}

palindrome(“eye”);

1 Like
2

Please put the link to the challenge back. That makes it easier for people to help you.

3

What does this part do?

4

It flips the chronology of the loop ( for idi, i = 1, d = 2, i = 3) becomes i (prev representing 3 = 1)

5

That’s what you intend for it to do, but what does that syntax literally mean?

6

the funny thing, but the expression will give the expected result.

7

No… It won’t give the desired result

8

5 - [2] // 3
i mean that

9

Right, but that’s not the entire purpose of that expression.

10

You’ve completely stumped me What does (adjStr[i] != adjStr.length-[i]) this actually do?

11

Yes. You’re right. I just looked at the piece of expression and missed the point.

12

This is a letter in the string, right?

Is this a letter from the string?

1 Like
13

Ive tried invalidating my presupposition but I can’t find anything to ```
var lastChar = myString[myString.length -1];

OK but now when I alter my code to become ```
if(adjStr[i]!=adjStr[adjStr.length-i])
         return false;

Im still left with errors

1 Like
14

Careful here. This is close, but you want to start at

1 Like
15 
// Your a saint for walking me through this.
function palindrome(str) {
  
   let adjStr = str.toLowerCase().replace(/[^a-z0-9]/g, "");
   for(let i=0;i<adjStr.length; i++) {
     if(adjStr[adjStr.length + i] !=adjStr[adjStr.length-i]){
         return false;
     }
    else{
      return true;
    }
  
   }}
  

palindrome("eye");
16

But it now only picks up some of the palindromes

correction now i can find palindromes but anything that isnt a palindrome is left undiscovered

17

which letters are you comparing?
will that let you compare the first and the last one?

it seems to me you have undefined lots of time in there.

1 Like
18

gotcha another loop is required

19

no, it’s not, you need to fix the comparison

adjStr[adjStr.length + i] !=adjStr[adjStr.length-i]

what are the values in here for each value of i?

1 Like
20 
function palindrome(str) {
  
    let adjStr = str.toLowerCase().replace(/[^a-z0-9]/g, "");
    for(let i = 0; i < adjStr.length; i++) {
      ***if(adjStr[i] !== adjStr[adjStr.length -1 - i]){***
***          return false;***
      }
     else{
       return true;
     }
   
    }}
   
 
 palindrome("eye");

// I’ve identified why my prev solution didnt work
//However, palindrome(“almostomla”)should returnfalse` . Is remaining a painful thorn in my side.