G-SAV - LE GESTIONNAIRE DES MAINTENANCES INFORMATIQUES

Commentaire de guismo1er le 16/12/2008 18:56:17

quand tu fais une recherche, faudrait faire un lien pour modifier ;)

Commentaire de spipod le 17/12/2008 10:12:39

Tout à fait d'accord avec TECLIS01.

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 : </FONT> 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.

Commentaire de Teclis01 le 17/12/2008 21:27:17

il y a toujours et encore des fautes. Pour ce qui est de la source, ne t'es tu pas dit que si cela t'avais prit 3 semaines en autodidacte, ça en prenais beaucoup moins en réalité ? Enfin toujours est il que cette source est posée. Gratuit et open source hum hum... je ne disserterais pas là dessus mais moi quand je poste c'est avec le c½ur... Mes sources je les offre et c'est de l'open source après chacun sa politique mais gratuit ça n'a rien a faire là si tu veux garder des droits dessus (avis personnel).
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)?

Commentaire de VicTeams le 18/12/2008 09:35:29

Merci pour vos réactions, j'ai beau examiner mon code dans tout les sens je ne voie pas ce qui cloche, c'est, plus ou moins, comme EsbaseAdmin, car je me suis baser sur ce code.... Ais-je fait une erreur ?

Commentaire de neigedhiver le 20/12/2008 21:45:52

Bonjour bonjour,

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...

Commentaire de kohntark le 21/12/2008 13:59:55

Salut,


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\"><b>N° :</b></font><br>$id</td>";
=> echo '<td width="200" height="100" class="texte" valign="top"><b>N° :</b></font><br>'.$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 '<b>Nom :</b> $result['nom'].'<br>';
  //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-

Commentaire de neigedhiver le 21/12/2008 18:17:51

Effectivement, pour setlocale, sur Linux/Unix c'est :
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

Commentaire de spipod le 22/12/2008 10:00:12

Il existe deux méthodes :

- 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.

Commentaire de gn1234567 le 17/01/2009 10:57:29

Bonjour,

Le forum ne fonctionne plus, dommage ...