Trying to access array offset on value of type bool. Как исправить?

Question

[Aleksandr Yurchenko]

Здравствуйте.
На одном из проектов встретил нотис:

Notice: Trying to access array offset on value of type bool in ...

Погуглив понял, что это новшество в 7.4, но вот не могу понять как его исправить.

Участок кода где появился нотис

$init = $_POST['external_url'] ?? null;
$preparedLink = prepareExternalLink($init);
$sth = db_query("SELECT external_url, created_short_id FROM result WHERE external_url='" . $preparedLink . "'")->fetch();

if ($sth) {
    $_COOKIE['unique_url'] = false;
} elseif (isset($_COOKIE['url_empty'])) {
    $_COOKIE['unique_url'] = false;
} elseif (isset($_COOKIE['url_incorrect'])) {
    $_COOKIE['unique_url'] = false;
} else {
    $_COOKIE['unique_url'] = true;
}

$short_url = "{$_SERVER['HTTP_HOST']}/r_{$sth['created_short_id']}";
if ($sth !== false) {
    $sth = array_merge($sth, ['short_link' => $short_url]);
    $sth = changeKeyExistArray($sth,
        ['external_url', 'created_short_id', 'short_link', 0],
        ['External_url', 'ID_short_url', 'Result_url', 'Unique_url']);
}

Нотис указывает на эту строчку:

$short_url = "{$_SERVER['HTTP_HOST']}/r_{$sth['created_short_id']}";

не подскажите как исправить?

Comments

[Aleksandr Yurchenko]

Pardon Me! Where Do I Find 4giveness?, именно с ней связано, я переключил версию на 7.3 - нотиса нет, на 7.4 носит присутствует. Погуглив наткнулся на несколько ссылок (1, 2), где возникает такие же проблемы при смене версии php на 7.4

Но нигде я не нашел, как убрать этот нотис.

[Aleksandr Yurchenko]

Pardon Me! Where Do I Find 4giveness?,
засунул

$short_url = "{$_SERVER['HTTP_HOST']}/r_{$sth['created_short_id']}";

в проверку, которая начинается ниже, ошибок ушла.

Просто с таким нотисом ранее не встречал. И решил разобраться, так как наткнулся на rfc и там четко было указано, что с следующей версии (8.0) это уже не будет нотисом, а уйдет в ошибку (как я понял). Вот цитата:

Trying to access array offset on value of type %s Notice Warning

Rationale: These diagnostics are generated when trying to use scalars as arrays. The first two occur in write contexts, the latter in read contexts. The latter was introduced in PHP 7.4 as a notice with express intention to elevate the severity in PHP 8.0. In line with the symmetrical case on objects, the write case is treated more severely here, as it usually implies data loss.

Спасибо за то что натолкнули на мысль (пункт 1) и как-то проблема сразу же решилась сама собой :)

[Aleksandr Yurchenko]

Pardon Me! Where Do I Find 4giveness?, я только ЗА! :) и всегда стараюсь тайп хинтить и по максимум запрещать себе то, что php в принципе разрешает)))
Просто когда у тебя не много опыта, ты решаешь проблемы по мере их возникновения). Хороший пример, теперь я буду по обращать внимание на подобные результаты.

Answer 1

[Rsa97]

Судя по всему, в $sth у вас записано булево значение (предположительно, false). Попасть в переменную это значение могло только при вызове функции db_query()->fetch(). Вот там и ищите проблему. Функция db_query к стандартным не относится, значит смотрите в своём коде, что она возвращает и почему.

Comments

[Aleksandr Yurchenko]

Судя по всему, в $sth у вас записано булево значение (предположительно, false)

Совершенно верно, возвращает false.

Функция db_query к стандартным не относится, значит смотрите в своём коде, что она возвращает и почему.

Эта функцию написал с целью "собрать в ней" подключение к базе, подготовку запроса и его непосредственное выполнение

db_query

function db_query(string $sql_query, array $params_execute = [])
{
    $pdh = db_connect();
    $sth = $pdh->prepare($sql_query);
    $verifiedParams = [];
    foreach ($params_execute as $placeholder => $item) {
        if (is_int($item)) {
            $sth->bindParam(count($params_execute), $placeholder, PDO::PARAM_INT);
            $verifiedParams[] = $item;
        } elseif (is_string($item)) {
            $sth->bindParam(count($params_execute), $placeholder,PDO::PARAM_STR); //data_type: PDO::PARAM_STR (default)
            $verifiedParams[] = $item;
        }
    }
    try {
        $sth->execute($verifiedParams);
    } catch (PDOException $exception) {
        echo 'Technical troubles in the site. Soon all fix.';
        file_put_contents(__DIR__ . '/../../logs/Errors_system.txt',
            date('Y-m-d H:i:s') . ' [db_query]: ' . $exception->getMessage() . PHP_EOL, FILE_APPEND);
    }
    return $sth;
}

Посмотрел эту функцию и не многое прояснилось, возвращается false, так как в sth ничего не возвращает ($_POST['external_url'] по умолчанию null):

$init = $_POST['external_url'] ?? null;
$preparedLink = prepareExternalLink($init);
$sth = db_query('SELECT external_url, created_short_id FROM result WHERE external_url=" . $preparedLink . "')->fetch();

Нужно добавить дополнительную проверку или начиная с 7.4 функция db_query() считается не корректной...

[Rsa97]

Aleksandr Yurchenko, При чём тут 7.4?
fetch() вернёт false, если по вашему запросу в базе ничего не найдено.

[Aleksandr Yurchenko]

Pardon Me! Where Do I Find 4giveness?, собственно если в инпут ($_POST['external_url']) не указать никакого адреса, значение $sth будет false.
Проблему решил засунув одну строчку под проверку $sth !== false

[Aleksandr Yurchenko]

Rsa97, потому что нотисом она стала только с версии 7.4, ранее возвращалось null rfc и соответственно на версиях ниже 7.4 её просто нет.

[Aleksandr Yurchenko]

Pardon Me! Where Do I Find 4giveness?, спасибо за замечание! Переделаю :)