Телеграмм чат группы laravel

$old_levels) и $new_levels - массивы

14:04пожаловаться #1

Vladimir Yefremov

состроил я такое:

        if (!empty($old_levels) && !empty($new_levels)) {
            $count = max(count($old_levels), count($new_levels));
        } elseif (empty($old_levels) && empty($new_levels)){
            $count = 0;
        }
        elseif (empty($old_levels)) {
            $count = count($new_levels);
        }elseif (empty($new_levels)) {
            $count = count($old_levels);
        }else {
            $count = 0;
        }

свою задачу выполняет, но как-то не кошерно выглядит :)

возможно ли это написать лаконичнее?

в отдельный метод и убрать elseif и сразу делать return

PSYTRGLES in Laravel для начинающих

14:05пожаловаться #2

Vladimir Yefremov

состроил я такое:

        if (!empty($old_levels) && !empty($new_levels)) {
            $count = max(count($old_levels), count($new_levels));
        } elseif (empty($old_levels) && empty($new_levels)){
            $count = 0;
        }
        elseif (empty($old_levels)) {
            $count = count($new_levels);
        }elseif (empty($new_levels)) {
            $count = count($old_levels);
        }else {
            $count = 0;
        }

свою задачу выполняет, но как-то не кошерно выглядит :)

возможно ли это написать лаконичнее?

а они всегда массивы? или null/false может быть?

14:06пожаловаться #3

PSYTRGLES

а они всегда массивы? или null/false может быть?

жду массивы, но может быть любое null/false - поэтому и проверки

14:07пожаловаться #4

или вот так $count = max(count((array)$old_levels), count((array)$new_levels));

PSYTRGLES in Laravel для начинающих

14:07пожаловаться #5

да, вариант с приведением, ибо читается не оч такая конструкция

14:08пожаловаться #6

Egor Gruzdev

или вот так $count = max(count((array)$old_levels), count((array)$new_levels));

в начале нормализовать в массивы и сделать все одной строкой

14:08пожаловаться #7

ух,

Спасибо!
пробую

14:09пожаловаться #8

Vladimir Yefremov

жду массивы, но может быть любое null/false - поэтому и проверки

а ожидание неуправляемое, не от твоего кода зависит

14:09пожаловаться #9

Egor Gruzdev

а ожидание неуправляемое, не от твоего кода зависит

приходит из формы, там элементы можно динамически добавить, а можно и не добавлять

14:10пожаловаться #10

Egor Gruzdev

или вот так $count = max(count((array)$old_levels), count((array)$new_levels));

отлично работает,

Спасибо!

4unkur in Laravel для начинающих

14:22пожаловаться #11

Andrey Helldar

Задача контроллера - контролировать, а не выполнять бизнес-логику. Поэтому, участок с бизнес-логикой, а именно созданием новой записи в БД, вынесен в соответствующее место - в сервис.

В общем как началась эта беседа про ООП не ООП закинул в закладки и только сегодня получилось прочитать все сообщения.

этот доклад как то относится к тому что обсуждалось или это кое что другое?
И тут пример прям из реального проекта

https://www.youtube.com/watch?v=2_380DKU93U

Andrey Helldar in Laravel для начинающих

14:37пожаловаться #12

4unkur

Мне сложно сказать по нескольким причинам: очень долгое видео и я на слух не воспринимаю иностранную речь...

4unkur in Laravel для начинающих

14:39пожаловаться #13

кто еще топил за сервисы, может вы сможете оценить?