Променливата $_GET проверка

[eddy]

Това като проверка издържаливо ли е.

<?php 

if($_GET['a']==1 && is_numeric($_GET['a'])) {

echo '<div id="komentarite">
<a href="galeriq.php?a=2">СКРИИ КОМЕНТАРИ</a>
</div>';

include("comments.php"); 
}
elseif($_GET['a']==2 && is_numeric($_GET['a'])) {
		echo '<div id="komentarite">
<a href="galeriq.php?a=1">ПОКАЖИ КОМЕНТАРИ</a>
</div>';
}
	
else {
echo '<div id="komentarite">
<a href="galeriq.php?a=1">ПОКАЖИ КОМЕНТАРИ</a>
</div>';}

?>

 

[eddy]

А това е другия вариянт:

<?php 

if($_GET['a']==1 && (int)$_GET['a']){

echo '<div id="komentarite">
<a href="galeriq.php?a=2">СКРИИ КОМЕНТАРИ</a>
</div>';

include("comments.php"); 
}
elseif($_GET['a']==2 && (int)$_GET['a']) {
		echo '<div id="komentarite">
<a href="galeriq.php?a=1">ПОКАЖИ КОМЕНТАРИ</a>
</div>';
}
	
else {
echo '<div id="komentarite">
<a href="galeriq.php?a=1">ПОКАЖИ КОМЕНТАРИ</a>
</div>';}

?>

 

[RegexSux]

А това е другия вариянт:

<?php 

if($_GET['a']==1 && (int)$_GET['a']){

echo '<div id="komentarite">
<a href="galeriq.php?a=2">СКРИИ КОМЕНТАРИ</a>
</div>';

include("comments.php"); 
}
elseif($_GET['a']==2 && (int)$_GET['a']) {
		echo '<div id="komentarite">
<a href="galeriq.php?a=1">ПОКАЖИ КОМЕНТАРИ</a>
</div>';
}
	
else {
echo '<div id="komentarite">
<a href="galeriq.php?a=1">ПОКАЖИ КОМЕНТАРИ</a>
</div>';}

?>

Във всички ифо-ве второто условие ти е излишно

 

[eddy]

Проверявам дали е число .

 

[tedo96]

Проверявам дали е число .

Да и няма смисъл от това в случая... Ти така и иначе си проверяваш дали е 1. Пробвай без втората проверка.

 

[eddy]

, а кое е по-надежния начин за проверяване дали е число.

is_numeric или (int)

Тука прочетох:
http://www.htmlforums.com/php-programming/t-int-get-101064.html

че is_numeric не е най-добрият вариянт.

 

[tedo96]

is_numeric ,но в случая няма смисъл от тази проверка...

 

[eddy]

Ясно, да

За is_numeric прочети какво е писал erisco във линка по-горе.