Re: Value (not) being retuned by a function
| From: | Robert Cummings | Date: | Sun, 11 Dec 2005 20:03:21 +0000 |
| Subject: | Re: Value (not) being retuned by a function | ||
| References: | 1 | Groups: | php.general |
| Request: | Send a blank email to php-general+get-227316@lists.php.net to get a copy of this message | ||
On Sun, 2005-12-11 at 14:55, MARG wrote:
> Hi,
>
> I have this form:
> http://www.tuxdoit.com/newsletter.php
>
> If you care to check it, you'll see that
> http://www.tuxdoit.com/formsResult.php
>
> never returns the function value:
>
> formsResult.php:
> -------------------------------------
> <?php
> include('functions.php');
> ini_set("display_errors","On");
>
> $email = $_REQUEST['email'];
>
> $emailResult = isEmailOk($email);
>
> print($emailResult);
>
> if ($emailResult == 1)
> print('Email Ok');
> else
> print('Email is not Ok');
>
> ?>
> --------------------------------
>
> functions.php:
> --------------------------------
> <?php
>
> function isEmailOk($email) {
> print($email);
> if (!eregi
>
> ("^([-!#\$%&'*+./0-9=?A-Z^_
a-z{|}~^?])+@([-!#\$%&'*+/0-9=?A-Z^_a-z{|}~^?]+\\.)+[a-zA-Z]{2,6}\$",
> $email))
> return 1;
>
> } // is EmailOk
>
> ?>
> -------------------------------
>
> Why is that happening ?
I would wager because there's a ! operator preceding the ereg() call.
Also you have very dirty code, your isEmailOk() function doesn't return
a value in the case that the if expression fails, it assumes PHP will
return something useful (PHP returns null, but relying on that is asking
for trouble, and I doubt you knew so in the first place).
Cheers,
Rob.
--
.------------------------------------------------------------.
| InterJinn Application Framework - http://www.interjinn.com
|
:------------------------------------------------------------:
| An application and templating framework for PHP. Boasting |
| a powerful, scalable system for accessing system services |
| such as forms, properties, sessions, and caches. InterJinn |
| also provides an extremely flexible architecture for |
| creating re-usable components quickly and easily. |
`------------------------------------------------------------'