waarom wil mijn script niet toevoegen? - Forum

?

Onbekende gebruiker

17-12-2007 23:11

Misschien handig om code te geven? Mijn kristallen bol is nogal mistig vandaag... Misschien bij andere niet, maar dat betwijfel ik.
Het is trouwens uitgebreider, niet uitgebrijder.

H

Hylke

17-12-2007 23:11

Je zult met php code moeten komen ;-) want aan een webpagina kunnen wij niet zoveel zien!

s

sebas

17-12-2007 23:13

dit is het script
<html>
<head
<title>machine invoegen</title>
</head>
<body>

<?php
if ($_POST["knop"]
&& $_POST["machine_naam"]
&& $_POST["machine_merk"]
&& $_POST["machine_type"]
&& $_POST["bouwjaar"]
&& $_POST["leverancier"]
&& $_POST["straat"]
&& $_POST["nummer"]
&& $_POST["postcode"]
&& $_POST["plaats"]
&& $_POST["tel_nr"]
&& $_POST["fax_nr"]
&& $_POST["olie_filter_org"]
&& $_POST["olie_filter_ver"]
&& $_POST["olie_filter"]
&& $_POST["lucht_filter_org"]
&& $_POST["lucht_filter_ver"]
&& $_POST["lucht-filter"]
&& $_POST["gasolie_filter_org"]
&& $_POST["gasolie_filter_ver"]
&& $_POST["gasolie_filter"]
&& $_POST["hydr_filter_org"]
&& $_POST["hydr_filter_ver"]
&& $_POST["hydr_filter"]
&& $_POST["motor_olie_type"]
&& $_POST["motor_olie_inhoud"]
&& $_POST["motor_olie"]
&& $_POST["hydr_olie_type"]
&& $_POST["hydr_olie_inhoud"]
&& $_POST["hydro_olie"]
&& $_POST["eindaandr_olie_type"]
&& $_POST["eindaandr_olie_inhoud"]
&& $_POST["eindaandr_olie"]
&& $_POST["keuring_datum"]
&& $_POST["extra"]) {

$db= mysql_connect("localhost", "root", "******")
or die ("kan niet verbinden: ".mysql_error());
mysql_select_db("machinelijst", $db);
$sql = "INSERT INTO machine

(machine_naam,
machine_merk,
machine_type,
bouwjaar,
leverancier,
straat,
nummer,
postcode,
plaats,
tel_nr,
fax_nr,
olie_filter_org,
olie_filter_ver,
olie_filter,
lucht_filter_org,
lucht_filter_ver,
lucht-filter,
gasolie_filter_org,
gasolie_filter_ver,
gasolie_filter,
hydr_filter_org,
hydr_filter_ver,
hydr_filter,
motor_olie_type,
motor_olie_inhoud,
motor_olie,
hydr_olie_type,
hydr_olie_inhoud,
hydro_olie,
eindaandr_olie_type,
eindaandr_olie_inhoud,
eindaandr_olie,
keuring_datum,
extra)
VALUES ('".$_POST["machine_naam"]."',
'".$_POST["machine_merk"]."',
'".$_POST["machine_type"]."',
'".$_POST["bouwjaar"]."',
'".$_POST["leverancier"]."',
'".$_POST["straat"]."',
'".$_POST["nummer"]."',
'".$_POST["postcode"]."',
'".$_POST["plaats"]."',
'".$_POST["tel_nr"]."',
'".$_POST["fax_nr"]."',
'".$_POST["olie_filter_org"]."',
'".$_POST["olie_filter_ver"]."',
'".$_POST["olie_filter"]."',
'".$_POST["lucht_filter_org"]."',
'".$_POST["lucht_filter_ver"]."',
'".$_POST["lucht-filter"]."',
'".$_POST["gasolie_filter_org"]."',
'".$_POST["gasolie_filter_ver"]."',
'".$_POST["gasolie_filter"]."',
'".$_POST["hydr_filter_org"]."',
'".$_POST["hydr_filter_ver"]."',
'".$_POST["motor_olie_type"]."',
'".$_POST["motor_olie_inhoud"]."',
'".$_POST["motor_olie"]."',
'".$_POST["hydr_olie_type"]."',
'".$_POST["hydr_olie_inhoud"]."',
'".$_POST["eindaandr_olie_type"]."',
'".$_POST["eindaandr_olie_inhoud"]."',
'".$_POST["eindaandr_olie"]."',
'".$_POST["keuring_datum"]."',
'".$_POST["extra"].")";
if (!mysql_query($sql)) {
echo "helaas, de machine met de naam ".$_POST["machine_naam"]." kon niet worden toegevoegd aan de database. neem contact op met sebas!!!!";
} else {
echo "de machine met de naam <b>".$_POST["machine_naam"].",/b> is toegevoegd.";
}
mysql_close($db);
} else {
?>

<form method="post" action="<?php echo $_SERVER ["PHP_SELF"] ?>">
welke Machine toevoegen<br>

<tr>
<td width="133">naam</td>
<td width="155"><input type="text" name="machine_naam" size="20"></td>
<td width="93">merk</td>
<td><input type="text" name="machine_merk" size="20"></td>
</tr>
<tr>
<td width="133">type</td>
<td width="155"><input type="text" name="machine_type" size="20"></td>
<td width="93">bouwjaar</td>
<td><input type="text" name="bouwjaar" size="20"></td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">leverancier</td>
<td width="155"><input type="text" name="leverancier" size="20"></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">straat</td>
<td width="155"><input type="text" name="straat" size="20"></td>
<td width="93">nummer</td>
<td><input type="text" name="nummer" size="20"></td>
</tr>
<tr>
<td width="133">postcode</td>
<td width="155"><input type="text" name="postcode" size="10"></td>
<td width="93">plaats</td>
<td><input type="text" name="plaats" size="20"></td>
</tr>
<tr>
<td width="133">tel nr</td>
<td width="155"><input type="text" name="tel_nr" size="20"></td>
<td width="93">fax nr</td>
<td><input type="text" name="fax_nr" size="20"></td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"><b>motor filter</b></td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">orgineel</td>
<td width="155"><input type="text" name="olie_filter_org" size="20"></td>
<td width="93">vervang nr</td>
<td><input type="text" name="olie_filter_ver" size="20"></td>
</tr>
<tr>
<td width="133">vervangen op</td>
<td width="155"><b>datum</b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"> <b>          
</b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"><b>gasolie filter</b></td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">orgineel</td>
<td width="155"><input type="text" name="gasolie_filter_org" size="20"></td>
<td width="93">vervang nr</td>
<td><input type="text" name="gasolie_filter_ver" size="20"></td>
</tr>
<tr>
<td width="133">vervangen op</td>
<td width="155"><b>datum</b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"><b>lucht filter</b></td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">orgineel</td>
<td width="155"><input type="text" name="lucht_filter_org" size="20"></td>
<td width="93">vervang nr</td>
<td><input type="text" name="lucht_filter_ver" size="20"></td>
</tr>
<tr>
<td width="133">vervangen op</td>
<td width="155"><b>datum</b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"><b>hydro filter</b></td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">orgineel</td>
<td width="155"><input type="text" name="hydr_filter_org" size="20"></td>
<td width="93">vervang nr</td>
<td><input type="text" name="hydro_filter_ver" size="20"></td>
</tr>
<tr>
<td width="133">vervangen op</td>
<td width="155"><b>datum</b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"><b>motor olie</b></td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">type</td>
<td width="155"><input type="text" name="motor_olie_type" size="20"></td>
<td width="93">hoeveelheid</td>
<td><input type="text" name="motor_olie_inhoud" size="20"></td>
</tr>
<tr>
<td width="133">vervangen op</td>
<td width="155"><b>datum</b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"><b>hydrolic olie</b></td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">type </td>
<td width="155"><input type="text" name="hydro_olie_type" size="20"></td>
<td width="93">hoeveelheid</td>
<td><input type="text" name="hydro_olie_inhoud" size="20"></td>
</tr>
<tr>
<td width="133">vervangen op</td>
<td width="155"><b>datum</b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"><b> </b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"><b>eindaandrijving olie</b></td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">type</td>
<td width="155"><input type="text" name="eindaandr_olie_type" size="20"></td>
<td width="93">hoeveelheid</td>
<td><input type="text" name="eindaandr_olie_inhoud" size="20"></td>
</tr>
<tr>
<td width="133">vervangen op</td>
<td width="155"><b>datum</b></td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133"> </td>
<td width="155"> </td>
<td width="93"> </td>
<td> </td>
</tr>
<tr>
<td width="133">gekeurd tot</td>
<td width="155"><b>datum</b></td>
<td width="93"> </td>
<td> </td>
</tr>
</table>
<p>extra<br>
<textarea rows="10" name="extra" cols="46"></textarea></p>
<input type="submit" name="knop" value="toevoegen">
</form>
<?php
}
?>
</body>
</html>

s

sebas

17-12-2007 23:14

ik was druk bezig maar jullie zijn zo snel met antwoorden

groet sebas

Jan Koehoorn

17-12-2007 23:22

Niet bumpen!

Die lange lijst met if's is een beetje gek in het begin. Beter zo:
<?php
if ($_SERVER['REQUEST_METHOD'] == 'POST') {
foreach ($_POST as $key => $value) {
$value = trim ($value)
if (empty ($value)) {
// hier de foutmeldingen verzamelen
}
}
}
?>

W

Wouter K

17-12-2007 23:23

Heb je error's .

Schrijf eens een andere verwerking ipv met phpserverself

dan werkt het misschien wel,

H

Hylke

17-12-2007 23:25

Leuke opmaak. Je gaat in ieder geval al de fout in met if($_POST .... blablablabla

Om dit script uit te voeren moeten al die dingen true zijn =S Gebeurd bijna nooit. Wat jij waarschijnlijk wilt is meer iets als dit:


<?php
function checkForm()
{
  foreach ($_POST as $waarde)
  {
    if ($waarde == "")
      return false;
  }
  return true;
}

if ($_SERVER['REQUEST_METHOD'] == "POST")
{
  if (!checkForm())
     echo "Niet alles is ingevult!";
  else
  {
     // De verwerking van je gegevens
  }
}
else
{
  // Het invulformulier
}
?>

Ik moet er bij zeggen dat dit niet een erg goeie manier is van controle van je formulier, en bovendien misschien ook niet aan jouw wensen voldoet :-) Dit is slechts een snel klein voorbeeldje. Ook niet getest.

Joren de Wit

17-12-2007 23:28

Offtopic: daarnaast klopt er ook vrij weinig van je datamodel. Als je problemen in de toekomst wilt voorkomen, doe je er verstandig aan om nu al te beginnen met normaliseren!