ГДЕ gid = $ gid LIMIT 26 не работает?

Question

Возможно, у меня нет лучшего запроса в запросе, и если кто-то может научить меня более эффективному способу, который будет оценен.

Но с тем, что у меня есть, я не могу понять, почему команда LIMIT не работает?

Я ОГРАНИЧИВАЯ миниатюры до 26, но я получаю 38?

Очень странно.

Если это помогает, я могу сказать, что в таблице галереи есть 7 опубликованных галерей, поэтому оттуда будет 7 идентификаторов, и я надеялся, что следующий запрос будет проходить через таблицу фотографий и вернуть все фотографии с помощью id из 7 галерей, которые я спросил выше справа?

Ну Theres мой код если кто может иметь смысл его ..

<?php 

// Check if gallery is published 

$query1 = "SELECT id,status FROM gallery WHERE status=1"; 
$result1 = mysql_query($query1) or die(mysql_error()); 
while($row1 = mysql_fetch_array($result1)) { 

$gid = $row1['id']; 

// now get the photo file names based on the above published gallery ids 

$query2 = "SELECT id,uid,gid,image,origimage FROM photo WHERE gid='$gid' LIMIT 20"; 
$result2 = mysql_query($query2) or die(mysql_error()); 
while($row2 = mysql_fetch_array($result2)) { 

?> 


<div style="float:left;"> 
<a class="featureGrid" href="public-photo-user.html?uid=<?php echo $row2['uid']; ?>&gid=<?php echo $row2['gid']; ?>&id=<?php echo $row2['id'];?>"> 
<img src="media/users/croppedthumbs/<?php echo $row2['uid']; ?>/<?php echo $row2['gid'] ?>/<?php echo $row2['image']; ?>" /> 
</a> 
</div> 

<?php }} ?> 

Благодаря любому, который может помочь пролить свет на это или предложить/научи меня лучший путь.

Cheers. John

Answer 1

Вы делаете несколько запросов, каждый из которых по отдельности ограничивается до 20.

Вместо объединить запросы в одном запросе с помощью объединения. Это более эффективно, чем зацикливание нескольких запросов, и вы можете положить предел комбинированных результатов:

SELECT photo.id, photo.uid, photo.gid, photo.image, photo.origimage 
FROM photo JOIN gallery ON gallery.id=photo.gid 
WHERE gallery.status=1 
LIMIT 20 

Вы также должны всегда быть осторожны, чтобы использовать mysql_real_escape_string() при вводе значений в строки запроса, или вы получите SQL-инъекцию дыры безопасности. Аналогично, весь текст, выводимый на HTML-страницу, должен быть закодирован с использованием htmlspecialchars(), чтобы избежать разметки-инъекции, а данные, вставленные в часть URL-адреса, должны быть закодированы с помощью rawurlencode().

ETA:

Можете ли вы мне ссылку на пример mysql_real_escape_string в сочетании с htmlspecialchars.

Ну, если вы еще делали это, это:

$query2 = "SELECT id,uid,gid,image,origimage FROM photo WHERE gid='$gid' LIMIT 20"; 

понадобились бы $gid в нем быть экранированы, в противном случае любой апостроф (или потенциально обратной косой черты) в значении приведет его взрывать вверх.

$query2 = "SELECT id,uid,gid,image,origimage FROM photo WHERE gid='".mysql_real_escape_string($gid)."' LIMIT 20"; 

Тогда:

href="public-photo-user.html?uid=<?php echo $row2['uid']; ?>... 

будет уязвим, если значение uid содержит двойные кавычки, и будет также не для других различных символов, которые не являются действительными, чтобы положить в URL.

Возможно, вы можете быть уверены, что ваши идентификаторы никогда не будут содержать знаков препинания, но могут быть использованы любые другие значения, поэтому лучше всегда использовать соответствующую кодировку всякий раз, когда вы вставляете строку текста в другой контекст, такой как SQL, HTML или URL.

Typing mysql_real_escape_string и htmlspecialchars все время немного утомительно, хотя, так что я, как правило, определить функции быстрого доступа, как:

function m($str) { return "'".mysql_real_escape_string($str)."'"; } 
function h($str) { echo htmlspecialchars($str); } 
function u($str) { echo rawurlencode($str); } 

который может быть использован как:

$query2 = "SELECT id,uid,gid,image,origimage FROM photo WHERE gid=".m($gid)." LIMIT 20"; 

Hello, <?php h($name); ?> 

<a class="featureGrid" href="public-photo-user.html?uid=<?php u($row2['uid']); ?>&amp;gid=<?php u($row2['gid']); ?>&amp;id=<?php u($row2['id']); ?>"> 

(примечание также & в значении атрибута должно быть экранировано до & для корректности и надежности HTML.)

Answer 2

У вас есть несколько галерей с status из 1 и, следовательно, проходят через внутренний контур более одного раза.