Discussion :

[asprog]

bonjour ,

je viens de développer un jeu en c++,j'ai trouvé quelques problèmes avec les pointeurs voila le message qui s'affiche "invalide conversion from int to int"
voici le code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 void Brique::collision(ball b){
                         if(i>0) /* si ya une brique,une collison avec la balle peux avoir lieu*/
                             {
                         if((b.rballe.x + b.rballe.w) >= rbrique.x
                                   &&
                          (b.rballe.x) <= (rbrique.x + rbrique.w)
                                    &&
                          (b.rballe.y + b.rballe.h) >= rbrique.y
                                    &&
                          (b.rballe.y) <= (rbrique.y + rbrique.h))
                                     {
                                  b.sens2=1;
      /* efface les briques selon le niveau*/

l'erreur se produit dans la ligne b.sens2=1 ;sachant que sens2 est un attribut déclarée comme pointeur dans la classe balle .merci d'avance

[LittleWhite]

[EDIT]J'avais mal lu, vikki donne la solution

[vikki]

Hello,
Effectivement, tu sembles vouloir affecter un entier à un pointeur, heureusement que le compilateur t'arrête Pourquoi avoir déclaré sens2 en tant que pointeur (pointeur vers quel type dalleurs?) alors que le but semble de stocker un entier en fonction du sens (peut être qu'un enum serait plus claire)?

[asprog]

Hello,
Pourquoi avoir déclaré sens2 en tant que pointeur (pointeur vers quel type dalleurs?) alors que le but semble de stocker un entier en fonction du sens (peut être qu'un enum serait plus claire)?

pointeur vers un entier;j'ai déclaré un pointeur car j'ai besoin de modifer la valeur elle meme du variable sens2 je vous souviens que sens2 est déclaré dans une autre classe Balle mais elle est aussi utilisée dans cette classe "brique"

[LittleWhite]

pointeur vers un entier;j'ai déclaré un pointeur car j'ai besoin de modifer la valeur elle meme du variable sens2 je vous souviens que sens2 est déclaré dans une autre classe Balle mais elle est aussi utilisée dans cette classe "brique"

Alors dans ce principe là, il faudrait mettre

Code :

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

*b.sens2 = 1;

Pourtant, vous raisonnement est faux.
Si vous voulez modifier une balle, ce n'est pas son sens qui doit être un pointeur, mais toute la balle, lors du passage dans la fonction qui fait la modification.
Sur ce principe là, votre fonction collision prends un pointeur sur ball (afin de modifier le sens de la balle par la suite):

Code :

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

b->sens2 = 1;

(En prennant en compte que sens2 est un int). Et hop, plus de problème

[asprog]

et si j'ai écris b->sens2=1 est ce que la variable sens2 prend la valeur 1 en mémoire sachant que dans une autre fonction dans la classe balle j'ai une autre utilisation de sens2 et qui prend une autre valeur.
bref sens2 est utilisée plusieurs fois dans mon programme et je veux bien conserver et récupéré sa dernière valeur c'est pourquoi j'ai pensé au pointeur

[LittleWhite]

et si j'ai écris b->sens2=1 est ce que la variable sens2 prend la valeur 1 en mémoire sachant que dans une autre fonction dans la classe balle j'ai une autre utilisation de sens2 et qui prend une autre valeur.
bref sens2 est utilisée plusieurs fois dans mon programme et je veux bien conserver et récupéré sa dernière valeur c'est pourquoi j'ai pensé au pointeur

Oui, cela fonctionnera.
En fait, dans votre application, vous ne devriez qu'avoir une seule balle.
Si une fonction à besoin d'avoir des informations sur la balle, on lui donne la balle en paramètre (sans pointeur (si possible avec une référence constante (voir FAQ C++))
Si une fonction a besoin de modifié les paramètre de la balle, alors dans ce cas, il faut passer un pointeur.

[asprog]

bon pour être plus précise dans ma question je vais vous montrer 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
class ball{
           int x,y;
 
      public:
             int sens1,sens2;
               ball();
             ball(int ,int);
             void deplaceBalle();
             void load();
             void affiche();
             void Limite();
             void collision(Raquete);
           };

ça c l'interface; voyant maintenant l'implémentation du méthode collision(Raquete)

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
 
   void ball::collision(Raquete r){
          { /* la collision entre la balle et la raqette*/
        if((rballe.x + rballe.w) >= r.rraquet.x
                 &&
        (rballe.x) <= (r.rraquet.x + r.rraquet.w)
                 &&
        (rballe.y + rballe.h) >= r.rraquet.y
                  &&
         (rballe.y) <= (r.rraquet.y + r.rraquet.h))
                     {
              sens2=0;
                   }
                    }
                    }

comme vous voyez sens2=0 ; passons maintenant à une autre classe Brique dans laquelle on va utiliser sens2

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
 void Brique::collision(ball *b){
                           for (y=0;y<10;y++)
                              {
                           for (x=0;x<10;x++)
                              {
                            rbrique.x=x*51;
                            rbrique.y=y*21;
                         if(i>0) /* si ya une brique,une collison avec la balle peux avoir lieu*/
                             {
                         if((b->rballe.x + b->rballe.w) >= rbrique.x
                                   &&
                          (b->rballe.x) <= (rbrique.x + rbrique.w)
                                    &&
                          (b->rballe.y + b->rballe.h) >= rbrique.y
                                    &&
                          (b->rballe.y) <= (rbrique.y + rbrique.h))
                                     {
                                  b->sens2=1;

et par la suite on va faire un test sur sens2 dans la classe balle: si elle prend la valeur 0 alors faire ceci sinon faire cela .
j'espère que je sois plus précise dans l'explication de mon problème cette fois ci

[LittleWhite]

Hum là, il n'y a plus de problème ? Si ? Compilation ou exécution ?

[asprog]

ma question d'abord: est ce que je vais déclarer mon variable sens2 en tant que pointeur ou pas??veuillez me répondre précisément et merci d'avance

[Emmanuel Deloget]

ma question d'abord: est ce que je vais déclarer mon variable sens2 en tant que pointeur ou pas??veuillez me répondre précisément et merci d'avance

Veuillez vous documenter sur ce qu'est un pointeur, une référence, et ne pas nous ordonner de manière brutale de répondre à vos questions. On est pas aux ordres.

[asprog]

et ne pas nous ordonner de manière brutale de répondre à vos questions. On est pas aux ordres.

je m'excuse vraiment j'avais pas l'intention de vous ordonner

[LittleWhite]

Voici un cours (il y en a d'autre) sur les pointeurs: http://chgi.developpez.com/pointeur/

Pour l'explication:

sens2 n'est pas un pointeur, meme si on veut channger sa valeur selon les fonctions. Je pensais l'avoir explique plutot, mais je vais recommencer.

Premierement, nous evitons les pointeurs le plus possible (surtout en C++)
Maintenant, l'histoire c'est que je conseille de passer une balle par pointeur (donc un pointeur sur balle, donc si nous changeons quelque chose a l'interieur, le changement est toujours present en sortant de la fonction (ce qui n'est pas le cas avec un passage par copie (sans pointeurs), car c'est une copie (comme le nom l'indique)))
Donc meme avec un pointeur sur une balle, on pourra modifie sens2

[asprog]

merci LittleWhite pour votre explication