[php/mysql] $blaat="" --> error

[EL_Loco]

ik heb dit scripje van een kleine zoekmachine:

PHP:

if ($soort == "Geen voorkeur")
        {
                //nix te doen
                $q_soort = "";
        }
        if ($soort == "Locomotief")
        {
                //loc
                $q_soort = "AND
                  catalogus.type_id = 1";
        }
        if ($soort == "Wagon")
        {
                //wagon
                $q_soort = "AND
                  catalogus.type_id = 2";
        }
        if ($soort == "Overig")
        {
                //overig
                $q_soort = "AND
                  catalogus.type_id = 3";
        }

de query

PHP:

$query = "
                  select * from catalogus, type_catalogus, merk_catalogus
                  WHERE
                  catalogus.merk_id=merk_catalogus.merk_id
                  AND
                  catalogus.type_id=type_catalogus.type_id
                  $q_art_nr
                  $q_int_expr
                  $q_soort
                  ";
        $result = mysql_query ($query)
        or die (mysql_error());

alles werkt, behalve dat wanneer mensen "Geen voorkeur" kiezen ze meteen deze error krijgen:

Notice: Undefined variable: q_soort in c:\program files\apache group\apache\htdocs\peter\final_1\pages800\zoek_adv.php on line 93

en dat is niet netjes.
php ziet de variable niet. Nu is mijn vraag, is er een teken waar mysql niet op let, maar waarbij php wel gewoon de $q_soort onthoudt
dit is natuurlijk wel een beetje vals spelen, als iemand een andere oplossing heeft graag

[Nielsz]

Spatie
En doe gewoon eens een switch($soort);

[EL_Loco]

met spatie ziet php hem ook niet
waarom moet ik switch gebruiken is dat sneller

[Grum]

www.php.net/isset www.php.net/switch www.php.net/array

[creative8500]

Haco schreef op 30 november 2002 @ 16:13:
met spatie ziet php hem ook niet
waarom moet ik switch gebruiken is dat sneller

Switch is een speciale soort if-elseif-else syntax voor als je alleen de waarde van een-en-dezelfde variabele wilt evalueren.

PHP:

switch($soort)
{
    default:
/* deze default-actie is heel belangrijk: zo krijg je, wanneer
er geen if-statement true geeft, toch een gedefinieerde variabele 'q_soort' */
    case "Geen voorkeur":
        $q_soort = "";
    break;

    case "Locomotief":
        $q_soort = "AND calogus.type_id = 1";
    break;

    case "Wagon":
        $q_soort = "AND catalogus.type_id = 2";
    break;

    case "Overig":
        $q_soort = "AND catalogus.type_id = 3";
    break;
}

$query = "
SELECT
    *
FROM
    catalogus,
    type_catalogus,
    merk_catalogus
WHERE
    catalogus.merk_id = merk_catalogus.merk_id
    AND
    catalogus.type_id = type_catalogus.type_id
".$q_art_nr.$q_int_expr.$q_soort;
$result = mysql_query($query) or die (mysql_error());

Voor meer informatie: PHP: switch - Manual

_{dankje, creative8500}

edit: Het beste zou natuurlijk zijn: i.p.v. een else of een default gebruiken: vooraan je script zetten:

PHP:

$q_soort = "";

[ Voor 45% gewijzigd door creative8500 op 01-12-2002 12:09 ]

[EL_Loco]

dank voor de uitleg , kwas net zelf op php.net en heb het switch onderdeel gelezen, maar dat lost nog niet m'n probleem op met mysql
-edit- met array en isset kom ik geloof ik ook niet verder,
_{tenzij ik iets over het hoofd gezien heb}

[ Voor 32% gewijzigd door EL_Loco op 30-11-2002 16:27 . Reden: edit ]

[Grum]

Onder het motto: TIMTOWTDI

PHP:

$theKinds = array( 'Locomotief', 'Wagon', 'Overig' );

if( !isset( $theKind ) || !in_array( $theKind, $theKinds ) || $theKind == 'Geen voorkeur' )
    $theString = '';
else
{
    $theTemp = array_flip( $theKinds );
    $theString = 'AND catalogus.type_id = '. $theTemp[ $theKind ] + 1;
}

[ Voor 64% gewijzigd door Grum op 30-11-2002 16:33 ]

[creative8500]

Haco schreef op 30 November 2002 @ 16:25:
dank voor de uitleg , kwas net zelf op php.net en heb het switch onderdeel gelezen, maar dat lost nog niet m'n probleem op met mysql
-edit- met array en isset kom ik geloof ik ook niet verder,
_{tenzij ik iets over het hoofd gezien heb}

Zie vorige reactie, vooral bij default. Om voortaan zelf te kijken waar het probleem ligt: zet bij elke if-statement een echo "nummer x", zo kun je zien welke if-statement true teruggeeft, en als je niks op je scherm krijgt: dan is er helemaal geen statement true.
De wijze les voor vandaag: maak _altijd_ gebruik van else { }

[thomaske]

zet eens bovenaan je script

PHP:

echo $soort;

Dat heet DEBUGGEN

[ Voor 50% gewijzigd door thomaske op 30-11-2002 16:34 ]

[creative8500]

thomaske schreef op 30 November 2002 @ 16:32:
zet eens bovenaan je script

PHP:

echo $soort;

Als hij het HTML-formpje verkeerd heeft gemaakt dan is het natuurlijk wel heel triest

[Apache]

Het is trouwens een notice, zelfs geen warning of error, die default zou het moeten oplossen, anders kan je hem initialiseren met een lege waarde.

Als je perse wil kan je ook het error reporting level in je php.ini iets lager zetten.

[EL_Loco]

dank allen, hier was ik al een paar uur op aan het puzzelen

[EL_Loco]

Apache schreef op 30 november 2002 @ 16:36:
Het is trouwens een notice, zelfs geen warning of error, die default zou het moeten oplossen, anders kan je hem initialiseren met een lege waarde.

Als je perse wil kan je ook het error reporting level in je php.ini iets lager zetten.

daar heb je gelijk in, maar als ik iets doe dan wil ik het foutloos doen , dus zonder error, warnigs, notice (wat is het meervoud hiervan )

[ Voor 1% gewijzigd door EL_Loco op 30-11-2002 16:38 . Reden: de = het ]

[thomaske]

creative8500 schreef op 30 November 2002 @ 16:33:
[...]

Als hij het HTML-formpje verkeerd heeft gemaakt dan is het natuurlijk wel heel triest

Haco schreef op 30 November 2002 @ 16:36:
dank allen, hier was ik al een paar uur op aan het puzzelen

[EL_Loco]

creative8500 schreef op 30 november 2002 @ 16:33:
[...]

Als hij het HTML-formpje verkeerd heeft gemaakt dan is het natuurlijk wel heel triest

m'n formpje was goed

[Apache]

Haco schreef op 30 November 2002 @ 16:37:
[...]

daar heb je gelijk in, maar als ik iets doe dan wil ik het foutloos doen , dus zonder error, warnigs, notice (wat is het meervoud hiervan )

true, ik code ook enkel php met error_reporting(E_ALL); bovenaan maar als iemand hier op GoT iets post heb je er geen idee van of hij net bezig is of niet, als beginner valt die niet aan te raden (die geven meestal ook geen shit om notices (wat volgens mij ook meteen het meervoud is ))