Amélioration de mon code PHP avec allegement de ma base de données

**guillaumeIOB** · 30/09/2018, 15h07

Bonjour à tous.

J'ai actuellement un script qui me permet de saisir les informations de chaque participant à un jeu.

Parmi ces informations l'on peut inscrire cette personne dans maximum 20 ligues, donc des que j'ouvre la fiche d'un joueur j'ai mon script qui créé les 20 lignes.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
 
$joueur=@$_GET['joueur']; 
 
$res_ligue = mysql_query("SELECT * FROM `".$sufftable."dem_ligue` where idJOUEUR='$joueur' order by nbligue asc");
for ($i = 2; $i <= 20; $i++) 
{
$res_nbligue = mysql_query("SELECT * FROM `".$sufftable."dem_ligue` where idJOUEUR='$joueur' and nbligue ='$i'");
$nb=mysql_num_rows($res_nbligue );
if ($nb==0){$modif=mysql_query("INSERT INTO `".$sufftable."dem_ligue` (`idJOUEUR`,`nbligue `) VALUES ('$joueur','$i')");}
}

Alors que dans mon admin PHP j’utilise un nombre aléatoire de lignes sans jamais savoir combien à l'avance mais toujours max 20.

La premier ligne est affichée, les autres sont cachées et s'affichent en cliquent sur un plus en queue de ligne.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
													echo '<div class="form-group normal">
														<div class="col-sm-1">Ligue</div>
														<div class="col-sm-1">niveau</div>
														<div class="col-sm-1-1">points</div>
														<div class="col-sm-1-1">Infos/CP</div>
													</div>';	
													$res_ligue = mysql_query("SELECT * FROM `".$sufftable."dem_ligue` where idJOUEUR='$joueur' order by nbligue asc");
													for ($i = 2; $i <= 20; $i++) 
													{
													      $res_nbligue = mysql_query("SELECT * FROM `".$sufftable."dem_ligue` where idJOUEUR='$joueur' and nbligue ='$i'");
													      $nb=mysql_num_rows($res_nbligue );
													      if ($nb==0){$modif=mysql_query("INSERT INTO `".$sufftable."dem_ligue` (`idJOUEUR`,`nbligue `) VALUES ('$joueur','$i')");}
													}													
													while($clligue = mysql_fetch_array($res_ligue , MYSQL_ASSOC))
													{
													      $nbligue =$clligue ['nbligue'];
													      if ($clligue ['nomligue'] =="1"){$ncl_nomligue="Bronze";}
													      if ($clligue ['nomligue'] =="2"){$ncl_nomligue="Argent";}
													      if ($clligue ['nomligue'] =="3"){$ncl_nomligue="Or";}
                                                                                                             $niveauligue=$clligue ['niveauligue'];
                                                                                                             $pointsligue=$clligue ['pointsligue'];
                                                                                                             $infosligue=$clligue ['infosligue'];
 
													echo '  	
													<div name="divligue_'.$nbligue.'" id="divligue_'.$nbligue.'">
 
														<div class="col-sm-1"><select name="nomligue_'.$nbligue.'" id="nomligue_'.$nbligue.'" onchange="envoi_ligue(\'nomligue_'.$nbligue .'\','.$joueur.',.$nbligue .');" ><option disabled>Organisme</option>';if ($clcred['credracsoc']<>''){echo '<option value="'.$clligue['credracsoc'].'" selected="selected">'.$ncl_nomligue.'</option>';} echo '<option value=""></option>'; require ('liste_ligue.php');echo '</select></div>
 
														<div class="col-sm-1"><div id="niveauligue_'.$nbligue.'" style="'; if ($clligue['niveauligue']=="10"){echo 'display:none';}echo '"><input type="text"  id="niveauligue_'.$nbligue.'" name="niveauligue_'.$nbligue.'" value="'.$clligue['niveauligue'].'" onblur="envoi_ligue(\'niveauligue_'.$nbligue.'\','.$joueur.','.$nbligue.')" ></div></div>
 
														<div class="col-sm-1"><div id="pointsligue_'.$nbligue.'" style="'; if ($clligue['pointsligue']=="10"){echo 'display:none';}echo '"><input type="text"  id="pointsligue_'.$nbligue.'" name="pointsligue_'.$nbligue.'" value="'.$clligue['pointsligue'].'" onblur="envoi_ligue(\'pointsligue_'.$nbligue.'\','.$joueur.','.$nbligue.')" ></div></div>
 
														<div class="col-sm-1"><div id="infosligue_'.$nbligue.'" style="'; if ($clligue['infosligue']=="10"){echo 'display:none';}echo '"><input type="text"  id="infosligue_'.$nbligue.'" name="infosligue_'.$nbligue.'" value="'.$clligue['infosligue'].'" onblur="envoi_ligue(\'infosligue_'.$nbligue.'\','.$joueur.','.$nbligue.')" ></div></div>
														';
														$i=$nbcred + 1;
														echo '
														<div class="col-sm-1-1"><span style="float:right;"><a href="javascript:;" onclick="document.getElementById(\'divligue_'.$i.'\').style.display = \'block\'"><b>+</b></a></span></div>														
													</div>';
													}

Chaque info tapée dans un champ d'une ligne est envoyée via une fonction javascript qui traite l'info en ajax et retourne un CSS sur le champ pour montrer que l'inf a été enregistrée.

Le problème est le suivant, pour chaque joueur j'enregistre à a chaque fois mes 20 lignes de codes dans ma base même si au final j'en utilise que 2 ou 3.

Quel serait le moyen pour alléger ce genre d'utilisation s'il vous plait.

Merci pour votre aide
Olivier

**badaze** · 30/09/2018, 16h06

Ne pas créer les 20 lignes quand tu ouvres un joueur. Mais seulement quand tu enregistres.
Faire comme tu fais actuellement et supprimer les lignes en trop quand tu as tout enregistré.

**CosmoKnacki** · 30/09/2018, 19h29

Tu devrais te mettre à PDO (d'urgence), aux requêtes préparées et ne pas ~~escmarmotter~~ escamoter la validation des données GET (et de tout ce qui vient de l'extérieur d'une manière générale), car écrire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$joueur=@$_GET['joueur']; 
 
$res_ligue = mysql_query("SELECT * FROM `".$sufftable."dem_ligue` where idJOUEUR='$joueur' order by nbligue asc");

c'est même plus un trou de sécurité, c'est un gouffre pour les injections SQL. (autrement dit si ce script est accessible en ligne, n'importe qui peut faire ce qu'il veut de ta base de données.)

**guillaumeIOB** · 30/09/2018, 21h06

Envoyé par CosmoKnacki

Tu devrais te mettre à PDO (d'urgence), aux requêtes préparées et ne pas ~~escmarmotter~~ escamoter la validation des données GET (et de tout ce qui vient de l'extérieur d'une manière générale), car écrire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$joueur=@$_GET['joueur']; 
 
$res_ligue = mysql_query("SELECT * FROM `".$sufftable."dem_ligue` where idJOUEUR='$joueur' order by nbligue asc");

c'est même plus un trou de sécurité, c'est un gouffre pour les injections SQL. (autrement dit si ce script est accessible en ligne, n'importe qui peut faire ce qu'il veut de ta base de données.)

Je comprends CosmoKnacki, mais il m'est impossible pour le moment de me mettre a PDO, j'ai essayé et c'est suffisamment compliqué pour que je ne puisse trouver le temps de m'y mettre.
Ensuite c'est un script qui fait partie d'une administration avec accès sécurisé dont l'URL de la page connexion n'est connue que des seuls professionnels qui l'utilisent, tous issus de la meme société.

Merci badaze pour cette information mais tant que l'année n'est pas terminée, l'on ne sait pas à l'avance combien il y aura de ligues, donc durant 12mois, meme une fois enregistré, l'on peut revenir régulièrement sur la fiche du joueur et y rajouter des ligues

**badaze** · 30/09/2018, 21h57

@kaolivier.

A ta place je ferai d'urgence ce que préconise CosmoKnacki ou tout du moins dans un premier temps contrôler la validité les paramètres passés aux pages ($_GET, $_POST). Même si l'url d'administration n'est connue que de quelques personnes, je suppose que le site est accessible à tous et que les failles de la partie admin se retrouvent dans la partie publique. Bref. Là ça n'est même pas urgent c'est plusqurgent.

Pour ce qui est de l'allègement de la bdd tu peux fort bien faire comme suit :

- Lecture des ligues associées au joueur (0 ligne lue lors de la création du joueur)
- Remplissage d'un tableau contenant les données des ligues du joueur.
- Complément du tableau jusqu'à 20 avec des valeurs par défaut.
- Parcours du tableau pour afficher les données.

Quand tu fais une modification ou un ajout tu insères ou mets à jour selon que tu trouves où pas des données dans la table.

**guillaumeIOB** · 30/09/2018, 22h17

Non il n'y a aucun accès public, l'admin est interne un peu comme un intranet, bien qu'il faille une connexion internet pour se connecter.

D'ici quelques mois je financerai un freelance pour qu'il me traduise tous ces scripts avec PDO, mais franchement je ne comprends pas cette urgence ???

OVH, et 1and1 qui sont les seuls serveurs sur lesquels tous mes scripts sont installés pour des centaines de personnes me disent eux-memes que le PHP 5.6 restera encore actif pour 2 ou 3 ans minimum et mysql fonctionnera encore durant cette période, alors pourquoi faudrait il se jeter de suite sur PDO ?? pour la sécurité c'est clair que les fonctions les plus sensibles sont accessibles dans mes scripts sur la base de séquence hachées ou cryptées, après tout reste piratable de toute façon.

- Lecture des ligues associées au joueur (0 ligne lue lors de la création du joueur)
- Remplissage d'un tableau contenant les données des ligues du joueur.
- Complément du tableau jusqu'à 20 avec des valeurs par défaut.
- Parcours du tableau pour afficher les données.

Quand tu fais une modification ou un ajout tu insères ou mets à jour selon que tu trouves où pas des données dans la table.

C'est ce que je fais déjà. En fait je cherchais plus une fonctions qui en cliquant sur le plus créé à la fois la ligne en BDD et en meme temps affiche la lignes pour la remplir ensuite.

**badaze** · 30/09/2018, 22h27

Ce n'est pas ce que tu fais puisque tu crées les enregistrements dans la table.

Mon algorithme ne crée rien. Il ne fait que lire et compléter avec des données en mémoire. Actuellement tu dois avoir une routine qui met à jour les données sans quoi ça ne fonctionnerait pas. Donc dans cette routine tu peux tester et voir s'il existe un enregistrement qui correspond ou pas.

Je ne savais pas si le site était public ou pas.
Pour ce qui est de PDO c'est à faire. Mais le plus urgent est de sécuriser les données transmises aux pages. Ensuite si les données sont "sécurisées" et qu'il n'y a plus de risque d'injection SQL le passage à PDO peut se faire plus tard.

Pour ce qui est de 1and1, il y a deux ou trois ans ils m'ont "forcé" à passer à PHP 7 car pour rester en PHP 5.x il fallait que je paie une rallonge.

**guillaumeIOB** · 30/09/2018, 22h57

Pour ce qui est de 1and1, il y a deux ou trois ans ils m'ont "forcé" à passer à PHP 7 car pour rester en PHP 5.x il fallait que je paie une rallonge.

Les boules...

Toutes les semaines, j’achète des hébergements sur 1and1 et OVH et je peux downgrader en 5.6 sans paiement supplémentaire..... franchement ils ont abusé de faire ça....

Pour reprendre mon sujet, en resumé tu me propose d'afficher mon tableau coté html et si les données que je rentre ne sont pas déja rentrées dans ma BDD il créé une ligne dans ma BDD et enregistre les données dans cette ligne ???

Le probleme est que dans mon exemple j'ai mis 3 champs par ligne mais si il y a 30 champs, il doit a chaque fois rechercher la combinaison des X champs pour verifier si je l'ai deja ????
ça va pas bouffer de la bande passante pour rien ça ?

Merci en tout cas de ton aide

**badaze** · 30/09/2018, 23h31

Actuellement tu modifies bien des lignes et ces modifications sont bien enregistrées. Comment fais-tu ?

Amélioration de mon code PHP avec allegement de ma base de données

PHP & Base de données

Discussions similaires

Partager

Partager