Discussion :

[gealex]

bonjour,

j'ai un problème avec mon programme de bataille navale :
Tout d'abord j'ai crée un bateau avec cette fonction :

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
29
30
31
32
33
34
35
36
 
typedef struct 
{
   int colonne;
   int ligne;
   int longueur;
   char orientation;
   int nbtouches;
}bateau;
 
typedef struct 
{
   bateau *contenu;
   int etat;
}Case;
 
typedef struct 
{
   Case tableau[9][9];
}grille;
 
bateau SaisirBateau (int l)
{
bateau b;
b.longueur = l;
printf("\nLigne 1ere case : ",b.ligne);
scanf("%d",&b.ligne); 
 
printf("\nColone 1ere case : ",b.colonne);
scanf("%d",&b.colonne);
 
printf("\nOrientation : ",b.orientation);
scanf("%c%c",&b.orientation,&b.orientation);
b.nbtouches=0;
return (b);
}

Ensuite j'utilise cette action qui permet de placer les informations d'un bateau pointé dans les cases correspondantes.
La voici :

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
 
void Ajouterbateau (bateau *pb, grille *g)
{
     int i,j,l,x;
     char o;
     l = (*pb).longueur;
     i = (*pb).ligne;
     j = (*pb).colonne;
     o = (*pb).orientation;
     (*pb).nbtouches=0;
 
     if (o=='b')
     {
        for (x=0;x<l;x++)
        {
          g->tableau[i][j].contenu = pb;
          g->tableau[i][j].etat = 1; 
        }
     }
     else if (o=='d')
     {
        for (x=0;x<l;x++)
        {
           g->tableau[i][j].contenu = pb;
           g->tableau[i][j].etat = 1; 
        }
     }
}

le probleme c'est que lorsque je retourne le pointeur sur un bateau placé à une case (i,j) du tableau avec cette fonction.

Code :

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

1
2
3
4
5
 
bateau* casebateau (grille g,int i, int j)
{  
   return(g.tableau[i][j].contenu);
}

je constate que ce pointeur est le même pour tous les bateaux rentrés !!!! Lorsque je veux modifier les informations d'un bateau à la case (i,j), il me modifie en fait celles du dernier bateau rentré. Comprend pô
En fait je voudrai que les pointeurs soient differents pour chaque bateau.

voila si vous pouvez me donner un ptit coup de main, c'est pas de refu

[diogene]

L'erreur est sans doute dans le code qui appelle SaisirBateau puis Ajouterbateau . Il a probablement la structure suivante :

Code :

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

1
2
3
4
5
 
bateau bat;
....
bat = SaisirBateau(....);
Ajouterbateau (&bat,...);

Si c'est le cas, tous les bateaux de la grille comportent l'adresse d'un même bateau, bat.
D'après ce que je crois comprendre, la logique voudrait que SaisirBateau non seulement initialise les données du bateau mais crée de façon dynamique les bateau :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
bateau  * SaisirBateau (int l)
{
bateau  * b = malloc(sizeof(bateau));
b->longueur = l;
....
return (b);
}
.....
bateau * bat;
.....
bat = SaisirBateau(....);
Ajouterbateau (bat,...);

Ne pas , dans ce cas oublier de détruire les bateau quand on n'en a plus besoin.

[crocodilex]

C'est certainement dut à ta fonction :

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
bateau SaisirBateau (int l)
{
   bateau b;
   b.longueur = l;
   printf("\nLigne 1ere case : ",b.ligne);
   scanf("%d",&b.ligne); 

   printf("\nColone 1ere case : ",b.colonne);
   scanf("%d",&b.colonne);

   printf("\nOrientation : ",b.orientation);
   scanf("%c%c",&b.orientation,&b.orientation);
   b.nbtouches=0;
   return (b);
}

La variable "b" étant locale à ta fonction, elle est indéfinie au retour.

[diogene]

crocodilex :

La variable "b" étant locale à ta fonction, elle est indéfinie au retour.

C'est exact, mais la fonction retourne une COPIE de la variable locale et la valeur de retour est elle bien définie (ce code ne retourne pas l'adresse de la variable locale)

[crocodilex]

crocodilex : C'est exact, mais la fonction retourne une COPIE de la variable locale et la valeur de retour est elle bien définie (ce code ne retourne pas l'adresse de la variable locale)

Tu as entièrement raison. Je me rends compte que je viens de dire une grosse bêtise (trop de précipitation). Merci de m'avoir corrigé...

[gealex]

merci de toutes vos réponses mais je dois faire ma bataille en statique. Et je crois que avec du malloc c'est dynamique

[crocodilex]

merci de toutes vos réponses mais je dois faire ma bataille en statique. Et je crois que avec du malloc c'est dynamique

Alors dans ce cas, crée un tableau de bateau

Code :

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

bateau bat[NB_MAX_BATEAU];

[gealex]

ok merci je vais essayer de faire ça

[gealex]

En fait, au lieu de faire un tableau de cases je fais un tableau de bateau c'est bien ça ?

[crocodilex]

En fait, au lieu de faire un tableau de cases je fais un tableau de bateau c'est bien ça ?

Ben je ne sais pas trop ce que tu veux faire. Si tu ne veux pas créer des bateaux de manière dynamique comme le propose Diogène, tu peux essayer comme cela (c'est un exemple incomplet) :

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
29
30
31
32
33
34
#define NB_BATEAU  3
 
void SaisirBateau (bateau *b)
{
    char c;
    printf("\nLongueur : ");
    scanf("%d",&b->longueur); 
    while ((c = getchar()) != '\n' && c != EOF);
 
    printf("\nLigne 1ere case : ");
    scanf("%d",&b->ligne); 
    while ((c = getchar()) != '\n' && c != EOF);
 
    printf("\nColone 1ere case : ");
    scanf("%d",&b->colonne);
    while ((c = getchar()) != '\n' && c != EOF);
 
    printf("\nOrientation : ");
    scanf("%c",&b->orientation);
    while ((c = getchar()) != '\n' && c != EOF);
    b->nbtouches=0;
}
 
int main (void)
{
    int i;
    bateau bat[NB_BATEAU];
 
    for (i=0; i<NB_BATEAU; i++) {
        SaisirBateau(&bat[i]);
    }
 
    return EXIT_SUCCESS;
}

J'ai fais quelques corrections dans la fonction "SaisirBateau", mais saches que la fonction scanf est déconseillée pour les débutants (d'ailleurs cela ne marchait pas dans le code original).

[gealex]

oki oki, je vais essayer comme ça. Encore une fois