Почему может возникать ошибка curl_setopt(): supplied resource is not a valid cURL handle resource?

Question

[Vlad]

Есть код:

class CurlAny extends BaseObject
{
    protected $_ch;
    
    protected $_timeout = 10;
    
    public function __construct(array $config = [])
    {
        $this->_ch = curl_init();

        parent::__construct($config);
    }
    
    public function send()
    {
        $this->_query('test.com', ['key' => 'value']);
    }
    
    private function _query(string $url, $data = false)
    {
        if (!$this->_ch) {
            throw new Exception('Empty $_ch in MtargetOffers.');
        }

        curl_setopt($this->_ch, CURLOPT_URL, $url);
        curl_setopt($this->_ch, CURLOPT_HEADER, false);
        curl_setopt($this->_ch, CURLOPT_TIMEOUT, $this->_timeout);
        curl_setopt($this->_ch, CURLOPT_NOBODY, 0);
        curl_setopt($this->_ch, CURLOPT_RETURNTRANSFER, true);

        if ($data) {
            curl_setopt($this->_ch, CURLOPT_POST, true);
            curl_setopt($this->_ch, CURLOPT_HTTPHEADER, [
                'Expect:',
            ]);
            curl_setopt($this->_ch, CURLOPT_POSTFIELDS, http_build_query($data));
        } else {
            throw new Exception('Пустые данные на нашей стороне', 500);
        }

        $page = curl_exec($this->_ch);
        $info = curl_getinfo($this->_ch);
        $err = curl_errno($this->_ch);
        $errmsg = curl_error($this->_ch);

        if ($this->_ch) {
            curl_close($this->_ch);
        }
        
        return $page;
    }
}

Код немного упрощен, но передано самое важное. Используется Yii2 на PHP 7.3

Время от времени отлавливаю такую ошибку

curl_setopt(): supplied resource is not a valid cURL handle resource

В чем может быть причина? Разве не достаточно проверки if (!$this->_ch) {...}?

P.S. На ~10000 запросов (с полной инициализацией данного объекта для каждого запроса) приходится 2-3 шт. указанной ошибки.

Answer 1

[Vlad]

В ходе нудной трассировки выяснилось, что (по неизвестным пока причинам) не создавался curl дескриптор
(CurlAny::$_ch) функцией curl_init() при создании нового объекта CurlAny

Answer 2

[ThunderCat]

Вангую что ошибка вываливается когда вы делаете 2 запроса используя 1 созданный объект ) Я бы на вашем месте не выносил инит в конструктор )

Разве не достаточно проверки if (!$this->_ch) {...}

там может быть например 1, что удовлетворяет условию, но нифига не ресурс. ЕМНИП is_resource будет как минимум правильнее. Но смысла особого в этой проверке нет.

Comments

[Vlad]

Каждый такой объект полностью одноразовый – либо совершает запрос и получает ответ, либо выбрасывается Exception.

[Vlad]

там может быть например 1

а если код не дает такой возможности? Все манипуляции с $_ch указаны в фрагменте кода. Как тут может быть 1?

[ThunderCat]

Vlad, причины могут быть разные, но смысл один - инициализация должна быть перед использованием, а проверка в том виде в каком она у вас сейчас вообще ниочем. Так делать не нужно вообще (разве что это флаговая переменная заранее заданного булева типа), это антипаттерн.

Каждый такой объект полностью одноразовый – либо совершает запрос и получает ответ, либо выбрасывается Exception.

И зачем такое? Это все равно что заряжать калаш запалом, порохом, пыжом и пулей через дуло.

[Vlad]

ThunderCat, понял, за это спасибо!

Answer 3

[Melkij]

Сложно сказать с не воспроизводимым примером кода. Впрочем легко заметить, что сейчас ваш метод _query строго одноразовый. На повторный запуск этого метода (в контексте одного объекта) вы получите такую ошибку. Потому что curl_close закрывает ресурс, но переменная остаётся типа ресурс и потому не отвечает условию !$this->_ch

Comments

[Vlad]

Каждый такой объект полностью одноразовый – либо совершает запрос и получает ответ, либо выбрасывается Exception.

[Melkij]

Для приведённого кода throw new Exception('Empty $_ch in MtargetOffers.'); недостижим никогда. Ни на первом ни на повторном вызове. Я уже написал почему.

[Vlad]

Melkij, правильно ли я понял, что такая замена

if (get_resource_type($this->_ch) !== 'curl') {
    throw new Exception('Empty $_ch in MtargetOffers.');
}

должна помочь? Разве дескриптор не будет уничтожен функцией curl_close после одного выполнения кода?

[Melkij]

Дескриптор будет уничтожен. Потому последующий curl_setopt и жалуется. Переменная - нет, и будет содержать пустой zend_resource с невалидным type.
Лучше обнулить _ch явно после curl_close. Понятнее будет.

Впрочем соглашусь с ThunderCat , дизайн класса вызывает вопросы.

[Vlad]

Melkij, понял, спасибо, попробую!

Возможно вся архитектура не слишком элегантная, просто приходится инициализировать динамически большое кол-во объектов, которые наследуются от этого класса, а затем через foreach происходит отправка данных этими объектами, если грубо описывать.

В дальнейшем планирую отказаться от такого подхода, но пока приходится пользоваться чем есть.