Problème initialisation tableau 2D

**MasterNico** · 18/10/2013, 10h49

Bonjour à tous mes lecteurs !

Voila j'ai un tout petit soucis de rien du tout, une fonction d'initialisation qui me fait une erreur de segmentation ... J'ai beau chercher je ne voit pas l'erreur.

voila le code que j'expliquerais juste après :

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
struct Case
{
    int x;
    int y;
    int numCaseL;
    int numCaseC;
};
 
void initTableau(Case tab[][4])
{
    int j = 0;
    for(int i=0;i<9;i++)
    {
        for(int i=0;i<5;i++)
        {
            tab[j][i].x = 25;
            tab[j][i].y = 20*i;
            tab[j][i].numCaseL = 20;
            tab[j][i].numCaseC = 20;
        }
        j++;
    }
}

Donc je cherche à initialiser un tableau de case qui est censé représenter une grille. l'initialisation fonctionnait jusqu'à présent mais depuis que j'ai rajouter les champs numCaseL et numCaseC et que j'essaie de les initialiser de la même manière, sa plante !

Merci de m'aider a résoudre ce petit problème.

EDIT : J'ai oublier de préciser que ce code reprends des principes du C mais est en réalité compilé comme du C++. Il parait que pédagogiquement il vaut mieux voir une bibliothèque qui n'est pas orientée objet au début ... J'utilise donc la SDL

**gangsoleil** · 18/10/2013, 12h06

Ta fonction prend tab[][4], mais dedans tu accedes a tab[j][i] avec i variant de 0 a 8...
Ca ne te choque pas ?

[Edit]
Tes deux boucles ont pour indice i... Il est tres peu probable que ce soit ca que tu veuilles faire.
Du coup, le j++ n'a pas sa place.

[Edit 2]
Non, en fait, c'est encore pire que ca, car tu ecris deux fois de suite for (int i...

Sauf si tu fais des trucs vraiment tordus, je te conseille de reprendre tranquillement les bases des boucles.

**MasterNico** · 18/10/2013, 13h01

euhh ...

certes il y a conflit ... cependant le code que j'ai posté est légèrement moins récent que celui que j'utilise et il ne fonctionne pas non plus ... :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void initTableau(Case tab[][4])
{
    for(int j=0;j<9;j++)
    {
        for(int i=0;i<5;i++)
        {
            tab[j][i].x = 25;
            tab[j][i].y = 20*i;
            tab[j][i].numCaseL = 20;
            tab[j][i].numCaseC = 20;
        }
    }
}

Je trouve que tu vas vite en besogne en me disant d'aller revoir les boucles ... Je code en six langages différent ... donc je n'ai pas beoins de revoir les boucles ou les structures conditionnelles

. Je partais du principe que si mon compilo (g++) ne me signale pas une redéclaration de i, il n'y avais pas de problème.

EDIT : Après vérif au débugguer, le conflit n'a pas lieu mais certes c'est plus compréhensible de cette manière je te l'accorde. Sa ne méritais tout de même pas un vulgaire "retourne voir le cours sur les boucles débutant"

EDIT 2 : Les deux lignes qui pose problème, ce sont les .numCase encore une fois. Commentées, l'algo tourne, on les dé-commente et on se retrouve avec une erreur de segmentation.

**gangsoleil** · 18/10/2013, 14h12

Envoyé par MasterNico

certes il y a conflit ... cependant le code que j'ai posté est légèrement moins récent que celui que j'utilise et il ne fonctionne pas non plus ...

Je trouve que tu vas vite en besogne en me disant d'aller revoir les boucles ... Je code en six langages différent ... donc je n'ai pas beoins de revoir les boucles ou les structures conditionnelles

. Je partais du principe que si mon compilo (g++) ne me signale pas une redéclaration de i, il n'y avais pas de problème.

Moi, je me suis base sur le code que tu postes, je ne peux pas deviner que ce n'est pas celui que tu executes. Et oui, le code que tu as posté montre des soucis dans l'utilisation des boucles qui necessitent de revoir les bases, et ce meme si tu connais 75 langages de programmation.

Regardons ce nouveau code, qui est par ailleurs incomplet, et ne nous permet pas de reproduire ton erreur.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void initTableau(Case tab[][4])
{
    for(int j=0;j<9;j++)
    {
        for(int i=0;i<5;i++)
        {
            tab[j][i].x = 25;
            tab[j][i].y = 20*i;
            tab[j][i].numCaseL = 20;
            tab[j][i].numCaseC = 20;
        }
    }
}

En parametre, tab[][4]. Mais on ne sait pas comment est alloue le tableau dans la fonction appelante : allocation dynamique ou statique ? Est-ce que tu alloues suffisamment d'espace ?
Si je te demande cela, c'est parceque tu fais iterer i sur 5 valeurs, ce qui risque deja de poser probleme vu que tu definies tab[][4].

[HS]
Lorsque je reponds a quelqu'un, je me base sur les elements que l'on me fournit. Toute personne postant le code que tu as poste aurait eu la meme reponse, toi comme mon voisin de boulot comme D. Ritchie.
En aucun cas ce n'est une remise en cause de tes competences, il nous arrive a tous de faire des erreurs grossieres et de ne pas les voir.

**ManusDei** · 18/10/2013, 14h13

On peut voir comment tu appelles la fonction initTableau ?
Et comment tu créé ta Case dans ton main ?

tab[][4] je trouve ça très bizarre comme écriture.
Tu as regardé les adresses mémoire des différents tab[i][j].x tab[i][j].y tab[i][j].numCaseL tab[i][j].numCaseC ?

**Médinoc** · 18/10/2013, 14h24

Envoyé par ManusDei

tab[][4] je trouve ça très bizarre comme écriture.

C'est une écriture parfaitement normale pour un tableau 2D dont au moins la seconde dimension est statique.

Seule la première dimension (ici vide) est ignorée par le compilateur.

**Sve@r** · 18/10/2013, 14h38

Envoyé par MasterNico

euhh ...

certes il y a conflit ... cependant le code que j'ai posté est légèrement moins récent que celui que j'utilise et il ne fonctionne pas non plus ... :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void initTableau(Case tab[][4])
{
    for(int j=0;j<9;j++)
    {
        for(int i=0;i<5;i++)
        {
            tab[j][i].x = 25;
            tab[j][i].y = 20*i;
            tab[j][i].numCaseL = 20;
            tab[j][i].numCaseC = 20;
        }
    }
}

Bonjour
Tu as un tableau de 4 éléments et tu tentes d'en remplir 5...

Envoyé par MasterNico

Je trouve que tu vas vite en besogne en me disant d'aller revoir les boucles ... Je code en six langages différent ... donc je n'ai pas beoins de revoir les boucles ou les structures conditionnelles

.

Ouais ouais t'es un super codeur. Ben retourne alors apprendre à compter combien il y a d'éléments entre 0 et 4 inclus...

Envoyé par MasterNico

Je partais du principe que si mon compilo (g++) ne me signale pas une redéclaration de i, il n'y avais pas de problème.

Moi je pars du principe que c'est à moi de réfléchir sans tout laisser gérer mon code par des outils pas forcéments parfaits...

Envoyé par MasterNico

Envoyé par gangsoleil

Sauf si tu fais des trucs vraiment tordus, je te conseille de reprendre tranquillement les bases des boucles.

Sa ne méritais tout de même pas un vulgaire "retourne voir le cours sur les boucles débutant"

Si tu veux citer, fais-le avec honnêteté. gangsoleil ne t'a pas dit "retourne voirs un cours sur les boucles" sur ce ton impératif, il t'a juste conseillé, en ne connaissant de toi que ton exemple de code, de reprendre tranquillement les boucles. Accessoirement si j'avais été toi, j'aurais moi dit "merci" plutôt que monter sur tes grands chevaux en disant "je code en six langages différents"...

**MasterNico** · 18/10/2013, 17h58

Il semblerai en effet que je me sois mal comporté ... Je présente mes plus plates excuse en particulier a gangsoleil qui a du suporter en premier ma suffisance. J'essaierais à l'avenir de bien surveiller mes réactions.

Il est bien évident aussi que je vous remercie ne serai-ce que pour prendre le temps de lire mon code ( dont ce n'est pas la meilleure représentation

)

Je n'ai pas encore tester la solution proposé mais je n'arrive pas à comprendre, dans le cas ou l'erreur serait bien là, pourquoi je n'ai pas d'erreur lorsque je commente ces satanés lignes ... Si cela m'avais levé une erreur, j'aurais vu de suite ma faute et je n'aurais pas fais c**** le monde !

Encore une fois ma réponse de tout a l'heure était rapide et ne reflète en aucun cas mes pensées profondes mais plutôt un ras-le-bol de la matinée.

Je teste dans la soirée et je vous dit si le sujet est résolu ...

**Sve@r** · 18/10/2013, 18h30

Envoyé par MasterNico

Il semblerai en effet que je me sois mal comporté ... Je présente mes plus plates excuse en particulier a gangsoleil qui a du suporter en premier ma suffisance. J'essaierais à l'avenir de bien surveiller mes réactions...
Encore une fois ma réponse de tout a l'heure était rapide et ne reflète en aucun cas mes pensées profondes mais plutôt un ras-le-bol de la matinée.

Arf, on a tous nos soucis qui parfois remontent inconsidérément.

Envoyé par MasterNico

Je n'ai pas encore tester la solution proposé mais je n'arrive pas à comprendre, dans le cas ou l'erreur serait bien là, pourquoi je n'ai pas d'erreur lorsque je commente ces satanés lignes ... Si cela m'avais levé une erreur, j'aurais vu de suite ma faute et je n'aurais pas fais c**** le monde !

Puisqu'on part tous sur de nouvelles bases, et si en remplaçant 5 par 4 ça ne fonctionne toujours pas, alors pour qu'on puisse t'aider il sera impératif que tu nous montres la définition exacte du tableau avant l'appel à init()...