PHP

Simplification de mon code et 2 questions

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Simplification de mon code et 2 questions

Calvein

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

Publicité

flo850

moi je

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++) ?

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

FORUM HardWare.fr

Programmation

PHP

Simplification de mon code et 2 questions

Sujets relatifs
code vba protection feuille excel sauf userform	effacer deux colones sur trois avec du code VBA dans Excel...
Plantage de Mozilla firefox a cause d'un code Javascript	LE code jules cesar en c++
Petit soucis avec mon code html et l'affichage sur google	Code source HTML suspect
aide pour corriger un code	[C/C++] Choisir sur quel processeur/coeur executer du code
code::blocks, fentre d'erreurs disparue	Appel d'une fonction mysql dans du code javascript
Plus de sujets relatifs à : Simplification de mon code et 2 questions

Page générée en 0.073 secondes