Could you explain a little better why this would make things better?
I don't understand how this would improve things.
Jim Lucas
----- Original Message -----
From: "Curt Zirzow" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Thursday, August 07, 2003 10:28 AM
Subject: Re: [PHP] Correct Coding
> * Thus wrote Christopher J. Crane ([EMAIL PROTECTED]):
> > Is this the best way to do this?
> >
> > if(isset($Task) && $Task == "Add") { Do something }
> >
> > I want to check if the variable is set and if so, if it is "Add".
>
> Yes, that is good. Remember, though, it is case sensitive so "ADD"
> wont match.
>
> Concerning the $Task == "Add", I'd like to make a comment. It can
> be a wise decision to compare your variables with strings like:
>
> if ("Add" == $Task)
>
> This can help preventing typo's like:
>
> if ($Task = "Add")
>
> I've seen people tear their hair out wondering why the if statement
> is always true even if $Task is not "Add". I don't see this method
> used very often but it can prevent serious logic typos.
>
>
> HTH,
>
> Curt
> --
> "I used to think I was indecisive, but now I'm not so sure."
>
> --
> PHP General Mailing List (http://www.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
