Re: Value (not) being retuned by a function

From: 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. | `------------------------------------------------------------'

« previous php.general (#227316) next »