What is wrong with this code

[Dylan Leblanc]

When this code is run the output is not what I expect.

<?

echo ( $id = (int) $_GET['id'] ? "[id{$id}]" : '' ) . ', ';
echo ( $id = (int) $_GET['id'] ? "?id={$id}" : '' );

?>


The output I want is:

[id12], ?id=12


But the output I get is:

[id12], ?id=[id12]


Can anyone tell what is wrong with the code? Is there something incorrect about the order of operations in the conditional expressions?

[firepages]

well


<?
echo ( $id = (int) $_GET['id'] ? "[id{$_GET['id']}]" : '' ) . ', ';
echo ( $id = (int) $_GET['id'] ? "?id={$_GET['id']}" : '' );
?>


gives the answer you are looking for ... as does


<?
echo ((int) $_GET['id'] ? "[id{$id}]" : '' ) . ', ';
echo ((int) $_GET['id'] ? "?id={$id}" : '' );
?>


lol actually with register_globals=off the second one does not


but not having a clue what you are doing I dont know if that helps :confused:

[V@no.]

what is (int) for? are u trying check if $id is a number not?
did u mean intval?
if so, probably this how it should be?<?
echo ( ($id = intval($_GET['id'])) ? "[id{$_GET['id']}]" : '' ) . ', ';
echo ( ($id = intval($_GET['id'])) ? "?id={$_GET['id']}" : '' );
?>

[Dylan Leblanc]

(int) reduces the value to an integer, so if the value was a string, the value would become 0, thus making the conditional expression false, and nothing would happen. It is a way of validating the incoming data.

Firepages, yes I can get some of your suggestions to work, kind of a combination of the two. Thanks. :)

[Spookster]

Originally posted by Dylan Leblanc
(int) reduces the value to an integer

The proper terminology is "typecasting" for us programmer geeks. :)

[V@no.]

Originally posted by Spookster
The proper terminology is "typecasting" for us programmer geeks. :)
hmmm...and why it's not published as a function at www.php.net ?
or it's not actualy function? :confused:

[firepages]

geek ! - speak for yourself Spooks :D - lol & I just been told I am a scripter not a programmer :( the shame ..... anyway

ok, also look at the '===' operator , it checks for type as well...


<?
$wha='1';
//$wha=1;
echo ((int)$wha=== $wha)?'true':'false';
?>

[firepages]

V@no , I dont think its technically a function as such , ? I dunno - anyway its just like typecasting in C which PHP is made from so it all makes sense eventually ~ check out
http://www.php.net/manual/en/language.types.type-juggling.php

[Spookster]

Originally posted by firepages
geek ! - speak for yourself Spooks :D - lol & I just been told I am a scripter not a programmer :( the shame ..... anyway

ok, also look at the '===' operator , it checks for type as well...


<?
$wha='1';
//$wha=1;
echo ((int)$wha=== $wha)?'true':'false';
?>


you are a GID. :D

[firepages]

GID ....

scared to ask :)

[Spookster]

Originally posted by firepages
GID ....

scared to ask :)

I was waiting for you to ask. lol

GID(Geek in Denial) :D