Would this solution be acceptable?

JavaScript
1 
function palindrome(str) {
    let rStr = str
    .toLowerCase()
    .split(/[_\W\s]+/g)
    .map(word => word.split("").reverse().join(""))
    .reverse()
    .join(""); 
    if(rStr === str.toLowerCase().split(/[_\W\s]+/g).join("")){
      return true;
    }
    return false; 
  }
  
  palindrome("A man, a plan, a canal. Panama");
2

Did it pass the tests?

3

If it passes, it’s acceptable.

You could tidy it up a bit though. You split, reverse, and join inside of a map for a split, reverse, and join. This means that you’re repeating yourself.

Repeating yourself is a bit of a red flag that you’re doing more work than you need to.

4

Yeah that’s what I was wondering. How do I make it more dry?

5

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.