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

Комментарии 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'…

raketa

Отдельное спасибо, за такие комментарии) Возьму себе в заметки

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

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

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

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

raketa

Спасибо. Учту. Использовал array_key_exists т.к. проверял существует ли элемент в массиве $data. С empty тоже работает. Но например если использовать empty, когда на форме есть checkbox, то выдает ошибку.

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

← Назад | Дальше → Движется на Habravel