Discussion :

[Pecose]

Bonjour,

Voilà en gros mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
for (x = 0; x < 12; x++) {
 
    if (boule[x] == null) {
 
        Objet boule[x] = new Objet();
    }
 
    for (y = 0; y < 12; y++) {
 
        // Là je récupère en gros les Shape de boule[x] et boule[y]
        collideBoules = shapeBouleX.intersects(shapeBouleY);
    }
}

Mon problème est la présence de toutes ces imbrications de for().

Est-ce possible de faire par exemple une classe en static qui s'occuperait toute seul de comparer les intersections ?

Je souhaiterais surtout supprimer la deuxième boucles for().

Merci d'avance pour votre aide.

[Nico02]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
for (x = 0; x < 12; x++) {
 
    if (boule[x] == null) {
 
        Objet boule[x] = new Objet();
    }
 
    for (y = 0; y < 12; y++) {
 
        // Là je récupère en gros les Shape de boule[x] et boule[y]
        collideBoules = shapeBouleX.intersects(shapeBouleY);
    }
}

Je n'ai rien compris, désolé

Pourquoi ça t’embête tant que ça d'avoir 2 boucles imbriquées ?

[Pecose]

Le problème est que je n'ai pas seulement 2 boucles imbriquées, je dois en avoir 10 ou 12 et j'ai peur que ça ralentisse la machine.
Et puis je me demandais si c'était une façon normale d'arranger sont code. Il y a peut être plus simple non ?

[Nico02]

Si le traitement impose d'avoir 10 ou 12 boucles, eh bien soit, ça ne ralentira la machine que si tu fais des grosses opérations à répétition (dans ce cas là oui il faut commencer à réfléchir à comment optimiser).

Après non avoir 10 boucles imbriquées dans une même fonction c'est pas très propre, tu peux toujours découper tes traitements en fonctions.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while(-)
{
  traitement 1
  while(-)
  {
     // traitement 2
     while(-)
     {
        // traitement 3
     }
  }
}

Peut s'écrire

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
22
while(-)
{
  traitement 1
   fonctionTraitement2(param1);
}
 
fonctionTraitement2(param1)
{
  while(-)
  {
     // traitement 2
     fonctionTraitement3(param2);
  }
}
 
fonctionTraitement3(param2)
{
  while(-)
  {
     // traitement 3
  }
}

Mais c'est un exemple, ça ne sert à rien de tout découper si ça n'a pas de réelle signification.

A mon sens, un code "bien écrit" est un code qui doit être compris par le développeur qui va peut-être passer après moi (même si bien sûr il faut respecter les conventions de nommage, indentation, etc..).

Si on arrive à suivre ta logique au travers des 10 boucles, eh bien soit (même si j'en doute un peu ). Par contre découper ton code dans tous les sens n'a pas beaucoup d’intérêt si ça devient un jeu de piste pour suivre ton code.

En espérant avoir été clair

Cdt.

[Pecose]

Oui très clair, merci.

[toutgrego]

Pour moi tu peux avoir 10/12 if imbriqués mais avoir 10/12 while/for imbriqués c'est qu'il y a un gros problème d'algorithme x)

[TIFéç]

Bonjour,

"Comment bien coder?", vaste question... Une façon de s'en approcher peut être de mettre en place des outils qui vont "mesurer" la qualité du code produit (non, "compiler n'est pas gagner" ) : checkstyle, PMD par exemple. Monter le tout dans un sonar / sonarQube pour avoir une interface agréable des problèmes potentiels n'est pas qu'un plus, cela devrait, de mon point de vue, être un incontournable pour tout dev java se voulant professionnel. Le temps investi à outiller le projet est largement récompensé par le gain de qualité et l'aide à la montée en compétence des développeurs qui vont y découvrir que tout un tas de comportements qu'ils utilisent régulièrement sont déconseillés voir carrément à proscrire. De plus la note de mesure de la qualité est très stimulante, les collègues se challengeant amicalement entre eux pour améliorer cette note et avoir le projet le "mieux" codé.

@toutgrego, avoir 10 ou 12 if imbriqués est plutôt déconseillé, cela fait monter la complexité cyclomatique : http://fr.wikipedia.org/wiki/Nombre_cyclomatique

My two cents...