<?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...


Iemand die me hier mee kan helpen?

Mvg, Mark
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
<?php
include('../safe_admin.php');
include('../begin.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>';}

// --EINDE ID GEVULT--
// --zo niet dan gewoon een overzicht plaatsen--
} else {
$sql = "SELECT * FROM jeugdvoorpagina";
$result = mysql_query ($sql);
if ($result) {
while ($array = mysql_fetch_array($result)) {
echo '<div style="float: right;">
<a href="http://puntnl.markvdlinden.nl/login/jeugd/voorpagina.php?opdracht=verberg&id=' . $array['id'] . '">Verbergen</a> |
<a href="http://puntnl.markvdlinden.nl/login/jeugd/voorpagina.php?opdracht=bewerk&id=' . $array['id'] . '">Bewerk</a> |
<a href="http://puntnl.markvdlinden.nl/login/jeugd/voorpagina.php?opdracht=verwijder&id=' . $array['id'] . '">Verwijder</a>
</div>
<h1><b> ' . $array['titel'] . '</b></h1>' . $array['tekst'];
}
} else {
echo 'Error: ' . mysql_error();
}
}
// --EINDE OVERZICHT

include('../eind.php');
?>
na elke { een tab inspringen
net voordat elke } een tab terug springen
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 ;-).
een PHP Beautifier? Wijzigt dit niet je hele script waardoor het niet meer doet? of voegt hij alleen de tabs toe?

Ik gebruik nu PSPad die werkt ook met kleurtjes;) En bovendien wijzig ik alles gelijk via de ftp van PSPad;) en hij is gratis...

Reageren