How can i merge the following into one regex?
str
.replace(/(\r\n|\n|\r)/gm,"") // get rid of line breaks
.replace(/\s/g, ""); // get rid of spaces
I want to call replace() once and take care of both in same regex
Asked
Active
Viewed 208 times
0
shmnsw
- 639
- 2
- 11
- 24
-
1Did you try anything? You could at least add another alternative to the end of the first pattern and check. However, `\s` matches `\n` and `\r`, so `\s` is enough. – Wiktor Stribiżew Dec 10 '15 at 10:41
-
http://stackoverflow.com/questions/6394416/replace-excess-whitespaces-and-line-breaks-with-php – brandelizer Dec 10 '15 at 10:41
-
str.replace(/(\r\n|\n|\r|\s)/gm,"") fixed it – shmnsw Dec 10 '15 at 10:50
-
@shmnsw You do not need to match `\r` and `\n`... See the link in my answer. – F.P Dec 10 '15 at 10:53
3 Answers
3
I guess you only need to be using \s... that normally includes [\r\n\t\f ], so the first replace is basically useless.
F.P
- 17,421
- 34
- 123
- 189
0
regexr.com is a great tool for testing regular expressions as it explains what each part does. for example if you hover over a /s it will tell you that it matches line breaks as well.
Gabor Magyar
- 926
- 2
- 9
- 20
