Discussion :

[Lulou40]

Bonjour!

J'ai un petit casse-tête à soumettre:

Le contexte:
Un aquarium contient des éléments ( cailloux, algues, poissons) qui barbotent gaiement dans l'eau. Les poissons (éléments mobiles) se déplacent vers une destination calculée aléatoirement à l'aide d'une méthode go().

Un timer (classe Time) applique dans une boucle infinie la méthode go() sur un aquarium. La méthode go() est donc "relancée à chaque tic".

Le problème:
La destination aléatoire est recalculée à chaque "tic". La méthode de déplacement (move) n'est appelée qu'une fois avant que tout soit relancé et la destination recalculée... Le poisson n'a jamais la possibilité d'arriver à destination.
La contrainte principale c'est que target(), qui calcule cette position, ne peut être appliquée que sur un MobileItem ( et bien sur il y a dans le vector parcouru des "non MobileItem" en plus des MobileItems.

J'ai l'impression d'avoir tout essayé dans tous les sens : des while, des if, rien y fait. Je désespère, alors je fait appel à vous. J'espère que quelqu'un pourra m'aider un peu.

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
23
24
25
 public void go(){    // fait bouger (move) les objets vers une destination (target)
 
 
		 Point destination = null;
 
		 for(int i=0; i < this.items.size() ; i++){  // on parcourt la collection d'objet
 
 
		   if(this.items.elementAt(i) instanceof MobileItem){  // si l'objet est mobile
 
			 destination = ((MobileItem) this.items.elementAt(i)).target(null);  // on définit une destination aléatoire pour l'objet
 
 
 
			   ((MobileItem) this.items.elementAt(i)).move(destination);  // l'objet se déplace vers la destination	 
 
		   }
 
  try{
		          Thread.sleep(100);
		        }
		        catch(InterruptedException e){}
 
		        this.repaint();
		 }

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
23
24
25
26
27
 
	public boolean move(Point destination){  // cette fonction déplace l'item vers une destination, et retourne un booléen indiquand s'il est arrivé à destination
 
 
 
 
 
			if( this.position.x != destination.x){
				if(this.position.x < destination.x){
 
					this.position.x += 5;
 
					}
				else
					this.position.x -= 5;
 
			}
 
// même chose pour les y
 
 
		if (this.position == destination)
		return true;
		else
			return false;
 
	}

[Invité]

Hello,

Alors une petite idée comme ça en passant : pourquoi ne pas ajouter un attribut "destination" a ton MobileItem ? Ainsi :

• Au départ tu définis une valeur "destination" a chaque poisson (monPoisson.setDestination(destination) ou via le constructeur, je ne sais pas comment ta liste de poissons est gérée)
• Dans ta méthode go(), tu vérifies pour chaque poisson s'il est arrivé a destination (positionActuelle == monPoisson.getDestination()).
  
  • Si c'est le cas, alors tu génères une nouvelle destination et tu l'associes a ton instance de MobileItem via un monPoisson.setDestination(nouvelleDestination).
  • Sinon bah rien de spécial.

Voila comment je ferais
En espérant t'avoir aidé

[LittleBean]

Salut,

J'ai l'impression d'avoir tout essayé dans tous les sens : des while, des if, rien y fait. Je désespère, alors je fait appel à vous. J'espère que quelqu'un pourra m'aider un peu.

A mon sens et comme le dit pierreAd, tu as plus un problème de conception que d'algorithmique.

Le comportement de ton objet mobilImpl (poisson) doit être géré par lui même et non par son conteneur (aquarium).Ce qui est plus cohérent avec la réalité ... c'est le poisson qui choisit et sait où il va

Ta méthode principale devrait être de la forme suivante

Code :

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

1
2
3
4
5
6
7
 
 if(this.items.elementAt(i) instanceof MobileItem){ 
 
 Mobilitem lMobilItem = ((MobileItem) this.items.elementAt(i));
lMobilItem.target(); 
lMobilItem.move();  
}

avec la méthode target comme ci-dessous :

Code :

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

1
2
3
4
5
6
7
8
9
 
public voit target(){
 
if (destination == null){
// choisir la destination
destination = toto;
}
// sinon rien
}

Par contre il me semble curieux que le calcul de destination soit dans la même boucle que l'action de déplacement .... les premiers sont avantagés

à ta place je ferrais 2 boucles :
- les poissons choisissent leurs cibles
- puis ils bougent
enfin avec 2 boucles si tu as des prédateurs ils ne mangeront jamais :p

Bonne chance