note 84604 added to function.implode
| From: | morticae at example dot com | 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