| 
	
 | 
 Posted by Chris Shiflett on 03/17/05 20:51 
AndreaD wrote: 
> I have about 10 text boxes each taking in value (ages), The code I have 
> checks for a value and if it is set it then sets the cookie to that value. 
> The else just clears the value. On the next page I use a foreach to get the 
> values but I think there must be a quicker way to collect the data appart 
> from 10 if-else staements. 
>  
> if (isset($andrea){ 
>  
> setcookie("cookie[andrea]", "$andrea"); 
> } 
>  
> else {setcookie("cookie[$andrea]", ""); 
>  
> } 
 
Let's look at your two setcookie() calls: 
 
setcookie("cookie[andrea]", "$andrea") 
setcookie("cookie[$andrea]", "") 
 
Just for debugging purposes, assume $andrea is initialized prior to this  
as follows: 
 
$andrea = 29; 
 
So, your code says that you want a cookie with the following name and  
value (available on the next request): 
 
$_COOKIE['cookie[andrea]'] = 29; 
 
Is that really what you want? 
 
Of course, it's very odd that if $andrea is not set, you want a cookie  
like this: 
 
$_COOKIE['cookie[]'] = ''; 
 
In addition, you will be generating a notice, because you're using  
$andrea in the name of the cookie, although you're first making certain  
that $andrea isn't set. 
 
I'm also a bit concerned about where $andrea originates. Is this coming  
from the user, and you're trusting it? That's a very dangerous practice. 
 
If you explain your problem, we might be able to offer some help. 
 
Chris 
 
--  
Chris Shiflett 
Brain Bulb, The PHP Consultancy 
http://brainbulb.com/
 
  
Navigation:
[Reply to this message] 
 |