АR
и так придется вставлять в каждый метод
вы хотите избежать дублирования 2ух строк, чтоб потом лечиться от костылей ?)
Size: a a a
2020 January 14
MM
вы хотите избежать дублирования 2ух строк, чтоб потом лечиться от костылей ?)
то есть для каждого метода приемлимо такой try ?
АR
логика на такой клиент должна быть
try {
$this->request();
} catch(Exception $e) {
$this->auth();
$this->request();
}
try {
$this->request();
} catch(Exception $e) {
$this->auth();
$this->request();
}
1) у вас же не рутовый эксепшн падает, если не авторизованы. Перепишите на try { ... } catch (UnauthException $exception) {auth; request;} catch(Exception $exception) {$this->logger->critical($exception->getMessage()); }
MM
я накидал образно, конечно exception будет выплевываться связанные с авторизацией. значит это дублирование не критично ?
AG
логика на такой клиент должна быть
try {
$this->request();
} catch(Exception $e) {
$this->auth();
$this->request();
}
try {
$this->request();
} catch(Exception $e) {
$this->auth();
$this->request();
}
нет, чувак. Выкидывай нахрен. try\catch никак не должен заменять операторов условий
MM
просто был бы обычный апи можно было токен чекать по его времени жизни и не парится перед запросом, а здесь нечего чекать.. только смотреть header
MM
нет, чувак. Выкидывай нахрен. try\catch никак не должен заменять операторов условий
я у мамы юморист? 😂
AG
причём здесь юмор. У тебя там напрашивается if\else, но никак не try\catch
MM
причём здесь юмор. У тебя там напрашивается if\else, но никак не try\catch
пример
АR
причём здесь юмор. У тебя там напрашивается if\else, но никак не try\catch
чтоб проверить авторизацию, надо дать запрос, если я правильно понял задачу
MM
чтоб проверить авторизацию, надо дать запрос, если я правильно понял задачу
просто мы вначале пробуем сделать запрос к нужной странице и получить определенные данные и если там выплюнет exception связанные с тем что кидает на авторизацию (header), то мы словим в catch и выполним auth (запрос) и снова повторим запрос на получение данных
AG
когда у тебя нет авторизации - ты всё равно получаешь ответ. С кодом 401, наверно. Это НЕ должно вызывать исключение. Это НОРМАЛЬНОЕ поведение скрипта.
Как правильно в данной ситуации:
1. отправляешь запрос.
2. Проверяешь код ответа.
3. Если код ответа 401 - выполняешь авторизацию и посылаешь запрос ещё раз
Как правильно в данной ситуации:
1. отправляешь запрос.
2. Проверяешь код ответа.
3. Если код ответа 401 - выполняешь авторизацию и посылаешь запрос ещё раз
AG
крч, фактически тоже самое. Но исключение надо бросать в ненормальных ситуациях. А "неавторизован" - это нормальная ситуация
MM
когда у тебя нет авторизации - ты всё равно получаешь ответ. С кодом 401, наверно. Это НЕ должно вызывать исключение. Это НОРМАЛЬНОЕ поведение скрипта.
Как правильно в данной ситуации:
1. отправляешь запрос.
2. Проверяешь код ответа.
3. Если код ответа 401 - выполняешь авторизацию и посылаешь запрос ещё раз
Как правильно в данной ситуации:
1. отправляешь запрос.
2. Проверяешь код ответа.
3. Если код ответа 401 - выполняешь авторизацию и посылаешь запрос ещё раз
и посылаешь запрос еще раз и там снова 401 и снова выполняет запрос ?) рекурсия ?)
AG
Можешь и рекурсивно, если хочется) Можешь остановиться. А можешь сделать 3 попытки получения данных
MM
тут еще надо учесть тот момент что данные авторизации (логин и пароль) могут быть не правильны. как это обыграть ?
MM
будет же крутиться по кругу
AG
допустим, $this->request() вернёт false,если авторизация не прошла.
for ($requestCount = 0; $requestCount < 3; ++$requestCount) {
$result = $this->request();
if ($result) {
break;
}
}YD
Привет. Хочу нормализовать базу данных. Создать новую таблицу и вынести в нее значение колонки и связать с новой колонкой по айди. Как правильно это сделать в симфони. После того как я выполню миграции нужно запустить скрипт который пройдеи по всем старым записям и конвертнет их. Как правильно писать этот скрипт через команду? Может есть где-то мануал по таким случаям? я не нашел
AG
3 попытки запроса. Если все 3 попытки отбреются - данных не будет и больше запросы пытаться слать тоже не будем