Simplification de mon code et 2 questions

Marsh Posté le 19-10-2008 à 22:10:22

Je viens de me replonger dans le PHP (et le SQL surtout) pour une petite appli, mon code est bon mais je pense que je peux le faire beaucoup plus simplement.

J'utilise le modèle MCV, voilà mes parties :

Vue :

Code :

<form method="post" action="index.php">
<fieldset>
<input type="hidden" name="Logiciel" value="<?php echo $codeLogi; ?>" />
<legend>Salle ou le logiciel n'est pas installé</legend>
<?php
if (tabVide($lesSalles)){
echo "<div class=\"enteteTab\">
<p>
<span>N° Salle</span>
<span>Nb Postes</span>
<span class=\"last\">Installer</span>
</p>
</div>";
foreach ($lesSalles as $laSalle) {
echo "<div class=\"corpsTab\">
<p>
<span>".$laSalle['num']."</span>
<span>".$laSalle['nbPostes']."</span>
<span class=\"lastRight\"><input type=\"checkbox\" name=\"InstallerSalle[]\" value=\"".$laSalle['num']."\" /></span>
</p>
</div>";
}
}
else{
echo "Toute les salles sont équipées du logiciel de code ".$codeLogi;
}
?>
</fieldset><p></p>
<fieldset>
<legend>Salle ou le logiciel est installé</legend>
<?php
if (tabVide($lesSallesLog)){
echo "<div class=\"enteteTab\">
<p>
<span>N° Salle</span>
<span>Nb Postes</span>
<span class=\"last\">Installer</span>
</p>
</div>";
foreach ($lesSallesLog as $laSalle) {
echo "<div class=\"corpsTab\">
<p>
<span>".$laSalle['num']."</span>
<span>".$laSalle['nbPostes']."</span>
<span class=\"lastRight\"><input type=\"checkbox\" name=\"DesinstallerSalle[]\" checked=\"checked\" value=\"".$laSalle['num']."\" /></span>
</p>
</div>";
}
}
else{
echo "Il n'y a aucune salle avec le logiciel de code ".$codeLogi;
}
?>
</fieldset>
<p><br />
<input type="submit" value="Valider" />
</p>
</form>

Controle :

Code :

//Installer des logiciels
if (isset($_POST["InstallerSalle"])){
$tabInstall = $_POST["InstallerSalle"];
$codeLogi = $_POST["Logiciel"];
foreach ($tabInstall as $tab){
installerLogiciel($connexion, $tab, $codeLogi);
}
}
//Desinstaller des logiciels
if (isset($_POST["DesinstallerSalle"])){
$tabDesinstall = $_POST["DesinstallerSalle"];
$codeLogi = $_POST["Logiciel"];
foreach ($tabDesinstall as $tab){
desinstallerLogiciel($connexion, $tab, $codeLogi);
}
}

Modèle :

Code :

// void installerLogiciel($idConnexion, $tab, $codeLogi)
// Installe le logiciel dans la bdd en fonction de la salle
function installerLogiciel($idConnexion, $tab, $codeLogi){
$req = "SELECT * FROM Installer WHERE numsalle='".$tab."' AND codeLogiciel='".$codeLogi."'";
$jeuResultat=mysql_query($req, $idConnexion);
if($jeuResultat){
$req="INSERT INTO `Installer` (`numSalle` ,`codeLogiciel`)VALUES ('".$tab."', '".$codeLogi."')";
mysql_query($req, $idConnexion);
}
mysql_free_result($jeuResultat);
}
// void desinstallerLogiciel($idConnexion, $tab, $codeLogi)
// Desinstalle le logiciel dans la bdd en fonction de la salle
function desinstallerLogiciel($idConnexion, $tab, $codeLogi){
$req = "SELECT * FROM Installer WHERE numsalle='".$tab."' AND codeLogiciel='".$codeLogi."'";
$jeuResultat=mysql_query($req, $idConnexion);
if($jeuResultat){
$req="DELETE FROM Installer WHERE numSalle = '".$tab."' AND codeLogiciel = '".$codeLogi."'";
mysql_query($req, $idConnexion);
}
mysql_free_result($jeuResultat);
}

Je pense qu'il est assez simplifiable mais c'est la première fois que j'utilise le modèle MCV et le retour au PHP est assez laborieux.

De plus, j'aurai aimé pouvoir faire en sorte que les logiciels installés soient, par défaut, checker dans le checkbox et c'est en les decheckant qu'il se désintalleraient (là, il sont vide et il faut les checker pour les désintallé, comme pour ceux à installé).

Et aussi, j'aurai aimé faire en sorte que mon dernier foreach ai une class supplémentaire (c'est un tableau en span, j'ai désactivé la bordure du bas et j'aimerai la mettre quand j'arrive à la fin de mon instruction).

Merci

Reply

Marsh Posté le 19-10-2008 à 22:10:22

Reply

Marsh Posté le 20-10-2008 à 09:40:45

1- ça pourrait être plus lisible en encapsulant tout ça dans des classes

2- tu affiches les checkbox avec '<input type="checkbox" name="InstallerSalle" value="'.$laSalle['num'].'" checked /> et dans ton traitement, tu compare la liste des logiciels avec celle qui est checkée. Les soft qui sont dans ta liste, mais qui n'ont pas de checkbox sont a desinstaller
Au passage, pourquoi ne pas appliquer la classe lastRight directement sur la checkbox ?

3-tu teux pas le remaplcer en for($cpt = 0 ; $cpt < sizeof($lesSallesLog ) $cpt++) ?

---------------

Reply

Simplification de mon code et 2 questions

Sujets relatifs:

Leave a Replay