par
ALphaman » 01 mars 2007, 18:03
Bonjour tous le monde,
J'aimerais avoir votre avis sur un bout de code, les optimisations que je pourais lui apportais car celui me "parrait" assez lourd .
//On vérifie que le membre est connecté.
if (isset($_SESSION['logged']) && $_SESSION['logged'] === true)
{
$postprixble = $_POST['prix_ble'] ;
$postprixo = $_POST['prix_avoine'] ;
$postprixa = $_POST['prix_oignon'] ;
//vente//
//-------------------------- Recupe nombre denré ------------------
$recup_denre = mysql_query ("SELECT id, ble, oignon, avoine FROM membres WHERE pseudo='$pseudo'");
$denre_recup = mysql_fetch_array ($recup_denre) or die (mysql_error());
$query_commerce_ble = mysql_query ("SELECT prix_denree, nombre_denree FROM commerce WHERE id_objet='1' AND prix_denree='$postprixble'");
while ($vm = mysql_fetch_array ($query_commerce_ble))
{
$vmprix = $vm['prix_denree'];
$vmnomb = $vm['nombre_denree'];
}
$query_commerce_oignon = mysql_query ("SELECT prix_denree, nombre_denree FROM commerce WHERE id_objet='2' AND prix_denree='$postprixo'");
while ($vo = mysql_fetch_array ($query_commerce_oignon))
{
$voprix = $vo['prix_denree'];
$vonomb = $vo['nombre_denree'];
}
$query_commerce_avoine = mysql_query ("SELECT prix_denree, nombre_denree FROM commerce WHERE id_objet='3' AND prix_denree='$postprixa'");
while ($va = mysql_fetch_array ($query_commerce_ble))
{
$vaprix = $va['prix_denree'];
$vanomb = $va['nombre_denree'];
}
//---------------Fin ------------------------------------------------
//Necessaire à l'update du ble
if (isset($_POST['ble']))
{
if ( $denre_recup['ble'] < $_POST['ble'] )
{
echo'Mise sur le marché impossible';
}
else
{
if ($_POST['ble'] > 0 && $_POST['prix_ble'] > 0 )
{
$nombre_ble = $_POST['ble'];
$prix_ble = $_POST['prix_ble'];
$noms_objet = ble ;
$total = $denre_recup['ble'] - $nombre_ble;
$id=$denre_recup['id'];
mysql_query("UPDATE membres SET ble='$total' WHERE id='$id'");
if($vmprix == $_POST['prix_ble'])
{
$denree_ble = ($vmnomb + $_POST['ble']);
mysql_query("UPDATE commerce SET nombre_denree='".$denree_ble."' WHERE id_objet='1' AND prix_denree='$postprixble'");
}
else
{
mysql_query('INSERT INTO commerce VALUES ("", "'.$_POST['ble'].'", "'.$_POST['prix_ble'].'", "1")');
}
}
}
}
// fin de lupdate du ble
Les variable POST proviennent d'une autre pages, et les 2 des 3 boucles qui parraissent inutile servent enfaite à repeter le Necessaire à l'update du ble mais avec l'oignon et l'avoine .
Merci à vous .
Bonjour tous le monde,
J'aimerais avoir votre avis sur un bout de code, les optimisations que je pourais lui apportais car celui me "parrait" assez lourd .
[php]//On vérifie que le membre est connecté.
if (isset($_SESSION['logged']) && $_SESSION['logged'] === true)
{
$postprixble = $_POST['prix_ble'] ;
$postprixo = $_POST['prix_avoine'] ;
$postprixa = $_POST['prix_oignon'] ;
//vente//
//-------------------------- Recupe nombre denré ------------------
$recup_denre = mysql_query ("SELECT id, ble, oignon, avoine FROM membres WHERE pseudo='$pseudo'");
$denre_recup = mysql_fetch_array ($recup_denre) or die (mysql_error());
$query_commerce_ble = mysql_query ("SELECT prix_denree, nombre_denree FROM commerce WHERE id_objet='1' AND prix_denree='$postprixble'");
while ($vm = mysql_fetch_array ($query_commerce_ble))
{
$vmprix = $vm['prix_denree'];
$vmnomb = $vm['nombre_denree'];
}
$query_commerce_oignon = mysql_query ("SELECT prix_denree, nombre_denree FROM commerce WHERE id_objet='2' AND prix_denree='$postprixo'");
while ($vo = mysql_fetch_array ($query_commerce_oignon))
{
$voprix = $vo['prix_denree'];
$vonomb = $vo['nombre_denree'];
}
$query_commerce_avoine = mysql_query ("SELECT prix_denree, nombre_denree FROM commerce WHERE id_objet='3' AND prix_denree='$postprixa'");
while ($va = mysql_fetch_array ($query_commerce_ble))
{
$vaprix = $va['prix_denree'];
$vanomb = $va['nombre_denree'];
}
//---------------Fin ------------------------------------------------
//Necessaire à l'update du ble
if (isset($_POST['ble']))
{
if ( $denre_recup['ble'] < $_POST['ble'] )
{
echo'Mise sur le marché impossible';
}
else
{
if ($_POST['ble'] > 0 && $_POST['prix_ble'] > 0 )
{
$nombre_ble = $_POST['ble'];
$prix_ble = $_POST['prix_ble'];
$noms_objet = ble ;
$total = $denre_recup['ble'] - $nombre_ble;
$id=$denre_recup['id'];
mysql_query("UPDATE membres SET ble='$total' WHERE id='$id'");
if($vmprix == $_POST['prix_ble'])
{
$denree_ble = ($vmnomb + $_POST['ble']);
mysql_query("UPDATE commerce SET nombre_denree='".$denree_ble."' WHERE id_objet='1' AND prix_denree='$postprixble'");
}
else
{
mysql_query('INSERT INTO commerce VALUES ("", "'.$_POST['ble'].'", "'.$_POST['prix_ble'].'", "1")');
}
}
}
}
// fin de lupdate du ble[/php]
Les variable POST proviennent d'une autre pages, et les 2 des 3 boucles qui parraissent inutile servent enfaite à repeter le Necessaire à l'update du ble mais avec l'oignon et l'avoine .
Merci à vous .