2011-04-26 107 views
0

好的,所以我有这个功能,我在页面中使用了很多时间,但只有当我调用函数一次,或者如果一个带有结果的函数我称它为第一...在php中自己的函数,多个调用不同的参数问题

这是查询:

$resultClientes = mysql_query("SELECT nombre.idNombre 
    , nombre.nombre AS nombreNombre 
    , comuna.nombre AS nombreComuna 
    , giro.nombre AS nombreGiro 
    , provincia.nombre AS nombreProvincia 
    , provincia.region_idRegion AS idRegion 
FROM nombre 
INNER JOIN comuna ON nombre.Comuna_idComuna = comuna.idComuna 
INNER JOIN giro ON nombre.Giro_idGiro = giro.idGiro 
INNER JOIN provincia ON comuna.Provincia_idProvincia = provincia.idProvincia 
ORDER BY nombreProvincia, nombreComuna, nombreGiro, nombreNombre"); 

,这是功能:

function listaClientesPorRegion($numReg,$query){ 
    $nombreProvincia = ""; 
    $nombreComuna = ""; 
    $nombreGiro = ""; 
    $nombreNombre = array(); 
    $i=0; 

    while ($row = mysql_fetch_assoc($query)) { 
    if($row['idRegion']== $numReg){ 
     if ($nombreProvincia == $row['nombreProvincia']) { 
     if ($nombreComuna == $row['nombreComuna']) { 
      if ($nombreGiro == $row['nombreGiro']) { 
      $nombreNombre[] = $row['nombreNombre']; 
      } 
      else { //nombreGiro 
      echo '<li>' . implode('</li><li>', $nombreNombre).'</li></ul></li></ul>'; 
     $nombreGiro = $row['nombreGiro']; 
     echo '<ul class="clientes_giro"><li>'.$nombreGiro.'<ul 
        class="clientes_nombre">'; 
     $nombreNombre = array($row['nombreNombre']); 
      } 
     } 
     else { // nombreComuna 
      echo '<li>' . implode('</li><li>', $nombreNombre). 
       '</li></ul></li></ul></li></ul>'; 
      $nombreComuna = $row['nombreComuna']; 
      echo '<ul class="clientes_comuna"><li>'.$nombreComuna; 
      $nombreGiro = $row['nombreGiro']; 
      echo '<ul class="clientes_giro"><li>'.$nombreGiro.'<ul 
       class="clientes_nombre">'; 
      $nombreNombre = array($row['nombreNombre']); 
     } 
     } 
     else { // nombreProvincia 
     if (!empty($nombreNombre)) { 
      echo '<li>' . implode('</li><li>', $nombreNombre). 
       '</li></ul></li></ul></li></ul></li></ul></div>'; 
     } 
     $class = $i++ % 2 ? 'clientes_floatEven' : 'clientes_floatOdd'; 

     $nombreProvincia = $row['nombreProvincia']; 
     echo '<div id="'.$class.'"><ul class="clientes_provincia"> 
       <li><div class="underline_yellow">'.$nombreProvincia.'</div>'; 
     $nombreComuna = $row['nombreComuna']; 
     echo '<ul class="clientes_comuna"><li>'.$nombreComuna; 
     $nombreGiro = $row['nombreGiro']; 
     echo '<ul class="clientes_giro"><li>'.$nombreGiro.'<ul 
     class="clientes_nombre">'; 
     $nombreNombre = array($row['nombreNombre']); 
     } 
    } 
    } 
    if (!empty($nombreNombre)) { 
    echo '<li>' . implode('</li><li>', $nombreNombre). 
     '</li></ul></li></ul></li></ul></li></ul></div>'; 
} 
} 

,这是HTML的一部分

<div id="atacama"> 
    <ul class="regiones_nomb_container"> 
    <li class="atacama"></li> 
    </ul> 
    <?php 
    listaClientesPorRegion(3,$resultClientes); 
    ?> 
</div> 

<div id="coquimbo"> 
    <ul class="regiones_nomb_container"> 
    <li class="coquimbo"></li> 
    </ul> 
    <?php 
    listaClientesPorRegion(4,$resultClientes); 
    ?> 
</div> 

<div id="valparaiso"> 
    <ul class="regiones_nomb_container"> 
    <li class="valparaiso"></li> 
    </ul> 
    <?php 
    listaClientesPorRegion(5,$resultClientes); 
    ?> 
</div> 

起初我只用一次时间和一个查询,我知道会有记录,如listaClientesPorRegion(9,$ resultClientes),然后我尝试了一个数字,我知道不会有记录,都工作,所以我打电话给我的网站所有的div多次,但现在不工作,我不知道为什么... (我是新的PHP和东西),谢谢你的帮助!

+0

我首先想到的是,从函数返回一个数组/对象和显示结果分别代替在函数内'echo'ing的。或者至少返回一串结果......我只是不喜欢函数/方法中的'echo'或'print'。 – 2011-04-26 01:10:29

+0

对于这个建议,我可能会受到批评,但请看[Smarty](http://www.smarty.net/documentation)。将代码从代码中分离出来是让代码可重用和健壮(如果你做得对)的重要一步。 – 2011-04-26 01:14:50

+0

还有[Flexy](http://pear.php.net/package/HTML_Template_Flexy/docs),它更像是一个模板引擎,但我觉得这个文档很累。 – 2011-04-26 01:22:08

回答

1

代码有一些问题。
首先,它看起来像你正在发送IdRegion和查询的结果作为($ numReg,$ query)你的函数。
这意味着您将不得不通过解析结果集中的每个记录以显示您正在查找的内容。

将查询移入该函数并仅传递idRegion。
其次,对$ numReg进行一些错误检查,以确保某些事情能够通过。
第三,在查询中放入$numReg值以自动限制结果。
这样你只能拉你打算使用的记录。

下面是示例代码:

function listaClientesPorRegion($numReg = null){ 
    if(!$numReg) { 
     // handling here (i.e. return null, echo 'Error!';, etc.) 
    } 
    $nombreProvincia = ""; 
    $nombreComuna = ""; 
    $nombreGiro = ""; 
    $nombreNombre = array(); 
    $i=0; 

    $resultClientes = mysql_query("SELECT nombre.idNombre 
     , nombre.nombre AS nombreNombre 
     , comuna.nombre AS nombreComuna 
     , giro.nombre AS nombreGiro 
     , provincia.nombre AS nombreProvincia 
     , provincia.region_idRegion AS idRegion 
    FROM nombre 
    WHERE provincia.region_idRegion = $numReg 
    INNER JOIN comuna ON nombre.Comuna_idComuna = comuna.idComuna 
    INNER JOIN giro ON nombre.Giro_idGiro = giro.idGiro 
    INNER JOIN provincia ON comuna.Provincia_idProvincia = provincia.idProvincia 
    ORDER BY nombreProvincia, nombreComuna, nombreGiro, nombreNombre"); 

    while ($row = mysql_fetch_assoc($query)) { 
    ... 
+0

thanksss,那工作! – Gmo 2011-04-26 02:12:21