Телеграмм чат группы phpgeeks страница 46160

Sergey

ну логично что где-то выше по коду

О_о

19:05пожаловаться #1

KP

Я понимаю ловить такие ексепшены гдето в методах делающих конкретную фугкцию, но еще выше я ожидаю чтотт рабочее а не эксепшены вылавливать

Г

Я понимаю ловить такие ексепшены гдето в методах делающих конкретную фугкцию, но еще выше я ожидаю чтотт рабочее а не эксепшены вылавливать

ну хз. у нас в проекте слушаются ексепшены и при их появлении в зависимости от вида соответствующий http ответ прилетает. То есть считай на самом верху

19:08пожаловаться #3

Г

а методы с конкретными функциями их только генерируют

19:09пожаловаться #4

KP

Гена

ну хз. у нас в проекте слушаются ексепшены и при их появлении в зависимости от вида соответствующий http ответ прилетает. То есть считай на самом верху

Т.е. ошибка об отсутствии юзеров в списке по заданным критериям поиска выдается как ошибка всего приложения?

Сергей Предводителев... in phpGeeks

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

СП

Гена

А если вернулся Null из запроса, то мы не узнаем что получить не вышло?

Узнаем, тут вопрос, как удобнее работать :)

Сергей Предводителев... in phpGeeks

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

СП

Kolya Pot

Т.е. ошибка об отсутствии юзеров в списке по заданным критериям поиска выдается как ошибка всего приложения?

Зачем ошибка приложения, код который запрашивал юзера должен обработать это исключение

19:11пожаловаться #7

S

Sergey in phpGeeks

Kolya Pot

Т.е. ошибка об отсутствии юзеров в списке по заданным критериям поиска выдается как ошибка всего приложения?

Сделай исключение DbNotFoundException какое-нибудь и выкидывай, когда ошибка при выборке,а метод оборачивай в трай кеч

Г

Т.е. ошибка об отсутствии юзеров в списке по заданным критериям поиска выдается как ошибка всего приложения?

Это API-проект. Так что да запрос на пользователя которого нет выдаст 404 статус и
{
"message": "error.not_found"
}

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

A

Andrey in phpGeeks

Сергей Предводителев

Узнаем, тут вопрос, как удобнее работать :)

Нет, тут вопрос принятых соглашений на проекте. По строгой логике да, метод getCustomer должен вернуть кастомера или выбросить исключение, если он не может этого сделать. Но то, что удобнее проверить ответ на нул, чем делать трай-кэтч, — это, по-моему, очевидно.

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

Г

Нет, тут вопрос принятых соглашений на проекте. По строгой логике да, метод getCustomer должен вернуть кастомера или выбросить исключение, если он не может этого сделать. Но то, что удобнее проверить ответ на нул, чем делать трай-кэтч, — это, по-моему, очевидно.

+1

19:15пожаловаться #11

S

Sergey in phpGeeks

Нет, тут вопрос принятых соглашений на проекте. По строгой логике да, метод getCustomer должен вернуть кастомера или выбросить исключение, если он не может этого сделать. Но то, что удобнее проверить ответ на нул, чем делать трай-кэтч, — это, по-моему, очевидно.

+

Сергей Предводителев... in phpGeeks

19:15пожаловаться #12

СП

Нет, тут вопрос принятых соглашений на проекте. По строгой логике да, метод getCustomer должен вернуть кастомера или выбросить исключение, если он не может этого сделать. Но то, что удобнее проверить ответ на нул, чем делать трай-кэтч, — это, по-моему, очевидно.

Совсем не очевидно. В try catch обернуть - вполне удобно тоже

19:16пожаловаться #13

Г

а пустой кастомер исключительная ситуация?

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

KP

Нет, тут вопрос принятых соглашений на проекте. По строгой логике да, метод getCustomer должен вернуть кастомера или выбросить исключение, если он не может этого сделать. Но то, что удобнее проверить ответ на нул, чем делать трай-кэтч, — это, по-моему, очевидно.

Вот у меня такое всегда вернет customer))) у которого есть isLoaded

19:17пожаловаться #15

A

Andrey in phpGeeks

Kolya Pot

Вот у меня такое всегда вернет customer))) у которого есть isLoaded

Глубокая у вас кроличья нора)

19:18пожаловаться #16

Г