PHP 中数组未定义索引错误（注意）

Question

我有这个函数：

function coin_matrix($test, $revs) {
    $coin = array();

    for ($i = 0; $i < count($test); $i++) {
        foreach ($revs as $j => $rev) {
            foreach ($revs as $k => $rev) {
            if ($j != $k && 
                $test[$i][$j] != null && 
                $test[$i][$k] != null) {

                $coin[$test[$i][$j]][$test[$i][$k]] += 1 / ($some_var - 1);
                }
            }
        }
    }
    return $coin;
}

问题

$test = array(
array('3'=>'1','5'=>'1'),
array('3'=>'2','5'=>'2'),
array('3'=>'1','5'=>'2'),
array('3'=>'1','5'=>'1'));

是

$revs = array('3'=>'A','5'=>'B');

，当我运行它时，它返回这些错误（通知）：

注意：未定义的索引：第 10 行的 1

注意：第 10 行的未定义的索引：1

注意：未定义的索引：第 2 行的 2 10

注意：第 10 行未定义索引：2

注意：第 10 行未定义索引：2

注意：第 10 行未定义索引：1 即

这一行：$coin[$test[$i][$j]] [$test[$i][$k]] += 1 / ($some_var - 1);

问题是最后函数返回正确的矩阵（数组），如果我测试是否$coin[$test[$i][$j]][$test[$i][$k]] 存在，则不再返回它。

任何建议将不胜感激！

谢谢！

Answer 1

$coin[$test[$i][$j]][$test[$i][$k]] += 1 / ($some_var - 1);

该警告是由 += 生成的。 += 需要在添加元素之前查找该元素，并且您第一次访问 $coin 中的任何元素时都没有初始化它们。

Answer 2

您可以/应该进行测试以确保在增加值之前设置 $coin[$test[$i][$j]][$test[$i][$k]] 。这不会改变代码的功能，但会使通知消失（这是一个很好的做法）。

function coin_matrix($test, $revs) {
    $coin = array();

    for ($i = 0; $i < count($test); $i++) {
        foreach ($revs as $j => $rev) {
            foreach ($revs as $k => $rev) {
            if ($j != $k && 
                $test[$i][$j] != null && 
                $test[$i][$k] != null) {

                    // new tests go here
                    if(!isset(coin[$test[$i][$j]])) 
                    {
                        coin[$test[$i][$j]] = array(); 
                    }
                    if(!isset(coin[$test[$i][$j]][$test[$i][$k]])) 
                    {
                        coin[$test[$i][$j]][$test[$i][$k]] = 0; 
                    }

                    $coin[$test[$i][$j]][$test[$i][$k]] += 1 / ($some_var - 1);
                }
            }
        }
    }
    return $coin;
}

Answer 3

我认为问题是你试图使用 $coin 作为二维数组。

如果你希望它是二维的， $coin 必须是数组的数组。

function coin_matrix($test, $revs) {

    $coin = array();

    for ($i = 0; $i < count($test); $i++) {
        foreach ($revs as $j => $rev) {
            foreach ($revs as $k => $rev) {
            if ($j != $k && 
                $test[$i][$j] != null && 
                $test[$i][$k] != null) {
                // add this.
                if ($coin[$test[$i][$j]] == null){
                    $coin[$test[$i][$j]] = array();
                }
                // end
                $coin[$test[$i][$j]][$test[$i][$k]] += 1 / ($some_var - 1);
                }
            }
        }
    }
    return $coin;
}

Answer 4

我不太明白，但我可以建议你使用

function coin_matrix($test, $revs) {
    $coin = array();

    for ($i = 0; $i < count($test); $i++) {
        foreach ($revs as $j => $rev) {
            foreach ($revs as $k => $rev) {
            if (($j != $k) && 
                ($test[$i][$j] != null) && 
                ($test[$i][$k] != null)) {

                $coin[$test[$i][$j]][$test[$i][$k]] += 1 / ($some_var - 1);
                }
            }
        }
    }
    return $coin;
}

Answer 5

您是否考虑过将 for 循环替换为 foreach 循环？
例如：

foreach( $tests as $i => $test )

这样做的好处是在 $test[ $i ] 中有一个值。然后，在您的 $test[ $i ][ $j ] == null 块中，放置以下内容：

        if ($j != $k && 
            // I suspect that this will cause errors too.
            // Do yourself a favor and add this sanity check.
            isset( $test[$j] ) && $test[$j] != null && 
            isset( $test[$k] ) && $test[$k] != null) {

                $currentK = $test[$k];
                $currentJ = $test[$j];
                // Use debug lines if setting things directly won't work
                if( !isset( $coin[ $currentJ ] ) || !is_array( $coin[ $currentJ ] ) )
                {
                    // $coin[ $currentJ ] = array();
                    die( "<pre>$currentK $currentJ \n" .  print_r( $coin ) );
                }
                $currentCoin =& $coin[ $currentJ ];
                if( !isset( $currentCoin [ $currentK ] ) || 
                    !is_array( $currentCoin [ $currentK ] ) )
                {
                    // Just curious, but when doing these checks before,
                    // did you remember to assign a numeric value here?
                    // 
                    // $currentCoin[ $currentK ] = 0;
                    die( "<pre>$currentK $currentJ \n" .  print_r( $coin ) );
                }
                $currentCoin[ $currentK ] += 1 / ($some_var - 1);
            }
        }