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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
| <?php
define('MAJ','27.01.2021 09:45:42');
// Initialisations des sessions
session_start();
if (!isSet($_SESSION['site'])) {
$siteID = '';
$sitePrefixe = '';
$siteName = '';
} else {
$siteID = $_SESSION['site']['id'];
$sitePrefixe = $_SESSION['site']['prefixe'];
$siteName = $_SESSION['site']['nom'];
}
// Inclusion des fichiers principaux
define('INCLUDES_FILES', array('header'=>'header.php',
'footer'=>'footer.php',
'variables'=>'variables.php',
'fonctions'=>'fonctions.php'
));
foreach (INCLUDES_FILES as $index => $files) {
if (!file_exists('includes/'.$files)) { header('Location: alerte.php?warning=500'); }
if ($index == 'variables' || $index == 'fonctions') { include('includes/'.$files); }
}
/*
CREATE TABLE adm_sites (
id BIGINT UNSIGNED NOT NULL AUTO_INCREMENT,
prefixe VARCHAR(200) NULL,
nom VARCHAR(200) NOT NULL,
infos TEXT NULL,
PRIMARY KEY (id),
UNIQUE (prefixe)
);
*/
// Fonction d'insertion
function inserersolution($prefixe, $nom, $infos) {
try {
$connexion = C_PDO::getC();
$requete = "INSERT INTO adm_sites (prefixe, nom, infos) VALUES(?,?,?)";
$stmt = $connexion->prepare($requete);
$stmt->execute([$prefixe, $nom, $infos]);
// Trouve l'id auto incrémenté
$id = $connexion->lastInsertId ();
}
catch(PDOException $e) {
exit($e->getMessage());
}
return $id;
}
// Fonction de modification
function modifiersolution( $id, $prefixe, $nom, $infos) {
try {
$connexion = C_PDO::getC();
$requete = "UPDATE adm_sites SET prefixe = ?, nom = ?, infos = ? WHERE adm_sites.id = ?";
$stmt = $connexion->prepare($requete);
$stmt->execute([$prefixe, $nom, $infos, $id]);
$count = $stmt->rowCount();
}
catch(PDOException $e) {
exit($e->getMessage());
}
return $count ? "Modification réussie" : "Impossible de modifier";
}
// Fonction de suppression
function supprimersolution($id) {
try {
$connexion = C_PDO::getC();
$requete = "DELETE FROM adm_sites WHERE adm_sites.id = ?";
$stmt = $connexion->prepare($requete);
$stmt->execute([$id]);
$count = $stmt->rowCount();
}
catch(PDOException $e) {
exit($e->getMessage());
}
return $count ? "Supression réussie" : "Impossible de supprimer";
}
// function de vérification commune à l'ajout et à la modification
function verifpost($post) {
$erreur = empty($post['nom']) ? "Le nom doit être renseignée" : null;
$erreur = empty($post['prefixe']) ? "Le préfixe doit être renseignée" : $erreur;
return $erreur;
}
// Ajout en provenance du formulaire
if (isset($_POST["ajout"]) && $_POST["ajout"] == "ajouter") {
// Enregistre le post en session pour éviter d'avoir à ressaisir tous les champs en cas d'erreur
$_SESSION['ajout_sites'] = $_POST;
//J'applique trim sur les éléments du tableau pour supprimer les espaces blancs en début et fin de chaine qui pourraient être rentrés inopinément par l'utilisateur
$post = array_map('trim',$_POST);
//Fonction de vérification
$erreur = verifpost($post);
// Si pas d'erreur on lance l'insertion en récupèrant l'id auto incrémenté
$id = empty($erreur) ? inserersolution($post['prefixe'], $post['nom'], $post['infos']) : null;
// On affiche un message d'erreur uniquement si nécéssaire. Si OK le header renverra vers l'élément inséré
$_SESSION['message_sites'] = isset($erreur) ? $erreur : null;
// Header pour éviter les multiples post et le messgae du navigateur si on réaffiche cette page. L'ancre id sert pour aller à l'élément inséré
header('Location: '.$_SERVER['PHP_SELF'].'#lent'.$id);exit;
}
// Modification en provenance du formulaire
if (isset($_POST["modif"]) && $_POST["modif"] == "modifier") {
$_SESSION['modif_sites'] = $_POST;
$post = array_map('trim',$_POST);
$erreur = verifpost($post);
$_SESSION['message_sites'] = empty($erreur) ? modifiersolution($post['id'], $post['prefixe'], $post['nom'], $post['infos']) : $erreur;
header('Location: '.$_SERVER['PHP_SELF'].'#lent'.$post['id']);exit;
}
// Suppression en provenance du formulaire
if (isset($_POST["suppr"]) && $_POST["suppr"] == "supprimer") {
$_SESSION['message_sites'] = supprimersolution($_POST['id']);
header('Location: '.$_SERVER['PHP_SELF']);exit;
}
// Variables qui pourront être affichées dans le html pour les ajouts, modifications et suppressions
$message = isset($_SESSION['message_sites']) ? $_SESSION['message_sites'] : null;
$modif = isset($message,$_SESSION['modif_sites']) ? $_SESSION['modif_sites'] : null;
// $id_modif pour cibler le formulaire de modification
$id_modif = isset($_SESSION['modif_sites']['id']) ? $_SESSION['modif_sites']['id'] : null;
// On fait également un test sur message pour ne pas remplir les champs du formulaire d'ajout si l'insertion est réussie
$ajout = isset($message,$_SESSION['ajout_sites']) ? $_SESSION['ajout_sites'] : null;
// On efface les variables de session pour qu'elles ne soient définies qu'une fois
unset($_SESSION['message_sites'],$_SESSION['modif_sites'],$_SESSION['ajout_sites']);
//Affichage des résultats en alimentant (hydratant) les champs des formulaires de modification et suppression
$select = "SELECT id, prefixe, nom, infos FROM adm_sites ORDER BY id";
$connexion = C_PDO::getC();
// Pas besoin de requête préparée ici puisqu'il n'y a pas de variables utilisateur dans la requête
$read = $connexion->query($select);
?>
<!DOCTYPE html>
<html lang="fr-CH">
<head>
<title>Administrations</title>
<link rel="stylesheet" href="images/designs/styles.css">
<script src="https://kit.fontawesome.com/2e9b34a81e.js"></script>
<style>
form p {
margin:0.5em;
padding:0;
}
form .label {
display:inline-block;
width:120px;
}
#message {
border:1px solid black;
background:#0CF;
font-size:1.5em;
text-align:center;
}
.saisie {
width: 400px;
}
button {
padding: 3px;
width: 100px;
}
</style>
</head>
<body>
<? include('includes/'.INCLUDES_FILES['header']); ?>
<h1>Gestion des Sites</h1>
<div align="justify" style="width: 600px;">
Concernant le nom du site ainsi que son préfixe, il serait important de ne pas mettre d’accents,
de caractères spéciaux ou d’espace. Il faudrait plutôt jouer entre les majuscules
et les minuscules pour différencier les mots. En revanche pour les infos, aucun soucis, tous les
caractères sont acceptés et présenteront aucun danger niveau codage.
</div>
<br />
<a href="sites.php?gestion=new" target="_parent" title="Ajouter une entrée"><i class="fas fa-folder-plus" style="font-size: 30px;"></i></a>
<br />
<?php
// On fait afficher ici les messages de suppression et d'ajout mais pas ceux de modification qui seront affichés dans le formulaire concerné
if(isset($message) && empty($id_modif)) echo '<p id="message">'.htmlspecialchars($message).'</p>';
//if (!empty($_GET['gestion'])) {
?>
<br />
<form action="#" method="POST">
<fieldset style="width: 600px;">
<legend>Ajouter</legend>
<p><label><span class="label">Nom : </span><input type="text" name="nom" value="<?=isset($ajout['nom']) ? htmlspecialchars($ajout['nom']) : null;?>" class="saisie" /></label></p>
<p><label><span class="label">Prefixe : </span><input type="text" name="prefixe" value="<?=isset($ajout['prefixe']) ? htmlspecialchars($ajout['prefixe']) : null;?>" class="saisie" /></label></p>
<p><label><span class="label">Infos : </span><textarea name="infos" class="saisie"><?=isset($ajout['infos']) ? htmlspecialchars($ajout['infos']) : null;?></textarea></label></p>
<div align="center"><button type="submit" name="ajout" value="ajouter">Ajouter</button></div>
</fieldset>
</form>
<br />
<?php
//}
foreach ($read as $row) {
?>
<!-- l'id sert d'ancre de redirection pour les ajouts / modifications (Attention de ne pas utiliser d'id statique dans ces lignes puisque c'est une boucle)-->
<br />
<form action="#" method="post" style="margin-bottom:0.5em" id="lent<?= isset($row->id)? htmlspecialchars($row->id) : ''?>">
<fieldset style="width: 600px;">
<!-- message des modifications -->
<legend>Modifier/Supprimer : <b><?=htmlspecialchars($row->nom);?></b></legend>
<?= isset($id_modif, $message) && $row->id == $id_modif ? '<p style="color:blue">'.htmlspecialchars($message).'<p>' : ''?>
<input type="hidden" name="id" value="<?= htmlspecialchars($row->id)?>" >
<p><label><span class="label">Nom : </span><input type="text" name="nom" value="<?= $row->id == $id_modif ? htmlspecialchars($modif['nom']) : htmlspecialchars($row->nom) ?>" class="saisie" /></label></p>
<p><label><span class="label">Prefixe : </span><input type="text" name="prefixe" value="<?= $row->id == $id_modif ? htmlspecialchars($modif['prefixe']) : htmlspecialchars($row->prefixe) ?>" class="saisie" /></label></p>
<p><label><span class="label">Infos : </span><textarea name="infos" class="saisie"><?= $row->id == $id_modif ? htmlspecialchars($modif['infos']) : htmlspecialchars($row->infos) ?></textarea></label></p>
<div align="center">
<button type="submit" name="modif" value="modifier">Modifier</button>
<button type="submit" name="suppr" value="supprimer" style="margin-left:2em">Supprimer</button>
</div>
</fieldset>
</form>
<?php
}
?>
<? include('includes/'.INCLUDES_FILES['footer']); ?>
</body>
</html> |
Revision d'un vieux site en PHP/MySQL avec le format actuel
Donc, je vais avoir besoin d'aide pour transformer certaines lignes de codes dans un format plus correct et je me dis qu'il serait aussi bien d'opter pour une manière plus aérée et plus lisible que mes gros pâtés de codes qui ne veulent plus rien dire. Donc, je vais devoir refaire intégralement mon site principal (www.gwanda.ch). Donc, si vous voulez m'aider, c'est avec grand plaisir de recevoir vos remarques et vos corrections
Répondre avec citation
Envoyé par MaitrePylos
Et je m'ennuyait tout le temps à utiliser des tableaux HTML, et lorsque j'ai vu que tu utilisais le CSS pour remplacer les tableaux, je me suis dit, mais quel bonne idée. Bref, tu as fait une heureuse aujourd'hui et pas qu'un peu, j'ai le sourire croché et des étoiles pleins les yeux ! 
Je pensais que par sécurité, il était préférable de convertir tous les caractères spéciaux avant l'enregistrement dans la DDB afin d'éviter que par exemple un utilisateur hack la base de données. Et j'utilisais d'office htmlentities car d'après la bible PHP de 2002, htmlentities convertit les caractères accentués contrairement à htmlspecialchars. Après, la bible PHP n'est plus d'actualité vu qu'elle est sortie en 2002 
Ceci-dit, j'ai corrigé mes short tag.
Sur le moment, j'avais vaguement compris le système, mais grâce à toi, je le comprends bien mieux à présent !! Je pense qu'il me fallait un exemple visuel plutôt que de vagues explications comme le tuto sur lequel j'étais tombée. Grâce à l'exemple que tu m'as donné, c'est directement devenu plus clair et plus compréhensible et je t'en remercie ! Bon, pour l'instant, cela ne me sert pas à grand chose, car pour ma table 
Partager