Discussion :

[bogoss91]

Salut, à tous.

Je suis en train de faire un jeu, et dans la classe qui représente la carte, j'ai un tableau à deux dimensions, contenant les détails du terrain.

Le problème, c'est que j'en ai besoin avec d'autres classes pour gerer les collisions, et donc j'utilisais le mot clé friend, pour rendre plusieurs classes amies.

Mais maintenant, j'ai envie de rendre ça plus "propre", et donc, je veux faire une fonction qui renvoie ce tableau, mais je ne sais as quoi spécifier comme retour, car quand je mets :

Code :

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

1
2
3
4
int Carte::getMap()
{
    return m_carte;
}

Je reçois cette erreur : "Cannot convert int(*)[15] to int". J'ai essayé plein d'autres trucs qui n'ont pas marché.

Merci de m'aider

[rikau2]

si tu fais int mafonction() oui tu retournes un int.
si ton tableau est de la forme myTab[][], c'est donc un int **

essaies comme ca....

[bogoss91]

"Cannot convert int(*)[15] to int** in return"

[rikau2]

Peux tu me donner la declaration de m_carte dans la classe Carte

[bogoss91]

Dans le .h je mets ça en private :

int m_carte[1000][15];

[rikau2]

Code :

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

1
2
3
4
int Carte::getMap()
{
    return m_carte;
}

Je te conseil ceci, test et dit moi si ca marche bien, et pas seulement a la compilation:

Code :

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

1
2
3
4
int Carte::getMap()
{
    return (int **)m_carte;
}

[Laurent Gomila]

int** et int[][] ne sont pas compatibles, au contraire de int* et int[]. Ici il faut renvoyer un int[1000][15].

[rikau2]

mais ceci est equivalent : char *argv[] et char **argv, je pensais que par extension....
le prototype de fonction devient : int[1000][25] get... ? c'est un peu statique comme truc

[bogoss91]

J'ai essayé de faire ça :

Code :

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

1
2
3
4
int[1000][15] Carte::getMap() 
{  
         return m_carte; 
}

Et ça n'a pas marché : "Expected unqualified id before '[' token"

[rikau2]

Apres test en effet avec un cast (int**) on obtient un access vio.
Laurent ? Il n'est pas possible de definir un type de retour comme int[20][30] mafonction()....?

[Laurent Gomila]

Code :

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

1
2
3
4
int mafonction()[1000][15]
{
 
}

Avec un typedef c'est mieux :

Code :

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

1
2
3
4
5
6
typedef int MonType[1000][15];
 
MonType mafonction()
{
 
}

[Sylvain Togni]

typedef int MonType[1000][15];

MonType mafonction() { }

sauf que même avec un typedef, on ne peut pas retourner des tableaux C, ce ne sont pas des objets assignables.

Mais on peut retourner un tableau dans une structure, ou utiliser boost::array<>.

[Laurent Gomila]

... Je retourne me coucher ...

[rikau2]

m_carte est bien un pointeur de tableau? on ne peut pas le balader?

Ce qui veut dire, s'il veut pouvoir retourner un tableau, il doit utiliser un int**?

[bogoss91]

Donc moi là je dois faire comment?

[rikau2]

Ce que tu peux faire c'est de modifier ta declaration de tableau en int **
A ce moment la dans le constructeur de ta classe tu lui alloues l'espace mémoire requis.
Allocation de memoire dynamique

Apres ta methode pourra retourner un int ** et tu n'aura plus ce problème.

[Laurent Gomila]

Dans ce cas autant gérer un int[1000*15] et renvoyer un int*. Ca évitera la double allocation dynamique et tous ses problèmes.

[Alp]

Pour préciser ce qu'a dit Laurent, tu peux gérer un tableau à une dimension comme un tableau à deux dimensions.

Code :

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

1
2
3
4
int at(int i, int j)
{
   return tab[i*1000 + j*15];
}

A moins que ce soit l'inverse, je ne me rappelle plus.

[koala01]

Salut,

si le tableau est de l'ordre de int tab[1000][15], la formule est

Code :

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

1
2
3
4
int LaClasse::fonction(int ligne, int col)
{
    return tab[ligne*15+col]; 
}

De manière générale, c'est element=numero_ligne*MAX_COLONE+numero_colone

[bogoss91]

Oulà, ça devient compliqué tout ça De toute façon pour l'instant ça urge pas, donc je vais liasser ça comme ça, et si ça me gêne plus tard je changerai.