Que pensez vous de ce code

ZGoblin · Février 13, 2006, 2:54

Voici un bout de code sur lequel je dois travailler, j’aimerai vous demander votre avis, ca vient de moi où c’est plus agréable de se faire pendre en public que de travailler avec ce code ?

[codebox]
Sub spe1(WS_EnTraitement As Worksheet)

Dim i As Long
Dim j As Long
Dim NNI As String
NNI = "0"
For i = 9 To Fct.F_L_LastRow(ActiveSheet)
WS_EnTraitement.Activate

  If NNI <> ThisWorkbook.Worksheets("Temp").Cells(i, "D").Text Then
     NNI = ThisWorkbook.Worksheets("Temp").Cells(i, "D").Text
     If CTRL.F_B_ControleFormatDateJJMMAAAA(

ThisWorkbook.Worksheets(“Temp”).Cells(i, “A”).Text) Or
ThisWorkbook.Worksheets(“Temp”).Cells(i, “A”).Text <> “” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “A”, “La date n’est pas valide”,
“Date de Demande”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “A”).Text,
“La date doit etre renseignée au format JJMMAAAA”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If Not Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “B”).Text) = 4 Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “B”, “Le domaine n’est pas valide”, “Domaine”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “B”).Text,
“Le domaine doit etre du texte sur 4 caractéres”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If Not (Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “C”).Text) = 4 Or
ThisWorkbook.Worksheets(“Temp”).Cells(i, “A”).Text = “”) Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “C”, “Le sous domaine n’est pas valide”,
“Sous Domaine”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “C”).Text,
“Le domaine doit etre du texte sur 4 caractéres”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If (Not (Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text) <= 12 And
(Left(ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text, 1) = “9” Or
Not IsNumeric(Left(ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text, 1))))) Or
ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text = “” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”, WS_EnTraitement.Name,
CStr(i), “D”, “Le NNI n’est pas valide”, “NNI”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text,
"Le NNI doit etre du texte sur 12 caractéres et commencer par une lettre ou un ““9"””,
“Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If (Not Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “E”).Text) <= 40) Or
ThisWorkbook.Worksheets(“Temp”).Cells(i, “E”).Text = “” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”, WS_EnTraitement.Name,
CStr(i), “E”, “Le nom n’est pas valide”, “Nom”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “E”).Text,
“Le nom doit etre du texte sur 40 caractéres maxi”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If (Not Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “F”).Text) <= 40) Or
ThisWorkbook.Worksheets(“Temp”).Cells(i, “F”).Text = “” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “F”, “Le prenom n’est pas valide”,
“Prenom”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “F”).Text,
“Le nom doit etre du texte sur 40 caractéres maxi”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If Not Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “G”).Text) <= 10 Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “G”, “Le prenom n’est pas valide”,
“Bureau”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “G”).Text,
“Le Bureau doit etre du texte sur 10 caractéres maxi”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If Not Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “H”).Text) <= 10 Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “G”, “Le prenom n’est pas valide”,
“Bâtiment”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “H”).Text,
“Le Batiment doit etre du texte sur 10 caractéres maxi”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If Not CTRL.F_B_COM_NumeroTelephoneFrance(
ThisWorkbook.Worksheets(“Temp”).Cells(i, “I”).Text, " ") Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “I”, “Le telephone n’est pas valide”,
“Téléphone”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “I”).Text,
“Le Téléphone n’est pas au bon format”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If

     If Not CTRL.F_B_COM_NumeroTelephoneFrance(

ThisWorkbook.Worksheets(“Temp”).Cells(i, “J”).Text, " ") Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “J”, “La telecopie n’est pas valide”,
“Télécopie”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “J”).Text,
“La Télécopie n’est pas au bon format”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If

     If Not CTRL.F_B_COM_AdresseMail(

ThisWorkbook.Worksheets(“Temp”).Cells(i, “K”).Text) Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “K”, “Le mail n’est pas valide”, “Mail internet”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “K”).Text, “Le Mail n’est pas valide”,
“Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If

     If Not CTRL.F_B_DATE_FormatJJMMAAAA(

ThisWorkbook.Worksheets(“Temp”).Cells(i, “L”).Text) Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “L”, “La Date n’est pas valide”, “Date début”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “L”).Text,
“La date n’est pas valide format attendu JJMMAAA”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
ElseIf Trim(ThisWorkbook.Worksheets(“Temp”).Cells(i, “L”).Text) = “” And
Left(ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text, 1) = “9” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”, WS_EnTraitement.Name,
CStr(i), “L”, “La Date est Obligatoire pour ce NNI”, “Date début”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “L”).Text,
“La date est obligatoire pour les Interim et prestataire”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If

     If Not CTRL.F_B_DATE_FormatJJMMAAAA(

ThisWorkbook.Worksheets(“Temp”).Cells(i, “M”).Text) Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”, WS_EnTraitement.Name,
CStr(i), “M”, “La Date n’est pas valide”, “Date Fin”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “M”).Text,
“La date n’est pas valide format attendu JJMMAAA”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
ElseIf Trim(ThisWorkbook.Worksheets(“Temp”).Cells(i, “M”).Text) = “” And
Left(ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text, 1) = “9” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “M”, “La Date est Obligatoire pour ce NNI”, “Date début”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “M”).Text,
“La date est obligatoire pour les Interim et prestataire”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If Not CTRL.F_B_DATE_DateInferieurA(
ThisWorkbook.Worksheets(“Temp”).Cells(i, “L”).Text,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “M”).Text) Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”, WS_EnTraitement.Name,
CStr(i), “M”, “La Date de debut doit etre inferieur a celle de fin”, “Date début”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “M”).Text, “Les dates ne sont pas coherante”,
“Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If

     If Not (ThisWorkbook.Worksheets("Temp").Cells(i, "N").Text <> "") Then
        Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal("LOG", WS_EnTraitement.Name,

CStr(i), “N”, “L’Adresse n’est pas valide”, “Adresse”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “N”).Text,
“L’adresse est obligatoire et doit etre du texte sur 42 caractéres”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If

     If Not AppRange("adressListe", ThisWorkbook.Worksheets("Temp").Cells(i, "N").Text) Then
        Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal("LOG",

WS_EnTraitement.Name, CStr(i), “N”, “L’Adresse n’est pas valide”, “Adresse”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “N”).Text,
“L’adresse doit appartenir a la liste”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If

     If AppRange("listee", ThisWorkbook.Worksheets("Temp").Cells(i, "O").Text) > 12 Or

ThisWorkbook.Worksheets(“Temp”).Cells(i, “N”).Text = “” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “O”, “Le DA n’est pas valide”,
“Parametre domaine d’activité”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “O”).Text,
“Le DA est obligatoire et doit etre du texte sur 12 caractéres”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
NNI = ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text
Else

     If (Not (Len(ThisWorkbook.Worksheets("Temp").Cells(i, "D").Text) <= 12 And

(Left(ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text, 1) = “9” Or
Not IsNumeric(Left(ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text, 1))))) Or
ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text = “” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”, WS_EnTraitement.Name,
CStr(i), “D”, “Le NNI n’est pas valide”, “NNI”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “D”).Text,
"Le NNI doit etre du texte sur 12 caractéres et commencer par une lettre ou un ““9"””,
“Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If (Not Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “E”).Text) <= 40) Or
ThisWorkbook.Worksheets(“Temp”).Cells(i, “E”).Text = “” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”, WS_EnTraitement.Name,
CStr(i), “E”, “Le nom n’est pas valide”, “Nom”,
ThisWorkbook.Worksheets(“Temp”).Cells(i, “E”).Text,
“Le nom doit etre du texte sur 40 caractéres maxi”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
If (Not Len(ThisWorkbook.Worksheets(“Temp”).Cells(i, “F”).Text) <= 40) Or
ThisWorkbook.Worksheets(“Temp”).Cells(i, “F”).Text = “” Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(i), “F”, “Le prenom n’est pas valide”,
“Prenom”, ThisWorkbook.Worksheets(“Temp”).Cells(i, “F”).Text,
“Le nom doit etre du texte sur 40 caractéres maxi”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If

  End If

Next i

For i = 15 To Fct.F_L_LastCol(ThisWorkbook.Worksheets(“Temp”))
For j = 9 To Fct.F_L_LastRow(ThisWorkbook.Worksheets(“Temp”))

     If ThisWorkbook.Worksheets("Temp").Cells(j - 1, "D").Text <>

ThisWorkbook.Worksheets(“Temp”).Cells(j, “D”).Text Then
If (ThisWorkbook.Worksheets(“Temp”).Cells(6, i).Text = CST_GestVent Or _
ThisWorkbook.Worksheets(“Temp”).Cells(6, i).Text = CST_Appro1 Or _
ThisWorkbook.Worksheets(“Temp”).Cells(6, i).Text = CST_Appro2)
And (ThisWorkbook.Worksheets(“Temp”).Cells(j, Fct.F_S_ColNumber2String(i)).Text
<> “”) Then

           If ThisWorkbook.Worksheets("Temp").Cells(j, "I").Text = "" Then

              Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal("LOG",

WS_EnTraitement.Name, CStr(j), Fct.F_S_ColNumber2String(i),
“Pour les Appro et les gestionnaires le num de tel doit etre present”,
ThisWorkbook.Worksheets(“Temp”).Cells(6, i).Value,
ThisWorkbook.Worksheets(“Temp”).Cells(j, Fct.F_S_ColNumber2String(i)).Text,
“renseigner les colonnes obligatores”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1

           End If

        End If


        If ThisWorkbook.Worksheets("Temp").Cells(6, i).Text = CST_Prescrip And

(ThisWorkbook.Worksheets(“Temp”).Cells(j, Fct.F_S_ColNumber2String(i)).Text <> “”) Then

           If ThisWorkbook.Worksheets("Temp").Cells(j, "P").Text = "" _
              Or ThisWorkbook.Worksheets("Temp").Cells(j, "Q").Text = "" _
              Or ThisWorkbook.Worksheets("Temp").Cells(j, "R").Text = "" _
              Or ThisWorkbook.Worksheets("Temp").Cells(j, "S").Text = "" Then

              Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal("LOG",

WS_EnTraitement.Name, CStr(j), Fct.F_S_ColNumber2String(i),
“Pour les prescripteurs les champs P,Q,R et S doivent etre renseignés”,
ThisWorkbook.Worksheets(“Temp”).Cells(6, i).Value,
ThisWorkbook.Worksheets(“Temp”).Cells(j, Fct.F_S_ColNumber2String(i)).Text,
“renseigner les colonnes obligatores”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1

           End If

        End If

     End If

     If ThisWorkbook.Worksheets("Temp").Cells(j, i).Text <> "" Then
        If Not AppListe(ThisWorkbook.Worksheets("Temp").Cells(j, i).Value,

Fct.F_S_ColNumber2String(i)) Then
Call ERREUR.P_Ecrire_Ligne_Log_FichierLocal(“LOG”,
WS_EnTraitement.Name, CStr(j), Fct.F_S_ColNumber2String(i),
“La données n’est pas dans la liste”,
ThisWorkbook.Worksheets(“Temp”).Cells(6, i).Value,
ThisWorkbook.Worksheets(“Temp”).Cells(i, Fct.F_S_ColNumber2String(i)).Text,
“La donnée doit appartenir à la liste”, “Bloquant”, Fct)
NB_ERREUR = NB_ERREUR + 1
End If
End If
Next j
Next i
End Sub
[/codebox]

Je précise, qu’il ne s’agit que d’une partie d’un programme mais que le reste est du même genre, je ne donne pas le contexte pour des raisons de confidentialité.

benur · Février 13, 2006, 3:01

je crois que j ai arrete de lire a la 5eme ligne.
Tout ce que je peux te souhaiter c’est bon courage

Bussiere · Février 13, 2006, 3:04

des variables avec des noms qui ne correspondent a rien et pas de commentaires …

Bonne chance
Bussiere

PERECil · Février 13, 2006, 3:45

J’ai vu pire… au moins chez toi c’est indenté et “presque lisible”.

ZGoblin · Février 13, 2006, 3:53

[quote]Cil’ date=’ 13 Feb 2006, 16:45’ post=‹ 445161 ›]
J’ai vu pire… au moins chez toi c’est indenté et « presque lisible ».[/quote]

Pour l’indentation, j’ai un outil qui s’en occupe (heureusement). Sinon, ce qui me lourde vraiment, c’est : les noms de variables/procédures/constantes qui ne respectent aucunes règles de nommage, les multiples for et if imbriqués, le manque de commentaires et le fait que l’auteur de ce code m’a fait remarquer que, je cite :

ElRed · Février 13, 2006, 4:01

“Fait ce que je dis, ne fait pas ce que je fais”…

d’un autre coté il m’est arrivé de coder comme cela , j’avais 16 ans, une collique et j’étais le seul au courant d’un complot visant à asseoir la supremacie des cornichons dans le pays des potagers…

nolan · Février 13, 2006, 4:26

Et sinon le code il marche?

ZGoblin · Février 13, 2006, 4:33

Comment dire, …, non.

Je dois le modifier et je ne sais même pas par où commencer, je vais attendre demain matin pour être un peu plus motivé, là il est trop tard.

BodySplash · Février 13, 2006, 4:36

Ca pue le code écrit sur le pouce pendant la pause déjeuner pour un non informaticien ayant appris vba dans un lavomatique en attendant ses fringues

Phil · Février 13, 2006, 4:54

[quote=« ZGoblin, post:5, topic: 27493 »]quote Cependant, il (moi) doit prêter plus d’attention au respect des normes de développement établies sur le projet, le non respect de ces normes rendant plus difficile la maintenance du code. Les développements réalisés sur le projet sont en effet amenés à être mutualisés et réutilisés, le non respect des normes expose au risque de ne pouvoir maintenir les outils.[/quote] :P[/quote]C’est ce qu’on te reproche ?! On croit rêver surtout si le monsieur est l’auteur de ce code pourri

En voyant le code c’est : comment écrire un code qui devrait faire 10 lignes propres en 158 lignes répetitives.

Ne pètes pas les plombs: au bout du compte ça sera toujours de ta faute surtout si le mec est de mauvaise foi patente. Et donc si tu n’arrêtes pas râler, avec raison quand je vois ce code et d’autres trucs que tu as raconté, tu vas passer pour l’emmerdeur de service. Courage. En tout cas pour ne pas te sentir seul, on a tous rencontré des cas comme cela…et dans ces cas extrêmes tu as toutes les chances de te faire planter car ils sont de très mauvaises foi et n’ont aucune auto-critique.

ZGoblin · Février 13, 2006, 4:56

Pour faire avancer les choses et dire autre chose que du mal car finalement, je m’entend bien avec la personne si il ne regarde pas mon code et inversement je vais donner quelques explications.

Je comprend très bien que l’on code mal pour des raisons de timing. La plupart du temps, il faut aller vite, le travaille est toujours urgent, et ce code a été écrit dans ce contexte, une personne ne faisait pas son travail correctement a pénalisé tout le monde, y compris le développeur qui a écrit ce code en plusieurs fois à l’arrache sans trop savoir où il allait.
Ca je comprend, ce que je déplore, c’est que c’est devenu une habitude et que la personne en question trouve plus lisible un code regroupé dans un unique module qui se lit du début à la fin en bloc.

Non ! , j’arrive parfaitement à travailler dans les délais en faisant un code mieux organisé, plus propre où je prend le temps de commenter et de bien tout séparer dans plusieurs modules. Même si au début c’est plus long, je regagne le temps perdu par la suite lorsque je dois effectuer des modifications ou reprendre un code pour faire des traitements similaires.

C’est vrai que pour quelqu’un d’extérieur, il faut plus le temps de comprendre l’utilité de chaque module plutot que de voir en gros ce que fait une unique méthode énorme comme dans l’exemple. Mais si on nomme correctement les modules, les variables et les procédures, ca ne doit pas poser de problèmes où alors le mec est vraiment une quiche.

Pensez aux autres développeur, prenez le temps de commenter, de réfléchir, de bien choisir les noms utilisés dans vos prog, de bien organiser le code, de respecter des normes de nommages, c’est vraiment important.

Zekiller · Février 13, 2006, 8:31

Oh j’en ai vu du code de porc, ça oui

3500 lignes pour générer un rapport en html. Si.
Une page web avec le même recopié 3 fois en entier. Une fois en readonly, une fois en modifiable, et une fois en gris. J’en pleure toujours.
Une fonction en PL/SQL avec 150 parametres, il avait recopié 15 fois les même 10 parametres avec des chiffres à la fin. Devait pas connaitre les LOOP.

Bref … je pourrais continuer un moment, des fois ça fait tellement pitié que tu te croirais presque « bon programmeur » :P.

Phil · Février 14, 2006, 10:10

[quote=“ZGoblin, post:11, topic: 27493”]Pour faire avancer les choses et dire autre chose que du mal car finalement, je m’entend bien avec la personne si il ne regarde pas mon code et inversement :P[/quote]mouais. Autant je ne prends pas mal de me faire planter par une personne si j’ai fait une erreur, autant j’aurais plus de mal à me faire planter, en plus par écrit, pour un travail que j’estime bien fait surtout par une personne avec qui je m’entends bien.

coccobill · Février 14, 2006, 10:19

Bienvenue dans le monde de la maintenance informatique

Plaignez-vous, vous n’avez pas eu à maintenir du code C manifestement écrit par un developpeur cobol.

Mon plus mauvais souvenir, une variable globale qui s’appelle « liste ». Va rechercher dans tout le code toutes les occurence du mot « liste ». Va chercher bonheur.

Et puis je suis plus indulgent maintenant avec l’expérience. Moi même j’ai commis quelques bouses, mais quand le marketing considère les specs comme un truc qu’on peut changer tout le temps, ou demande des modifs qui n’ont rien à voir avec ce qui est écrit. C’en devient même interessant d’essayer de comprendre comment un code est devenu une bouse au gré des changements de developpeur, des surcouche ajoutées, des modifs de dernière minute.

« On fait ça vite fait dans un coin on refera propre quand on aura le temps ». HA HA HA.

GloP · Février 14, 2006, 10:27

Ca, c’est la phrase a mettre dans un cadre avec un lisere dore. C’est genial

ZGoblin · Février 14, 2006, 10:45

Grave !

En fait, je n’ai pas tout mis, [mode ma vie]Ce matin, j’ai eu une évaluation, c’est à dire que mon chef et une autre responsable m’ont présenté oralement le document (celui qui contenait le paragraphe ci-dessus) qui explique ce qu’il pense de mon travail, et à vrai dire, ca c’est très bien passé, sur 5 points : j’ai eu 2 « dépasse les objectifs » et 3 « Atteint les objectifs » ce qui est une excellente évaluation. Le paragraphe était juste une remarque négative parmis pleins de compliments B) . Mais je maintiens mes convictions, NON ! on ne me fera jamais programmer comme dans l’exemple, même sous la torture.

[Edit]
Après une rapide réflexion, en fait si je vais programmer vite fait ce qu’on me demande pour être tranquil, mais je décline toutes responsabilités en cas de problème. A mon avis, je ne dois pas être le seul à faire ça…
[/Edit]

HackerZ · Février 14, 2006, 11:45

Mouarf, ca me fait penser au dernier truc que j’ai eu, je devais faire les fonctions de modifications de pages en ligne sur un site, et le web"master" avait fait tout le site avec des javascript , en tableaux et tout…
(exemple d’un des 10fichiers javascript : ici
Bref j’ai du tout refaire le site.

TwinSidE · Février 14, 2006, 12:16

HackerZ > atta ça je parie que c’est pas le code source avec lequel le webMaster a travaillé, ça c’est la version de prod compressé pour gagner un minimum de bande passante et ptet pour éviter que tout le monde copie le code au passage.

même les plus grand sado du code ne peuvent pas concevoir un programme en une ligne =)

HackerZ · Février 14, 2006, 2:45

[quote=« TwinSidE, post:18, topic: 27493 »]HackerZ > atta ça je parie que c’est pas le code source avec lequel le webMaster a travaillé, ça c’est la version de prod compressé pour gagner un minimum de bande passante et ptet pour éviter que tout le monde copie le code au passage.

même les plus grand sado du code ne peuvent pas concevoir un programme en une ligne =)[/quote]

J’ai recu les sources comme ca
En fait c’est plutot le gars qui utilise des outils qui sortent des javascript mais bon c’est pas le top… et ces 9 fichier .js servaient uniquement a afficher un menu dynamique

holybout · Février 14, 2006, 3:55

Je resiste à la tentation de poster bien bien pire. la seule solution dans ces cas là, trouver ce que fais le code et le refaire de A à Z. Déjà, comprendre mon propre code sans commentaire d’il y a deux semaines, j’ai du mal, alors celui des autres…