Про простоту и незамысловатость
Но когда я вижу вот такой код, то я готов завести ещё 5 блогов и писать ещё более простые вещи ещё более примитивным языком:
<?php
$idmode = $userdata->user_idmode;
if ($idmode == 'nickname')
$user_identity = $userdata->user_nickname;
if ($idmode == 'login')
$user_identity = $userdata->user_login;
if ($idmode == 'firstname')
$user_identity = $userdata->user_firstname;
if ($idmode == 'lastname')
$user_identity = $userdata->user_lastname;
if ($idmode == 'namefl')
$user_identity = $userdata->user_firstname.
' '.$userdata->user_lastname;
if ($idmode == 'namelf')
$user_identity = $userdata->user_lastname.
' '.$userdata->user_firstname;
if (!$idmode)
$user_identity = $userdata->user_nickname;
?>
$idmode = $userdata->user_idmode;
if ($idmode == 'nickname')
$user_identity = $userdata->user_nickname;
if ($idmode == 'login')
$user_identity = $userdata->user_login;
if ($idmode == 'firstname')
$user_identity = $userdata->user_firstname;
if ($idmode == 'lastname')
$user_identity = $userdata->user_lastname;
if ($idmode == 'namefl')
$user_identity = $userdata->user_firstname.
' '.$userdata->user_lastname;
if ($idmode == 'namelf')
$user_identity = $userdata->user_lastname.
' '.$userdata->user_firstname;
if (!$idmode)
$user_identity = $userdata->user_nickname;
?>
Код я этот откопал в одном популярном блоговом движке, не буду называть его, чтобы не портить людям настроение.
Как бы я написал? Просто: switch() и в дефолте $user_identity = $userdata->user_nickname;
Кстати, знаете, как я понимаю, что написанный код действительно хорош? Когда после N-ной правки я удаляю половину кода, а функциональность не теряется и не ухудшается. Чаще всего так получается сделать, если сделегировать часть работы какой-то другой и более грамотно написанной программе: броузеру, апачу, май-эс-кью-элю, самой операционке... И тогда я слышу чей-то шепот за спиной: keep it simple, stupid :]
0 комментов:
Отправить комментарий