优化PHP代码-函数显示和未定义变量的PHP注意事项

任何建议，下面是玩具变量的函数-函数loadProductVarients

目前其显示：-

a） 仅玩具变体名称，其中我打算实现产品名称和变体名称。例如，如果Hotwheels是公司，法拉利是产品，Zsi是其变体

现在下面的函数只显示Zsi作为变体名称，我打算实现法拉利Zsi作为变体名称

我认为需要在$temp1中添加产品变量，但尽管使用了许多组合，我还是无法实现它

b） 第二个是在函数loadproductvarients中获取PHP通知：未定义变量：temp1和temp2通知

我正在学习PHP-非常感谢您的帮助和建议

编辑

你好，洛德

原函数

   function loadProductVarients($id,$minprice,$maxprice,$fuel_type){
        $mainframe =& JFactory::getApplication();
        $option = JRequest::getCmd('option');
        $database =& JFactory::getDBO();
        global $Itemid; 
        $Vcond="";
        if($minprice!="" and $maxprice!=""){
        $Vcond.=" and (v_price between ".$minprice." and ".$maxprice.")";
        }elseif($minprice){ $Vcond.=" and v_price >= ".$minprice."";
        }elseif($maxprice){ $Vcond.=" and v_price <= ".$maxprice."";}

        if($fuel_type!="")
            $Vcond.=" and v_fuel_type='$fuel_type' ";

        $sql = "Select *  from #__newcar_variants  Where v_prod_id='".$id."' $Vcond and v_status='1'";

        $database->setQuery($sql);
        $rows = $database->loadObjectList();
    $list="";
        if($rows){
            foreach($rows as $row){
                if($row->v_small_img!=""){
                    $img = "uploads/variants/".$row->v_big_img ;
                }else{
                    $img="templates/pioneer_home/images/dvd1.jpg";
                }       

    $temp1.='<li><a href="index.php?newcar&id='.$row->v_prod_id.'&vid='.$row->v_id.'">'.$row->v_name.'</a></li>';
    $temp2.='<li>Rs. '.$row->v_price.'</li>';
            }
            $list.='<div class="sliding-box-middle"><ul>'.$temp1.'</ul></div>';
            $list.='<div class="sliding-box-right"><ul>'.$temp2.'</ul></div>';

        }else{
            $list.='<p>No Variants.</p>';
        }
        return $list;
    }


  $sql = "Select *  from #__newcar_variants  Where v_prod_id='".$id."' $Vcond and v_status='1'";

函数loadProductVarients（$id、$minprice、$maxprice、$fuel\u类型）{
$mainframe=&JFactory:：getApplication（）；
$option=JRequest:：getCmd（'option'）；
$database=&JFactory:：getDBO（）；
全局$Itemid；
$Vcond=“”；
如果（$minprice！=”“和$MAXPICE！=”“）{
$Vcond.=”和（v_价格介于“$minprice.”和“$maxprice.”之间）；
}elseif（$minprice）{$Vcond.=”和v_price>=“$minprice.”；
}elseif（$maxprice）{$Vcond.=”和v_price=“.minprice.””；
}elseif（$maxprice）{$Vcond.=”和v_price在未设置的变量上使用串联

替换

 $temp1.='<li><a href="index.php?new&id='.$row->v_prod_id.'&vid='.$row->v_id.'">'.$row->v_name.'</a></li>';
 $temp2.='<li>Rs. '.$row->v_price.'</li>';

$temp1.='
'；
$temp2.='
Rs.$row->v_价格'
'；

借

$temp1='
'；
$temp2='
Rs.$row->v_价格'
'；

或

$temp1=''；
$temp2=''；
//代码
$temp1.='
'；
$temp2.='
Rs.$row->v_价格'
'；

不过，我没有发现您的第一个问题。
谢谢，但它不起作用。使用上面的代码虽然删除了错误通知，但它只显示一个产品变体。我的意思是说，法拉利有3个变体-zsi、lsi、msi。因此，使用上面的代码只显示一个变体，如我所说，我没有发现您的第一个问题。变体是我吗n v_name字段，对吗？是否尝试在foreach（$rows as$row）中回显某些内容循环查看是否确实有3个变量？您确定您的查询返回3行吗？是的，变量是v_name。我在第一次查询中问的问题：-目前变量中的字段仅显示变量名称，即zxi、lxi、exi、hxi。而我想要实现的是在变量名称下显示产品和变量名称。我指的是法拉利zxi、ferrari lxi、ferrari exi和ferrari hxi。原始sql查询正确显示-但仅显示变体。但是，产品名称，即prod_id=“$Carmodel”以及v_名称变体不会显示-尽管尝试了许多组合。请使用Joomla编码标准来处理您的数据库问题：Hello Lodder，建议您重新编写t他按照编码标准编码，但收到了新的通知。May pl Advice！！感谢Joomla编码标准的链接，将
$db->quote（v_status='1'）
更改为
$db->quote（'v_status=1'））
谢谢，很抱歉再次打扰您。虽然通知已经消失，但是没有针对模型显示变体。之前显示了变体。可能会建议-如果缺少任何内容！！请尝试
var\u dump（$rows）
查看是否实际从数据库中获取了任何结果。我认为这与查询的
->where（）
部分有关。
$Vcond.=“and v_fuel_type='$fuel_type'”；
因此它是从
和
开始的
 $temp1.='<li><a href="index.php?new&id='.$row->v_prod_id.'&vid='.$row->v_id.'">'.$row->v_name.'</a></li>';
 $temp2.='<li>Rs. '.$row->v_price.'</li>';

$temp1='<li><a href="index.php?new&id='.$row->v_prod_id.'&vid='.$row->v_id.'">'.$row->v_name.'</a></li>';
$temp2='<li>Rs. '.$row->v_price.'</li>';

$temp1 = '';
$temp2 = '';
// CODE
$temp1.='<li><a href="index.php?new&id='.$row->v_prod_id.'&vid='.$row->v_id.'">'.$row->v_name.'</a></li>';
$temp2.='<li>Rs. '.$row->v_price.'</li>';