<?php // Actie voor verbergen
if ($opdracht == "verberg"){
echo "verberg";
}
if ($opdracht == "bewerk"){
echo "bewerk";
}
if ($opdracht == "verwijder"){
echo "verwijder";
}
// --EINDE ACTIE VERBERGEN
// Als opdracht leeg is of niet bestaat of er is een foute opdracht opgegeven, dan deze foutmelding geven:
else{
echo 'Er is helaas iets fout gegaan, als deze fout zich vaker voor doet, neemt u dan aub contact op met <a href="mailto:[email protected]">Mark vd Linden</a>!
<br><a href="http://puntnl.markvdlinden.nl/login/jeugd/voorpagina.php">Klik hier om verder te gaan<a>';}
?>
Wat dit doet is moet doen is eerst kijken of opdracht verberg is, zo niet dan kijken of opdracht bewerk is, zo niet dan kijken of opdracht verwijder is, zo niet dan een fout melding geven. Maar hij maakt er dit van:
Kijken of het verberg is, zo ja verberg en foutmelding laten zien
Kijken of het bewerk is, zo ja bewerk en foutmelding laten zien
Kijken of het verwijder is, verwijder laten zien
Opdracht, is niks dan foutmelding laten zien...
Ja, ik weet het, die van mij zijn altijd on overzichtelijk en werken altijd maar net...
Ik krijg nooit een echt goed script, iemand een tip hoe ik dit bv overzichtelijk kan krijgen?
Ik knip dit straks in 4 bestanden denk, overzicht, verberg, bewerk en verwijder.php
echo '<br><h3>Voorpagina jeugd wijzigen</h3><br>
<br><h4>Wat op het moment de voorpagina is:</h4>';
$opdracht = $_GET['opdracht'];
// Kijk of id gevult is
if (isset($_GET['id']) && trim($_GET['id']) != '') {
// Actie voor verbergen
if ($opdracht == "verberg"){
echo "verberg";
// EINDE ACTIE VERBERG
// Begin actie bewerk
} elseif ($opdracht == "bewerk") {
echo "bewerk";
// EINDE ACTIE BEWERK
// Begin actie verwijder
} elseif ($opdracht == "verwijder") {
$id = $_GET['id'];
$sql = "
DELETE FROM
jeugdvoorpagina
WHERE
id = ' . $id . '
";
if(!$res = mysql_query($sql))
trigger_error(mysql_error().'<br />In query: '.$sql);
elseif(mysql_affected_rows() == 0)
echo 'Geen records verwijderd. <br />Query: '.$sql;
else
echo 'Er zijn '.mysql_affected_rows().' records verwijderd uit de database.';
// --EINDE ACTIE VERWIJDER
// Als opdracht leeg is of niet bestaat of er is een foute opdracht opgegeven, dan deze foutmelding geven:
} else {
echo 'Er is helaas iets fout gegaan, als deze fout zich vaker voor doet, neemt u dan aub contact op met <a href="mailto:[email protected]">Mark vd Linden</a>!
<br><a href="http://puntnl.markvdlinden.nl/login/jeugd/voorpagina.php">Klik hier om verder te gaan<a>';}
Gebruik een goede editor (ik raad je phpDesigner 2008 aan). Een editor waar kleurtjes duidelijk zichtbaar zijn, en waar je - als je bij een { of } gaat staan - aangegeven wordt bij welk tegenovergesteld teken ({ of }) hij hoort. Bovendien zit in phpDesigner2008 de 'PHP Beautifier', 1 klik en heel je php script is prachtig overzichtelijk ;-).