[php]if constructie werkt niet naar behoren

donderdag 4 augustus 2011 12:26

Acties:

0 Henk 'm!

Topicstarter

ik ben bezig met een beginnerscursus php en nu heb ik een oefening gedaan waarbij er wordt gekeken of een formuliertje is ingevuld. Dit ging goed totdat de php code (van de controle) en de html code(van het formuliertje) samengevoegd werden. De bedoeling is dat het formulier moet worden getoond als er geen gegevens worden meegestuurd en als er wel gegevens zijn (m.b.v. de post methode) dan moet de controle worden uitgevoerd. Het probleem is dat het formulier nooit wordt getoond. Dit is mijn code:

PHP:

<html>
<head>
<title>formuliertje</title>
</head>
<body>
<?php
if (isset($_POST)){
    if (empty($_POST['naam'])) {
    $fout['naam']= TRUE;
    }
    if (empty($_POST['email'])){
    $fout['email'] = TRUE;
    }
    if (isset($fout)) {
    echo 'een van de velden is niet goed ingevuld';
    }
    else {
    echo 'alles is goed ingevuld!';
    }
}
else {
?>
<form action="formuliertje.php" method="post">
Naam: <input type="text" name="naam" /><br />
Email: <input type="text" name="email" /><br />
<input type="submit" value="Verzenden" />
</form>
<?php
}
?> 
</body>
</html>

ik heb al op google gezocht maar kon niet echt een vergelijkbaar voorbeeld vinden.
kan iemand mij vertellen wat hier fout gaat? alvast hartelijk bedankt.

donderdag 4 augustus 2011 12:29

Acties:

0 Henk 'm!

Xesxen

doe eens op regel 7: if(isset($_POST['naam']) || isset($_POST['email'])) {

Rare vogel in spe

donderdag 4 augustus 2011 12:30

Acties:

0 Henk 'm!

djexplo

williewonka03 schreef op donderdag 04 augustus 2011 @ 12:26:

PHP:

<html>
<head>
<title>formuliertje</title>
</head>
<body>
<?php
if(isset($_POST['naam']) || isset($_POST['email'])) {
    if (empty($_POST['naam'])) {
           $fout['naam']= TRUE;
    }
    if (empty($_POST['email'])){
           $fout['email'] = TRUE;
    }
    if (isset($fout)) {
           echo 'een van de velden is niet goed ingevuld';
    }
    else {
           echo 'alles is goed ingevuld!';
    }
}
else {
       echo '<form action="formuliertje.php" method="post">';
       echo 'Naam: <input type="text" name="naam" /><br />';
       echo 'Email: <input type="text" name="email" /><br />';
       echo '<input type="submit" value="Verzenden" />';
       echo '</form>';
}
?> 
</body>
</html>

ik heb al op google gezocht maar kon niet echt een vergelijkbaar voorbeeld vinden.
kan iemand mij vertellen wat hier fout gaat? alvast hartelijk bedankt.

'if it looks like a duck, walks like a duck and quacks like a duck it's probably a duck'

donderdag 4 augustus 2011 12:32

Acties:

0 Henk 'm!

williewonka03

Topicstarter

Xesxen schreef op donderdag 04 augustus 2011 @ 12:29:
doe eens op regel 7: if(isset($_POST['naam']) || isset($_POST['email'])) {

hartelijk bedankt! dat werkte. dit staat niet in de cursus, foutje in de cursus dus.

djexplo schreef op donderdag 04 augustus 2011 @ 12:30:
[...]

pardon maar wat is de waarde hiervan?

[ Voor 22% gewijzigd door williewonka03 op 04-08-2011 12:33 ]

donderdag 4 augustus 2011 12:35

Acties:

0 Henk 'm!

djexplo

Dit is een ranzige constructie ...

code:

else {
?>
Blaat
<?php
}

Als ik jou was zou ik ook PHP error reporting aan zetten, zodat je zelf ziet wat je fout doet ...

'if it looks like a duck, walks like a duck and quacks like a duck it's probably a duck'

donderdag 4 augustus 2011 12:38

Acties:

0 Henk 'm!

williewonka03

Topicstarter

djexplo schreef op donderdag 04 augustus 2011 @ 12:35:
Dit is een ranzige constructie ...
code:
1
2
3
4
5
else {
?>
Blaat
<?php
}
Als ik jou was zou ik ook PHP error reporting aan zetten, zodat je zelf ziet wat je fout doet ...

uhm hoe zet je dat aan in notepad ++?

donderdag 4 augustus 2011 12:40

Acties:

0 Henk 'm!

djexplo

http://php.net/manual/en/function.error-reporting.php

'if it looks like a duck, walks like a duck and quacks like a duck it's probably a duck'

donderdag 4 augustus 2011 12:40

Acties:

0 Henk 'm!

Melodic Metal

djexplo schreef op donderdag 04 augustus 2011 @ 12:35:
Dit is een ranzige constructie ...
code:
1
2
3
4
5
else {
?>
Blaat
<?php
}
Als ik jou was zou ik ook PHP error reporting aan zetten, zodat je zelf ziet wat je fout doet ...

Lang niet zo ranzig als complete html blocks echo'en met php.

Error reporting staat in de manual.

donderdag 4 augustus 2011 12:50

Acties:

0 Henk 'm!

RobIII

Admin Devschuur®

^ Romeinse Ⅲ ja!

Php

williewonka03 schreef op donderdag 04 augustus 2011 @ 12:38:
[...]

uhm hoe zet je dat aan in notepad ++?

uhm, doe zelf maar eens wat moeite
[google="php error reporting"]

Het is nu over en uit met de helpdesktopics. Koop een goed boek, ga naar de bibliotheek of zoek een fatsoenlijke online tutorial; ik weet niet welke tutorial je nu aan 't volgen bent maar die is waardeloos of je leest 'm maar half. You decide.

[ Voor 26% gewijzigd door RobIII op 04-08-2011 12:53 ]

There are only two hard problems in distributed systems: 2. Exactly-once delivery 1. Guaranteed order of messages 2. Exactly-once delivery.

Je eigen tweaker.me redirect

Over mij

donderdag 4 augustus 2011 14:13

Acties:

0 Henk 'm!

NMe

Quia Ego Sic Dico.

Php

Even door het slotje heen posten om de bad practices af te schieten.

Xesxen schreef op donderdag 04 augustus 2011 @ 12:29:
doe eens op regel 7: if(isset($_POST['naam']) || isset($_POST['email'])) {

Liever doe je if ($_SERVER['REQUEST_METHOD'] == 'POST') om te kijken of het een postback betreft, en vervolgens met isset per postvariabele kijken of de waarde wel of niet goed ingevuld is om zo zinnige foutmeldingen te kunnen geven.

djexplo schreef op donderdag 04 augustus 2011 @ 12:35:
Dit is een ranzige constructie ...

En dát is jouw persoonlijke mening, geen waarheid. Ik vind zoals hierboven al gezegd wordt tientallen echo's veel ranziger. En ook dát is een persoonlijke mening, geen waarheid.

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.

Onderwerpen