- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
static public function checkFingerprint($fingerprint = null)
{
$user = ServiceFactory::getServiceUsers()->getByID((int)$_COOKIE['__MW__accountID']);
$fingerprint = (isset($fingerprint)) ? $fingerprint : Users::getFingerprint();
$flag = (bool)($fingerprint == $user->generateFingerprint());
if ( $flag )
return $user->userID;
else
return 0;
}
не только индусские разработчики бывают весёлыми - это творение взрослого афроамериканского собрата. Перестраховался везде где только мог. Особенно доставляет приведение к bool результата сравнения :)
Lure Of Chaos 19.05.2010 16:53 # +3
Lure Of Chaos 19.05.2010 19:54 # +2
mrbig66 19.05.2010 21:40 # +1
mrbig66 19.05.2010 19:21 # 0
guest 20.05.2010 01:33 # +1
а вообще последние 5 строк функции можно сократить в один тернарный оператор.
newmindcore 20.05.2010 07:15 # 0
сейчас это выглядит так
постарался сохранить максимум авторского колорита :)
cheef 20.05.2010 13:00 # +2
newmindcore 20.05.2010 13:13 # 0
cheef 20.05.2010 13:24 # 0
newmindcore 20.05.2010 13:39 # 0
При желании можно назвать это "поменял один говнокод на другой", но я называю это некоторым изяществом, которое позволяет php.
cheef 20.05.2010 13:41 # 0
Lure Of Chaos 20.05.2010 15:32 # +1
} тоже нечитаемы
один прочитает и все понятно, другому нужна куча промежуточных переменных
cheef 21.05.2010 16:13 # 0
guest 20.05.2010 14:47 # +2
mrbig66 20.05.2010 15:52 # +3