Get Text Between - getBtw()

[Seige]

I'm trying to create a function that'll get text in between a string
that returns an Array. It should allow patterm matching so it's
necessary to use the match() function.

However, this function I made doesn't work on MSIE for particularly
unknown reasons. Can anybody take a look at it?

function getBtw(x, y, z) {
var p = new Array();
y = x.match(y);
for (i in y){
x = x.substr(x.indexOf(y[i]) + y[i].length);
z = x.match(z);
if (typeof (z) != "string") { z = z[0]; }
p[i] = x.substring(0, x.indexOf(z));
}
return p;
}

where
x is the entire string,
y is the start pattern,
z is the end pattern

The original function can be found here, but it only returns a string
of the first occurance of the text:
http://javascript-library.blogdrive.com/archive/13.html

I suspect this might be the inconsistency in browser using the match
function.

[Seige]

I found the problem, but its still a mystery.

function getBtw(x, y, z) {
var p = new Array();
y = x.match(y);
for (i in y){
x = x.substr(x.indexOf(y[i]) + y[i].length); //x.indexOf(y[i])
returns -1 in MSIE
z = x.match(z);
if (typeof (z) != "string") { z = z[0]; }
p[i] = x.substring(0, x.indexOf(z));
}
return p;

}

at the line where I left a comment, it returns -1 in MSIE. Does anybody
know how to get pass this?

[Seige]

Does anybody have any suggestions?

Seige wrote:

I found the problem, but its still a mystery.

function getBtw(x, y, z) {
var p = new Array();
y = x.match(y);
for (i in y){
x = x.substr(x.indexOf(y[i]) + y[i].length); //x.indexOf(y[i])
returns -1 in MSIE
z = x.match(z);
if (typeof (z) != "string") { z = z[0]; }
p[i] = x.substring(0, x.indexOf(z));
}
return p;

}

at the line where I left a comment, it returns -1 in MSIE. Does anybody
know how to get pass this?

[Lasse Reichstein Nielsen]

"Seige" <bl******@gmail.com> writes:

Seige wrote:

y = x.match(y);
for (i in y){
x = x.substr(x.indexOf(y[i]) + y[i].length); //x.indexOf(y[i])
returns -1 in MSIE .... at the line where I left a comment, it returns -1 in MSIE. Does anybody
know how to get pass this?

Does anybody have any suggestions?

What strings and regexps are you testing with?
What is the value if x and y[i] where it fails?

/L
--
Lasse Reichstein Nielsen - lr*@hotpop.com
DHTML Death Colors: <URL:http://www.infimum.dk/HTML/rasterTriangleDOM.html>
'Faith without judgement merely degrades the spirit divine.'

[Lasse Reichstein Nielsen]

Lasse Reichstein Nielsen <lr*@hotpop.com> writes:

"Seige" <bl******@gmail.com> writes:

Seige wrote:

y = x.match(y);
for (i in y){
x = x.substr(x.indexOf(y[i]) + y[i].length); //x.indexOf(y[i])
returns -1 in MSIE

Does anybody have any suggestions?

.... What is the value if x and y[i] where it fails?

And, more importantly, what is the value of "i"?

From MSDN:
---
The array returned by the match method has three properties, input,
index and lastIndex.
---
You are using "for(i in y)" to iterator over the properties of the
array in "y", but that array has been extended with three extra
properties.

Try doing "for(i = 0; i < y.length; i++)" instead, then it should
only interate over the actual matched strings.

/L
--
Lasse Reichstein Nielsen - lr*@hotpop.com
DHTML Death Colors: <URL:http://www.infimum.dk/HTML/rasterTriangleDOM.html>
'Faith without judgement merely degrades the spirit divine.'

[Seige]

Thanks Lasse Reichstein Nielsen, that's a nice name.

You're right, I changed it as you suggested and it worked on IE.
However I still don't understand why.

function getBtw(x, y, z) {
var p = new Array();
y = x.match(y);
for (i in y){
x = x.substr(x.indexOf(y[i]) + y[i].length); //x.indexOf(y[i])
returns -1 in MSIE
z = x.match(z);
if (typeof (z) != "string") { z = z[0]; }
p[i] = x.substring(0, x.indexOf(z));
}
return p;
}

Now that this set of codes work, I found several other problems... if
there isn't a match for y or z, it'll fail... And since I'm trying to
avoid using try{}catch(e){}, I'll have to recode it.

Will update here once I get it done.
Lasse Reichstein Nielsen wrote:

Lasse Reichstein Nielsen <lr*@hotpop.com> writes:

"Seige" <bl******@gmail.com> writes:

Seige wrote:
y = x.match(y);
for (i in y){
x = x.substr(x.indexOf(y[i]) + y[i].length); //x.indexOf(y[i])
returns -1 in MSIE

Does anybody have any suggestions?

...

What is the value if x and y[i] where it fails?

And, more importantly, what is the value of "i"?

From MSDN:
---
The array returned by the match method has three properties, input,
index and lastIndex.
---
You are using "for(i in y)" to iterator over the properties of the
array in "y", but that array has been extended with three extra
properties.

Try doing "for(i = 0; i < y.length; i++)" instead, then it should
only interate over the actual matched strings.

/L
--
Lasse Reichstein Nielsen - lr*@hotpop.com
DHTML Death Colors: <URL:http://www.infimum.dk/HTML/rasterTriangleDOM.html>
'Faith without judgement merely degrades the spirit divine.'

[Seige]

I fixed it and here is the updated version: Sadly after all the trouble
I went through I didn't use the for() function at all :p

getBtw = function (x, y, z) {
var p = new Array();
while ( x.match(y) && x.match(z) ){
var xy = x.match(y);
if (typeof (xy) != "string") { xy = xy[0]; }
x = x.substr(x.indexOf(xy) + xy.length);
var xz = x.match(z);
if (xz){
if (typeof (xz) != "string") { xz = xz[0]; }
xz = x.indexOf(xz);
p[p.length] = x.substring(0, xz);
x = x.substr(xz);
} //if xz
} //while
return p; }

This function will allow you to get some text between a string and it
supports pattern matching. It'll return first occurance of each match,
and returns an array.

[Seige]

Get Text Between - getBtw()
http://javascript-library.blogdrive.com/archive/13.html
Seige wrote:

I fixed it and here is the updated version: Sadly after all the trouble
I went through I didn't use the for() function at all :p

getBtw = function (x, y, z) {
var p = new Array();
while ( x.match(y) && x.match(z) ){
var xy = x.match(y);
if (typeof (xy) != "string") { xy = xy[0]; }
x = x.substr(x.indexOf(xy) + xy.length);
var xz = x.match(z);
if (xz){
if (typeof (xz) != "string") { xz = xz[0]; }
xz = x.indexOf(xz);
p[p.length] = x.substring(0, xz);
x = x.substr(xz);
} //if xz
} //while
return p; }

This function will allow you to get some text between a string and it
supports pattern matching. It'll return first occurance of each match,
and returns an array.