方法はマスターに異常の指導を依頼してください

方法、エラーを教えてください

この投稿は ymkacscc20 によって最終編集されました: 2012-12-13 02:53:59

<br />
<?php<br />
function get_employees_by_hierarchy( $_employee_id = 0,$_depth = 0,$_org_array = array() ) {<br />
	if ( $this->org_depth < $_depth ) {<br />
        $this->org_depth = $_depth;<br />
	}<br />
    $_depth++;<br />
    $_query = "SELECT * FROM employees WHERE ";<br />
    if ( !$_employee_id ) {<br />
        $_query .= "employee_manager_id IS NULL OR employee_manager_id = 0";<br />
    }<br />
   	else { <br />
       	$_query .= "employee_manager_id = " . $this->dbh->quoteSmart( $_employee_id );<br />
    }<br />
   	$_result = $this->query( $_query );<br />
<br />
   	while ( $_row = $_result->fetchRow() ) {<br />
       	$_row['depth'] = $_depth;<br />
       	array_push( $_org_array, $_row );<br />
       	$_org_array = $this->get_employees_by_hierarchy(<br />
                       $_row['employee_manager_id'],<br />
                       $_depth,<br />
                       $_org_array<br />
                     );<br />
   				}<br />
   return $_org_array;<br />
}<br />
?><br />
这段代码还可以做怎么样的优化啊? 3-5行为什么很重要？ 17行为什么很重要？ 小弟刚开始学php，求大神的指教哦~<br />
<br />

-----解決策---------
このコードのみが重要です$this->dbh->quoteSmart( $_employee_id )
ただし、quoteSmart は文字通りエスケープ機能のみを提供します。ただ逃げるだけなら余計だ。数値をエスケープする必要がないためです。文字列の場合、構築されたクエリ文字列
内で引用符で囲まれません。
$this->dbh->quoteSmart( $_employee_id ) が $_employee_id
の元の値を返す場合 次に、「employee_manager_id = " 。 $this->dbh->quoteSmart( $_employee_id ) は
を返します。 employee_manager_id=$_employee_id
のすべてのレコード 以下で再帰的に呼び出す場合、 $_employee_id = $_row['employee_manager_id']
$_employee_id は変更されていないため、無限ループに陥ります

したがって、 $this->dbh->quoteSmart( $_employee_id ) は $_employee_id
とは異なる値を返す必要があります。 したがって、彼はとても重要です