Как бороться со адресной строкой браузера?

[AgentVlad]

Народ, вот такая вот проблема.

Суть в том, что у меня есть строка - например:

мой сайт/others.php?type=type12&car=type4&pa ges=100

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

Как это можно избежать?

[wall66]

понизить уровень отображения ошибок

доработать скрипт others.php добавив условия на правильность выполнения запросов и прочего и отладочные сообщения

обрабаоывать входящие значения в массивах get и post для обеспечения правильности формата введенных данных

изменить вид адресной строки с помошью чпу (mod_rewrite) - всеравно придется добавлять скриптам устойчивости

изучить пхп и получить образование чтоб не задавать таких вопросов

[AgentVlad]

//пожалуйста, не надо переходить на личности! Модератор.
Вопрос остается в силе для тех, кто готов помочь:
Поясню немного: запрос посылается не формой, а прямой ссылкой.

<a href =" http://мой сайт/others.php?type=type15&pages=100 "> Внедорожник </a>

Запрос определяет 2 переменные $type and $pages
переменная $type может быть от type1 до type16, а $pages только сто.

Может задать какие-то условия, при которых выполняется запрос, то как бы лучше это сделать, не прописывать же для каждого type1, type2... type999 (:

Спасибо, если есть встречный вопрос задавайте.

[mv28jam]

Возможно это поможет

if ($_GET['type']==null) $_GET['type']=type1;
if ($_GET['pages']==null) $_GET['pages']=1;

После того как Вы нагло обращаетесь на ТЫ к незнакомым людям, помогать Вам не будут, да и модератор удалит Ваши истеричные оры

[SkyM@n]

Кроме предложений, изложенных в постах выше, советую использовать ф-цию:

PHP код:

function quote_smart($value){
    // если magic_quotes_gpc включена - используем stripslashes
    if (get_magic_quotes_gpc()) {
        $value = stripslashes($value);
    }
    // Если переменная - число, то экранировать её не нужно
    // если нет - то окружем её кавычками, и экранируем
    if (!is_numeric($value)) {
        $value = "'" . mysql_real_escape_string($value) . "'";
    }
    return $value;
} 


или рекурсивную обработку запроса:
сама функция:

PHP код:

function sanitize_input(&$request) {
    $request = mysql_real_escape_string(strip_tags($request));
}
так использовать:
array_walk_recursive($_REQUEST, 'sanitize_input'); 


Сообщение от wall66

понизить уровень отображения ошибок

А вот этого делать не советовал бы. Ошибки отображать по максимуму для обеспечения наименьшей возможности срабатывания человеческого фактора. Но с другой стороны третьи лица, воззрев сообщения об ошибке - может воспользоваться ним не на благо вам. Так что ставить error_reporting исключительно для себя и/или на время дебаггинга/девелопинга.

[AgentVlad]

Спасибо, что предложили свои варианты, но я видимо плохо объяснил.

Попробую еще раз, мож что придет на ум:

others.php?type=type15&pages=100

"type=" - она имеет только значения type1 до type15
&pages= - только 100

остальные значения - мимо.

Можно ли:

if ($type == 'type1' and $pages == '100') { Делать это! }
else { Ничего не найдено. }

Но как сделать, чтоб не прописывать для каждого type1 - type990

Сделать типо того, только правильно

if ($type == 'type1-type990' and $pages == '100') { Делать это! }
else { Ничего не найдено. }
(словами: если переменная type равна от тайпу1 до тайпу990(т.е. type1,type2...typeN...type990) и pages равна стам, то делать этот запрос, если нет - писать "ничего не найдено")

Понимаю, что туго объяснено, но мож кто поймет...

[mv28jam]

if ($type == 'type1-type990' and $pages == '100') { Делать это! }
else { Ничего не найдено. }
(словами: если переменная type равна от тайпу1 до тайпу990(т.е. type1,type2...typeN...type990) и pages равна стам, то делать этот запрос, если нет - писать "ничего не найдено")

так можно

PHP код:

if($pages == 100 and substr($type,0,4)=='type' and  substr($type,3)>=1 and substr($type,3)<=990){
  Tdosomething;
}else{
  echo 'sorry there is no such item';
} 


[SkyM@n]

А чем приведенный вами код не устраивает? Или вы хотите оптимизации?
Вы мне покажите что должно быть вместо этого:

// Делать это!

PHP код:

if ($type == 'type1' and $pages == '100') {
   //  Делать это!   
}else {
   // Ничего не найдено.  
} 


Но я таки настаивал на ЧПУ.
Создаете в корне сайта файл:
.htaccess

  <IfModule mod_rewrite.c>
       RewriteEngine On
       Options +FollowSymLinks

      RewriteRule ^others/([0-9]+)-([0-9]+)\.html$ others.php?type=type$1&page=$2 [L,NC]
  </IfModule>

И после этого заходите не как others.php?type=type1&pages=100, а как /others/1-100.html

[QunneD]

PHP код:

if (isset($type) and isset($pages))
{
   if (($type > 15 and $pages > 99) or ($type < 1 and $pages < 1))
   {
      echo "Такая страница не найдена!";
   }
   else
   {
      // Что делать...
   }
} 


Вот если правильно понял, писал в блокноте. На ошибки не проверял, но должно работать.

[bohdan4ik]

Сообщение от SkyM@n

Так что ставить error_reporting исключительно для себя и/или на время дебаггинга/девелопинга.

С этим я не соглашусь. error_reporting(E_ALL) - ставить всегда, не забыв, при этом, включить логи, а вот вывод ошибок в броузер (display_errors) - только на время разработки приложения (:
З.Ы.: Советую не подавлять ошибки, используя @ (согласен, может немного сократить размер кода, но так же может и безопасности повредить), а отлавливать их "человечным" способом