Comment coder correctement ?

Version imprimable

Bonjour,

Je suis actuellement entrain de creer un site web (HTML/CSS/PHP) et je viens souvent sur super forum pour trouver des infos afin de resoudre mes problemes et comme j'ai souvent eu de bon retour, je voudrais avoir quelques conseil sde 'bonnes pratiques' pour coder correctement c'est a dire en respectant un certain standard utilise.

Je n'ai jamais suivi de formation donc j'ai essaye d'apprendre en suivant différent tuto sur le net du coup mes pages font ce qu'elles doivent faire mais je pense que c'est loin d'etre propre.

Voici par exemple le code de ma page index.php. Pouvez vous me donner votre avis sur mon code

index.php

Code:

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
<!DOCTYPE html>
<html>
    <head>
	<meta http-equiv="Content-Type" content="text/html" charset="UTF-8" > 
        <link rel="stylesheet" href="style.css" />
        <title>Mon site</title>
    </head>
    <body>
        <div id="bloc_page">
            <header>
		<?php include ("titre_principal.php");?>
                <?php include ("menu.php");?>
            </header>
		<?php include ("baniere.php");?>
		<?php include ("authentifie.php");?>
		<?php include ("recherche_simple.php");?>
 
            <section>
                <article>
                    <h1><img src="images/ico_epingle.png" alt="" class="ico_categorie" />Acceuil</h1>
                    <p>Ce site est mon site</p>
		<div class="titre_rouge">
			</br>Au hasard
		</div>
		<?php
		try
		{
			$pdo_options[PDO::ATTR_ERRMODE] = PDO::ERRMODE_EXCEPTION;
			$bdd = new PDO('mysql:host=localhost;dbname=mon_projet', 'root', '', $pdo_options);
			$bdd->query("SET NAMES 'utf8'");
			$reponse = $bdd->query('SELECT * FROM film WHERE id >= RAND() * (SELECT MAX(id) FROM film) LIMIT 4');
			while ($donnees = $reponse->fetch())
			{
				echo '<strong><a href="fiche_film_generic.php?id='.$donnees['id'].'">'.$donnees['titre_original'].'</a></strong> <a href="fiche_film_generic.php?id='.$donnees['id'].'"> ('.$donnees['titre_francais'].')</a><br />';
			}
		}
		catch (Exception $e)
		{
			die('Erreur : ' . $e->getMessage());
		}
	?>
      </article>
   </section>
    <?php include ("footer.php");?>
 
</div>
</body>
</html>

Merci

16/04/2012, 10h07
Vil'Coyote

sans connaitre le contenu des include on peut supposer que cela à l'air normal et propre. le seul reproche que je ferais serait dans l'utilisation du php.

l'ouverture et fermeture intempestive peut avoir des conséquences en terme de ralentissement ou gestion serveur.

j'aurais tendance à tous passer en php.
à toi de voir.
16/04/2012, 16h22
Baldy

Tu n'utilises pas les commentaires pour agrémenter ton code? C'est toujours très apprécié si jamais tu partages ton code avec d'autres ou qu'il est repris par quelqu'un derrière toi.
16/04/2012, 16h55
bob633

J'ajouterais aussi la possibilité de faire tes requêtes SQL dans des fonctions ou des classes (je sais pas si tu bosses en php 4 ou 5) afin de séparer l'affichage des données du traitement.

Après en effet, sur ces quelques lignes, c'est correct
16/04/2012, 20h23
p_oum

Bonjour,

Merci pour vos reponses.
@bob633: Je suis en PHP5 enfin je ne connais pas trop les difference entrela version 4 et 5. Lorsque tu dis "requêtes SQL dans des fonctions ou des classes" que veux tu dire exactement ? Creer un fichier fonctions.php par exemple et ajouter dans ce fichier les lignes qui permettent de faire la connexion a la base et/ou les requetes ?

Aurais tu un exemple simple a me donner ?

Merci

Je ne suis pas développeur web mais on m'a dit qu'il était toujours bien de séparer le traitement (affichages, requêtes, ...).

Oui PDO est PHP5 mais tu peux utiliser des classes pour avoir la notion d'objet dans ton programme.

Au lieu d'avoir ta connexion BDD dans ton code actuel. Tu pourrais créer un fichier classConnexion.php par exemple dans lequel tu aurais ta méthode de connexion
Code:

1 2 3 4 5 6 7 8 9 10 11 class Connexion { private $connexion; function __construct() { // Ton code PDO de connexion return $this->connexion; } }
Ensuite une classFilm.php
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 class films { private $ListeFilm = array(); function __construct() { } function Afficherfilm($connexion) { // Ton code select return $this->ListeFilm ; // Tableau de tout tes films } }
Dans ta page principal :
Code:

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 <?php include_once('classConnexion.php'); include_once('classFilms.php'); ?> <!DOCTYPE html> <html> <head> <meta http-equiv="Content-Type" content="text/html" charset="UTF-8" > <link rel="stylesheet" href="style.css" /> <title>Mon site</title> </head> <body> <div id="bloc_page"> <header> <?php include ("titre_principal.php");?> <?php include ("menu.php");?> </header> <?php include ("baniere.php");?> <?php include ("authentifie.php");?> <?php include ("recherche_simple.php");?> <section> <article> <h1><img src="images/ico_epingle.png" alt="" class="ico_categorie" />Acceuil</h1> <p>Ce site est mon site</p> <div class="titre_rouge"> </br>Au hasard </div> <?php $connexion = new Connexion(); // constructeur de connexion $Films = new Films(); // Constructeur de films $tableauListeFilms = $Films->Afficherfilm($connexion); // Tu appelles ta méthode Afficherfilm, avec en paramètre la connexion BDD for($i=0;$i<count($tableauListeFilms);$i++) { echo $tableauListeFilms[$i]; // Afficher chaque ligne du tableau films } ?> </article> </section> <?php include ("footer.php");?> </div> </body> </html>
Voilà des classes simples, après tu peux regarder des cours PHP objet sur google ou même sur les tutos developpez.

J'ai écris le code rapidement mais dans l'idée c'est ca. Sur ta page php principale, tu déclares juste et appels juste tes méthodes de classes.

edit : j'ai modifié un peu mon code, c'était pas jolie :mrgreen:

17/04/2012, 09h20
Vil'Coyote

la principale différence réside dans la notion de classe. ce qu'il t'es indiqué est donc d'utiliser cette notion pour éviter la redondance dans ton code.

http://stephaneey.developpez.com/tut...p5_nouveautes/

donc là il ne s 'agirait pas de générer une simple page avec des fonction. mais une classe de connexion réutilisable.
17/04/2012, 19h01
p_oum

Ok merci beaucoup pour tout ces conseils je vais essayer de les appliquer au mieux