I keep making this sort of mistake: ```s.replace("...

# random

[Ellen Spertus]

I keep making this sort of mistake:

s.replace("\\s+", "")

[Dominaezzz]

Took me a while to parse that. Didn't think anyone actually relied on that feature.

[Ellen Spertus]

I call String.replace() with a regular expression a lot, although usually I use a pre-compiled pattern for efficiency. Do you see the bug in my code?

[Shawn]

no .toRegex() or Regex() call means you’re using .replace(String, String) instead of .replace(Regex, String), the former of which iirc is just a simple search and replace

[Ellen Spertus]

Yeah, I meant s.replace(Regex("\\s+"), "")

[LeoColman]

An alternative would be s.replace("\\s+".toRegex(), "")

[LeoColman]

Maybe this should be a warning from IntelliJ

[LeoColman]

If it could detect that you're trying to use the string as a regex

[Ellen Spertus]

@LeoColman Can you write a regex to detect that? 😉