Форум PHP

Принимаем от пользователя некий id, хочу найти оптимальный и простой способ обработки.
?id = x

$id = 0+(int)$_GET["id"]; if($id > 0){ //иначе ошибка ввели строку или 0 или отрицательное число или ничего }

Как я понимаю, от вредосных строк мы защищены
Но как защитится от слишком большого числа, или от числового выражения в адресной строке.
Нужно, сравнить число с тем которое может быть.
1 SELECT count(id) FROM table WHERE id = x
Выполнить, такой sql запрос с пришедшим возможно неверным id.
2 SELECT id FROM table WHERE 1
Собрать все id из таблицы и сверить с ними пришедший id. Но если данных очень много выполнять такой запрос не хотелось бы.
3 SELECT max(id) FROM table WHERE 1
Выбрать максимальные id и поставить условие.
Поставить условие типа if(id > 0 && id < $maxid)

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

а для чего вы хотите так делать? что будет, если id окажется большим?

Если п-ль ввел огромный id будет неверный sql запрос
Допустим у меня поле Id типа INT
У него есть свой предел, если число превысит этот предел то вместо него будет занесено значение соответсвующее границе предела.
По мойму оно не надо:)

в таком случае как вы защищаетесь от отсутствующих id в базе.
вот я введу id=4, а у вас такого нет. и что теперь? general protection fault как поговаривала винда.
во-вторых с чего вы взяли что при большом id будет неверным sql запрос?

Решение такое:
Запрос
&goodid=65&count=432

function CheckGoodid(){   $maxid = getoneresult("SELECT MAX(goodid) FROM goods WHERE 1");     if($this -> goodid > 0 && $this -> goodid < $maxid){      if(getoneresult("SELECT count(*) FROM goods WHERE goodid =".$this -> goodid)){       return TRUE;      }   }     return FALSE;  }  function CheckCount(){   global $maxgoodcount;     if($this -> count > 0 && $this -> count < $maxgoodcount){         return TRUE;   }     return FALSE;  } //getoneresult возвращает одно выбранное значение

вот я введу id=4, а у вас такого нет. и что теперь? general protection fault как поговаривала винда.

Сначала смотрим, что 4 больше 0 и меньше моего максимального id в базе,
а затем SELECT count(*) FROM goods WHERE goodid =4

>Если п-ль ввел огромный id будет неверный sql запрос

покажите пример такого запроса

INSERT INTO `basket` ( `sid` , `dateadd` , `goodid` , `count` )
VALUES (
'0', '0', '100000000000000000000000000', '0'
);


В таблице получается 0 0 2147483647 0

а где неправильный запрос? mysql_error() что то возвращает?
и что за такой id который пользователь сам должен делать?
получается что вы сами не знаете структуру вашей таблицы.
какая первоначальная задача? потому как решение очень странное

Задача корзина, интернет магазина, в пользователь вводит count, количество товара в текстовое поле, конечно он туда не введет 10000000000000, если он нормально делает заказ, но теоритически ему никто не мешает этого сделать. Так же как и ввести например отрицательное число. А зачем мне в таблице заказов бессмысленные значения?
mysql_error() конечно ничего не возвращает, но тем не менее если бы был не INT а TINYINT урезались бы уже достаточно небольшие числа. Почему бы не проверять большие значения?

Насчет goodid тут как и в любом каталоге, можно сделать просто SELECT count(*) FROM goods WHERE goodid =".$this -> goodid и далее товар/рубрика найден/не найден.

Но помойму, условие не повредит
$maxid = getoneresult("SELECT MAX(goodid) FROM goods WHERE 1");
if($this -> goodid > 0 && $this -> goodid < $maxid){
}
Если $this -> goodid > $maxid в данном случае имело место редактирование адресной строки и не подставлять $this -> goodid в запрос, по мойму уже плюс к безопасности...

проверить сколько есть на складе и дальше плясать уже от этого

проверить сколько есть на складе и дальше плясать уже от этого

Не ну можно в goods заделать поле count и через админку бы заполнялось для каждого товара его количество. Это тоже стандартно и мое решение легко расширяемо и для такого случая, но если заказчику это не нужно. Те админу проще не заполнять это поле. Я просто выношу в настройки $maxcount = 10000 согласовав с заказчиком, если сделали заказ на какое нибудь реальное число товаров, то его можно рассматривать, а если ввели например 10000000 то такой заказ даже не пройдет.


Как вы считаете?

Если вас беспокоит слишком большое число, да можно воспользоваться третьим решением, но обычно это не делают, так как пользователь вряд ли в этом случае что-то реально покупать будет...