@Zukrine

Какой код выглядит грамотнее?

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

class ArmoryController extends Controller
{
    public function armory_all()
    {
        foreach (Armory::where('use', 0)->get() as $item) {
            dump($item);
        }
    }
}


А вот тут немного длинее:

class ArmoryController extends Controller
{
    public function armory_all()
    {
        $items = Armory::where('use', 0)->get();
        foreach ($items as $item) {
            dump($item);
        }
    }
}


Как лучше будет? Чтобы сразу запомнить и тренироваться.
  • Вопрос задан
  • 163 просмотра
Решения вопроса 1
alexey-m-ukolov
@alexey-m-ukolov Куратор тега PHP
Второй, конечно. Потом у вас ещё одно условие появится и оно перестанет помещаться в одну строку.
Ответ написан
Пригласить эксперта
Ответы на вопрос 2
Fragster
@Fragster
помогло? отметь решением!
Совсем правильно может быть скоуп сделать https://laravel.com/docs/8.x/eloquent#local-scopes
Ответ написан
Комментировать
AmdY
@AmdY
PHP и прочие вебштучки
Напишите тесты и всё поймёте. Оба варианта ужасны.
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы