Question

我正在

“未定义的变量：html in the line $html .= generateOption($optstyle.'option', $level, $data, $padding);”

我的功能出了什么问题？

function generateOptions($parent, $level, $padding, $menu, $db)
{
    $result=$db->query("SELECT id, name FROM menu WHERE parent='$parent' AND showinmenu='$menu'");
    $spacer = '&nbsp;&nbsp;'; 
    $padding = str_repeat($spacer, $level);
    while($data=$result->fetch_row()){      
        $children_html = generateOptions($data[0], $level+1, $padding, $menu,$db);
        $optstyle = empty($children_html) ? 'std' : 'bold';
        $html .= generateOption($optstyle.'option', $level, $data, $padding); (this line)
        $html .= $children_html;
    }
    return $html;
}

Answer 1

在尝试使用它之前，你没有定义$ html。

尝试在$html = "";

之后添加$padding = str_repeat($spacer, $level);

Answer 2

在开始追加$html之前，您必须先对其进行初始化。考虑$html .= something和$html = $html . something一样，您应该看到问题。

此外，您的查询不安全。确保你逃避一切 - 以防万一。

Answer 3

在附加变量之前先声明变量。使用点运算符只需添加内容。如果您只是在循环之前使用$html = '';，则警告应该消失。

Answer 4

function generateOptions($parent, $level, $padding, $menu, $db)
{
    $html = ''; # define `$html` first
    $result = $db->query("SELECT id, name FROM menu WHERE parent='$parent' AND showinmenu='$menu'");
    $spacer = '&nbsp;&nbsp;'; 
    $padding = str_repeat($spacer, $level);

    while($data = $result->fetch_row())
    {      
        $children_html = generateOptions($data[0], $level+1, $padding, $menu,$db);
        $optstyle = empty($children_html) ? 'std' : 'bold';
        $html .= generateOption($optstyle.'option', $level, $data, $padding);
        $html .= $children_html;
    }
    return $html;
}

正如其他人所提到的，您应该转义$parent和$menu变量以防止SQL注入。

Answer 5

用这个开始代码： $html = "";

Answer 6

在开始行插入代码

if (!isset($html)) {
   $html = '';
};

Answer 7

        $html .= generateOption($optstyle.'option', $level, $data, $padding); (this line)
        $html .= $children_html;

删除第一行中第一个=前面的点。

PHP代码中未定义的变量通知

7 个答案: