Télécharger le zip
Salut,je viens de regarder rapidement, et je trouve que tu codes assez proprement. Mais quelques petites améliorations pourraient être faites.Par exemple, créer un fichier "header.inc.php" par exemple contenant toutes les informations fixes se trouvant toujours dans le haut de ton code (exemple avec le fichier creation_client.php) :fichier header.inc.php :<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"><html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr"><head><title><?php echo $titre; ?></title><meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" /><link rel="stylesheet" media="screen" type="text/css" title="titre du css" href="Design.css" /> </head><body> <?phpmysql_connect("localhost", "root", "");mysql_select_db("usine"); ?>tu n'aurais plus qu'à appeler dans chaque page cette page header.inc.php, comme ça, plus besoin de faire à chaque fois plusieurs connexion à sql dans la même page.ça reviendrai à ça pour ton fichier creation_client.php :<?php$titre = 'Nouveau Client';include 'includes/header.inc.php';//le reste du code?>Tu peux également faire la même chose avec le bas de page en incluant les balises de fermeture du body, html et de sql.Sinon, pense à tout le temps utiliser des simple côte pour tout ce qui n'est pas html, c'est plus lisible et ça améliore un peu la rapidité du script. Autre petite chose sur les echo, pas besoin d'en mettre à chaque ligne. Exemple :fichier creation_commande.php :echo "<option value=$id>";echo $j;echo "</option>"; peut être remplacé par :echo '<option value="' . $id . '">' . $j . '</option>\n';Tu mérite quand même une bonne note pour la lisibilité, la clarté (rare sur les premières sources :P).//Klick.
klicksoe, \n n'est pas evalue dans des '', mais seulement dans des ""
salut,# $annee = date("Y");# $mois = date("m");# $jour = date("d");# $date=$annee.'-'.$mois.'-'.$jour ;## echo $date;echo "<br/>"; tu peux faireecho date('Y - m -d'); tout simplement# $id_palette = isset($_POST['id_palette']) ? $_POST['id_palette'] : '';### if ($id_palette) {if(isset($_POST['id_palette') && !empty($_POST['id_palette')){# echo "<option value=$w>";# echo $j;# echo "</option>"; un seul echo suffit=>echo '<option value="'.$w.'">'.$j.'</option>';....-pense a faire une classe qui gère le sql ça va être plus propre, et attention aux failles xss. -tu dois t'habituer à indenter ton code (html et php)..voilà sinon pas mal :)
rt c domage de pas avoir un fichier config
j'ai regarder vite fait ton code , il me semble que tu a oublié de mettre une gestion d'erreur en cas de problème de connections a ta basse de donné et je confirme les Commentaires de klicksoe , mets un fichier externe , sa t'économisera des lignes, et plus rapide a modifier ton host , login et pass .... si un jour tu change d'hébergeur.sinon le code me semble propre , good joob.
encore une petite chose, avants d'enregistrer un nouveau client ou un nouvelle article , fait un control dans ta base histoire de voir ci ce client ou article n'y est pas déjà.
Se souvenir du profil
Mot de passe oublié ? / Activation de compteCréer un compte
1 873 466 membres 274 nouveaux aujourd'hui 16 151 membres club