Может войдёшь?
Черновики Написать статью Профиль
Eskono

Eskono +4

Вступил в наши ряды: 20 февраля 2017

Замечен в последний раз: 13 ноября 2018

Оставил на форуме: 28 сообщений

Последнее сообщение: 24 октября 2018

Вы сможете отправить письмо, если войдёте

Комментарии (5)

Eskono

public function isAuthor($user) {
if ($user) {
if($this->trashed()) {
return null;
}

if ($this->users()->find($user->id)) {
return true;
} else {
return false;
}
}
return'mismatch';
}

чем меньше вложенность ифов, тем удобнее читать код, лучше так:

if (!user) {
return 'mismatch';
}
...тут остальное без вложенности

и еще — лучше все тексты по типу «mismatch» выносить куда-нибудь в константы

Eskono

$data = [];
$data ['title'] = $this->title;
$data ['articles'] = $articles_trash_user;
$data ['auth_user'] = Auth::user();

почему не так?
$data = [
'title' ⇒ $this->title,
......
];

Eskono

array_key_exists(«filename»,$data) && $data['filename']

а не проще поставить проверку !empty($data['filename']) вместо вышеуказанного?

Eskono

К чему этот пост на главной?