class API
{
public $apiKey = '** key here **';
public $entryPoint = 'https://api.novaposhta.ua/v2.0/json/';
public function getRegions()
{
return $this->requestToAPI('getAreas');
}
public function getCities()
{
return $this->requestToAPI('getCities');
}
private function requestToAPI($method, $model = 'Address')
{
$params = array(
'modelName' => $model,
'calledMethod' => $method,
'apiKey' => $this->apiKey
);
$ch = curl_init();
curl_setopt($ch, CURLOPT_URL, $this->entryPoint);
curl_setopt($ch, CURLOPT_HEADER, 0);
curl_setopt($ch, CURLOPT_POST, 1);
curl_setopt($ch, CURLOPT_POSTFIELDS, json_encode($params));
curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, false);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
$response = curl_exec($ch);
curl_close($ch);
$decodedResponse = json_decode($response, true);
return $decodedResponse;
}
}
-
С какой целью спрашиваете? что то не работает? Или с точки зрения красоты и опрятности кода?– Kostiantyn OkhotnykCommented 2 мар. 2017 в 16:21
-
Спрашиваю с точки реализации кода и его валидности– Валера БондаренкоCommented 2 мар. 2017 в 16:25
Добавить комментарий
|
2 ответа
- Публичные поля класса
$apiKey
,$entryPoint
не очень правильно. CURLOPT_SSL_VERIFYPEER, false
всё же лучше проверять- Отсутствует проверка на ошибки
curl_
. А вдруг хост не найден или запрос составлен не верно? - Ответы от сервера можно не тупо в массив перегонять, а в класс (но это зависит от задачи).
-
По поводу второго пункта: если ставить проверку на сертификат то у меня получается curl_exec === false Commented 6 мар. 2017 в 15:29
-
Правильно у вас получается. Нужно сам сертификат подставлять в опцию
CURLOPT_CAINFO
Commented 6 мар. 2017 в 15:37 -
А сам сертификат мне надо получить у Новой Почты ? Commented 6 мар. 2017 в 15:51
-
Как вариант можно сделать вот так: Выделить абстракцию API:
class Api {
protected function send($url, $params) {
$ch = curl_init();
curl_setopt($ch, CURLOPT_URL, $url);
curl_setopt($ch, CURLOPT_HEADER, 0);
curl_setopt($ch, CURLOPT_POST, 1);
curl_setopt($ch, CURLOPT_POSTFIELDS, $params);
curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, true);
curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, false);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
if(curl_exec($ch) === false) {
throw new Exception('Нет ответа от сервера');
}
$response = curl_exec($ch);
curl_close($ch);
$decodedResponse = json_decode($response, true);
return $decodedResponse;
}
}
Наследовать класс, так как потом вам надо будет получать не только от новой почты, но и от других API и не только постом:
class ApiNewPost extends Api {
private $apiKey = '** key here **';
private $entryPoint = 'https://api.novaposhta.ua/v2.0/json/';
public function requestToAPI($method, $model = 'Address')
{
$params = array(
'modelName' => $model,
'calledMethod' => $method,
'apiKey' => $this->apiKey
);
return $this->send($this->entryPoint, json_encode($params));
}
}
определить интерфейс ответа от АПИ:
interface ResponseApi {
public function getData();
}
А для регионов и городов создать отдельные классы:
class Regions implements ResponseApi {
private $api;
function __construct(Api $api) {
$this->api = $api;
}
public function getData() {
return $this->api->requestToAPI('getAreas');
}
}
class Cities implements ResponseApi {
private $api;
function __construct(Api $api) {
$this->api = $api;
}
public function getData() {
return $this->api->requestToAPI('getCities');
}
}
Вот такой вот вариант
$cities = new Cities(new ApiNewPost());
$regions = new Regions(new ApiNewPost());