优化PHP代码 - 功能显示和未定义的变量

Question

任何建议PHP通告，以下是玩具变体的功能 - 功能loadProductVarients

目前其显示： -

一个）玩具变异名称只是，其中i打算实现产品名称和变体名称。例如，如果Hotwheels是公司，法拉利是产品，Zsi是其变种。

现在以下功能只显示ZSI作为变种的名字，我打算实现法拉利ZSI的变种名称

我认为需要增加在$ temp1目录产品的变量，但是尽管采用多种组合 - 我是无法实现它。

b）第二越来越PHP公告：未定义的变量：temp1中& TEMP2注意在功能loadproductvarients

我在学习PHP的 - 你提供帮助和建议将非常感激！

编辑

你好Lodder，

原函数

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'"; 

下面是修改后的代码与变化劝写： -

 function loadProductVarients($id,$minprice,$maxprice,$fuel_type){ 
     $db = JFactory::getDBO(); 
     $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' "; 

     $query = $db->getQuery(true); 
     $query->select('*'); 
     $query->from('#__newcar_variants');  
     $query->where($db->quote($Vcond), $db->quote($id), $db->quote('v_status=1'));  

     $db->setQuery($query); 
     $rows = $db->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; 
    } 

Answer 1

您在未设置的变量上使用连接。

更换

$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>'; 

我没有，虽然得到你的第一个问题。