[PHP] Controleren op 0 en isset conflict

[verytallman]

Ik zat net enorm te klote met een variabele die een getal kan zijn (van 0 tot oneindig) of ook geen waarde kan bevatten.

Ik wou controleren of de variabele een waarde bevat, maar dat gaat fout want php ziet 0 als geen waarde. Om uiteindelijk de waarde te weten te komen heb ik 3 if statements nodig....:

PHP:

if ($id == 0 && is_numeric($id)) {
   // waarde is 0
}

if (!isset($id)) {
   // geen waarde
}

if (!empty($id) && is_numeric($id)) {
   // andere waarde
}

Als je hiernaar kijkt klopt php ook niet, want: bij die eerste if doe ik een check is_numeric(), maar waarom geeft php daarop wel true als hij eigenlijk vind dat $id geen waarde is? (Daarmee bedoel ik, hij zegt bij isset($id) => false).
Php vind dus wel dat $id = 0 een waarde is want is_numeric($id) geeft true, maar ook weer niet want isset($id) geeft false.

Doe ik nu erg moeilijk of is dit de enigste manier met php ?

[x-force]

Moet je dan niet met === gaan werken?
http://nl3.php.net/manual....operators.comparison.php

[Verwijderd]

Wat wil je nu precies weten?
Wat is het doel van je script?

[4Real]

PHP:

if ( isset($id) && int_val($id) )
{
     // goed (Y)
}

Als die niet geset is stopt die er al mee en als die wel gezet is maar geen int is doet hij ook niks

[NMe]

Klopt niet, 4Real. int_val() maakt er een integer van. Deze lijkt me wel voldoende?

PHP:

if (!isset($id) || !is_int($id) || $id == '') {
  //ongeldig
} else {
  //geldig
}

Empty kun je niet gebruiken omdat die bij 0 true returnt. Waarom PHP bij is_numeric en is_int true geeft als je er een lege string in stopt is me niet duidelijk.

[ Voor 54% gewijzigd door NMe op 12-03-2006 18:36 ]

[Verwijderd]

-NMe- schreef op zondag 12 maart 2006 @ 18:34:
Waarom PHP bij is_numeric en is_int true geeft als je er een lege string in stopt is me niet duidelijk.

Is volgens mij ook niet waar?

PHP:

$test = (string) "";
echo is_numeric($test); // false

[ Voor 17% gewijzigd door Verwijderd op 12-03-2006 18:53 ]

[XWB]

Waarom PHP bij is_numeric en is_int true geeft als je er een lege string in stopt is me niet duidelijk.

Ik krijg gewoon een false terug

[NMe]

Wat hij bij mij zegt weet ik niet, want ik heb het niet geprobeerd. Ik lees wel het volgende in de topicstart:

verytallman schreef op zondag 12 maart 2006 @ 17:30:
Als je hiernaar kijkt klopt php ook niet, want: bij die eerste if doe ik een check is_numeric(), maar waarom geeft php daarop wel true als hij eigenlijk vind dat $id geen waarde is?

Maar als het waar is wat jullie zeggen, dan heeft TS ergens iets fout gedaan.

[XWB]

Ik denk dat TS bedoelt dat $id = 0 true geeft, en dat niet kan omdat 0 geen waarde is (volgens mij gaat TS er dus vanuit dat met $id = 0, $id leeg is). Maar 0 is niet hetzelfde als NULL

[ Voor 32% gewijzigd door XWB op 12-03-2006 19:30 ]

[verytallman]

Die eerdere redenering van mij klopt idd niet...heb me vergist.

Maar ik weet wel dat checken op niet niet feilloos gaat, vooral die === laat het afweten.

Ik heb er nu maar iets anders van gemaakt.

[XWB]

Maar ik weet wel dat checken op niet niet feilloos gaat, vooral die === laat het afweten.

Hoezo? Wat lukt je niet dan? Met === vergelijk je types.

PHP:

<?php
$int = 1;
$string = 1; 
if ($int === $string)
{
    echo 'waar';
}
else
{
    echo 'onwaar';
}
?>

Bovenstaande krijg je true terug, want beide zijn integers.

PHP:

<?php
$int = 1;
$string = '1';
if ($int === $string)
{
    echo 'waar';
}
else
{
    echo 'onwaar';
}
?>

En bij deze krijg je false terug, want je hebt een integer en een string.

[ Voor 42% gewijzigd door XWB op 12-03-2006 20:01 ]

[NMe]

Weet je wel wat === (de identity operator) doet dan? Die heeft het bij mij namelijk nog nooit laten afweten bij goed gebruik.

[verytallman]

De === werkte bij mij toch echt niet. Ik heb daar mogelijk wel een verklaring voor.
De $id verkrijg is door:
$id = $_GET['id'];

Dit is geen int maar een string , daarom werkt de === niet. Dat is mijn enige verklaring.

Met intval() kan je dan wel van die $_GET['id'] een int maken, maar intval geeft een 0 terug bij een lege variabele. En dat wou ik niet want dan kan ik niet meer die gevallen eruit halen waarbij $id geen waarde had.

[XWB]

De === werkte bij mij toch echt niet. Ik heb daar mogelijk wel een verklaring voor.
De $id verkrijg is door:
$id = $_GET['id'];

Dit is geen int maar een string , daarom werkt de === niet. Dat is mijn enige verklaring.

Klopt, maar dan maak je er toch een integer van? $id = (int)$_GET['id'];

[steffex]

hier krijg ik waarde terug, dus dit werkt prima:

code:

<?php

$id = '0';

if( (ctype_digit($id)) AND (is_numeric($id)) AND (isset($id)) ) {

    echo 'waarde';

}


else {

    echo 'geen waarde';

}

?>

[NMe]

Hacku schreef op zondag 12 maart 2006 @ 20:00:
Klopt, maar dan maak je er toch een integer van? $id = (int)$_GET['id'];

Mjah, maar daarna heeft een check om te kijken òf het een integer was ook geen zin meer natuurlijk.

[Verwijderd]

Als $id uit $_GET komt, kan je zoiets doen:

PHP:

if(array_key_exists('id', $_GET)) {
    $id = int_val($_GET['id']);
} else {
    $id = null;
}

Dan kan je kijken of $id === 0 (is 0) of $id === null (bestaat niet).

[T-MOB]

Of je maakt een eigen variant op empty() die 0 niet als leeg beschouwd

PHP:

function strEmpty(&$in) {
    return (empty($in) && $in != '0');
}

[Verwijderd]

Weet niet of het met deze kwestie te maken heeft, maar ter informatie: een variabele verkregen via GET of POST is nooit en te nimmer een integer en wordt altijd als een string behandeld, volgens mij.

_{Een klein testje bevestigde dit zojuist.}

[ Voor 14% gewijzigd door Verwijderd op 13-03-2006 08:50 ]

[JayVee]

Hacku schreef op zondag 12 maart 2006 @ 20:00:
[...]
Klopt, maar dan maak je er toch een integer van? $id = (int)$_GET['id'];

Is nog hartstikke veilig ook als je $id wilt gebruiken in queries etc!

[Michali]

stef-o.nl schreef op zondag 12 maart 2006 @ 20:01:
hier krijg ik waarde terug, dus dit werkt prima:

code:

<?php

$id = '0';

if( (ctype_digit($id)) AND (is_numeric($id)) AND (isset($id)) ) {

    echo 'waarde';

}


else {

    echo 'geen waarde';

}

?>

Het lijkt me beter om als eerst te controleren of hij wel bestaat, met isset, ipv. op het laatst. Dat ivm. short circuit evaluation. In dit geval krijg je een error (of notice) als hij niet bestaat. Bovendien is zo veel wit ruimte en al die haakjes ook niet echt duidelijk:

PHP:

if ( isset($id) && ctype_digit($id) && is_numeric($id) ) {
    echo 'waarde';
}
else {
    echo 'geen waarde';
}