PHP

Est-ce normal d'avoir 200 lignes de code pour en afficher 4 ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Est-ce normal d'avoir 200 lignes de code pour en afficher 4 ?

Master_Jul

Hello,

voilà je suis en train de faire un « petit » script pour gérer des cours et leur numéro (première partie d'un exercice), voilà le résultat :

http://masterjul.homeip.net/tp_sql/gest_cours.php

Et voilà... le code :
http://masterjul.homeip.net/tp_sql/source_cours.php

Soit un peu plus de 200 lignes (avec les commentaires et le HTML certes). Je précise avant de voir arriver les vautours que le but n'est pas pour le moment de faire du code W3C compliant mais les remarques sont les bienvenues, quand même.

Ma question est surtout de savoir si l'on ne pas simplifier ce code ou c'est une taille normale. Et puis s'il y a des erreurs à la con, je veux bien des remarques aussi !

Je travaille en registe_global à off et error_reporting E_ALL.

Merci !

EDIT 17 AVRIL :

Je viens de pondre la gestion des centres, préparez-vous psychologiquement avant de regarder le code :

http://masterjul.homeip.net/tp_sql/gest_centres.php
http://masterjul.homeip.net/tp_sql/source_centres.php

Message édité par Master_Jul le 17-04-2004 à 11:14:21

---------------
En français, on écrit "connexion", pas "connection".

Publicité

drasche

Tu pourrais virer tous ces echo et taper le code HTML en brut avec juste des echo ici et là pour afficher les données, ça me semble mieux et beaucoup plus lisible (pis tu te ferais pas chier avec l'échappement de quotes).

---------------
Whichever format the fan may want to listen is fine with us – vinyl, wax cylinders, shellac, 8-track, iPod, cloud storage, cranial implants – just as long as it’s loud and rockin' (Billy Gibbons, ZZ Top)

Arjuna

Aircraft Ident.: F-MBSD

Ce serait bien de te mettre d'accord avec toi-même : soit tu utilise le ' soit tu utilise le " pour les chaînes PHP, mais évite de mélanger les deux, quitte à mettre des caractères d'échappement à l'intérieur. Parceque c'est passablement illisible.

Dj YeLL

$question = $to_be || !$to_be;

Bon, pour ce qui est du nombre de ligne de code, c'est pas evident de te répondre, je ne sais pas s'il y a des choses superflues...

Sinon voici comment moi je procède (mais bon, chacun sa méthode) :

Code :

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<title>Gestion des cours</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<link href="tp_sql.css" rel="stylesheet" type="text/css">
<script type="text/javascript" language="javascript">
var confirmMsg = 'Êtes-vous sûr(e) de vouloir supprimer ';
</script>
<script src="libraries/functions.js" type="text/javascript" language="javascript"></script>
</head>
<body background="images/fond1.gif">
<div align="center">
<p class="police_std_titre">GESTION DES COURS</p>
<?PHP
//Paramètres de connexion à la base MySQL
require 'connexion.inc.php';
//Connexion
$connex = connex_base();
if (!$connex)
{
echo "Connexion à la base impossible, réessayez plus tard.";
exit;
}
/*********************************************************\
Fontion pour tester la présence d'un cours dans la base
Test basé sur le numéro du cours
\*********************************************************/
function presence_cours($cours)
{
$sql= "SELECT cours FROM cours WHERE cours=$cours";
$req= mysql_query($sql) or die('Erreur SQL : '.$sql.'<br>'.mysql_error());
$line = mysql_fetch_array($req);
if($line)
{
return false;
}
else return true;
}
// On extrait les variables passées dans l'URL
extract($_GET,EXTR_OVERWRITE);
// Les variables sont définies histoire de coder proprement
if(!isset($action)) $action=null;
if(!isset($cours_existant_value)) $cours_existant_value=null;
if(!isset($tri)) $tri=null;
/******************\
GESTION DU TRI
\******************/
if($tri=='intitule')
$order_by="ORDER BY intitule";
else
$order_by="ORDER BY cours";
/********************\
AJOUT D'UN COURS
\********************/
if($action=='ajouter')
{
extract($_POST,EXTR_OVERWRITE);
if(presence_cours($cours))
{
$intitule=strtoupper(trim($intitule));
$sql= "INSERT INTO cours VALUES ($cours,'$intitule')";
$req= mysql_query($sql) or die('Erreur SQL : '.$sql.'<br>'.mysql_error());
if(!$req) echo "Echec de l'insertion.";
} else
{
echo '<p class="header_table">Le cours '.$cours.' existe déjà !</p>';
$cours_existant_value=$intitule;
}
}
/*************************\
MODIFICATION D'UN COURS
\*************************/
elseif($action=='modifier_valide')
{
extract($_POST,EXTR_OVERWRITE);
$intitule=strtoupper(trim($intitule));
$sql= "UPDATE cours SET cours=$cours,intitule='$intitule' WHERE cours='$cours_mod'";
$req= mysql_query($sql) or die('Erreur SQL : '.$sql.'<br>'.mysql_error());
if(!$req) echo "Echec de la modification.";
}
/*************************\
SUPPRESSION D'UN COURS
\*************************/
elseif($action=='supprimer')
{
$sql= "DELETE FROM cours WHERE cours=$cours";
$req= mysql_query($sql) or die('Erreur SQL : '.$sql.'<br>'.mysql_error());
if(!$req) echo "Echec de la suppression.";
}
/************************\
CREATION DE LA LISTE
\************************/
$sql= "SELECT * FROM cours $order_by";
$req= mysql_query($sql) or die('Erreur SQL : '.$sql.'<br>'.mysql_error());
// Entête du tableau
?>
<table width="60%" border="0" align="center" bgcolor="#FFFFFF">
<tr bgcolor="#4E76A4" class="header_table">
<td><div align="center"><a href="gest_cours.php" class="header_table">N°</a></div></td>
<td><div align="center"><a href="gest_cours.php?tri=intitule" class="header_table">Intitulé</a></div></td>
<td><div align="center">Modif.</div></td>
<td><div align="center">Suppr.</div></td>
</tr>
<?
// Compteur pour couleurs alternées
$i=0;
while($line = mysql_fetch_array($req))
{
// La couleur de fond de la ligne change une fois sur deux
if($i%2!=0) $bgcolor='#9FBED0'; else $bgcolor='#C2D6E2';
$i++;
echo ' <tr bgcolor="'.$bgcolor.'" class="police_std">'."\n";
// Si l'on est en train de modifier une valeur
if($action=='modifier' && $cours_mod==$line['cours'])
{
?>
<form name="modif_cours_form" method="post" action="gest_cours.php?action=modifier_valide&cours_mod=<?=$cours_mod?>">
<td>
<input name="cours" type="text" size="3" value="<?=$line['cours']?>">
</td>
<td>
<input name="intitule" type="text" size="35" value="<?=$line['intitule']?>">
</td>
<td colspan="2">
<div align="center">
<input type="submit" name="Submit" value="Modifier ce cours">
</div>
</td>
</form>
<?
} else

Dj YeLL

$question = $to_be || !$to_be;

Sinon j'ai aussi l'impression que tout est mélangé ... tu devrais peut etre séparer les differentes choses dans plusieurs fichiers, et faire des includes ... (ajout, suppression, edition etc...)

++

Master_Jul

Nouvelle version, suivant vos conseils :

http://masterjul.homeip.net/tp_sql/source_cours.php

Sinon Dj YeLL, je dois tout faire en un seul fichier, et je ne saurai de toute façon pas trop comment dispatcher les tâches dans d'autres fichiers, à part 3 requêtes SQL tout reste assez « collé » à la page.

Message édité par Master_Jul le 15-04-2004 à 13:57:45

---------------
En français, on écrit "connexion", pas "connection".

drasche

ça va pas marcher ton lien

Master_Jul

drasche a écrit :

ça va pas marcher ton lien

Je viens de m'en appercevoir en le collant dans le premier post.

---------------
En français, on écrit "connexion", pas "connection".

Master_Jul

Y'a 3 lignes de code qui ne servent à rien :

if(!$req) echo "Echec de la modification.";

On ne peut pas les atteindre...

---------------
En français, on écrit "connexion", pas "connection".

Dj YeLL

$question = $to_be || !$to_be;

Euh ... si tu mettais ton code dans un TXT on pourrait peut etre le lire ... parceque la, on ne verra que le HTML (c'est tout l'interet du PHP d'ailleurs)

++

Edit : OUPS, j'aurais mieux fait de cliquer avant de parler

Message édité par Dj YeLL le 15-04-2004 à 14:44:56

Publicité

kalex

Si c'est le nombre de ligne qui te tracasse, oui y a moyen de faire plus court.

Par exemple, remplace :

Code :

if($tri=='intitule')
$order_by="ORDER BY intitule";
else
$order_by="ORDER BY cours";

Par

Code :

$order_by = 'ORDER BY '.($tri == 'intitule' ? $tri : 'cours');

Tu peux appliquer ça plusieurs fois dans ton code (ça marche aussi pour return...).

Et les commentaires façon entête sont un peu gros, il veut mieux expliquer au début du fichier ce que tu vas faire et ensuite commenter le minimum (des noms explicites pour les variables et fonctions suffisent souvent).

Enfin, ce n'est que mon avis.

Message édité par kalex le 15-04-2004 à 14:59:31

nico168

"Est-ce normal d'avoir 200 lignes de code pour en afficher 4 ?"

moi je vois pas de pertinence dans le rapport entre le nombre du ligne du code source et le nombre de ligne en sortie.

Code :

<?php while(1) echo "toto<br>"; ?>

Dj YeLL

$question = $to_be || !$to_be;

Pourtant il y en a ... de la pertinence à cette question ...

C'est pas le fait d'avoir 200 lignes de code pour en afficher 4 qui importe ici ....

C'est d'avoir 200 lignes de code alors qu'on pourrait peut-etre en avoir 100 ...

On appelle ça de l'optimisation ...

Suivant ton exemple, si je te met :

echo "t";
echo "o";
echo "t";
echo "o";

Tu crois pas qu'il serait plus interessant de le transformer en echo "toto"; ?

Donc oui, sa question est pertinente

CQFD

++

drasche

disons que la formulation du titre du topic est erronée.

Master_Jul

kalex a écrit :

Si c'est le nombre de ligne qui te tracasse, oui y a moyen de faire plus court.

Par exemple, remplace :

Code :

if($tri=='intitule')
$order_by="ORDER BY intitule";
else
$order_by="ORDER BY cours";

Par

Code :

$order_by = 'ORDER BY '.($tri == 'intitule' ? $tri : 'cours');

Je n'ai pas très bien compris comment marchait cette syntaxe, j'ai essayé de trouver des trucs sur Google à « syntaxe courte » mais je n'ai rien trouvé. Enfin si, juste le echo court.

---------------
En français, on écrit "connexion", pas "connection".

R3g

fonctionnaire certifié ITIL

Master_Jul a écrit :

Je n'ai pas très bien compris comment marchait cette syntaxe, j'ai essayé de trouver des trucs sur Google à « syntaxe courte » mais je n'ai rien trouvé. Enfin si, juste le echo court.

ça s'apelle opérateur ternaire et ça sert à remplacer un if. en gros ça s'écrit condition ? action_si_vrai : action_si_faux

---------------
Au royaume des sourds, les borgnes sont sourds.

kalex

C'est bien ça.
La doc (voir en bas) :
http://www.unixtech.be/docs/php-fr [...] rison.html

Master_Jul

Ok merci ! En fait j'avais du mal à comprendre à cause du $tri, j'aurai mieux compris comme ça :

$order_by = 'ORDER BY '.($tri == 'intitule' ? 'intitule' : 'cours');

---------------
En français, on écrit "connexion", pas "connection".

jagstang

Pa Capona ಠ_ಠ

Master_Jul a écrit :

Ok merci ! En fait j'avais du mal à comprendre à cause du $tri, j'aurai mieux compris comme ça :

$order_by = 'ORDER BY '.($tri == 'intitule' ? 'intitule' : 'cours');

c'est pas de l'optimisation ça... ni pour la machine, et surtout pas pour le lecteur...

Dj YeLL

$question = $to_be || !$to_be;

Surtout si l'intitulé est $tri = "Bienvenue dans la 1ère partie de notre courcernant la symbiose incorrecte entre les lapins géants d'Afrique du Sud et la tourtelle à 3 patte du Corridor Argenté"...

jagstang

Pa Capona ಠ_ಠ

mysql_close();

Tu peux le faire bien avant la fin de la page ! Tout le monde fait la même erreur

kalex

JagStang a écrit :

c'est pas de l'optimisation ça... ni pour la machine, et surtout pas pour le lecteur...

Pour le lecteur, je veux bien (et encore c'est pas parce qu'on le voit rarement en php que ce n'est pas extrêmement utile, justement pour alléger la syntaxe), mais pour la machine si ! Même si ici, les deux manières de faire sont si rapides qu'on ne verra jamais la différence.

Message édité par kalex le 16-04-2004 à 15:17:41

jagstang

Pa Capona ಠ_ಠ

aucune différence à mon sens. PHP étant un language interprété, puis le résultat de celui-ci envoyé via un média instable comme internet, ce genre d'"optimisation" s'apparente _vraiment_ à de l'enc**age de mouche.

a+

Master_Jul

et sinon une petite question conne, au début j'avais fait

//modif
if {}
//ajout
if {}
// suppr
if {}

Et après, je me suis dit que je pouvais mettre d'un point de vu plus "logique" des elseif mais est-ce que celà change quelque chose du point de vu du code et de sa rapidité/façon d'éxécution ?

À priori, je ne vois pas de différence si je ne termine pas la série de elseif par un else, ce dont je n'ai pas besoin ici.

---------------
En français, on écrit "connexion", pas "connection".

jagstang

Pa Capona ಠ_ಠ

attend c'est du PHP là. c'est pas comme si ce code devait être exécuté des milliards de fois...

c'est clair que des if c'est plus long. mets un switch si tu veux.

Mais ça reste de l'optimisation à 2 balles je m'excuse... Le gars devra de toute façon attendre quelques ms avant de recevoir la page par le réseau.

kalex

JagStang a écrit :

Tu dois pas aimer Perl toi ?
Non, ça permet souvent d'ogmenter la lisibilité et la simplicité du code.
Je préfère faire :

Code :

do_func(($var == 'ok' ? TRUE : FALSE), $var2, $var3);

Que :

Code :

if($var == 'ok')
do_func(TRUE, $var2, $var3);
else
do_func(FALSE, $var2, $var3);

C'est peut-être con pour toi, mais pour moi c'est plus clair et logique. Et surtout je ne vais dire à personne, que ne pas faire comme moi est stupide...

De plus que PHP soit un langage interprété est plutôt une bonne chose pour ce genre de truc...

Master_Jul

Je viens de tranposer un return et ça fait quand même beaucoup plus propre [:mlc] :

Avant :

if($line)
{
return false;
}
else return true;

Après :

return ($line) ? false : true;

---------------
En français, on écrit "connexion", pas "connection".

Master_Jul

Le code est mis à jour et je pense pour ainsi dire bouclé, merci à vous. :jap:

http://masterjul.homeip.net/tp_sql/source_cours.php

---------------
En français, on écrit "connexion", pas "connection".

jagstang

Pa Capona ಠ_ಠ

Master_Jul a écrit :

Je viens de tranposer un return et ça fait quand même beaucoup plus propre [:mlc] :

Avant :

if($line)
{
return false;
}
else return true;

Après :

return ($line) ? false : true;

encore mieux. remplace

return ($line) ? false : true;

par

return ($line) ;

Master_Jul

Ça ne serait pas plutôt return(!$line); ?

---------------
En français, on écrit "connexion", pas "connection".

jagstang

Pa Capona ಠ_ಠ

oui bien vu. Alors ça marche ?

Master_Jul

Impéc, merci. :jap:

Bon, je vais passer au gros morceau, la gestion des centres.

---------------
En français, on écrit "connexion", pas "connection".

kalex

Ça commence à devenir bien propre tout ça.
Encore un enculage de mouches :
return (!mysql_fetch_array($req));

Master_Jul

Je viens de pondre la gestion des centres, préparez-vous psychologiquement avant de regarder le code :

http://masterjul.homeip.net/tp_sql/gest_centres.php
http://masterjul.homeip.net/tp_sql/source_centres.php

Déjà, je ne vais faire qu'une fonction pour le changement de date, en utilisant un second argument.

Message édité par Master_Jul le 17-04-2004 à 11:13:51

---------------
En français, on écrit "connexion", pas "connection".

Publicité

FORUM HardWare.fr

Programmation

PHP

Est-ce normal d'avoir 200 lignes de code pour en afficher 4 ?

Sujets relatifs
Extraire le code ISRC d'un CD audio	[CSS]Pourriez vous m'aider pour ce code (CSS ELITE)
Afficher les dernières entrée d'une table ...	trouver son temps de connexion internet et l'afficher
[PHP] Faire son code source en unicode...	[MFC] Afficher une seconde vue sur un document besoin d'aide
un code écrit en assembleur est-il exécutable sur n'importe quel OS ?	[pda pocket pc C#] code touche
[PHP] Probleme pour afficher un champ (bd) dans une page PHP	afficher les sous ensembles d un ensembles. (gray inside)
Plus de sujets relatifs à : Est-ce normal d'avoir 200 lignes de code pour en afficher 4 ?

Page générée en 0.135 secondes