RegExp problem - How do I remove punctuation and whitespaces?

Question

I am trying to solve a palindrome problem. The palindrome can contain letters and numbers, but everything else must be removed. My code below is not returning the correct result.

function palindrome(str) {
  const regex = /[\s]|[[:punct:]]/gi;
  //   const regex = /age/gi;

  const string = str.replace(regex, "").toLowerCase().split("");

  console.log(string);

  let aPointer = 0;
  let bPointer = string.length - 1;

  while (aPointer <= bPointer) {
    if (string[aPointer] !== string[bPointer]) {
      return false;
    }
    aPointer += 1;
    bPointer -= 1;
  }
  return true;
}

console.log(palindrome("My age is 0, 0 si ega ym."));

The output removes the spaces but not the punctuation. Am I getting the syntax wrong?

`:punct:` is a POSIX thing, you can't use those character classes in JavaScript regular expressions. See [How can I strip all punctuation from a string in JavaScript using regex?](https://stackoverflow.com/questions/4328500/how-can-i-strip-all-punctuation-from-a-string-in-javascript-using-regex) — Wyck, Jun 07 '20 at 18:52

score 2 · Answer 1 · answered Jun 07 '20 at 19:21

2

You could use the following:

const regex = /[^a-zA-Z0-9]/gi;

This matches everything that is not a letter or number. You can just add any other exclusions that you may need.

answered Jun 07 '20 at 19:21

FrostyZombi3

619
5
13

score 0 · Answer 2 · edited Jun 07 '20 at 19:07

0

Try the below regex:

const regex = str.replace(/(~|`|!|@|#|$|%|^|&|\*|\(|\)|{|}|\[|\]|;|:|\"|'|<|,|\.|>|\?|\/|\\|\||-|_|\+|=)/g,"");

edited Jun 07 '20 at 19:07

rene

41,474
78
114
152

answered Jun 07 '20 at 19:02

Yaqoob

13
1
6

What about characters that are not on the list? @ScottWells said they wanted only letters and numbers. If some character happens to not be on the list it could still slip through. – jkrei0 Jun 07 '20 at 19:31

jkrei0 · Answer 3 · 2020-06-08T23:17:47.703

0

Your regex expression looks unusual. I've never heard of :punct: for matching punctuation.

When I run that regex, it matches (space), :], [], p], u], n], c], and t]. A screenshot of the syntax highlighting for the expression you used (from RegExr):

So that regex will match either whitespace OR any character in [:punct and then a ].

Changing your regex to: /[^A-Za-z0-9]/gi should solve your problem.

This regex will match anything that is NOT in the ranges A-Z, a-z, or 0-9 so you can remove it.

Edit: Apparently PCRE (PHP) regex allows using [:punct:] for matching punctuation. Regex in JavaScript does not use this and matches it literally.

edited Jun 08 '20 at 23:17

answered Jun 07 '20 at 19:27

jkrei0

188
1
9

Also, I highly recommend RegExr for checking your regex expressions. It has syntax highlighting, will automatically test your expression against text you put in, and gives you an explanation of what your regex does. [Regexr](https://regexr.com) – jkrei0 Jun 07 '20 at 19:47
I have been using regex101.com. That's how I built my regex initially. I'll have to give this a shot. – Scott Wells Jun 07 '20 at 20:05
Oh... I think I may have figured out why you were using `[:punct:]`. regex101.com uses PCRE (PHP) by default, which uses `[:punct:]`. If you choose the javascript flavor it then works as it does in javascript code, and matches `[:punct:]` literally. – jkrei0 Jun 07 '20 at 20:12

RegExp problem - How do I remove punctuation and whitespaces?

3 Answers3