DD
с частными банками
Size: a a a
2020 October 28
AK
ну в целом да
AK
С
доброе утро
DD
AK
доброе утро
С
Я написал базовый сервис для получения температуры в Москвабаде через ЯндексПогоду.
Там имеется апи, которое в таком формате примерно:
Я написал сервис:
https://github.com/forszaken/delivery/blob/master/app/Services/WeatherService.php
Где делается запрос на это апи. Параметры к запросу в отдельном методе подготавливаются, сами параметры редактируются в config/weather.php
Как можно сделать лучше, в чем минусы подхода?
Там имеется апи, которое в таком формате примерно:
GET https://api.weather.yandex.ru/v2/informers?
lat=<широта>
& lon=<долгота>
& [lang=<язык ответа>]
X-Yandex-API-Key: <значение ключа>
Я написал сервис:
https://github.com/forszaken/delivery/blob/master/app/Services/WeatherService.php
Где делается запрос на это апи. Параметры к запросу в отдельном методе подготавливаются, сами параметры редактируются в config/weather.php
Как можно сделать лучше, в чем минусы подхода?
С
Я написал базовый сервис для получения температуры в Москвабаде через ЯндексПогоду.
Там имеется апи, которое в таком формате примерно:
Я написал сервис:
https://github.com/forszaken/delivery/blob/master/app/Services/WeatherService.php
Где делается запрос на это апи. Параметры к запросу в отдельном методе подготавливаются, сами параметры редактируются в config/weather.php
Как можно сделать лучше, в чем минусы подхода?
Там имеется апи, которое в таком формате примерно:
GET https://api.weather.yandex.ru/v2/informers?
lat=<широта>
& lon=<долгота>
& [lang=<язык ответа>]
X-Yandex-API-Key: <значение ключа>
Я написал сервис:
https://github.com/forszaken/delivery/blob/master/app/Services/WeatherService.php
Где делается запрос на это апи. Параметры к запросу в отдельном методе подготавливаются, сами параметры редактируются в config/weather.php
Как можно сделать лучше, в чем минусы подхода?
Пока единственное что меня отталкивает, это $headers для запроса к Апи в конструкторе, чисто интуитивно. Но я хз как лучше иначе.
AK
С
https://github.com/forszaken/delivery/blob/1ea7cd0d488c1aa316d67d55089f9fcafa981d68/app/Services/WeatherService.php#L16
лучше зависеть от интерфейса http клиента
лучше зависеть от интерфейса http клиента
ты предлагаешь газзл выбросить или чо?
AK
https://github.com/forszaken/delivery/blob/1ea7cd0d488c1aa316d67d55089f9fcafa981d68/app/Services/WeatherService.php#L46
апи урл я бы лучше получал в конструкторе или вообще принимал бы уже погодный клиент с настроенным базовым урлом
апи урл я бы лучше получал в конструкторе или вообще принимал бы уже погодный клиент с настроенным базовым урлом
AK
ты предлагаешь газзл выбросить или чо?
просто использовать интерфейс поверх
AK
в целом оч не гибкая штука у тебя получается что у тебя один раз хардкодятся широта и долгота в конфиге. почему просто в методе получения погоды не принимать их как параметры, а вызывающая сторона уже пусть хранит это как хочет?
С
https://github.com/forszaken/delivery/blob/1ea7cd0d488c1aa316d67d55089f9fcafa981d68/app/Services/WeatherService.php#L46
апи урл я бы лучше получал в конструкторе или вообще принимал бы уже погодный клиент с настроенным базовым урлом
апи урл я бы лучше получал в конструкторе или вообще принимал бы уже погодный клиент с настроенным базовым урлом
тут апи урл + параметры к урлу ес чо
Но как я понял ты предлагаешь сделать $this->url = config('weather.api_url'), чтобы это было по умолчанию в самом классе базово и потом в getUrl дальше дополнялось через buildParams()?
Но как я понял ты предлагаешь сделать $this->url = config('weather.api_url'), чтобы это было по умолчанию в самом классе базово и потом в getUrl дальше дополнялось через buildParams()?
AK
и ответ от погодного сервиса я бы обернул в какой нибудь объект, в котором уже будут явные поля
а то у тебя типа миксед идет и хер знает че оттуда вернется пока не попробуешь запустить https://github.com/forszaken/delivery/blob/1ea7cd0d488c1aa316d67d55089f9fcafa981d68/app/Services/WeatherService.php#L25
а то у тебя типа миксед идет и хер знает че оттуда вернется пока не попробуешь запустить https://github.com/forszaken/delivery/blob/1ea7cd0d488c1aa316d67d55089f9fcafa981d68/app/Services/WeatherService.php#L25
AK
тут апи урл + параметры к урлу ес чо
Но как я понял ты предлагаешь сделать $this->url = config('weather.api_url'), чтобы это было по умолчанию в самом классе базово и потом в getUrl дальше дополнялось через buildParams()?
Но как я понял ты предлагаешь сделать $this->url = config('weather.api_url'), чтобы это было по умолчанию в самом классе базово и потом в getUrl дальше дополнялось через buildParams()?
нет я предлагаю сделать:
public function __construct(string $apiUrl) { $this->apiUrl = $apiUrl; }AK
или даже
public function __construct(WeatherClient $client) { $this->client = $client; }AK
где в клиенте уже будет настроен урл до апи
AK
в общем главный поинт, не нужно обращаться к глобальному стейту в твоем сервисе
AK
(под глобальным стейтом я имею ввиду вызов функции config())