Robert Cummings wrote: > On Sun, 2007-01-21 at 16:27 +0100, Jochem Maas wrote: >> Robert Cummings wrote: >>> On Sun, 2007-01-21 at 16:02 +0100, Jochem Maas wrote: >>>> <?php >>>> >>>> $initial = (is_string($forename) && strlen($forename) > 0) >>>> ? $forename[0] >>>> : '' >>>> ; >>>> >>>> echo trim($initial.' '.$surname); >>>> >>>> ?> >>> That sure is verbose Jochem... >> agreed, it was done on purpose in the spirit of 'give the OP >> a hint about not assuming anything about the input', I could have done this: >> >> <?= trim(@$forename[0].' '),$surname; ?> >> >> which is as short as I can make it :-) it also assumes that the OP >> actually santized the incoming data ($forename) before doing *anything* with >> it. > > Oh, I didn't make mine as short as I could, I sanitized it, and didn't > cheat by using the error suppression operator ;) consider it my evil streak :-) indeed yours is the better. > >>> echo trim( substr( (string)$forename, 0, 1 ).' '.$surname ); now we can get on with having a flame war as to the 'best' way to style your code, because obviously this is better ;-) ... echo trim(substr((string)$forename, 0, 1).' '.$surname); > > Cheers, > Rob. -- PHP General Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php