Hulp nodig met een berekening in php
Quote:
if($_POST['bestellen']){
$query = mysql_query("SELECT `id`, `product`, `prijs` FROM `producten_overige` ORDER BY `id` ASC");
while($row = mysql_fetch_object($query)){
if(${product.$row->id} == "on"){
if(${product.$row->id."_periode"} < 1){
$prijs = $row->prijs;
$periode = 0;
}
else{
$prijs = ($row->prijs * ${product.$row->id."_periode"});
$periode = ${product.$row->id."_periode"};
}
winkelmand($session_klantnr, 1, $row->product, 0.00, $prijs, $config['btw'], $periode, 0);
}
echo "<META HTTP-EQUIV=\"refresh\" CONTENT=\"1; URL=".$config['url']."klantenpaneel/winkelmand\">";
echo "<p>".$lang['BESTELLEN_SUCCESVOL']."</p>";
}
}
else{
?>
<b> Betaling per:</b>
<select name="product_periode">
<option value="1">1
</option>
<option value="3">3
</option>
<option value="6">6
</option>
<option value="12">12
</option>
<option value="24">24
</option>
</select>
$query = mysql_query("SELECT `id`, `product`, `prijs` FROM `producten_overige` ORDER BY `id` ASC");
while($row = mysql_fetch_object($query)){
if(${product.$row->id} == "on"){
if(${product.$row->id."_periode"} < 1){
$prijs = $row->prijs;
$periode = 0;
}
else{
$prijs = ($row->prijs * ${product.$row->id."_periode"});
$periode = ${product.$row->id."_periode"};
}
winkelmand($session_klantnr, 1, $row->product, 0.00, $prijs, $config['btw'], $periode, 0);
}
echo "<META HTTP-EQUIV=\"refresh\" CONTENT=\"1; URL=".$config['url']."klantenpaneel/winkelmand\">";
echo "<p>".$lang['BESTELLEN_SUCCESVOL']."</p>";
}
}
else{
?>
<b> Betaling per:</b>
<select name="product_periode">
<option value="1">1
</option>
<option value="3">3
</option>
<option value="6">6
</option>
<option value="12">12
</option>
<option value="24">24
</option>
</select>
Gewijzigd op 05/08/2011 19:24:36 door Angelo Kreikamp
- gebruik van shortags gaat tegen je werken
- het gebruik van _fetch_object
- het gebruik van backticks in sql statements
- het gebruik van brackets
- bereking van aantal * prijs in php ipv de query
- foutieve naamgeving van variabelen
- geen foutafhandeling op mysql_query
- if(!$row2->contract == 0){, slaat ook als een tang op een varken
- escapen is nooit nodig
- onoverzichtelijk coderen
En je mag zelf bedenken hoe je je select op basis van jaar kan maken
Oh en je titel slaat natuurlijk ook nergens op
- een ieder die hier een topic opent heeft ergens advies/raad/hulp nodig, dat hoef je niet in je title te verwerken
- titel en vraag sluiten niet op elkaar aan
Gewijzigd op 05/08/2011 19:37:45 door Noppes Homeland