note 84604 added to function.implode

From: Date: Mon, 21 Jul 2008 16:51:13 +0000
Subject: note 84604 added to function.implode
Groups: php.notes 
Request: Send a blank email to php-notes+get-142431@lists.php.net to get a copy of this message
re: m dot ford Sorry to be picky, but your solution doesn't account for the base-case empty array. If you called: implode_array('before', 'after', 'glue', array()); you would get: 'before after' <?php function implode_wrapped($before, $after, $glue, $array) { if(!empty($array)){ return $before . implode($after.$glue.$before, $array) . $after; } return ''; } ?> Webmaster's solution works because substr will return an empty string instead of an error. This is not the case in all languages. ---- Server IP: 64.71.164.2 Probable Submitter: 72.48.145.242 ---- Manual Page -- http://www.php.net/manual/en/function.implode.php Edit -- https://master.php.net/note/edit/84604 Del: integrated -- https://master.php.net/note/delete/84604/integrated Del: useless -- https://master.php.net/note/delete/84604/useless Del: bad code -- https://master.php.net/note/delete/84604/bad+code Del: spam -- https://master.php.net/note/delete/84604/spam Del: non-english -- https://master.php.net/note/delete/84604/non-english Del: in docs -- https://master.php.net/note/delete/84604/in+docs Del: other reasons-- https://master.php.net/note/delete/84604 Reject -- https://master.php.net/note/reject/84604 Search -- https://master.php.net/manage/user-notes.php

« previous php.notes (#142431) next »