Door
Frits van Leeuwen
op 24-08-2017 21:31
gewijzigd op 24-08-2017 23:15
6.311 views
Hallo allemaal,
Dit is merkwaardig.
Ik heb een case die reageert op 0 records, 1 record en meer dan1 record.
Als er geen record is, en dat is ook zo, toont hij een form.(administratie0.php)
Die verlaat ik na het invullen door op opslaan te klikken en dan wordt een query uitgevoerd:
Dit werkt, want daarna zie ik in de tabel 1 record staan.
Maar aansluitend wil ik de informatie in een ander form weer tonen. Maar die blijkt dan leeg te zijn, terwijl ik toch echt een gevuld record in de tabel zie staan. Met F5 krijg ik het wel in beeld.
Als ik het hele programma opnieuw start, kom ik weer door de case, maar dit keer ziet hij dat er 1 record is en toont hij die met het zelfde form als dat ik eerder gebruikte nadat ik een record had toegevoegd. Maar dit keer zie ik het record wel in het form. Hier volgt de code om het ingevulde record te tonen.
administratie1.php
?php
$query01 = "SELECT * FROM tabel_administraties";
$stmt = $dbh->prepare($query01);
$stmt->execute();
//PDO::FETCH_ASSOC: returns an array indexed by column name as returned in your result set
$record = $stmt->fetch(PDO::FETCH_ASSOC);
echo "<table>";
echo "<tr><td>Administratie van: </td><td>" . $record['administraties_naam'] . "</TD></TR>";
echo "<tr><td>adres is: </td><td>" . $record['administraties_straat'] . " " . $record['administraties_huisnr'] . "</TD></TR>";
echo "<tr><td></td><td>" . $record['administraties_postcode'] . " " . $record['administraties_plaats'] . "</TD></TR>";
echo "<tr><td>Telefoonnr.: </td><td>" . $record['administraties_telefoonnr'] . "</TD></TR>";
echo "<tr><td>K.v.K. nr.: </td><td>" . $record['administraties_kvk'] . "</TD></TR>";
echo "<tr><td>B.T.W. nr.: </td><td>" . $record['administraties_btwnr'] . "</TD></TR>";
echo "<tr><td>e-mail adres: </td><td>" . $record['administraties_email'] . "</TD></TR>";
echo "</table>";
?>
Ik denk dat ik beter toevoegen en muteren kan scheiden. Want nu ik zo aan het puzzelen ben, krijg ik na op opslaan geklikt te hebben weer een leeg formulier. Hij heeft dan ook niets bewaard. Ik heb de database leeg gegooid en ben een nieuwe begonnen. Maar nu werkt ook het toevoegen weer niet.
Ik denk dat ik ergens wat inzicht mis. Dit is de code die ik nu heb. Als iemand een aanwijzing heeft waar het mis gaat verneem ik het graag.
administraties0.php
<?php
include("databaseopenen.php");
// functie die de formuliervelden valideert en foutmeldingen aanmaakt
function validate($naam, $email) {
$errors = array();
// validatieregels voor de naam
if(strlen($naam) < 1) {
$errors[] = 'U heeft geen naam ingevuld.';
}
// validatieregels voor het mailadres
if(!strlen($email)) :
$errors[] = 'U heeft geen email adres ingevuld.';
elseif(!filter_var($email, FILTER_VALIDATE_EMAIL)):
$errors[] = 'U heeft een ongeldig email adres ingevuld.';
endif;
// geef de array met foutmeldingen terug
return $errors;
}
$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$errors = array();
if (!empty($_GET["id"])) {
// indien het formulier verstuurd is
if($_SERVER['REQUEST_METHOD'] == 'POST') {
// valideer de ingevulde gegevens
$errors = validate($naam, $email);
// als er dan geen fouten zijn ga na het opslaan naar index.php
if(!count($errors)) {
$query03 = "UPDATE 'tabel_administraties'
SET 'administraties_naam' = :administraties_naam,
'administraties_straat' = :administraties_straat,
'administraties_huisnr' = :administraties_huisnr,
'administraties_postcode' = :administraties_postcode,
'administraties_plaats' = :administraties_plaats,
'administraties_telefoonnr' = :administraties_telefoonnr,
'administraties_kvk' = :administraties_kvk,
'administraties_btwnr' = :administraties_btwnr,
'administraties_website' = :administraties_website,
'administraties_email' = :administraties_email
WHERE 'administraties_id' = :administraties_id";
$stmt = $dbh->prepare($query03);
$stmt->execute();
//PDO::FETCH_ASSOC: returns an array indexed by column name as returned in your result set
$record = $stmt->fetch(PDO::FETCH_ASSOC);
//gegevens opslaan in de database
$stmt->bindParam(':administraties_id',$adm_id);
$stmt->bindParam(':administraties_naam',$_POST["adm_naam"]);
$stmt->bindParam(':administraties_straat',$_POST["adm_straat"]);
$stmt->bindParam(':administraties_huisnr',$_POST["adm_huisnummer"]);
$stmt->bindParam(':administraties_postcode',$_POST["adm_postcode"]);
$stmt->bindParam(':administraties_plaats',$_POST["adm_plaats"]);
$stmt->bindParam(':administraties_telefoonnr',$_POST["adm_telefoonnr"]);
$stmt->bindParam(':administraties_kvk',$_POST["adm_kvk"]);
$stmt->bindParam(':administraties_btwnr',$_POST["adm_btwnr"]);
$stmt->bindParam(':administraties_website',$_POST["adm_website"]);
$stmt->bindParam(':administraties_email',$_POST["adm_email"]);
$link1 = "index.php?id=" . $adm_id;
$_GET['id'] = null;
header('Location: '.$link1);
exit;
}
} else {
//Dit kan alleen uitgevoerd worden als id is ingegeven bij de url
$adm_id = intval($_GET['id']);
$query02 = "SELECT * FROM tabel_administraties WHERE administraties_id=" . $adm_id . " LIMIT 1";
$stmt = $dbh->prepare($query02);
$stmt->execute();
//PDO::FETCH_ASSOC: returns an array indexed by column name as returned in your result set
$record = $stmt->fetch(PDO::FETCH_ASSOC);
$adm_naam = $record['administraties_naam'];
$adm_straat = $record['administraties_straat'];
$adm_huisnummer = $record['administraties_huisnr'];
$adm_postcode = $record['administraties_postcode'];
$adm_plaats = $record['administraties_plaats'];
$adm_telefoonnr = $record['administraties_telefoonnr'];
$adm_kvk = $record['administraties_kvk'];
$adm_btwnr = $record['administraties_btwnr'];
$adm_website = $record['administraties_website'];
$adm_email = $record['administraties_email'];
}
} else {
// indien het formulier verstuurd is
if($_SERVER['REQUEST_METHOD'] == 'POST') {
// valideer de ingevulde gegevens
$errors = validate($naam, $email);
// als er dan geen fouten zijn ga na het opslaan naar index.php
if(!count($errors)) {
$query03 = "INSERT INTO tabel_administraties (
administraties_naam, administraties_straat, administraties_huisnr, administraties_postcode, administraties_plaats, administraties_telefoonnr, administraties_kvk, administraties_btwnr, administraties_website, administraties_email
) VALUES (
:administraties_naam, :administraties_straat, :administraties_huisnr, :administraties_postcode, :administraties_plaats, :administraties_telefoonnr, :administraties_kvk, :administraties_btwnr, :administraties_website, :administraties_email
)";
$stmt = $dbh->prepare($query03);
$stmt->execute();
//PDO::FETCH_ASSOC: returns an array indexed by column name as returned in your result set
$record = $stmt->fetch(PDO::FETCH_ASSOC);
//gegevens opslaan in de database
$stmt->bindParam(':administraties_naam',$_POST["adm_naam"]);
$stmt->bindParam(':administraties_straat',$_POST["adm_straat"]);
$stmt->bindParam(':administraties_huisnr',$_POST["adm_huisnummer"]);
$stmt->bindParam(':administraties_postcode',$_POST["adm_postcode"]);
$stmt->bindParam(':administraties_plaats',$_POST["adm_plaats"]);
$stmt->bindParam(':administraties_telefoonnr',$_POST["adm_telefoonnr"]);
$stmt->bindParam(':administraties_kvk',$_POST["adm_kvk"]);
$stmt->bindParam(':administraties_btwnr',$_POST["adm_btwnr"]);
$stmt->bindParam(':administraties_website',$_POST["adm_website"]);
$stmt->bindParam(':administraties_email',$_POST["adm_email"]);
$link1 = "index.php";
$_GET['id'] = null;
header('Location: '.$link1);
exit;
}
} else {
//id is niet meegegeven bij de url dus is het een nieuw record
$adm_id = "nieuw";
$adm_naam = "";
$adm_straat = "";
$adm_huisnummer = "";
$adm_postcode = "";
$adm_plaats = "";
$adm_telefoonnr = "";
$adm_kvk = "";
$adm_btwnr= "" ;
$adm_website = "";
$adm_email = "";
}
}
?><!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<title>administraties0</title>
<link rel="stylesheet" type="text/css" href="css/style.css">
</head>
<body>
<?php
if(count($errors)) {
echo '<ul id="errors">';
foreach($errors as $error) {
echo '<li>' . $error . '</li>';
}
echo '</ul>';
}
?>
<table class="empty">
<tr>
<td width="250" class="vak1">
</td>
<td class="programmadeel">
<form action="" method="post">
<h1><B>Administratie (<?php echo $adm_id; ?>)<B></h1>
<table>
<tr><td>
<div class="pf">Naam : </div></td><td><input type="text" id="adm_naam" name="adm_naam" tabindex="1" size="107" value="<?php echo $adm_naam; ?>">
</td></tr><tr><td>
<div class="pf">Straat + Huisnummer : </div></td><td><input type="text" id="adm_straat" name="adm_straat" tabindex="2" size="90" value="<?php echo $adm_straat; ?>">
<input type="text" id="adm_huisnummer" name="adm_huisnummer" tabindex="3" size="10" value="<?php echo $adm_huisnummer; ?>">
</td></tr><tr><td>
<div class="pf">Postcode + Plaats : </div></td><td><input type="text" id="adm_postcode" name="adm_postcode" tabindex="4" size="7" value="<?php echo $adm_postcode; ?>">
<input type="text" id="adm_Plaats" name="adm_plaats" tabindex="5" size="93" value="<?php echo $adm_plaats; ?>">
</td></tr><tr><td>
<div class="pf">Telefoonnr : </div></td><td><input type="text" id="adm_telefoonnr" name="adm_telefoonnr" tabindex="6" size="20" value="<?php echo $adm_telefoonnr; ?>">
</td></tr><tr><td>
<div class="pf">KvK : </div></td><td><input type="text" id="adm_kvk" name="adm_kvk" tabindex="7" size="20" value="<?php echo $adm_kvk; ?>">
</td></tr><tr><td>
<div class="pf">BTWnr : </div></td><td><input type="text" id="adm_btwnr" name="adm_btwnr" tabindex="8" size="20" value="<?php echo $adm_btwnr; ?>">
</td></tr><tr><td>
<div class="pf">Website : </div></td><td><input type="text" id="adm_website" name="adm_website" tabindex="9" size="107" value="<?php echo $adm_website; ?>">
</td></tr><tr><td>
<div class="pf">e-mail : </div></td><td><input type="text" id="adm_email" name="adm_email" tabindex="10" size="107" value="<?php echo $adm_email; ?>">
</td></tr>
</table>
<table>
<tr><td>
<input type="reset" value="Velden legen" id="knop_reset" name="knop_reset" alt="Alle velden terug zetten in de default waarde" tabindex="11">
<input type="submit" value="Opslaan" id="knop_opslaan" name="knop_opslaan" alt="Administratie opslaan" tabindex="12">
</table>
</form>
</td>
</tr>
</table>
</body>
</html>
Wat ik zie is dat je dit doet:
1. prepare
2. execute
3. bind
Dat is toch niet logisch? Execute doe je helemaal aan het einde. Dus prepare, bind, execute. Ook vang je ook geen fouten af, waardoor je dit soort dingen niet ziet. Exceptions die gegooid worden moet je wel even vangen. Toegegeven, niet helemaal verplicht, maar wel zo handig.
Waarom gebruik je eigenlijk names als $query03, $query03 etc? Wat is er mis met gewoon 1 naam aan te houden als $query of het meer gebruikte $sql?
Ben, daar heb ik wat aan. Ik ga er steeds meer van begrijpen.
Doordat ik ging splitsen kwam ik daar ook achter (de volgorde).Maar wel heel fijn dat je het me verteld, want dan weet ik het zeker.
Het afvangen wil ik zelf het liefst beperken tot de echt belangrijke punten. Maar ik ga er wel steeds meer over denken om het toch te doen.
$query met nummering doe ik om onderscheid te houden. Soms heb ik meer queries op 1 pagina. Dan zie ik graag welke ik waar gebruik. Door het gebruik van Prepare, heb ik gelijk een variabele waarmee ik ze bij elkaar kan zetten. En zo heb ik ook een overzicht dat ik mogelijk nodig kan hebben.
Ik heb het toevoegen weer onder controle. Nu het bewerken nog. Als ik ook die weer onder controle heb, ga ik ze toch weer samenvoegen.
[size=xsmall]Toevoeging op 03/09/2017 22:57:51:[/size]
Ook dat deel werkt nu. Ik ga ze samenvoegen en dan meld ik me met het resultaat.
Elke query is belangrijk. Dat is het punt. Het feit dat je geen correcte afhandeling toepast houdt hier in dat je geen foutmeldingen ziet die er wel degelijk zijn. Elke query moet dus de juiste foutafhandeling hebben. Je applicatie werkt immers al niet meer correct als ook maar 1 query mislukt. Toegegeven, in sommige gevallen kun je mislukte queries negeren, maar dat zijn erg specifieke situaties.
>> $query met nummering doe ik om onderscheid te houden. Soms heb ik meer queries op 1 pagina. Dan zie ik graag welke ik waar gebruik
Dat zie je al door gewoon naar je code te kijken, of door logische namen voor je queries te gebruiken. $query02, $query03 etc zijn net zo veelzeggend als overal $sql gebruiken, wat veel logischer is. Al is het alleen maar omdat je nu de scope ontzettend vervuilt en geheugen weggooit.
