Просмотр поста

# ramzes (25.08.2016 / 16:07)
а подробнее?

Я сразу оговорюсь, у нас событие!!!111!!!111
Я вот прямо тут сейчас буду первый раз в жизни писать на пхп перлы

Так что если я напишу какую-нибудь хню, то надо будет мне вежливо обьяснить - мол Петя у тебя тута говнокод :настав:
Смысла вышеупомянутого недоговнокода я не знаю, поетому ограничюсь только коментариями относящимися к простоте чтения кода.

Итак, первое чтобы я сделал, это вынес бы имя файла в переменную, а не копипастил бы его

$filename = app::storage()->path('avatar').'/'.$id.'.png';

Ну потом можно и старый код оставить, так как его стало легко читать

if($type==true){
     return app::storage()->fileExists($filename) ? '/'.$filename : null;

 }else{
     return '/'.$filename;

 }

Или при наличии пива и помутнения разума я бы мог еще написать более гавнистую вычурную строчку.

return ($type==true && app::storage()->fileExists($filename) == false)? null:'/'.$filename;

И да, по имени аргумента 'type' хрен поймешь почему он должен форсать наличие файла