Javascript Regex Replace - Not replacing all the required strings

Question

I have no idea what is happening here..

model.attributes.data.Path.replace('/\\/g',""), @options.path.replace('/\\/g',"")

When doing :

console.log model.attributes.data.Path.replace('/\\/g',""), 
@options.path.replace('/\\/g',"")

the data is:

T2/T2_2, T2/T2_2

It returns this:

T2T2_2, T2/T2_2

So only the first path was replaced, but not the second one? Why would that be?

no, thats not what I am doing. Im doing two replaces, its not one string. Im just logging them next to each other — Harry
– Harry, Commented Jan 23, 2013 at 14:10
114 questions and did not learn how to format your code in your questions? — Engineer
– Engineer, Commented Jan 23, 2013 at 14:10

Cerbrus · Accepted Answer · 2013-01-23 14:15:08Z

2

Aside from the fact you're matching backslashes (\\ = \), instead of forward slashes (\/ = /), Don't put your regexes into the replace function as strings.

Use:

.replace(/\//g,"");

Instead of

.replace('/\//g',"");

Then it'll work just fine:

"T2/T2_2 , T2/T2_2".replace(/\//g,"");
// returns: "T2T2_2 , T2T2_2"

Otherwise, it'll just try to literally find the string '/\//g'.

Also, to replace both forward and backslashes in 1 regex, try this:

"T2/T2_2 , T2\T2_2".replace(/\/|\\/g,"");
// returns: "T2T2_2 , T2T2_2"

# \/|\\ Matches:
# \/  - Forward slash
# |   - Or
# \\  - Backslash

edited Jan 23, 2013 at 14:15

answered Jan 23, 2013 at 14:10

Cerbrus

73.3k19 gold badges138 silver badges151 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

CD.. · Accepted Answer · 2013-01-23 14:09:20Z

1

Try .replace(/\//g,"") instead of .replace('/\\/g',""), (the regex is not a string).

answered Jan 23, 2013 at 14:09

CD..

74.4k25 gold badges160 silver badges170 bronze badges

Comments

ATOzTOA · Accepted Answer · 2013-01-23 15:48:24Z

1

Try:

model.attributes.data.Path.replace(/\//g,"")
@options.path.replace(/\//g,"")

/\\/g matches a backslash and /\//g matches a forward slash.

edited Jan 23, 2013 at 15:48

answered Jan 23, 2013 at 14:08

ATOzTOA

36.1k23 gold badges101 silver badges119 bronze badges

2 Comments

Harry Over a year ago

That is actually the problem, I missed it and neglected to write it in the question. The first one contains a backslash and the second a forward. it was actually T2\T2_2 , T2/T2_2

Cerbrus Over a year ago

@Harry, noticed, but only part of the problem :P

Collectives™ on Stack Overflow

Javascript Regex Replace - Not replacing all the required strings

3 Answers 3

Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related