Лагает проверка на входящие данные.

Ты говоришь: "Если оно не равно один, или если не равно два". Но если оно равно один, то оно не может быть равно два! И наоборот. Попробуй, лучше, так:

Код (Text):

1. if ($option!="1" AND $option!="2")

Мы говорим: "если оно не равно один и не равно два"
В кавычки взял, чтобы не было преобразования типа из string в int

 

PHP:

1. <?php
2.  
3. // Берем модуль числа для проверки
4. $option = abs(intval($_GET['option']));
5.  
6. if($option && $option < 2)
7.     header("Location: index.php");
8.  
9. ?>

 

Apple, * if(!$option || $option > 2)

 

Ах да, чёрт!
Мне показалось при условии 1 или 2 переадресовывать.

 

Кстати

PHP:

1. <?
2.  
3. $option = intval($_GET['option']);
4.  
5. switch($option) {
6.     case 1: {
7.         // ...
8.    
9.         break;
10.     }
11.  
12.     case 2: {
13.         // ...
14.    
15.         break;
16.     }
17.  
18.     default:
19.         header("Location: index.php");
20. }
21.  
22. ?>
23.  

Так легче будет условия добавлять и их обработку
Ну это дело автора.

 

=)

PHP:

1.  
2. <?php
3.  
4. if (in_array($option, array(1,2)) === false)
5. ...
6.  
7.  

 

А если $option меньше нуля или с десятичной частью?

Тогда уж
if (!ctype_digit($option) OR $option < 1 OR $option > 2)
...

А еще нужно проверить на существование через isset

 

Мне вариант со switch-case нравится