我有这段代码,它确实不是最漂亮和最高效的。该代码有效,但我觉得有办法避免所有循环并组合这些查询并可能构成一个数组。我只是想知道是否有人对如何改进这一点有任何指示
$sql = "SELECT p.image, group_concat(pi.image) as additional_images
FROM product as p
JOIN product_image as pi on pi.product_id=p.product_id
WHERE p.product_id = '{$product['standard']['product_id']}'";
$result = mysql_query($sql);
$images_before = mysql_fetch_assoc($result);
$images = array();
if($images_before['additional_images'] != ""){
$images = explode(",", $images_before['additional_images']);
}
array_push($images, $images_before['image']);
$sql2 = "SELECT p.image, group_concat(pi.image) as additional_images
FROM product as p
JOIN product_image as pi on pi.product_id=p.product_id
WHERE p.product_id = '{$product['professional']['product_id']}'";
$result2 = mysql_query($sql2);
$images_before2 = mysql_fetch_assoc($result2);
$images2 = array();
if($images_before2['additional_images'] != ""){
$images2 = explode(",", $images_before2['additional_images']);
}
array_push($images2, $images_before2['image']);
$sql3 = "SELECT p.image, group_concat(pi.image) as additional_images
FROM product as p
JOIN product_image as pi on pi.product_id=p.product_id
WHERE p.product_id = '{$product['premium']['product_id']}'";
$result3 = mysql_query($sql3);
$images_before3 = mysql_fetch_assoc($result3);
$images3 = array();
if($images_before3['additional_images'] != ""){
$images3 = explode(",", $images_before3['additional_images']);
}
array_push($images3, $images_before3['image']);
$counter = 0;
foreach($images as $image) {
if ($counter == 0) {
echo "<a id='show_{$product['standard']['product_id']}' style='display:none;' href='http://somesite.com/shop_possystems/image/{$image}' rel='prettyPhoto[show_{$product['standard']['product_id']}]'></a>";
$counter++;
}else{
echo "<a style='display:none;' href='http://somesite.com/shop_possystems/image/{$image}' rel='prettyPhoto[show_{$product['standard']['product_id']}]'></a>";
$counter++;
}
}
$counter2 = 0;
foreach($images2 as $image) {
if ($counter2 == 0) {
echo "<a id='show_{$product['professional']['product_id']}' style='display:none;' href='http://somesite.com/shop_possystems/image/{$image}' rel='prettyPhoto[show_{$product['professional']['product_id']}]'></a>";
$counter2++;
}else{
echo "<a style='display:none;' href='http://somesite.com/shop_possystems/image/{$image}' rel='prettyPhoto[show_{$product['professional']['product_id']}]'></a>";
$counter2++;
}
}
$counter3 = 0;
foreach($images3 as $image) {
if ($counter3 == 0) {
echo "<a id='show_{$product['premium']['product_id']}' style='display:none;' href='http://somesite.com/shop_possystems/image/{$image}' rel='prettyPhoto[show_{$product['premium']['product_id']}]'></a>";
$counter3++;
}else{
echo "<a style='display:none;' href='http://somesite.com/shop_possystems/image/{$image}' rel='prettyPhoto[show_{$product['premium']['product_id']}]'></a>";
$counter3++;
}
}
?>
而不是使用 where p.product_id = <value>
, 使用 WHERE IN
语法:
$sql = "SELECT ...
WHERE p.product_id IN (
{$product['standard']['product_id']},
{$product['professional']['product_id']},
{$product['premium']['product_id']}
)";
我是一名优秀的程序员,十分优秀!