Salut
Par definition, si ca fonctionne, ben c'est correct.
Apres si tu veux parler de "beaute du code" ou d'optimisation... Pas trop d'avis mais je peux faire quelques commentaires.
Le deux trucs que je ferais forcement remarquer si je voyais un code comme ca au boulot, c'est:
- l'initialisation de ton tableau a 10 elements en dur: si ton resultat en a moins, t'as des elements qui servent a rien et peuvent induire en erreur selon ce que tu fais du resultat. Si ton resultat est cense avoir plus de 10 elements, ton code va planter a l'execution -> tu devrais faire un new int[max-min+1] plutot qu'un new int[10]
- si tes parametres ne sont pas coherents (min et/ou max en dehors du "vrai" nombre d'elements de la liste) tu vas aussi planter - tu devrais soit faire une gestion d'exception dans ta fonction (i.e. verifier si min et max sont coherents par rapport a ta liste, et faire un traitement special le cas echeant), soit au pire indiquer que tu peux renvoyer une exception (pour forcer l'appelant a gerer les cas a probleme).
Apres quelques autres remarques, mais c'est plutot du chipotage:
- ArrayList est un type specifique de liste - tu pourrais facilement rendre ta fonction utilisable avec tout type derive de List en changeant ton parametre (surtout que tu utilises toi meme une List dans le code).
- en Java "recent" c'est bien vu de declarer ton type de liste - dans ton cas, ArrayList<Integer> list et List<Integer> sousListe (mais si t'as un vieux JDK ca reconnaitra pas cette syntaxe)
- la conversion de sousListe.get(i) d'un Integer en un int (dans ta boucle) est couverte silencieusement par Java mais un code "bien ecrit" la ferait apparaitre explicitement pour eviter tout malentendu
- tu pourrais aussi eviter de recourir a une sous-liste et sortir les elements directement de ta liste (par exemple avec une boucle for(int i=min, i<=max; i++)). Mais soyons honnetes, ca n'est pas forcement plus rapide a l'execution (ca depend de comment la liste est implementee)
J'pense que c'est tout...
A+
Message édité par lasnoufle le 03-01-2014 à 23:43:29
---------------
C'était vraiment très intéressant.