G-SAV - LE GESTIONNAIRE DES MAINTENANCES INFORMATIQUES
cs_guismo1er
Messages postés
76
Date d'inscription
vendredi 21 mars 2003
Statut
Membre
Dernière intervention
12 mars 2009
-
16 déc. 2008 à 18:56
VicTeams Messages postés 63 Date d'inscription vendredi 20 janvier 2006 Statut Membre Dernière intervention 27 février 2011 - 7 févr. 2009 à 11:00
VicTeams Messages postés 63 Date d'inscription vendredi 20 janvier 2006 Statut Membre Dernière intervention 27 février 2011 - 7 févr. 2009 à 11:00
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/48720-g-sav-le-gestionnaire-des-maintenances-informatiques
https://codes-sources.commentcamarche.net/source/48720-g-sav-le-gestionnaire-des-maintenances-informatiques
7 févr. 2009 à 11:00
6 févr. 2009 à 11:43
17 janv. 2009 à 10:57
Le forum ne fonctionne plus, dommage ...
22 déc. 2008 à 12:07
Je comprends bien que tu ne cherches pas à le descendre et que tu préfères qu'il se dépasse, progresse, etc. Mais comme tu dis, chacun sa méthode. D'autres se sont déjà vu faire des réflexions sur leur manière de s'exprimer dans les commentaires. Il ne s'agissait pas du fond, mais de la forme... Là, c'est un peu ça... Mets la forme ;)
@Girardeau : On peut trouver pas mal de tutos et d'articles sur le sujet, un peu partout sur le net.
http://www.google.fr/search?q=convention+codage+php
L'important quand on écrit du code est de penser à la personne qui le lira plus tard. A partir de là, le bon sens devrait suffire...
22 déc. 2008 à 10:00
- 1 Oui, c'est bien, bravo, mais tu devrais …
- 2 Nul, fait mieux et revoir me voir après.
Le but n'est pas de décourager, mais de Booster.
Si êtes franc avec lui, il n’est pas possible de dire que c’est bien. J’ai rencontré bien des nanars en informatique pour savoir qu’il existe des causes perdues d’avance. A lui de me montrer le contraire, je serais très heureux de voir que je me suis trompé, comme pour NLION qui à bien rebondi après mon agression et qui à finaliser un très bon travail.
Chacun sa méthode.
- Un template payant n’est pas forcement un bon template.
- Prends le temps de survoler les fonctions PHP : www.php.net
Prends un exemple de recherche.php
if ($search NULL or $search '%'){
} else {
…/…
}
Tu ne sais pas inverser une condition ? … Je ne sais pas quoi en penser …
Sinon, voici la réponse simple (a + b)\ = a\ . b\ :
If ( $search <> NULL and $search <>’%’)
Victeams, étonnes-moi ! Et je te ferais de plates excuses, promis.
22 déc. 2008 à 09:27
Je viens de lire cet échange de réponses sur la bonne orthographe mais aussi sur la bonne écriture du code PHP.
Etant un débutant qui cherche à progresser, existe t-il une FAQ regroupant les régles pour l'écriture du code php mais aussi sur les bons usages en matière de programmation
Merci par avance
Laurent Girardeau
21 déc. 2008 à 18:17
setlocale(LC_TIME, 'fr_FR');
Bon je dis ça de mémoire...
Il faut savoir qu'on peut cumuler les noms de langues, comme autant de paramètres à la fonction, ou alors dans un tableau.
setlocale utilisera alors la première langue qui sera reconnue sur le système.
En gros, on peut faire comme ça :
setlocale(LC_TIME, 'FRA', 'fra', 'fr', 'fr_FR');
http://fr.php.net/setlocale
21 déc. 2008 à 14:02
le setlocale() est paramétré pour windows, il me semble que c'est différent pour les autres OS.
Kohntark-
21 déc. 2008 à 13:59
Je viens de jeter un oeil à ton code.
Voici quelques remarques non exhaustives :
- ton code HTML date un peu :
tu devrais utiliser les feuilles de style
les tableaux ne sont pas fait pour faire de la mise en page
Tu devrais revoir tous tes include :
il y a include("includes/header.php"); et require("includes/config.php"); dans
index.php, hors header.php fait déjà un include de config.php
Même chose dans fonction_suppr_fiche.php :
<?php
include("includes/header.php");
include("includes/config.php"); // déjà inclus dans header.php
?>
... et un peu partout
Tu devrais revoir la gestion des connexions que tu fais au serveur mySql :
Tu te connectes dans index.php alors que tu l'as déjà fait en début de
script avec header.php (qui inclus config.php)
Tu utilises mysql_close() = > ça ne sert à rien, la connexion étant
fermée à la fin du script
Je suis d'accord pour ne pas traiter la sécurité sur une appli comme celle là, à condition, bien sur, qu'elle ne soit pas accessible de l'extérieur ou
qu'elle soit protégée par un htaccess. Par contre tu devrais contrôler tous les champs de ton formulaire, non pas contre les injections SQL, mais
pour éviter les erreurs de saisie. Rien n'empêche d'entrer "tartanpion" dans le champ numéro de téléphone, ou d'oublier de remplir un champ (ou tous d'ailleurs)
Tu ne fais quasiment aucun contrôle des erreurs qui peuvent survenir !!
Il y a des centaines de lignes comme celle là :
echo "<td width="200" height="100" class="texte" valign="top"> N° :
$id</td>";
=> echo '<td width= "200" height="100" class="texte" valign="top"> N° :
'.$id.'</td>';
... et puis tu n'es pas obligé de faire un echo pour chaque ligne, un seul suffit (fiche.php, fiche_list.php, ...)
Ton code n'est pas correctement indenté
???
$query ="SELECT * FROM fiche ORDER BY nom ";
$result= mysql_query($query);
$num = mysql_num_rows ($result);
mysql_close();
while ($i < $num) {
$nom = mysql_result($result,$i,"nom");
$prenom = mysql_result($result,$i,"prenom");
$total = mysql_result($result,$i,"total");
$id = mysql_result($result,$i,"id");
$id = mysql_result($result,$i,"id");
?>
<table border="0"class="table" class="texte">
[...]
++$i;
}
=> c'est lourd et "complexe" sans raison !if (false $query mysql_query("SELECT nom, prenom, total, id FROM fiche ORDER BY nom")) die('erreur de la requête ...');
while ($result = mysql_fetch_array($query, MYSQL_ASSOC)) {
echo ' Nom : $result['nom'].'
';
//etc ...
}
La modification d'un numéro de téléphone ne fonctionne pas
Pourquoi ces 30 lignes de code ?
$day_l = date("l");
$mois = date("F");
$day_f = date("d");
$year = date("Y");
switch ($day_l)
{
case "Monday" : $day_l2 = "Lundi"; break;
case "Tuesday" : $day_l2 = "Mardi"; break;
case "Wednesday" : $day_l2 = "Mercredi"; break;
case "Thursday" : $day_l2 = "Jeudi"; break;
case "Friday" : $day_l2 = "Vendredi"; break;
case "Saturday" : $day_l2 = "Samedi"; break;
case "Sunday" : $day_l2 = "Dimanche"; break;
}
switch ($mois)
{
case "January" : $mois2 = "Janvier"; break;
case "February" : $mois2 = "Février"; break;
case "March" : $mois2 = "Mars"; break;
case "April" : $mois2 = "Avril"; break;
case "May" : $mois2 = "Mai"; break;
case "June" : $mois2 = "Juin"; break;
case "July" : $mois2 = "Juillet"; break;
case "August" : $mois2 = "Aôut"; break;
case "September" : $mois2 = "Septembre"; break;
case "October" : $mois2 = "Octobre"; break;
case "November" : $mois2 = "Novembre"; break;
case "December" : $mois2 = "Décembre"; break;
}
[...]
<?php echo("$day_l2 $day_f $mois2 $year"); ?>
à remplacer par ces 2 lignes :
setlocale(LC_TIME, "fra");
echo strftime("Nous sommes le %A %d %B %Y");
... et ce n'est qu'un coup d'oeil, il y a encore de très nombreuses erreurs :)
Bon courage,
Kohntark-
20 déc. 2008 à 22:21
... étonnant d'ailleurs quand on lit les autres posts de Spipod.
Menfin je n'en fais pas une maladie, il faut juste rappeler que chacun a son niveau, et tout le monde ici est passé par la case débutant. Alors envoyer sur les roses les neuneus (et il y en a !!): OUI; décourager les personnes qui veulent apprendre et qui font des efforts: NON !
Kohntark-
P.S. 2 : j'ai essayé de faire le premier commentaire sans faute de la page lol...
Loupé !! ;)
"Je ne peut qu'approuver" => Je ne peux qu'approuver
20 déc. 2008 à 21:45
Pour ma part, je n'ai même pas regardé une seule ligne de code... J'ai juste envie de réagir sur des questions de fond qui ressortent de cette discussion.
Concernant l'orthographe, tout d'abord, je ne prendrai pas position. Je me contenterai de faire remarquer que les propos de Teclis concernant l'orthographe de VicTeams représentent 2 lignes sur... un certain nombre (qui varie suivant la résolution de l'écran... et j'ai la flemme de compter les mots). S'attarder là-dessus comme c'est fait me semble un peu exagéré : il n'a pas non plus disserté sur la nécessité d'apporter beaucoup de soin à l'orthographe (des correcteurs orthographiques existent, on n'est ici pas sur un chat, on a le temps de les utiliser si besoin est). Quant aux réactions en elles-mêmes, si je les trouve un peu exagérées, leurs propos restent tout à fait justes et pertinents : VicTeam ne fait pas d'erreurs de syntaxe, on comprend donc ce qu'il dit.
Bref. Le mieux serait sûrement de mettre de côté l'orthographe... Au lieu de trop s'attarder dessus, indiquer à VicTeam quelles erreurs il a faites me paraît plus judicieux.
Bon... Sur phpcs, on trouve toutes sortes de sources. Des bonnes (notamment celles de Malalam, Codefalse, Coucou747, pour ne citer que les principaux habitués), et des mauvaises (pas de liste, ça ferait beaucoup).
On (ou plutôt les admins) pourrait supprimer les sources de mauvaise qualité (bon celles qui sont vraiment trop pourries en général ne font pas long feu). Mais quel serait alors l'intérêt pour leur auteur ? Qu'en retirerait-il sinon qu'il s'est fait détruire ?
Au contraire, ici, on (et là je parle de tout le monde) privilégie (en général...) l'aspect didactique (avec plus ou moins de diplomatie, mais c'est une autre question). C'est à dire aider l'auteur d'une source à l'améliorer quand il a fait des erreurs relatives :
- à la sécurité
- aux bonnes pratiques de programmation
- à la syntaxe et/ou la lisibilité du code (indentation, commentaires, etc)
Le fait qu'on trouve de tout sur phpcs est à la fois un défaut et une qualité pour le site... Tout dépend ce qu'on cherche en tant qu'auteur de source, ou en tant qu'utilisateur...
Je ne peut qu'approuver à 200% la fin du commentaire de Zeguizmo... On n'a pas du tout le même point de vue quand on débarque ici pour poster une source et quand on y passe ses journées (tout est relatif...) depuis longtemps.
@Kohntark : je suis en partie d'accord avec le dernier paragraphe de ton commentaire. Je suis d'accord sur le fait que la réponse dans sa forme n'est pas vraiment correcte... Mais sur le fond, rien à dire... Si on me dit après 3 semaines de boulot intense sur quelque chose que c'est particulièrement nul (de manière diplomatique), je saurai à quoi m'en tenir : ça fera mal, mais ce sera honnête (si tant est que ce soit justifié, ce qu'a rapidement fait Spipod).
Ce qui nous ramène au commentaire de Zeguizmo... Ici, les critiques sont rarement méchantes gratuitement (ça arrive, hein, personne n'est parfait).
Si j'ai un peu de temps demain, je jette un oeil au code source pour lister un peu quelques défauts et proposer des améliorations (ou au moins des pistes ou des idées). Je vous invite à faire de même :)
Bonne soirée, bon week end
P.S. : j'espère que ça fait pas trop le gars qui fait la morale, c'est pas du tout le but... La discussion est intéressante et peut même soulever un débat de fond très... intéressant également (désolé pour la répétition).
P.S. 2 : j'ai essayé de faire le premier commentaire sans faute de la page lol... J'ai pu laisser échapper des fautes de frappe ou d'inattention...
20 déc. 2008 à 10:56
Combattre le langage SMS est certes louable sur le net, mais il ne faut pas pousser le bouchon trop loin, là ce n'est pas du langage SMS.
=> tout à fait d'accord, "descendre" une personne comme Victeams qui semble faire des efforts d'orthographe et de grammaire est franchement limite, nous ne sommes pas sur un forum de "bonne pratique de la langue Française".
Autant j'adhère à ce genre de réaction lorsqu'un intervenant s'exprime délibérément dans un langage limite compréhensible, autant sur ce coup je ne suis pas d'accord.
"il y a toujours et encore des fautes" => "il" s'écrit "Il", c'est un début de phrase, il y a donc une majuscule.
Franchement, à quoi même ce genre de réflexion ?
... il n'y a pas un post sans 1 faute sur cette page !!
Pour ce qui est de "... C'est de la crotte.", "Allez, 1 / 10 pour l'effort de publier." ce n'est même plus limite, c'est incorrect. Que dirais tu si, dans un autre domaine, VicTeams t'adressait cette remarque sur un travail qui t'a pris 3 semaines ?
Cordialement,
Kohntark-
18 déc. 2008 à 09:35
18 déc. 2008 à 01:06
Combattre le langage SMS est certes louable sur le net, mais il ne faut pas pousser le bouchon trop loin, là ce n'est pas du langage SMS. Il y a des gens pour qui l'orthographe n'est pas une chose facile. J'ai personnellement côtoyé un véritable génie des matières scientifiques, survolant sans effort les programmes de classe prépa où les plus balèzes se pètent les dents, et pourtant, il n'était pas capable d'écrire une phrase sans faire de fautes. Il a beaucoup travaillé là dessus, mais rien à faire... et de la matière grise, il en a a revendre. Depuis ce temps je regarde les gens qui font des fautes avec un oeil totalement différents des gens qui écrivent avec un langage SMS, qui eux, sont de véritables porcs.
Désolé pour le hors sujet mais je commence a voir de plus en plus de réactions et de plus en plus violentes contre l'orthographe qui me semblent injustifiées.
Pour la source, les réactions sont malheureusement justifiées. C'est le problème de ce site, il y a beaucoup de débutant qui on eu le mérite d'apprendre tout seul le langage, de travailler dur pour débuguer leur appli (qui est souvent la première et donc... je n'en dirai pas plus) et qui sont tout contents de partager leur travail. Erreur fatale :) Ceux qui vous lisent sont chevronnés et en ont marre de rabâcher 1000 fois la même chose. Faut pas leur en vouloir, sont pas méchants ^^ Juste prendre les commentaires techniques et surtout modifier la source, pour pas laisser traîner de mauvais exemples sur le site.
Bon courage,
Guiz
17 déc. 2008 à 21:27
si tu as acheté le template, c'est qu'il est soumis certainement à un copyright as tu vérifié que tu avais le droit de le redistribuer (et ce gratuitement)?
17 déc. 2008 à 16:17
Mais les fautes, certes, je fais de mon mieux, pour sa que j'ai créer un forum pour les différents problèmes.
17 déc. 2008 à 10:12
On voit bien que tu as fait ça pour toi, et non pas pour en faire profiter quelqu’un. Tu as déposé ton source pourquoi ?
Je vois une grande utilité à ton programme, c’est l’exemple à ne pas suivre.
C’est pas propre, il n’y a pas de test correct (isset, empty …/…), des fonctions ‘fonction_creation_fiche.php’ qui n’en sont pas. Du code HTML mal structuré, il y a des balises mises en vrac : après un </TD>
….. C’est de la crotte. Il faudrait une journée complète pour te lister les anomalies.
Bref, tu comprends que ce n’est pas présentable comme projet.
Allez, 1 / 10 pour l’effort de publier.
16 déc. 2008 à 20:34
L'orthographe non plus de toute évidence ...
Plus franchement le code est crade, pas indenté, des solutions existes et beaucoup plus élégantes... donc script inutile autant pour les débutants pour leur donner des mauvaises habitudes que pour ceux qui ont besoin d'un gestionnaire de ticketing qui s'arracheront les cheveux en relisant le code.
16 déc. 2008 à 18:56