tableau deux dimensiosn

Version imprimable

voici un bout de code que j'aimerais faire marcher :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 #include <iostream> using namespace std; int** gettab(int tab[][2]){ return (int**)tab; } int main() { int a[4][2] = {{0, 0}, {1, 0}, {2, 0}, {2, 1}}; int ** b = gettab(a); cout << b[0][0]; return 0; }
j'ai un segmentation fault à l'instruction : cout << b[0][0];

est ce que l'erreur vient du cast : return (int**)tab; ?

21/12/2009, 15h43
r0d

Si tu ne nous le dit pas, on ne peut pas deviner ce que tu veux faire. Pour l'instant, la fonction gettab ne fait pas grand chose, et nous ne pouvons pas deviner ce que tu veux qu'elle fasse.

Citation:

Envoyé par toto828

est ce que l'erreur vient du cast : return (int**)tab; ?

Non, l'erreur vient du fait que tu essaie d'accéder à des éléments d'un tableau qui n'a pas été alloué. C'est une des raisons pour laquelle en c++ on n'utilise pas des int* (ou int[]), et encore moins des int** (ou int[][]), ça c'est en C. En C++, on utilise les conteneurs de la lib standart, std::vector par exemple (voir FAQ).

sur Stackoverflow on a répondu à ma question, voici la réponse :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
int (*gettab(int tab[][2]))[2] {
  return tab;
}
// or:
template<class T> struct identity { typedef T type; };
identity<int(*)[2]>::type gettab(int tab[][2]) {
  return tab;
}
 
int main() {
  int a[4][2] = {{0, 0}, {1, 0}, {2, 0}, {2, 1}};
  int (*b)[2] = gettab(a);
  cout << b[0][0];
}

intéressant. Moi j'aurais plutôt fait comme suit:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
typedef int N;
typedef N* pN;
template<class T, typename pN> struct identity { typedef T type; };
identity<int(*)[*pN]>::type gettab(const int tab[][*pN]) {
  return reinterpret_cast<identity, pN>( (void*) const_cast( tab ) );
}
 
int main() {
  int a[4][2] = {{0, 0}, {1, 0}, {2, 0}, {2, 1}};
  int (*b)[2] = gettab(a);
  cout << b[0][0];
}

21/12/2009, 16h49
toto828

merci pour la réponse. Je crois que le plus sage c'est d'utiliser std::vector ou Boost, parce que niveau lisibilité du code c'est quand même chaud pour un débutant.
21/12/2009, 19h34
r0d

Citation:

Envoyé par toto828

Je crois que le plus sage c'est d'utiliser std::vector ou Boost, parce que niveau lisibilité du code c'est quand même chaud pour un débutant.

:ccool:
Effectivement, c'est de loin la meilleure solution.
A moins d'avoir besoin de quelque chose un peu spécifique, normalement le vector de la STL devrait suffire.