Здравствуйте. Пишу нейкое обратное API в своих целях, написал следующий код (ниже будет). Вопрос, все ли нормально в данном коде? PHP: <?php $member_id = $_GET['id']; if(!$member_id) exit('Error API #1 - Not passed'); # Отправляем запрос на сервер $communityUrl = 'https://forum.demure-rp.ru/'; $apiKey = '62df69baed4cb9cc75670ec482fa9b0f'; $curl = curl_init( $communityUrl . 'api/index.php?core/members/' . $member_id ); curl_setopt_array( $curl, array( CURLOPT_RETURNTRANSFER => TRUE, CURLOPT_HTTPAUTH => CURLAUTH_BASIC, CURLOPT_USERPWD => "{$apiKey}:" ) ); $response = curl_exec( $curl ); echo $response; # Проверяем наличие ошибок и прочего $find_error1 = strripos($response, 'errorCode'); $find_error2 = strripos($response, 'errorMessage'); if($find_error1 === false && $find_error2 === false) exit('Все хорошо, выведем потом нужные данные'); else exit('Найдены ошибки, выводим их'); --- Добавлено --- P.s. я не совсем гуру в PHP, поэтому если есть какие-то недочеты, или какие-то другие предложения, буду раз выслушать!
нет проверки, что в $_GET действительно есть член id в 7 пхп надо писать так: $member_id=$_GET['id'] ?? 0; это и проверка, и присваивание дефолтного значения вот так просто exit в случайном месте приводит потом к тому, что неудобно делать всякое лучше воспользоваться неким роутером или фреймворком число пробелов разное в начале строки работать надо в IDE, использовать автоформатирование, чтоб всё одинаковое было всегда юзай фигурные скобки