Форум программистов, компьютерный форум, киберфорум
PHP: ООП
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.50/4: Рейтинг темы: голосов - 4, средняя оценка - 4.50
Заблокирован

Вернуть объект или nullable?

06.06.2024, 12:32. Показов 1170. Ответов 13
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Есть следующий метод и два варианта его использования:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
// первый вариант
 
public function getRouteGroup(string $group): RouteGroup
{
    return $this->routeGroups[$group] ?? new RouteGroup();
}
 
$middleware = $this->getRouteGroup('bla-bla-bla')->getMiddleware();
 
// второй вариант
 
public function getRouteGroup(string $group): ?RouteGroup
{
    return $this->routeGroups[$group] ?? null;
}
 
$middleware = $this->getRouteGroup('bla-bla-bla')?->getMiddleware() ?? [];
Какой правильнее использовать?

p.s. стоит ли ради того чтобы добраться до пустого массива RouteGroup::getMiddleware, создавать экземпляр RouteGroup, если его нет в массиве routeGroups (первый вариант)
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
06.06.2024, 12:32
Ответы с готовыми решениями:

Это теперь стандарт - <Nullable>enable</Nullable>?
При создании нового проекта в VS2022 на базе Net6 в файле проекта по умолчанию стоит &lt;Nullable&gt;enable&lt;/Nullable&gt; Это теперь...

Что оптимальнее - nullable bool или enum на три значения
Изначально нужна была переменная, чтобы хранить 2 значения. Но потом понадобилось добавить еще третье - дефолтное значение, из-за чего я...

Недопустимый неполный тип , или как передать в функцию указатель на объект класса , или просто объект
Други мои , кодеры. Не могу я ни как передать в функцию объект класса , или что еще лучше(для меня , так как учусь работать с указателями)...

13
Эксперт PHP
3899 / 3237 / 1353
Регистрация: 01.08.2012
Сообщений: 10,909
07.06.2024, 12:39
Лучший ответ Сообщение было отмечено HP Fedora как решение

Решение

Зависит от предназначения класса и метода. На первый взгляд вариант 1 выглядит сомнительно, возвращать объект вместо null ради последующего сокращения кода...

Можно сделать отдельный метод getRouteGroupMiddleware:

PHP
1
$middleware = $this->getRouteGroupMiddleware('bla-bla-bla') ?? [];
1
 Аватар для sad67man
2604 / 1508 / 689
Регистрация: 23.08.2015
Сообщений: 3,834
07.06.2024, 16:43
Лучший ответ Сообщение было отмечено HP Fedora как решение

Решение

Цитата Сообщение от Jodah Посмотреть сообщение
Можно сделать отдельный метод getRouteGroupMiddleware:
Тогда уж ?? не надо, можно вернуть пустой массив.

Цитата Сообщение от Jodah Посмотреть сообщение
На первый взгляд вариант 1 выглядит сомнительно, возвращать объект вместо null ради последующего сокращения кода...
Почему? часто такой прием использую. Все зависит от ситуации.
По запросу "почему нельзя возвращать NULL" вы найдете кучу статей и даже есть целые книги. Советуют вместо null кидать исключение, либо отдавать nullObject. В основном это касается Java. Я думаю это связано с тем, что в Java даже при указании типа, все равно можно вернуть или принимать null. И если где-то возникает ошибка - потом сложно найти концы.

Цитата Сообщение от HP Fedora Посмотреть сообщение
Какой правильнее использовать?
Вы должны учитывать, что если возвращаете null, то вы будете обязаны везде делать проверку и принимать решение, что делать в случае null.. Если в каждом случае будет выкидываться исключение, то логичнее это исключение поместить в сам метод. Если каждый раз вы будете эмулировать логику объекта, то имеет смысл возвращать заглушку.

В данном конкретном случае меня смущает обращение к несуществующим группам по ключу. Пока не могу до конца представить такую ситуацию, из-за чего сложно однозначно сказать что лучше. Что если вы допустите опечатку? потом долго будете искать ошибку. Но вы как-то уж слишком закапываетесь. Если хотите отложить этот вопрос, то добавьте специальный метод, как выше предложили.
1
Заблокирован
08.06.2024, 08:01  [ТС]
Цитата Сообщение от sad67man Посмотреть сообщение
Что если вы допустите опечатку?
у роутов одной группы будет одинаковое имя группы, опечатки быть не может

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
// назначаем группу
public function group(callable $callback, string $groupName = ''): RouteGroup
{
    $groupName = $groupName ?: uniqid();
 
    if (array_key_exists($groupName, $this->routeGroups)) {
        throw new Exception("Уже существует группа роутов с именем: $groupName");
    }
 
    $routeGroup = new RouteGroup($this, $groupName);
 
    $this->routeGroups[$groupName] = $routeGroup;
 
    $callback($routeGroup);
 
    return $routeGroup;
}
 
// достаём
$routeResult = $router->matchRoutes($request); // имя группы получаем методом RouteResult::getGroupName
 
$routeGroupMiddleware = $this->getRouteGroup($routeResult->getGroupName())?->getMiddleware() ?? [];
Цитата Сообщение от sad67man Посмотреть сообщение
Тогда уж ?? не надо, можно вернуть пустой массив.
я смысл понял - логика ?? внутри такого метода должна быть
0
Эксперт PHP
3899 / 3237 / 1353
Регистрация: 01.08.2012
Сообщений: 10,909
08.06.2024, 12:51
Цитата Сообщение от sad67man Посмотреть сообщение
Тогда уж ?? не надо, можно вернуть пустой массив.
Тогда не будет возможности разграничить ситуации "такой группы нет" и "у роута нет middleware". Хотя нужно ли здесь оно...

Цитата Сообщение от sad67man Посмотреть сообщение
Все зависит от ситуации.
Вот да, какой вариант не предложи, сразу в голове возникают примеры, когда он будет и не будет подходить. Без понимания проекта это просто гадание.
1
Заблокирован
08.06.2024, 14:49  [ТС]
Цитата Сообщение от Jodah Посмотреть сообщение
Тогда не будет возможности разграничить ситуации "такой группы нет" и "у роута нет middleware". Хотя нужно ли здесь оно...
правильно понимаете на этом моменте не хотелось бы останавливать выполнение кода, вернув пустой массив: ну нет промежуточного у группы (роут вне группы, имя группы у такого роута отсутствует, следовательно такой группы - нет) ну и ладно, скрипт продолжит выполняться без такового
0
 Аватар для sad67man
2604 / 1508 / 689
Регистрация: 23.08.2015
Сообщений: 3,834
10.06.2024, 13:46
Цитата Сообщение от HP Fedora Посмотреть сообщение
правильно понимаете на этом моменте не хотелось бы останавливать выполнение кода, вернув пустой массив: ну нет промежуточного у группы (роут вне группы, имя группы у такого роута отсутствует, следовательно такой группы - нет) ну и ладно, скрипт продолжит выполняться без такового
Видимо не до конца поняли нюанс. Что если routeResult вернул какое-то имя группы, а такой группы у вас не существует? Считается ли это критической ошибкой?
1
Заблокирован
10.06.2024, 15:50  [ТС]
sad67man, если роут не находится в группе, то у него никогда не будет имени группы, имя группы задаётся только если роут в группе, следовательно такой группы нет и промежуточного у неё нет, значит нужно вернуть пустой массив, не останавливая скрипт сообщением, что имя группы роута отсутствует в массиве групп роутов

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

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
public function group(callable $callback, string $groupName = ''): RouteGroup
{
    $groupName = $groupName ?: uniqid();
 
    if (array_key_exists($groupName, $this->routeGroups)) {
        throw new Exception("Уже существует группа роутов с именем: $groupName");
    }
 
    $routeGroup = new RouteGroup($this, $groupName);
 
    $this->routeGroups[$groupName] = $routeGroup;
 
    $callback($routeGroup);
 
    return $routeGroup;
}
routes.php

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
return function (App $app) {
 
    $app->group(function (RouteGroup $routeGroup) {
 
        $routeGroup->get('/{referrer_id:[1-9]\d*}', GetReferrerAction::class);
        $routeGroup->get('/', GetHomeAction::class);
        $routeGroup->get('/signin', GetSigninAction::class);
        $routeGroup->get('/forgot', GetForgotAction::class);
 
        $routeGroup->post('/', PostHomeAction::class);
        $routeGroup->post('/signin', PostSigninAction::class);
        $routeGroup->post('/forgot', PostForgotAction::class);
    }, 'Guest')
        ->addMiddleware(RedirectToAuthMiddleware::class); // промежуточное группы, будет применено при вызове любого роута группы
 
    $app->group(function (RouteGroup $routeGroup) {
 
        $routeGroup->get('/menu', GetMenuAction::class);
    }, 'Auth')
        ->addMiddleware(RedirectToGuestMiddleware::class); // промежуточное группы, будет применено при вызове любого роута группы
};
у меня промежуточное организовано отдельно для приложения, для группы роутов и для роутов (в этом же порядке и выполняется)
0
 Аватар для sad67man
2604 / 1508 / 689
Регистрация: 23.08.2015
Сообщений: 3,834
10.06.2024, 16:41
Цитата Сообщение от HP Fedora Посмотреть сообщение
если роут не находится в группе, то у него никогда не будет имени группы
Опять все переворачиваете. Вопрос: "Что должно происходить в случае возникновения такой ситуации". Вы отвечаете, что такая ситуация никогда не возникнет - так программисты не рассуждают. Мы по априори предполагаем, что все возможно, в том числе все возможные ошибки и баги.. Вот вы сейчас пишете метод и как раз нужно принять решение, как он должен себя вести в случае, когда вы пытаетесь достать несуществующую группу.. если такой ситуации быть не должно, значит это критическая ошибка, т.е. должны кидать исключение.

Добавлено через 8 минут
HP Fedora, Это так можно всю систему построить на все-возможных допущений. И будет очень сложно потом расследовать инциденты и искать концы.. Т.е. мы предполагаем, что система должна работать неким образом, а по факту она вдруг неожиданно работает по другому..
0
Заблокирован
10.06.2024, 16:53  [ТС]
sad67man, в вопросе всё описано

Добавлено через 4 минуты
sad67man,

PHP
1
2
3
4
return function (App $app) {
 
    $app->get('/test', GetTestAction::class); // у этого роута нет имени группы, метод getRouteGroup должен вернуть пустой массив промежуточного ПО без останоаки кода какими-то сообщениями об имени группы
};
0
 Аватар для sad67man
2604 / 1508 / 689
Регистрация: 23.08.2015
Сообщений: 3,834
10.06.2024, 16:59
Цитата Сообщение от HP Fedora Посмотреть сообщение
// у этого роута нет имени группы, метод getRouteGroup должен вернуть пустой массив промежуточного ПО без останоаки кода какими-то сообщениями об имени группы
};
Цитата Сообщение от sad67man Посмотреть сообщение
как он должен себя вести в случае, когда вы пытаетесь достать несуществующую группу
Мы про какой метод сейчас говорим? Вы видите разницу между методами $result->getGroupName() и $this->getRouteGroup('bla-bla-bla')..
0
Заблокирован
10.06.2024, 17:00  [ТС]
sad67man, вижу, при остсутствии имени группы будет $this->getRouteGroup('')

это чётко прописано в посте 4

PHP
1
2
3
$routeResult = $router->matchRoutes($request); // имя группы получаем методом RouteResult::getGroupName
 
$routeGroupMiddleware = $this->getRouteGroup($routeResult->getGroupName() /** имени группы нет, если роут вне группировки */)?->getMiddleware() ?? [];
0
 Аватар для sad67man
2604 / 1508 / 689
Регистрация: 23.08.2015
Сообщений: 3,834
10.06.2024, 17:17
HP Fedora, Ну если посмотреть на сам метод getRouteGroup
Цитата Сообщение от HP Fedora Посмотреть сообщение
PHP
1
2
3
4
public function getRouteGroup(string $group): ?RouteGroup
{
    return $this->routeGroups[$group] ?? null;
}
То мы должны рассмотреть множество всех возможных значений.. это если $group пустой, существующий и несуществующий ключ.
Ну вы напрочь не хотите рассматривать вариант с несуществующим ключом, а ведь именно он дает ответ, как должен себя вести метод..

Ну не хотите, и ладно) А то я что-то нервничать начинаю)
0
Заблокирован
10.06.2024, 17:23  [ТС]
sad67man, забей
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
10.06.2024, 17:23
Помогаю со студенческими работами здесь

Вернуть объект из функции
Здравствуйте. Начну с кода: Dim CON As New ADODB.Connection Dim RS As New ADODB.Recordset Private Function ПодключитьБД() ...

Вернуть объект класса
Как вернуть объект класса через return this? Класс управляемый. Конструкция return *this вызывает ошибки.

Вернуть созданный объект
Приветствую! Есть такой кусок кода В модуле класса Private m_Form As Object Private Property Let clsForm_add_Frame(ByVal...

Вернуть ссылку на объект
есть ф-я, которая должна возвращать ссылку: class CValue { ..... }; ........ CValue&amp; COrders::operator(LPCTSTR Name) { ...

Как в методе вернуть объект?
Здравствуйте, хочу реализовать хранение данных пользователя, некоторые данные хранить в БД а некоторые в ini файле. В одном из методов...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
14
Ответ Создать тему
Новые блоги и статьи
Реалии
Hrethgir 01.03.2026
Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru