[php] Is Mail Correct...

[jenne]

Ik heb het volgende:

PHP:

<?
require("info.php");

if (
$bestemming             <> "Selecteer" and 
$soorttoegangskaart         <> "" and 
$aantalkaarten             <> "" and 
$toegangskaartenvoordag     <> "" and 
$vertrekdat             <> "" and 
$aandagreis             <> "Selecteer" and 
$aanpers             <> "Selecteer" and 
$vervoer             <> "Selecteer" and 
$opstapplaats             <> "n.v.t." and 
$anverz             <> "Selecteer" and 
$reisverz             <> "Selecteer" and 
$hoofdmanvrouw             <> "Selecteer" and 
$hoofdnaam             <> "" and 
$hoofdgeb             <> "" and 
$hoofdadres             <> "" and 
$hoofdpost             <> "" and 
$hoofdwoon             <> "" and 
$hoofdtelprive             <> "") {
echo "U heeft de volgende verplichte velden niet correct ingevuld:<br>";

if ($bestemming == '') {
echo "Grand Prix Bestemming.<br>";
}
// Hier komen de rest van de verplichte velden dan ook nog

} else {
mail("[e-mail]", "$subject", "
Grand Prix Bestemming: $bestemming
Soort Toegangskaart: $soorttoegangskaart
Aantal kaarten: $aantalkaarten

Toegangs kaarten voor dag: $toegangskaartenvoordag

Ken jullie van: $kenvan
Anders namelijk: $andernl

Vertrekdatum: $vertrekdat
Aantel Daagse Reis: $aandagreis
Aantal Personen: $aanpers
Vervoer: $vervoer
Opstapplaats: $opstapplaats
Annulerings Verzekering: $anverz
Reisverzekering: $reisverz
Opmerkingen: $opmer

Hoofdboeker:
Naam: $hoofdmanvrouw $hoofdnaam
Geboorte Datum: $hoofdgeb
Adres: $hoofdadres
$hoofdpost $hoofdwoon
Telefoonnummer Prive: $hoofdtelprive
Telefoonnummer Werk: $hoofdtelwerk
E-Mail Adres: $hoofdemail", "From: $hoofdemail");
}
?>

Het is de bedoeling dat hij controleert of de velden zijn ingevuld, en zo niet neerzet welke velden niet, en als ze wel allemaal zijn ingevoerd dat hij een mailtje stuurt, maar hij doet helemaal niks, hij schrijft steeds Hoofdboeker op.

om te kijken wat hij doet surf naar http://www.rtlgrandprixreizen.nl/html/boek.html

[Nielsz]

ik zou om elke 'case' eerst eens haakjes zetten. dus

code:

if (
($a=="b") and 
($b=="a")
)

[chem]

waarom zo'n belachelijk if-set?

dit lijkt me mooier:

PHP:

<?
if(notempty($var1, $var2, $var3, ...) &amp;amp;&amp;amp; notValue('Selecteer', $var4, $var5, $var5)) {
    ..
}
?>

waarbij:

PHP:

<?
function notEmpty()
{
    $numargs = func_num_args();
    $arg_list = func_get_args();
    for ($i = 0; $i < $numargs; $i++) {
        if($arg_list[$i] == '') 
            return false;
    }
    return true;
}
?>

en

PHP:

<?
function notValue($value)
{
    $numargs = func_num_args();
    $arg_list = func_get_args();
    for ($i = 1; $i < $numargs; $i++) {
        if($arg_list[$i] == $value)
        return false;
    }
    return true;
}
?>

[drm]

allereerst zou ik je willen aanraden jezelf de C-style aan te wennen.

dus ipv:

code:

if ( $a <> $b and $c <> $d ) 
   ;

zo:

code:

if ( $a != $b && $c != $d )
   ;

zie verder de manual wat gebruikelijk is
waarom, vraag je je misschien af? Gewoon, omdat het de meest gangbare manier is, en 99,9% van de PHP-ers het gewend is.

Verder is het beter om per veld na te gaan of het correct ingevuld is:

PHP:

<?
if ( $bestemming != "Selecteer" )
   $missing [] = "Bestemming";
if ( /* etcetera */ )
   ;
// ...

if ( count ( $missing ) > 0 )
   foreach ( $missing as $field )
      echo "Het veld $field is niet (correct) ingevuld<br>";
?>

Dan is namelijk ook beter na te gaan waar de fout precies zit.

Kom je er dan nog niet uit, zet dan even bovenaan je mailscript:

PHP:

<?
echo "<pre>", var_dump ( $HTTP_POST_VARS ), "</pre>";
// of als je method="get" gebruikt in je form:
echo "<pre>", var_dump ( $HTTP_GET_VARS ), "</pre>";
?>

Dan zie je of de variabelen uberhaupt wel binnenkomen in je mailscript

hth

[drm]

Nielsz:
ik zou om elke 'case' eerst eens haakjes zetten. dus

code:

if (
($a="b") and 
($b="a")
)

Warning: condition is always true at line 2 in Nielsz.php
Warning: condition is always true at line 3 in Nielsz.php

[Nielsz]

Op maandag 21 januari 2002 10:37 schreef drm het volgende:

[..]

Warning: condition is always true at line 2 in Nielsz.php
Warning: condition is always true at line 3 in Nielsz.php

Auw!

[chem]

Op maandag 21 januari 2002 10:36 schreef drm het volgende:Verder is het beter om per veld na te gaan of het correct ingevuld is:
[..]
Dan is namelijk ook beter na te gaan waar de fout precies zit.

doe het dan zo:

PHP:

<?
$required = array('bestemming','aankomst');

foreach($required as $test)
{
    if($$test != "Selecteer" )
        $missing[] = $test;
}
// ...

if ( count ( $missing ) > 0 )
   foreach ( $missing as $field )
      echo "Het veld $field is niet (correct) ingevuld<br>";
?>

[jenne]

Op maandag 21 januari 2002 10:38 schreef Nielsz het volgende:

[..]

Auw!

ja want bv a=1 en b=2 en je dan zecht dan a=b is a 2 dan b=a 2, wan tje hebt net gezecht dat a 2 is, dat ken ik ook nog wel

[Nielsz]

Op maandag 21 januari 2002 10:45 schreef Decapitator het volgende:

[..]

ja want bv a=1 en b=2 en je dan zecht dan a=b is a 2 dan b=a 2, wan tje hebt net gezecht dat a 2 is, dat ken ik ook nog wel

Grapjas, dat snap ik ook wel hoor

[jenne]

Op maandag 21 januari 2002 10:47 schreef Nielsz het volgende:

[..]

Grapjas, dat snap ik ook wel hoor

Na waarom zeg je het dan, beetje mij op het verkeerde spoor te zetten

[jenne]

Uhmm Chem hij doet het niet...

[drm]

chem:
doe het dan zo:

PHP:

<?
/* meuk */
?>

Tja, * drm doet 't meestal met een MySQL tabelletje.

maar zonder sql zou ik het zo doen:

PHP:

<?
$fields = array ( // Veldnaam => defaultwaarde
   "bestemming" => "Selecteer",
   "soorttoegangskaart" => ""
   // etc
);

foreach ( $fields as $name => $default )
   if ( strcasecmp ( $HTTP_POST_VARS [ $name ], $default ) == 0 )
      $missing [] = $name;
// etc
?>

ben niet zo weg van variabele variabeles...

[chem]

Op maandag 21 januari 2002 10:51 schreef Decapitator het volgende:
Uhmm Chem hij doet het niet...

oh.

[chem]

Op maandag 21 januari 2002 10:59 schreef drm het volgende:

[..]

Tja, * drm doet 't meestal met een MySQL tabelletje.

maar zonder sql zou ik het zo doen:

PHP:

<?
$fields = array ( // Veldnaam => defaultwaarde
   "bestemming" => "Selecteer",
   "soorttoegangskaart" => ""
   // etc
);

foreach ( $fields as $name => $default )
   if ( strcasecmp ( $HTTP_POST_VARS [ $name ], $default ) == 0 )
      $missing [] = $name;
// etc
?>

ben niet zo weg van variabele variabeles...

htf wil je dit doen met mysql

maar jouw oplossing mag ook hoor

Zelf gebruik ik eigenlijk nooit variabele variabelen, behalve bij tijdelijke scriptjes om tijd te besparen (bv. ff een conversiedingetje maken).

Omdat ik nl. niet dit 'gewend' wil raken en me met Obj-C wil gaan bezighouden

[drm]

chem:
htf wil je dit doen met mysql

• fields-tabelletje met:

• fieldID (autoincrement tinyint)
• name (varchar(15))
• description (varchar (40))
• type (enum: 'text','textarea','select',etc...)<li> default

</li>

• fieldValues tabbeletje

• fieldValueID (autoincrement tinyint)<li> value (varchar (30))

</li>

• koppeltabelletje fieldValues met fields

• fieldValueID <li> fieldID

</li>

die laatste 2 natuurlijk alleen interessant voor multiple-velden zoals select, checkboxes en radiobuttons

Snappie? Dan kun je ook mooi je formulier op basis van de tabellen genereren, en hoef je niet het hele form te kloppen

[jenne]

Dit && is het teken van 'and' maar wat was die van 'or' nou dan, want ik bedacht me opeens dat hij al moet stoppen als een veld niet is ingevuld en niet als ze allemaal niet zijn ingevuld.

Dus het moet iniedergeval zijn 'or' en niet 'and'

[Nielsz]

Op maandag 21 januari 2002 11:19 schreef Decapitator het volgende:
Dit && is het teken van 'and' maar wat was die van 'or' nou dan, want ik bedacht me opeens dat hij al moet stoppen als een veld niet is ingevuld en niet als ze allemaal niet zijn ingevuld.

Dus het moet iniedergeval zijn 'or' en niet 'and'

AND == &&
OR == ||

[drm]

Op maandag 21 januari 2002 11:19 schreef Decapitator het volgende:
Dit && is het teken van 'and' maar wat was die van 'or' nou dan, want ik bedacht me opeens dat hij al moet stoppen als een veld niet is ingevuld en niet als ze allemaal niet zijn ingevuld.

Dus het moet iniedergeval zijn 'or' en niet 'and'

hehehe, jah, da's ook een goed punt dat probleem heb je dus niet als je alle velden afzonderlijk checkt ipv alles in 1 if

Zie ook de manual / operators / Logical operators

all you need is there

[jenne]

Op maandag 21 januari 2002 11:22 schreef Nielsz het volgende:

[..]

AND == &&
OR == ||

Thnx hij doet het

[wasigh]

je mag wel iets meer moeite doen om zelf dingen te zoeken!