これはphpと$ _POST変数で2つのmysqlテーブルを照会する正しい方法ですか？

Question

私はデータベースから注文を返す最良の方法を見つけようとしています。私はこれを思いついて、私が望むように働いています（私は思っています）それは私に探している結果を与えますが、正しいか、それとも良い方法があるかを知りたかったのです。

<?php 

$conn = mysql_connect('', '', ''); 
if (!$conn) 
{ 
die('Could not connect: ' . mysql_error()); 
} 
mysql_select_db($dbs, $conn); 

$Order_ID = $_POST['Order_ID']; 
//$Order_ID = '1001'; 

    $queryOrderHead = "SELECT * FROM Orders WHERE Order_ID = '$Order_ID' "; 

    $queryOrderLines = "SELECT * 

    FROM Order_LineDetails 
    WHERE Order_LineDetails.Order_ID = '$Order_ID' 

"; 
     if ($queryRunHead = mysql_query($queryOrderHead)){ 

       while ($info_HEAD = mysql_fetch_array($queryRunHead))  
       { 
        $OrderID_HEAD = $info_HEAD['Order_ID'];   
        $User_ID_HEAD = $info_HEAD['User_ID']; 
        $Customer_ID_HEAD = $info_HEAD['Customer_ID'];       
        echo $OrderID_HEAD.' '.$User_ID_HEAD.' '.$Customer_ID_HEAD.'<br>'; 
       } 

       $queryRunLines = mysql_query($queryOrderLines); 
       while ($info = mysql_fetch_array($queryRunLines))  
       { 
        $OrderID = $info['Order_ID'];   
        $OrderLineID = $info['OrderLineItem_ID'];   
        echo $OrderID.' '.$OrderLineID.'<br>'; 
       } 

     } else { 
     echo mysql_error();   
     } 

mysql_close($conn); 
?> 

だから、それが何、それは$ _POSTからOrder_IDのvalを使用し、それが同じOrder_IDを使用し、2番目のクエリをループし、成功に最初のクエリを実行し、別のテーブルからすべてのOrder_LineDetailsを取得しています。 mysql_real_escape()タグ以外

....

任意のポインタやアイデア???

Answer 1

任意のポインタまたはアイデア???

最初に注文を選択してから項目を選択することに問題はありません。しかし、あなたは（両方の変数とデータベース・スキーマ用）の配列構造にデータを整理するだけでなく、より良い命名規則を次の恩恵を受けるだろう：

$orderId = $_POST['order_id']; 

// order_id should be an INT, so no quotes. 
// Also look into parameterized queries with PDO as the mysql_* functions are archaic! 
$sqlOrder = "SELECT * 
      FROM orders 
      WHERE order_id = ".mysql_real_escape_string($orderId); 

$order = array(); 
if ($resOrder = mysql_query($sqlOrder)) { 

    if ($rowOrder = mysql_fetch_array($resOrder)) { 
     $order = $rowOrder; 
     // echo $rowOrder['order_id'].' '.$rowOrder['user_id'].' '.$rowOrder['customer_id']."<br />\n"; 

     $sqlOrderLines = "SELECT * 
        FROM order_lines 
        WHERE order_lines.order_id = ".mysql_real_escape_string($orderId); 

     if ($resOrderLines = mysql_query($sqlOrderLines)) { 
      $order['order_lines'] = array(); 
      while ($rowOrderLines = mysql_fetch_array($resOrderLines)) { 
       $order['order_lines'][] = $rowOrderLines;     
       // echo $rowOrderLines['order_id'].' '.$rowOrderLines['order_line_id']."<br />\n"; 
      } 
     } 
    } else { 
     echo 'Order not found'. 
    } 



} else { 
    echo mysql_error();   
} 

// debug 
print_r($order);