simple question

[Michael]

Hi,

can anyone tell me why the second 'alert()' is not executed here?
The first 'alert()' pops up (so I know I am inside the 'if'), but not the
second one, and orderNum is not set....

function addbook(bookname){
var orderNum = 0;
var cookies = document.cookie;
var val = cookies.split(";");
var i = 0;
var val2;
while(val != null){
val2 = val.split("=");
if(val2[0].search(/(order)(\d)/) > -1){
alert();
orderNum = $2;
alert("orderNum = "+$2); <<<<<<<<------- this alert
is not displayed
}
i++;
}
orderNum++;
alert("new cookie is: "+"order"+orderNum+"="+bookname+";");
document.cookie = "order"+orderNum+"="+bookname+";";
return true;
}


Thanks for any help

Michael

 

[Ian Collins]

Hi,

can anyone tell me why the second 'alert()' is not executed here?
The first 'alert()' pops up (so I know I am inside the 'if'), but not the
second one, and orderNum is not set....

function addbook(bookname){
var orderNum = 0;
var cookies = document.cookie;
var val = cookies.split(";");
var i = 0;
var val2;
while(val != null){
val2 = val.split("=");
if(val2[0].search(/(order)(\d)/) > -1){
alert();
orderNum = $2;

$2?

 

[scriptguru]

Try to replace $2 with RegExp.$2

Val Polyakh
http://trickyscripter.com

 

[Michael]

Thanks for that.

Regards
Michael

Hi,

can anyone tell me why the second 'alert()' is not executed here?
The first 'alert()' pops up (so I know I am inside the 'if'), but not
the second one, and orderNum is not set....

function addbook(bookname){
var orderNum = 0;
var cookies = document.cookie;
var val = cookies.split(";");
var i = 0;
var val2;
while(val != null){
val2 = val.split("=");
if(val2[0].search(/(order)(\d)/) > -1){
alert();
orderNum = $2;

$2?

It's a perfectly valid variable name. In this case a global variable since
the OP doesn't appear to have defined it anywhere in the posted code, and
it probably has an undefined value since it looks as though the OP has it
confused with the placeholders in a regular expression substitution.

Either the match method on the String, or the exec method on the RegExp
would work better:

var match = /(order)(\d)/.exec(val2[0]);
if (match) {
ordernum = m[2];
}

although only of course for orders 0-9.

 

[Dr John Stockton]

Try to replace $2 with RegExp.$2

Val Polyakh
http://trickyscripter.com

Why is a scriptguru recommending a deprecated form?

<URL:http://developer.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Deprecated_Features>

It's a good idea to read the newsgroup and its FAQ. See below.