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)