Collision avec wpf

**tomlev** · 13/01/2009, 23h49

Ah ok, je vois le problème...
En fait le Clip est une géométrie relative au contrôle : le centre de ton ellipse, c'est (150,150) par rapport à ton PacMan, pas par rapport au canvas. Il faudrait donc convertir cette géométrie dans le référentiel du Canvas.

Pour ça il suffit de décaler le Clip (ou plutôt une copie du Clip) par rapport à la position du contrôle sur le Canvas, en appliquant une TranslateTransform :

Code C# :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Geometry g1 = element1.Clip.Clone();
double offset1X = Canvas.GetLeft(element1);
double offset1Y = Canvas.GetTop(element1);
g1.Transform = new TranslateTransform(offset1X, offset1Y);
 
Geometry g2 = element2.Clip.Clone();
double offset2X = Canvas.GetLeft(element2);
double offset2Y = Canvas.GetTop(element2);
g2.Transform = new TranslateTransform(offset2X, offset2Y);
 
Geometry intersect = Geometry.Combine(g1, g2, GeometryCombineMode.Intersect, null);
return !intersect.IsEmpty();

EDIT: comme ça risque d'être un peu lourd de cloner les géométries à chaque fois, je pense qu'il vaudrait mieux que chacun de tes UserControl définisse une propriété Geometry, initialisée en clonant le Clip. Ca renverrait toujours la même Geometry en faisant simplement varier la transformation :

Code C# :

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
 
private Geometry _geometry;
private TranslateTransform _transform;
public Geometry Geometry
{
    get
    {
        if (_geometry == null)
        {
            _geometry = this.Clip.Clone();
            _transform = new TranslateTransform();
            _geometry.Transform = _transform;
        }
        _transform.X = Canvas.GetLeft(this);
        _transform.Y = Canvas.GetTop(this);
        return _geometry;
    }
}

Et dans la méthode TestCollision, tu n'as plus qu'à tester sur la propriété Geometry au lieu de refaire le calcul à chaque fois

**dfiad77pro** · 14/01/2009, 00h08

J'ai essayé pour les 2 cas et ça ne marche pas,

pour le deuxième cas, j'ai mis ça dans ma méthode de collision:

Code :

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

  if (element1.Geometry == element2.Geometry) { return true; } else { return false; }

**tomlev** · 14/01/2009, 00h39

Envoyé par dfiad77pro

J'ai essayé pour les 2 cas et ça ne marche pas,

pour le deuxième cas, j'ai mis ça dans ma méthode de collision:

Code :

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

  if (element1.Geometry == element2.Geometry) { return true; } else { return false; }

Ben non, elles seront jamais égales les 2 géométries... Il faut toujours tester l'intersection, comme tu faisais avec le clip :

Code :

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

1
2
3
4
5
6
   public bool TestCollision(Pacman_c element1, Mur element2)
        {
            Geometry intersect = Geometry.Combine(element1.Geometry, element2.Geometry, GeometryCombineMode.Intersect, null);
            return !intersect.IsEmpty(); 
 
        }

Après, je suis pas certain que ça marche, j'ai pas vérifié... mais je suis relativement confiant

**dfiad77pro** · 14/01/2009, 00h57

il a l'air de me détecter des collisions, mais pas au bon endroit (comme si la géométrie était déformée).

**tomlev** · 14/01/2009, 01h15

Envoyé par dfiad77pro

il a l'air de me détecter des collisions, mais pas au bon endroit (comme si la géométrie était déformée).

Bizarre...
Il faudrait mettre un breakpoint dans TestCollision et dans la propriété Geometry pour voir ce qui se passe. Vérifie la propriété Bounds des 2 géométries pour voir si ça colle

**dfiad77pro** · 14/01/2009, 01h28

Oula lol ça me donne des bounds bizzares, ça va être chaud à débugger

bon j'essaye de corriger ça

**dfiad77pro** · 14/01/2009, 14h57

Bon j'ai passé la nuit dessus sans réussir à corriger ça.

Rien a faire, les collisions sont hasardeuses.

Si quelqu'un veut m'aider directement et qu'il a visual studio 2008 et .net 3.5sp1, qu'il n'hésite pas à me contacter (d'autant plus que le projet se compile se lance et est bien commenté).

Sinon je referais ça avec des méthodes basiques de calcul de distance.

**tomlev** · 14/01/2009, 16h36

Upload le projet quelque part et envoie le lien, je tacherai de jeter un oeil ce soir
C'était mon idée d'utiliser les Geometry alors je me sens un peu responsable

**tomlev** · 14/01/2009, 18h38

Alors... j'ai trouvé plusieurs problèmes :

- la taille : dans ton UserControl Pacman, tu définis un Clip elliptique de rayon 300 (d'ailleurs il faudrait mettre 150 : c'est le rayon, pas le diamètre). Mais quand tu crées une instance de Pacman, tu changes sa taille : il faudrait donc modifier aussi la taille du Clip (en code-behind ou par un binding). Une fois ce problème corrigé, la méthode TestCollision marche bien

- le test sur la collision : tu testes comme ça:

Code C# :

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

if (!TestCollision(this, E_collision)) { /* annuler le déplacement */ ... }

Mais c'est le contraire qu'il faut faire : on annule le déplacement s'il y a une collision, pas s'il n'y en a pas...

Code C# :

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

if (TestCollision(this, E_collision)) { /* annuler le déplacement */ ... }

- l'annulation du déplacement : tu modifies la valeur de PositionX ou de PositionY, mais pas Canvas.Left et Canvas.Top, donc le Pacman reste affiché là ou il est (et la position sur le Canvas n'est plus cohérente avec PositionX et PositionY). Il faut soit appeler Canvas.SetLeft/SetTop après avoir modifié PositionX/Y, soit transformer PositionX et Y en propriétés qui mettent à jour la position sur le Canvas :

Code C# :

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

1
2
3
4
5
public double PositionX
{
    get { return Canvas.GetLeft(this); }
    set { Canvas.SetLeft(this, value); }
}

(A mon avis cette dernière solution est la meilleure)

**dfiad77pro** · 14/01/2009, 19h42

Franchement merci 1000 fois, j'ai pu modifier mon programme et alléger le code

!

Bon pour le reste du travail, je vais essayer de continuer seul, mais j'aurais une dernière question juste pour enrichir mes connaissances:

Y a t 'il possibilité de faire simplement détecter une collision avec tout les objets de type mur ( pour éviter que je fasse un tableau contenant les noms des murs et une boucle de test sur ce tableau) ?

**tomlev** · 14/01/2009, 20h28

Envoyé par dfiad77pro

Y a t 'il possibilité de faire simplement détecter une collision avec tout les objets de type mur ( pour éviter que je fasse un tableau contenant les noms des murs et une boucle de test sur ce tableau) ?

Ce que je ferais à ta place, c'est un seul objet de type mur, dont la forme est définie par une géométrie (tu peux composer une géométrie "complexe" avec GeometryGroup)

**dfiad77pro** · 14/01/2009, 20h40

ok, et y a t-il une possibilité de dessiner les géométries à la main? ( avec expression ou autre chose)

**tomlev** · 14/01/2009, 20h50

aucune idée... a priori je dirais que non

**dfiad77pro** · 17/01/2009, 00h01

J'ai un autre petit soucis :

Chaque Pacman peut avoir ses touches clavier de déplacement propres, mais lorsque j' instanci deux Pacmans différents, seul un Pacman peu bouger à la fois.

Code :

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

1
2
3
4
5
private void UserControl_Loaded(object sender, RoutedEventArgs e)
        {
                App.Window.KeyDown += recuperer_touche_deplacement;
 
                   }

Je voudrais savoir comment permettre de récupérer plusieurs touches simultanément ( j'ai essayer les threads, mais je comprend pas trop comment les utiliser dans mon cas)

**tomlev** · 17/01/2009, 02h26

Dans l'évènement KeyDown, au lieu de ne tester que sur e.Key, teste toutes les touches qui t'intéressent avec e.KeyboardDevice.IsKeyDown.

**dfiad77pro** · 17/01/2009, 11h58

L'appui simultané marche avec ta méthode, mais lorsque je fait bouger le pacman n2, la pression de la touche arrête un petit moment le pacman n1.

**tomlev** · 17/01/2009, 17h49

Envoyé par dfiad77pro

L'appui simultané marche avec ta méthode, mais lorsque je fait bouger le pacman n2, la pression de la touche arrête un petit moment le pacman n1.

Normal, c'est à cause du délai avant répétition de la touche : quand tu appuies sur une touche pendant longtemps, la répétition ne commence pas tout de suite, pour éviter de taper accidentellement plusieurs caractères. Quand tu appuies sur la 2e touche, ça remet à zéro ce délai avant répétition, et donc ça arrête la répétition de la 1ere touche.

A mon avis, l'évènement KeyDown n'est pas vraiment adapté pour gérer les touches de contrôle du jeu, à cause de ce problème de répétition. Il vaudrait mieux faire un timer dans lequel tu testes l'état des touches toutes les 50 ou 100ms. Ou encore mieux, utiliser DirectInput, qui est fait pour ça...

**dfiad77pro** · 17/01/2009, 18h33

Voila, j'ai essayé ça mais ça marche pas (à noter que j' appel les fonctions de lecture d'événement clavier dans l'objet Pacman)

Code :

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

1
2
3
4
5
6
7
8
9
 
private void UserControl_Loaded(object sender, RoutedEventArgs e)
        {
            TimeSpan raf = TimeSpan.FromSeconds((Double)(0.1));
             DispatcherTimer dispatcherTimer = new System.Windows.Threading.DispatcherTimer();
            dispatcherTimer.Tick += new EventHandler(recuperer_touche_deplacement); // on charge la fonction de récupération des touches
            dispatcherTimer.Interval = raf;
            dispatcherTimer.Start();
}

Pour la fonction appelée par le timer:

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
 
 
 public void recuperer_touche_deplacement(object sender, EventArgs e)
        {if (e is KeyboardEventArgs)
            {
                KeyboardEventArgs clav = (KeyboardEventArgs)e;
 
 
                if (clav.KeyboardDevice.IsKeyDown(Touche_Droite))
                {
                    this.Deplacement_sens = "droite";
                    this.Deplacer("droite");
                }
                if (clav.KeyboardDevice.IsKeyDown(Touche_Gauche))
                {
                    this.Deplacement_sens = "gauche";
                    this.Deplacer("gauche");
                }///

Je pense que je lui envoi rien pour détecter l'événement, mais je ne vois pas trop comment faire avec ça. j'ai regardé les directinput, ça a l'air pas mal, mais un peu lourd à écrire.

**tomlev** · 17/01/2009, 21h00

Ca n'a pas de sens de faire comme ça... Le paramètre de l'évènement Tick est un EventArgs, ce ne sera jamais un KeyboardEventArgs !

Il faut récupérer l'état des touches autrement, par exemple avec la méthode statique Keyboard.IsKeyDown