E_NOTICE: type 8 -- Undefined offset: -1 -- at line 11. В чем проблема?

Question

[skornew33]

Вот код:

<?php

$text = "{Al|La}la bl{bl|l b|lbl}lb. Cl{cl|lc}lc.";
$result = "";

preg_match_all("#[\{]{1}(.[^\}]*)[\}]{1}#", $text, $stringArray);

$attachmentsArray = $stringArray;

function replacement ($count, $attachmentsArray, $result) {
    $tok = strtok($attachmentsArray[1][$count - 1], "|");
    
    while ($tok !== false) {
        $replaceArray[$count - 1] = $tok;
        
        $tok = strtok("|");
        
        if ($count > 0) {
            replacement($count - 1, $attachmentsArray, $result);
        } else {
            $replaceArray = array_reverse($replaceArray);
            $result .= str_replace($attachmentsArray[0], $replaceArray, $text); $result .= "<br>";
            $replaceArray = array_reverse($replaceArray);
        }
    }
}

replacement(count($attachmentsArray[1]), $attachmentsArray, $result);

//$result .= str_replace($stringArray[0], replacement(count($stringArray[1])) , $text); $result .= "<br>";

echo $text . "<br><br>"; echo $result; echo "<br>";

/*$tok = strtok($stringArray[1][1], "|");

while ($tok !== false) {
    echo "Word = $tok<br>";
    $tok = strtok("|");
}*/

?>

А вот результат:

Не могу понять, что это значит, что неверно и что делать?

Comments

[Дмитрий]

Надо использовать foreach для массивов, и не делать отрицательные ключи

[skornew33]

Дмитрий, но перебор по массиву не делается и отрицательных ключей здесь нет.

[Дмитрий]

skornew33, ключей нет, а обращение есть

[skornew33]

Дмитрий, Ключи 0, 1 и 2. Или их вообще нет?

[Дмитрий]

skornew33, ну а print_r посмотреть? Сделать echo $count - 1 для отладки?

[skornew33]

Смотрели, делали. По большей части в интернете говорят, что просто отключать E_NOTICE и все.

[nokimaro]

Из примера кода

str_replace($attachmentsArray[0], $replaceArray, $text)

Откуда внутри функции берётся переменная $text?
Вижу как-минимум undefined variable или это опечатка.

[nokimaro]

Если посмотреть рекурсивный вызов функции replacement то становится понятно откуда берётся -1
Видно что в какой-то момент $count = 0 и идёт вызов replacement($count = 0, ...
И есть такой код где (0-1) = -1
$attachmentsArray[1][$count - 1]

рекурсивные вызовы replacement()

replacement: $count = 3, $attachmentsArray = 2, $result = ""
replacement: $count = 2, $attachmentsArray = 2, $result = ""
replacement: $count = 1, $attachmentsArray = 2, $result = ""
replacement: $count = 0, $attachmentsArray = 2, $result = ""
replacement: $count = 0, $attachmentsArray = 2, $result = ""
replacement: $count = 1, $attachmentsArray = 2, $result = ""
replacement: $count = 0, $attachmentsArray = 2, $result = ""
replacement: $count = 0, $attachmentsArray = 2, $result = ""
replacement: $count = 2, $attachmentsArray = 2, $result = ""
replacement: $count = 1, $attachmentsArray = 2, $result = ""
replacement: $count = 0, $attachmentsArray = 2, $result = ""
replacement: $count = 0, $attachmentsArray = 2, $result = ""
replacement: $count = 1, $attachmentsArray = 2, $result = ""
replacement: $count = 0, $attachmentsArray = 2, $result = ""
replacement: $count = 0, $attachmentsArray = 2, $result = ""

[nokimaro]

Мне сложно понять что хочет автор сделать этим кодом, но лечится это точно не отключением ошибок, а исправлением ошибок в коде.

[skornew33]

nokimaro, Спасибо за указание на проблему. Заметил ошибку в условии вызова функции.

P.S.
$text - забыл указать. Позже добавил.

Answer 1

[Макс Кузнецов]

Код не запускал, но скорее всего в строке

replacement(count($attachmentsArray[1]), $attachmentsArray, $result);

`$attachmentsArray[1]` пустой. Поэтому внутрь функции передаётся $count=0, ну и дальше обращение к массиву с индексом $count - 1, то есть -1.

Проверьте, что preg_replace_all возвращает то, что нужно. И добавьте условие с проверкой до вызова функции, что $attachmentsArray именно того формата, что вам нужен.

Comments

[skornew33]

Спасибо большое. Проблема заключалась в if.
У меня было записано:

if ($count > 0) {
      replacement($count - 1, $attachmentsArray, $result);
}

А нужно:

if ($count > 1) {
      replacement($count - 1, $attachmentsArray, $result);
}