Hulp nodig met een berekening in php

Overzicht Reageren

Sponsored by: Vacatures door Monsterboard

Angelo Kreikamp

Angelo Kreikamp

05/08/2011 19:23:05
Quote Anchor link
Het script telt via x maanden, en ik zou het graag in jaren hebben, ik heb al wat dingen geprobeerd en het liefst doe ik het opnieuw, maar misschien weten jullie een betere oplossing...

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>
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?php if(!$row2->contract == 0){ ?>

<select name="product
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?= $row2->id; ?>
_periode">
<option value="1">1
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?= $lang['MAAND']; ?>

</option>
<option value="3">3
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?= $lang['MAANDEN']; ?>

</option>
<option value="6">6
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?= $lang['MAANDEN']; ?>

</option>
<option value="12">12
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?= $lang['MAANDEN']; ?>

</option>
<option value="24">24
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?= $lang['MAANDEN']; ?>

</option>
</select>
Gewijzigd op 05/08/2011 19:24:36 door Angelo Kreikamp
 
PHP hulp

PHP hulp

25/12/2024 14:45:05
 
Noppes Homeland

Noppes Homeland

05/08/2011 19:35:09
Quote Anchor link
gooi dit maar heel rap in de prullenak

- 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
 



Overzicht Reageren

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.