Le pire bout de code que vous ayez vu

**fredop** · 22/09/2010, 12h20

Envoyé par Gordon Fowler

les variables sont passées par copie de la valeur

Est ce une généralité à propos de Java ou spécifique à l'exmple ?
Car si c'est le cas un, je suis en désaccord.

Cela impliquerait qu'un objet uniquement passé en paramètre n'est jamais modifiable dans la méthode comme cela est possible en C avec la notion de pointeur.

Donc ce qui s'amuse à faire des codes du genre car l'objet à subit un traitement le modifiant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
   public Object   doSiomething(Object  object ){

      //code
        return object;

    }

n'ont pas compris comment sont traités les paramètres en Java.

Le return est inutile sauf quelques cas (objets immuables comme l'objet LocalDate de l'api JodaTime que je recommande vivement par ailleurs...)

**GanYoshi** · 22/09/2010, 13h27

Envoyé par fredop

Est ce une généralité à propos de Java ou spécifique à l'exmple ?
Car si c'est le cas un, je suis en désaccord.

En java c'est une généralité, toutes les variables sont passées par copie de leur valeur.

Envoyé par fredop

Cela impliquerait qu'un objet uniquement passé en paramètre n'est jamais modifiable dans la méthode comme cela est possible en C avec la notion de pointeur.

On ne peut donc pas passer un objet en paramètre.
On passe une copie de la référence à cette objet, donc on peut ensuite faire des modifications sur cette objet, mais toute réasignation de la copie de la référence à l'objet est sans effet sur la référence initiale.

Donc ce qui s'amuse à faire des codes du genre car l'objet à subit un traitement le modifiant:

Envoyé par fredop

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
   public Object   doSiomething(Object  object ){

      //code
        return object;

    }

n'ont pas compris comment sont traités les paramètres en Java.
Le return est inutile sauf quelques cas (objets immuables comme l'objet LocalDate de l'api JodaTime que je recommande vivement par ailleurs...)

C'est en effet inutile si ta variable "object" ne pointe pas vers un nouvel object.
Après je sais pas quel est la meilleur solution en terme de visibilité.

Invité · 22/09/2010, 14h02

Cette technique permet de faire des choses du style

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

FaireEncoreAutreChose(FareAutreChose(mafonction(monObjet)))

Plutot que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
mafonction(monObjet)
FareAutreChose(monObjet)
FaireEncoreAutreChose(monObjet)

Ce pattern à un nom et est vachement utilisé pour les requète NHibernate par exemple ou la majorité des fonction retourne this

**GanYoshi** · 22/09/2010, 14h12

Envoyé par Bourgui

...

Oh j'y avait pas pensé merci de l'info !

Invité · 22/09/2010, 16h56

j'ai trouvé comment ça s'appelle : "Fluent interface"

http://en.wikipedia.org/wiki/Fluent_interface#C.23

**hegros** · 22/09/2010, 19h54

Envoyé par Bourgui

Cette technique permet de faire des choses du style

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

FaireEncoreAutreChose(FareAutreChose(mafonction(monObjet)))

Plutot que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
mafonction(monObjet)
FareAutreChose(monObjet)
FaireEncoreAutreChose(monObjet)

Ce pattern à un nom et est vachement utilisé pour les requète NHibernate par exemple ou la majorité des fonction retourne this

C'est justement ce genre de code qu'il ne faut pas faire.

D'ailleurs puisque tu parles de pattern, c'est le pattern de responsabilité 'Protection des variations' ou plus connu peut-être sous le nom 'Ne parlez pas aux inconnus' qui me fait penser à cela.

le code me fait aussi penser au pattern décorateur mais poussé à l'extrême et mal utilisé

Bref, ton premier bout de code (pattern hibernate lol) est sale, je préfére de loin retrouver le second code

**Médinoc** · 23/09/2010, 07h07

Pourtant, en C++ c'est courant (sauf qu'il y a des points ou des chevrons à la place des appels imbriqués).

**GanYoshi** · 23/09/2010, 09h09

Envoyé par hegros

C'est justement ce genre de code qu'il ne faut pas faire.

D'ailleurs puisque tu parles de pattern, c'est le pattern de responsabilité 'Protection des variations' ou plus connu peut-être sous le nom 'Ne parlez pas aux inconnus' qui me fait penser à cela.

le code me fait aussi penser au pattern décorateur mais poussé à l'extrême et mal utilisé

Bref, ton premier bout de code (pattern hibernate lol) est sale, je préfére de loin retrouver le second code

Je vois pas en quoi le premier code parle plus aux inconnus que le second, c'est juste appeler plusieurs méthode sur un seul et même objet, quel est le problème ?

**tomtom83** · 06/10/2010, 10h17

Un magnifique code pour séparer la partie entière et la partie décimale en c# afin d'afficher un temps avec 3 chiffres pour les millisecondes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
                double valeur;
                [...]
                // Update the time label
                minute = (int)valeur / 60;
                seconde = (int)valeur - minute * 60;

                if (valeur.ToString().Split(',').Length >= 2)
                {
                    miniseconde = valeur.ToString().Split(',')[1];
                    if (miniseconde.Length > 3)
                        miniseconde = miniseconde.Substring(0, 3);
                    else if (miniseconde.Length > 2)
                        miniseconde = miniseconde.Substring(0, 2) + "0";
                    else
                        miniseconde = miniseconde.Substring(0, 1) + "00";
                }
                else
                {
                    miniseconde = "000";
                }

C'est pas beau ça ? Une convertion en string puis un split sur la virgule !
Je travaille sur un xp français et ça m'a pris un bon moment pour comprendre pourquoi l'appli ne fonctionnait pas correctement sur windaube embedded en anglais. Hé oui sur un os en angliche quand on convertit un double en string ça met un . pas une virgule...
remplacé en une ligne par miniseconde=String.Format("{0:000}", (valeur - minute * 60 - seconde) * 1000);

**Bluedeep** · 30/05/2011, 13h42

Envoyé par tomtom83

remplacé en une ligne par miniseconde=String.Format("{0:000}", (valeur - minute * 60 - seconde) * 1000);

La correction ne valant pas beaucoup plus cher que le code critiqué, puisqu'elle démontre l'égal ignorance du TimeSpan.

**Khoyo** · 09/11/2010, 15h21

Obligé de citer l'IOCCC...
http://www.de.ioccc.org/years.html

ex:

#include <stdio.h>
#include <stdlib.h>
#include <math.h>

#define _ ;double
#define void x,x
#define case(break,default) break[O]:default[O]:
#define switch(bool) ;for(;x<bool;
#define do(if,else) inIine(else)>int##if?
#define true (--void++)
#define false (++void--)

char*O=" <60>!?\\\n"_ doubIe[010]_ int0,int1 _ Iong=0 _ inIine(int eIse){int
O1O=!O _ l=!O;for(;O1O<010;++O1O)l+=(O1O[doubIe]*pow(eIse,O1O));return l;}int
main(int booI,char*eIse[]){int I=1,x=-*O;if(eIse){for(;I<010+1;I++)I[doubIe-1]
=booI>I?atof(I[eIse]):!O switch(*O)x++)abs(inIine(x))>Iong&&(Iong=abs(inIine(x
)));int1=Iong;main(-*O>>1,0);}else{if(booI<*O>>1){int0=int1;int1=int0-2*Iong/0
[O]switch(5[O]))putchar(x-*O?(int0>=inIine(x)&&do(1,x)do(0,true)do(0,false)
case(2,1)do(1,true)do(0,false)6[O]case(-3,6)do(0,false)6[O]-3[O]:do(1,false)
case(5,4)x?booI?0:6[O]:7[O])+*O:8[O]),x++;main(++booI,0);}}}

**ThomasR** · 09/11/2010, 15h29

Envoyé par Khoyo

Obligé de citer l'IOCCC...
http://www.de.ioccc.org/years.html

ex:

Haha, excelent, c'est des artistes : http://www.de.ioccc.org/2004/newbern.c

**jabbounet** · 09/11/2010, 17h35

Envoyé par Khoyo

Obligé de citer l'IOCCC...
http://www.de.ioccc.org/years.html

ex:

ça existe encore ce concours?

Mais bon c'est peu être limite hors sujet je pense que l'on parle de code utilisé dans le cadre du boulot.

je me souviens d'un sapin de noel, contenant du code c....

**6-MarViN** · 07/12/2010, 12h01

J'ai trouvé ce code dans un dans un de mes projets fait par un de mes binômes en C

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
if (write(1, buffer, buff_size) < 0)
{
   write(1, "write failed\n", 30);
   exit(0);
}

**Barsy** · 30/05/2011, 13h29

Ce matin, dans le projet que je suis en train de reprendre, je suis tombé sur ce code :

Code C# :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
int Id;
if (int.TryParse(maValeur.ToString(), out Id))
{
	if (Id.ToString().Contains("-"))
		Id = Convert.ToInt32(Id.ToString().Replace("-", String.Empty));
}