note 57381 added to language.variables.predefined

Follow up to "webdesign at benking dot com"'s solution: Why are you using a strpos method as a list when php comes with powerful arrays? This: $allowed_args = ',f_name,l_name,subject,msg,'; foreach(array_keys($_POST) as $k) { $temp = ",$k,"; if(strpos($allowed_args,$temp) !== false) { $$k = $_POST[$k]; } } Could easily be expressed as: $allowed_args = array('f_name', 'l_name', 'subject', 'msg'); foreach($allowed_args as $k) $$k = $_POST[$k]; Maybe arrays are slower than recursively srtpos-ing through strings? (Beats me) Note: Yes, variables == null may be created, but who the heck cares? and if that is a problem, then: foreach($allowed_args as $k) if (isset($_POST[$k])) $$k = $_POST[$k]; ---- Manual Page -- http://www.php.net/manual/en/language.variables.predefined.php Edit -- http://master.php.net/manage/user-notes.php?action=edit+57381 Delete: added to the manual -- http://master.php.net/manage/user-notes.php?action=delete+57381&report=yes&reason=added+to+the+manual Delete: bad code -- http://master.php.net/manage/user-notes.php?action=delete+57381&report=yes&reason=bad+code Delete: spam -- http://master.php.net/manage/user-notes.php?action=delete+57381&report=yes&reason=spam Delete: useless -- http://master.php.net/manage/user-notes.php?action=delete+57381&report=yes&reason=useless Delete: non-english -- http://master.php.net/manage/user-notes.php?action=delete+57381&report=yes&reason=non-english Delete: already in docs -- http://master.php.net/manage/user-notes.php?action=delete+57381&report=yes&reason=already+in+docs Delete: other reasons -- http://master.php.net/manage/user-notes.php?action=delete+57381&report=yes Reject -- http://master.php.net/manage/user-notes.php?action=reject+57381&report=yes Search -- http://master.php.net/manage/user-notes.php