fout met variabelen

Door Peter op 10-08-2006 15:09

293 views

Bij onderstaande code krijg ik vele notices
Ik heb zelf een aantal variabelen toegevoegd waar ik mee wil werken.

Notice: Undefined variable: gevonden
Notice: Undefined variable: actief
Notice: Undefined variable: gevonden
Notice: Undefined variable: waarschijnlijk
Notice: Undefined variable: nietgevonden
Notice: Undefined variable: niet
Notice: Undefined variable: niet2

Welke stap moet ik maken om dit op te lossen?

<?
$query = "SELECT regio FROM `regio` WHERE `postcode` = '$pcijfers' ";
//query is aangemaakt
$sql = mysql_query($query);
//query is uitgevoerd
while($record = mysql_fetch_object($sql)){
$gevonden="".$record->regio."";
echo"".$gevonden."<br>";
}
$query2 = "SELECT actief FROM `overzicht` WHERE `regionaam` = '$gevonden' ";
//query is aangemaakt
$sql2 = mysql_query($query2);
//query is uitgevoerd
while($record2 = mysql_fetch_object($sql2)){
$gevonden2="".$record2->actief."";
echo"".$gevonden2."";
if ($gevonden2 != "Nee" && $gevonden2 != "Ja") {
$nietgevonden = "Wij konden uw postcode niet vinden in ons bestand.";
$niet = "";
echo"".$nietgevonden."";}
else if ($gevonden2 == "Nee") {
$actief = "nog niet actief in de regio";
$waarschijnlijk = " <br>
maar gaat binnenkort van start.";
$niet = "<strong>dus nog niet </strong>";}
else if ($gevonden2 == "Ja") {
$actief = " actief in de regio";
$waarschijnlijk = ".";
$niet2 = "";}
}
echo '
<table width="100%" height="300" bgcolor="#CCCCCC" >
<tr>
<th scope="col"><h2> HARTELIJK DANK VOOR UW AANVRAAG.</h2>
<br>
'.$actief.' '.$gevonden.''.$waarschijnlijk.''.$nietgevonden.'
<p>De informatie wordt '.$niet.''.$niet2.'toegezonden aan:</p>
<br>
<table width="0%" border="1" align="center" cellpadding="3" cellspacing="3" bordercolor="#FFFFFF" >
<tr>
<th align="left" valign="top" scope="row">'. strtoupper($_POST['naam']) .'</th>
</tr>
<tr>
<th align="left" valign="top" scope="row">'. strtoupper($_POST["straat"]) .' '. $_POST["nummer"] . $_POST["toevoeg"] .' </th>
</tr>
<tr>
<th align="left" valign="top" scope="row">'. strtoupper($_POST["pcijfers"]) .' '. strtoupper($_POST["postletters"]) .' '. strtoupper($_POST["plaats"]) .'</th>
</tr>
</table>
<p> </p></th>
</tr>
</table> ' ;
{
}}
?>

DeleteAll DeletelAll

10-08-2006 15:13

Ik heb je script niet gelezen, maar lijkt mij gewoon dat je dingen opvraagd die er niet zijn...

Edit:
Begin dus gewoon met alles es te bekijken ;-)!
Om te zien of je nx bent vergeten.

PHP erik

10-08-2006 15:15

Je gebruikt inderdaad allemaal variabelen die niet gedefinieerd zijn. Dus je moet bovenaan je script even checken of ze bestaan ( isset() ) en eventueel een (lege) waarde er aan koppelen.

Bo az

10-08-2006 15:15

Wat mij meteen opvalt is dat je code niet duidelijk is, je springt nauwelijks in bij je PHP.

$gevonden="".$record->regio."";
kan ook zo:
$gevonden = $record->regio;

wordt '.$niet.''.$niet2.'toegezonden
kan ook zo:
wordt '.$niet.$niet2.'toegezonden

Als je code duidelijk is, zal je zien dat het debuggen ook een stuk eenvoudiger is.

Frank -

10-08-2006 15:15

Eerst met isset() controleren of een variabele wel bestaat.
<?php
if(isset($var)){ // if TRUE, doe iets
echo $var; // in dit geval een echo
}
?>
Dan nog een tip: Haal die gore backtics ` uit je queries, die zorgen alleen maar voor vervuiling en problemen. Nu kun je nog meer tikfouten maken en verboden (gereserveerde) namen gebruiken. Dat wil je echt niet!

Peter

10-08-2006 15:45

hoe moet ik in mijn code dan de isset() gebruiken?
bijvoorbeeld in dit stukje code

<?
while($record2 = mysql_fetch_object($sql2)){
$gevonden2="".$record2->actief."";
echo"".$gevonden2."";
if ($gevonden2 != "Nee" && $gevonden2 != "Ja") {
$nietgevonden = "Wij konden uw postcode niet vinden in ons bestand.";
$niet = "";
echo"".$nietgevonden."";}
?>

Frank -

10-08-2006 15:48

Je maakt nu de volgende query aan:
<?php
$query = "SELECT regio FROM `regio` WHERE `postcode` = '$pcijfers' ";
?>
Hierin staat de variabele $pcijfers. Voordat je de query aanmaakt, moet je dus eerst controleren of deze variabele wel bestaat.
<?php
if(isset($pcijfers)){
$query = "SELECT regio FROM regio WHERE postcode = '$pcijfers' "; // zonder backtics!!!
// uitvoeren van de query en nog een berg code
}
// nog meer code
?>
Het is een beginnetje, de rest laat ik aan jouw fantasie over.

Peter

10-08-2006 16:06

kom er niet uit op deze manier.
Geef eens een voorzetje voor de onderstaande code....

<?
while($record2 = mysql_fetch_object($sql2)){
$gevonden2="".$record2->actief."";
echo"".$gevonden2."";
if ($gevonden2 != "Nee" && $gevonden2 != "Ja") {
$nietgevonden = "Wij konden uw postcode niet vinden in ons bestand.";
$niet = "";
echo"".$nietgevonden."";}
?>

Frank -

10-08-2006 16:32

Die code ziet er niet lekker uit... Het aanmaken van extra variabelen is nergens voor nodig.

Heb de boel even herschreven en snap er weinig van:
<?
while($row = mysql_fetch_assoc($sql2)){ // assoc is sneller
echo $row['actief']; // geen extra variabelen aanmaken

if ($row['actief'] != 'Nee' && $row['actief'] != 'Ja') { // controleer dit in de query!
$nietgevonden = 'Wij konden uw postcode niet vinden in ons bestand.';
$niet = '';
echo $nietgevonden;
}

}
?>
Waarom doe je de vergelijking niet in de query? En waarom maak je de variabele $nietgevonden aan om hem vervolgens direct te echoen? En die "". is natuurlijk overbodig.

Wat ik echter volledig mis, is waar jij isset() zou willen toepassen. Ik zie niet waar dat hier nodig zou zijn.

PHP Newbie

10-08-2006 16:37

"". kun je gewoon weg laten, dat is nergens voor nodig

Wat ik vaak doe is:

$_GET['bla'] = isset($_GET['bla']) ? $_GET['bla'] : NULL;

Peter

11-08-2006 14:59

bedankt.

Reageren

Inloggen om te reageren