我在将一些代码从旧式mysql查询转换为准备时遇到问题。我假设问题是由于我正在使用多个whiles,每个whiles都有自己的查询导致问题,因为一次只能激活一个预准备语句。
编辑:如果有人关心,我只使用了2个循环 -
function createDeskMenu()
{
global $bookingTimes, $dbconn;
$day0 = mktime(0, 0, 0, date("m") , date("d"), date("Y"));
$query = "SELECT location FROM location";
$result = mysqli_query($dbconn,$query);
mysqli_num_rows($result);
while ($row = mysqli_fetch_array($result))
{
$location = $row['location'];
echo "<h3>$location</h3><div>";
$query = $dbconn -> prepare("SELECT COALESCE( CountDesk, 0 ) total, name, d.desk_id, phone, fax, dock, pc FROM desk d LEFT JOIN (SELECT COUNT(booked.desk_id) CountDesk, desk_id FROM booked WHERE booking_id >=?)b ON d.desk_id = b.desk_id WHERE location=?");
$query->bind_param("is",$day0, $location);
$query->execute();
$query->bind_result($totalCount,$name,$desk_id,$phone,$fax,$dock,$pc);
while($query->fetch()) {
$total = count($bookingTimes) * 14 - $totalCount;
echo '<a href="?page=desk&desk='.$desk_id.'"><div class="desk"><b>'.$name.'
('.$total.' Available Bookings)</b><li>Facilities:';
if($phone){echo " Phone,";}if($fax){echo " Fax Machine,";}if($dock){echo " Laptop Dock.";}if($pc){echo " Desktop Workstation.";}
echo '</li></div></a><hr />';
}
$query->close();
echo '</div>';
}
}
答案 0 :(得分:1)
当连接有等待从另一个语句获取的行时,您不能prepare()
语句。您必须先关闭以前的结果集或从中获取所有行。
我认为不需要在所有中检索location
的外部查询,因为它没有WHERE
子句。您正在选择所有位置,因此可以完全省略该部分。所有使用外部循环的方法是为每个位置创建一个<h3>
,这非常浪费(除了最初导致代码破损)
相反,执行一个查询并在获取循环中检查位置是否已更改。当它改变时,输出你的标题
echo "<h3>$location</h3><div>";
因此,删除外部查询并完全循环,并使用类似以下的模式来检测location
中的更改。请务必ORDER BY location
,以便为您排序。
不需要绑定参数。您可以使用query()调用执行此操作,因为该位置不再是变量,并且$day0
已知为mktime()
的时间戳。
// Substitute a query() call and $day0 can be inserted directly.
// This one query fetches all locations sorted...
$query = $dbconn->query("
SELECT
COALESCE( CountDesk, 0 ) total,
name,
d.desk_id,
phone,
fax,
dock,
pc
FROM
desk d
LEFT JOIN (
SELECT COUNT(booked.desk_id) CountDesk, desk_id FROM booked WHERE booking_id >= $day0
)b ON d.desk_id = b.desk_id
ORDER BY location");
// Store the last location in a variable which starts empty...
$location = "";
while($row = $query->fetch_assoc()) {
// on change of $location, update the variable.
if ($location !== $row['location']) {
$location = $row['location'];
// And output the new location value
echo "<h3>$location</h3><div>";
}
// Do the rest of your loop.
$total = count($bookingTimes) * 14 - $row['total'];
echo '<a href="?page=desk&desk='.$row['desk_id'].'"><div class="desk"><b>'.$row['name'].'
('.$total.' Available Bookings)</b><li>Facilities:';
if($row['phone']){
echo " Phone,";
}
if($row['fax']){
echo " Fax Machine,";
}
if($row['dock']){
echo " Laptop Dock.";
}
if($row['pc']){
echo " Desktop Workstation.";
}
echo '</li></div></a><hr />';
}
$row->close();
echo '</div>';
现在说明它失败的原因....当前一个语句或查询中还有剩余行时,你不能prepare()
一个新语句。您必须首先获取所有行,或者使用$stmt->close()
关闭语句。所以有效地你不能嵌套获取循环。
更好的方法是首先将所有行提取到数组中,然后遍历该数组:
while ($row = $first_query->fetch()) {
// Append all onto an array
$first_query_rows[] = $row;
}
// Then loop over that
foreach ($first_query_rows as $row) {
// Do a new query with $row
}
通常,这可以通过适当的JOIN
来解决。