klicksoe
Messages postés38Date d'inscriptionvendredi 9 mars 2007StatutMembreDernière intervention23 avril 2009
-
5 mars 2008 à 14:16
cs_slayer
Messages postés202Date d'inscriptiondimanche 1 septembre 2002StatutMembreDernière intervention18 mars 2008
-
8 mars 2008 à 23:45
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
cs_slayer
Messages postés202Date d'inscriptiondimanche 1 septembre 2002StatutMembreDernière intervention18 mars 2008 8 mars 2008 à 23:45
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à.
cs_slayer
Messages postés202Date d'inscriptiondimanche 1 septembre 2002StatutMembreDernière intervention18 mars 2008 8 mars 2008 à 02:01
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.
bentom32390
Messages postés25Date d'inscriptionmercredi 28 novembre 2007StatutMembreDernière intervention21 février 2009 7 mars 2008 à 18:15
rt c domage de pas avoir un fichier config
amezghal
Messages postés385Date d'inscriptionlundi 27 février 2006StatutMembreDernière intervention21 août 20155 5 mars 2008 à 20:20
salut,
# $annee = date("Y");
# $mois = date("m");
# $jour = date("d");
# $date=$annee.'-'.$mois.'-'.$jour ;
#
# echo $date;echo "
";
tu peux faire
echo 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 :)
coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 5 mars 2008 à 15:19
klicksoe, \n n'est pas evalue dans des '', mais seulement dans des ""
klicksoe
Messages postés38Date d'inscriptionvendredi 9 mars 2007StatutMembreDernière intervention23 avril 2009 5 mars 2008 à 14:16
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) :
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 :
8 mars 2008 à 23:45
8 mars 2008 à 02:01
sinon le code me semble propre , good joob.
7 mars 2008 à 18:15
5 mars 2008 à 20:20
# $annee = date("Y");
# $mois = date("m");
# $jour = date("d");
# $date=$annee.'-'.$mois.'-'.$jour ;
#
# echo $date;echo "
";
tu peux faire
echo 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 :)
5 mars 2008 à 15:19
5 mars 2008 à 14:16
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>
<?php
mysql_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.