erreur sur code "windows a renconteré une erreur"

**antoine0207** · 28/08/2006, 11h50

Bonjour
j'ai du faire une erreur toute bidon étant débutant mais je m'en rend pas compte
quand ça arrivé sur la derniere ligne copiée de mon .exe il s'arrete et il y a une erreur

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
19
20
21
22
23
24
25
26
27
28
struct cellule
{
    short sol;
    bool un;
    bool deux;
    bool trois;
    bool quatre;
    bool cinq;
    bool six;
    bool sept;
    bool huit;
    bool neuf;

};



cellule* remplir()
{


cellule* tableau;
short i=11;

while (i<100)
{

tableau[i].sol=0;

**Biosox** · 28/08/2006, 11h58

euh... il doit pas être complet le code que tu nous montre...

la boucle while ne se termine pas (pas de '}') et il n'y a pas de main.

Par contre un truc déja faux:

si tu définis:

Code :

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

cellule *tableau

alors tableau sera un pointeur vers une cellule, mais pas initialisé, et surtout, la mémoire nécessaire pour tes cellules n'est pas allouée!!!

tu devrais plutot faire par exemple

Code :

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

cellule tableau[100]

A ce moment, il y a effectivement la mémoire pour 100 cellules qui est allouée, et "tableau" pointe vers la "1ere case"

**antoine0207** · 28/08/2006, 11h59

non effectivement j'ai coupé le code parce que la suite n'etais pas interressante
le probleme doit etre effectivement la memoire non allouée

**hegros** · 28/08/2006, 12h00

Aie rien n'est alloué pour la variable cellule.Il faut définir un tableau automatique/statique/dynamique(voir new) comme dis précédent ou pourquoi pas utiliser un container?

**antoine0207** · 28/08/2006, 13h32

je suis débutant donc je n'utilise pas ce qui est le plus approprié ...

maintenant il ne veux plus me renvoyer dans ma fonction mon tableau[100]
car la fonction est définit par tableau*

**r0d** · 28/08/2006, 14h03

Bonjour,

alors je te propose quelques conseils pour éviter de prendre de mauvaises habitudes:

* définir une structure:

Utilise plutôt la syntaxe suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
typdedef struct
{
    short sol;
    bool un;
    bool deux;
    bool trois;
    bool quatre;
    bool cinq;
    bool six;
    bool sept;
    bool huit;
    bool neuf;
} cellule;

* Ne jamais utiliser de tableaux, mais des conteneurs.

Pour mon exemple, j'utiliserais les conteneurs de la STL (lib standart du c++):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <vector>   //inclusion du header de la classe vector
using namespace std;  //préviens le compilateur que nous allons utiliser l'espace de nommage std dans ce fichier

vector <cellule> tableau; //déclaration d'un tableau de cellules
for (int i=0; i<100; ++i)
{
    cellule tmpcell;  // definition d'une cellule temporaire
    tmpcell.sol = 0; // initialisation de la cellule temporaire
    tmpcell.un = ... // etc.

    tableau.push_back(tmpcell); //ajout de la cellule dans le tableau
}

//pour accéder à une cellule de ton tableau, tu peux, par exemple, utiliser: montableau.at(x);

L'utilisation des conteneurs est trés fortement conseillée pour plusieurs raisons qui sont expliquées un peu partout sur ce forum (sécurité, simplicité, gestion de la mémoire, etc...).

Ensuite, tu trouveras tout ce dont tu as besoin pour l'utilisation des vecteurs ici

Hope it helps.