Discussion :

[eatherquake]

Bonjour,

je suis tombé sur cette ligne de code bizarre :

Code :

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

int nbCar = Integer.parseInt("" + fBatch.length());

ou fbatch renvoit un int (longueur d'un fichier) et donc je suis surpris de ne pas voir tout simplement :

Code :

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

int nbCar = fBatch.length();

Y a t il une subtilité qui m'échappe ou alors est ce que celui qui a écrit cette ligne ferait bien de prendre des vacances?

[adiGuba]

Salut,

On va miser sur 3 bonnes semaines de vacances

Plus sérieusement ce genre de code inutile peut arriver à tout le monde pour diverses raisons (changement du type de la variable, copier/coller, etc...)


a++

[MrEddy]

Si c'est la méthode length() de l'objet File, celui ci retourne un long et pas un int.

Donc il faut le caster en int, meme si il aurait pu utiliser

Code :

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

int a = (int) f.length();

[eatherquake]

C'est bien ce que je pensais...
On sait jamais, le type faisait peut être du cobol avant, et utilisait une vieille technique de gourou qu'il a transposé au java...

Merci

[adiGuba]

@MrEddy : c'est fort probable en effet ! Bien vu

@eatherquake : il faut toujours se méfier des "vieille technique de gourou"... surtout lorsqu'elles viennent d'un autre langage !!!

a++

[thelvin]

Si c'est la méthode length() de l'objet File, celui ci retourne un long et pas un int.

Donc il faut le caster en int, meme si il aurait pu utiliser

int a = (int) f.length();

Notons que si c'est ça, la méthode du haut pour convertir un long en int va lancer une exception en cas d'overflow, ce qui est peut-être voulu. Un cast ne lance pas d'exception.

Je ne connais pas de méthode plus élégante pour faire ce genre de conversion en peu de lignes, à la place je me sers de bibliothèques externes ou je garde une classe utilitaire dans un coin pour ça.