Cookie question

[Jim Davidson]

I learning JavaScript and doing something wrong here, what I want to
do is store the contents of a variable to a cookie so that it would
show the next time the page was opened. I'm doing something wrong in
the code below because instead of saving the value of nLastBonus to
the cookie I'm saving "nLastBonus".

Can someone point me in the right direction?

// Get the cookie
var nLastBonus = get_cookie()

// Did the cookie exist?
if (nLastBonus) {

// If not, then store 100 to nLastBonus
// visit_number = 1
nLastBonus = 100
}
else {

// Otherwise, increment nLastBonus
nLastBonus++
}

// Set the cookie
document.cookie="last_bonus=nLastBonus; expires=" + expire_string

[Lasse Reichstein Nielsen]

ra*****@icubed.com (Jim Davidson) writes:

I'm doing something wrong in the code below because instead of
saving the value of nLastBonus to the cookie I'm saving
"nLastBonus".
I assume the rest works. Change:
document.cookie="last_bonus=nLastBonus; expires=" + expire_string

to
document.cookie="last_bonus="+escape(nLastBonus)"+ ; expires="+expire_string;

Generally, you should escape the values stored in cookies to make sure
they don't contain meaningfull characters. In this case, it isn't
necessary, since numbers are safe, but if nLastBonus was the string
"foo;expires=now;" (or something similar that works) then you will get
trouble.

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

[Janwillem Borleffs]

"Lasse Reichstein Nielsen" <lr*@hotpop.com> schreef in bericht
news:vf**********@hotpop.com...

document.cookie="last_bonus="+escape(nLastBonus)"+ ; expires="+expire_string;

And when you get an error on the above line, change it into:

document.cookie="last_bonus="+escape(nLastBonus)+" ; expires="+expire_string;
JW

[Jim Davidson]

"Janwillem Borleffs" <[e-mail]> wrote in message news:<3f**********************@news.euronet.nl>...

"Lasse Reichstein Nielsen" <lr*@hotpop.com> schreef in bericht
news:vf**********@hotpop.com...

document.cookie="last_bonus="+escape(nLastBonus)"+ ;

expires="+expire_string;

And when you get an error on the above line, change it into:

document.cookie="last_bonus="+escape(nLastBonus)+" ; expires="+expire_string;
JW

Thanks guys that worked!