Probleme avec constructeur

Version imprimable

Bonjour, je débute en C++ et je suis en train de construire un petit programme et je viens de rencontrer une embûche...

Je suis en train de construire une classe tableau qui contient des objets "cellules". Je ne semble pas être capable d'utiliser mes objets cellules dans ma classe tableau pour une raison ou une autre...

Voici le code:

TABLEAU.HPP

Code:

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
/*
*Classe qui représente un tableau
*
*
*
*
*/
 
#ifndef TABLEAU_HPP_
#define TABLEAU_HPP_
 
#include <iostream>
#include "Cellule.hpp"
 
using namespace std;
 
 
 
class Tableau
{
public:   
 
    //Constructeurs
    Tableau();
    Tableau(const Tableau& t);
 
    //Destructeur
    ~Tableau();
 
    //Opérateurs
    //Tableau& operator=(const Tableau& t);
    //bool operator==(const Tableau& t) const;
 
    //accesseurs
    int getNombreCellules(Tableau ** tab) const;
    bool estVide() const;
    Cellule& getCellule(int i , int j);
 
private:
 
     Cellule ** grille;
 
};
 
#endif

TABLEAU.CPP

Code:

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
/*
*Classe qui représente un tableau
*
*
*
*
*/
 
#include "Tableau.hpp"
 
const int largeur = 24;
const int longueur = 80;
 
//Constructeurs
Tableau::Tableau()
{    
    for(int i = 0 ; i < largeur ; i++)
    {  for (int j = 0 ; i < longueur ; j++)
       {
 
           grille[i][j] = new Cellule(i,j,false);
 
 
       }
    }
 
}
Tableau::Tableau(const Tableau& t)
{
    *this = t;
}
 
//Destructeur
Tableau::~Tableau()
{
    delete [] grille;
}
 
//Opérateurs
/*Tableau& Tableau :: operator=(const Tableau& t)
{
    
}
*/
/*bool Tableau:: operator==(const Tableau& t) const
{
    
}
*/
 
//accesseurs
int Tableau::getNombreCellules(Tableau ** tab) const
{
    int nombreCellules = 0;
 
    for(int i = 0 ; i < largeur ; i++)
    {  for (int j = 0 ; i < longueur ; j++)
       {
 
           cell = grille[i][j];
 
           if (cell.getEtat())
           {
               nombreCellules ++;
           }
 
       }
    }
    return nombreCellules;
 
}
Cellule& Tableau::getCellule(int i , int j) const
{
    return grille[i][j];
}
 
bool Tableau::estVide() const
{
    return Tableau.getNombreCellules()==0;
}

Merci à l'avance

10/02/2008, 16h35
xav181

Tu fais un appel dans ton constructeur grille[i][j] avant même d'avoir allouer grille=> plantage, alloue avant ou declare ta grille comme ça Cellule grille[Nlignes][Ncolonnes]
10/02/2008, 16h47
watiero

Je ne suis pas sûr de tout à fait comprendre car j'ai déclarer ma grille dans tableau.hpp comme variable private, je ne comprends pas pourquoi je devrais la déclarer une autre fois...
10/02/2008, 17h16
xav181

C'est juste que de la maniere dont tu l 'as déclaré, tu ne peux pas faire grille[i][j];

ex :
int tab[10];
tab[5] = 0; => ok
int* tab;
tab[5] = 0 =>plante
mais;
int* tab;
tab = new int(10);
tab[5] = 0 => ok
10/02/2008, 17h18
xav181

En plus, si tu veux mettre des pointeurs de cellules dans ton tableau il te faut grille***; parce que avec new cellule tu récupères un pointeur de cellules

Alors je crois que j'ai bien compris mais il me renvoit tout de même un message d'erreur... Merci pour ta patience...

Voici comment je l'ai modifié:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
//Constructeurs
Tableau::Tableau()
{    
 
    grille = new Cellule[largeur][longueur];
 
    for(int i = 0 ; i < largeur ; i++)
    {  for (int j = 0 ; i < longueur ; j++)
       {
 
           grille[i][j] = new Cellule(i,j,false);
 
 
       }
    }
 
}

10/02/2008, 21h41
Okdokey

Il y a un indice i qui traine dans la boucle sur j...
10/02/2008, 23h22
Invité
Hello watiero,

Code:

grille = new Cellule[largeur][longueur];

Ceci ne fait pas ce que tu attends. L'opérateur "new" ne prend qu'une seule paire de crochets. Ce que tu fais, c'est allouer un tableau de "largeur" cellules, puis référencer la "longueur+1"-ème valeur. Ce que tu devrais faire:
Code:

1 2 3 4 5 6 grille = new (Cellule*)[largeur]; for (int i = 0; i < largeur; ++i) { grille[i] = new Cellule[longueur]; }
Si je puis me permettre, tu devrais d'ailleurs plutôt utiliser std::vector, ou alors (si tu n'as pas besoin de performances poussées) std::map<std::pair<int, int>, Cellule*>.

Carl
12/02/2008, 21h16
watiero

Merci beaucoup, j'ai pu résoudre mon problème grace à toi!

Merci pour le conseil de la std mais en fait nous commencons à la voir. Ce sera pour mon prochain travail :king: