probleme avec un return

Version imprimable

Voici mon code :

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define TAILLE_PIECE 5
#define NB_PIECE_MAX 30
#define ZERO 48
#define UN 49
#define X_MAJ 88
#define ESPACE 32
 
char **piece_fichier ()
{
    FILE *ptr_piece;
    int i , j , num_piece = 0;
    char ligne_piece [TAILLE_PIECE];
    char piece [NB_PIECE_MAX] [TAILLE_PIECE];
    ptr_piece = fopen ("piece.txt", "r");
    while (!feof (ptr_piece))
	{
	    fgets (ligne_piece, sizeof (ligne_piece), ptr_piece);
 
	    for (i=0 ; i<TAILLE_PIECE-1 ; i++)
	    {
    		if (ligne_piece [i] == ZERO)
    		{
    		    piece [num_piece] [i] = ESPACE;
    		}
    		else if (ligne_piece [i] == UN)
    		{
    		    piece [num_piece] [i] = X_MAJ;
    		}
 
	    }
	    num_piece ++;
	}
	for (j = 0 ; j < NB_PIECE_MAX-1 ; j++)
	{
		piece [j] [TAILLE_PIECE-1] = 0;
	}
	return piece;   erreur : 41 I:\piece_fichier.c [Warning] return from incompatible pointer type
                            erreur : 41 I:\piece_fichier.c [Warning] function returns address of local variable
}
 
int main ()
{
    char piece_retour [NB_PIECE_MAX] [TAILLE_PIECE];
    int j;
 
    piece_retour = piece_fichier ();   erreur : 49 I:\piece_fichier.c incompatible types in assignment
    for (j = 0 ; j < NB_PIECE_MAX-1 ; j++)
    {
        printf ("%s\n" , piece_retour [j]);
    }
    return 0;
}

Si quelqu'un pouvait me dire comment faire pour que je puisse retourner le tableau de chaine de caractère dans le main. merci d'avance.

Allez, c'est mon jour de bonte... :

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
 
char **piece_fichier (int Nb, int Taille, int *NbReel)
{
    FILE *ptr_piece=NULL;
    char **piece = NULL ;
    int i , j , num_piece = 0;
    char *ligne_piece=NULL;
 
    *NbReel = 0 ;
    ligne_piece = calloc ( (Taille+1), 1 );
    if ( ligne_piece == NULL )
       return NULL ;
    piece = calloc ( Nb, sizeof(char *) );
    if ( piece == NULL )
    {
       free(ligne_piece);
       return piece ;
    }
    for ( i = 0 ; i < Nb ; i++ )
      {
         piece[i] = calloc ( (Taille+1), 1);
         if ( piece[i] == NULL )
          {
              if ( i != 0 )
                for ( j = (i-1) ; j >= 0 ; j-- )
                  free(piece[j]);
              free(piece);
              free(ligne_piece);
              piece = NULL ;
              return piece ; 
          }
      }
 
    ptr_piece = fopen ("piece.txt", "r");
    if ( ptr_piece != NULL )
      {
          if ( fgets (ligne_piece, Taille, ptr_piece) != NULL )
             {
                 for (i=0 ; i<Taille ; i++)
                 {
                    if (ligne_piece [i] == ZERO)
                    {
                       piece [num_piece] [i] = ESPACE;
                    }
                    else if (ligne_piece [i] == UN)
                    {
    	          piece [num_piece] [i] = X_MAJ;
                    }
                num_piece ++;
             }
        }
    }
 
    free(ligne_piece);
    *NbReel = num_piece ;
    return piece;
}
 
int main ()
{
    char **piece_retour=NULL;
    int j, NbReel;
 
    if ( (piece_retour = piece_fichier (NB_PIECE_MAX, TAILLE_PIECE, &NbReel)) != NULL )
    {
       for (j = 0 ; j < NbReel ; j++)
         {
           printf ("%s\n" , piece_retour [j]);
         }
 
      /* liberation */
      for (j = (NB_PIECE_MAX-1) ; j >= 0 ; j--)
         free(piece_retour[j]);
      free(piece_retour);
    }
 
    return 0;
}

21/02/2008, 19h44
chamco

oula. ça va être chaud patate à comprendre mai je vais m'y atteler.
Sinon il n'y avait vraiment pas moyen en changeant assez peu mon code d'y arriver?
merci beaucoup en tout cas.
21/02/2008, 19h46
souviron34

ben je fais quasi la meme chose, sauf que pour retourner une variable locale, la seule chose qui aie une valeur a l'exterieur est un pointeur.... pointant sur une zone allouee...

Donc, au lieu de tenter de manipuler( avec insucces prevu) un tableaud etaille fixe, je passe les parametres de dimensions a la routine qui alloue a la bonne dimension, c'est tout...

tu alloues la mémoire pour ton tableau dans ton main, donc autant donner ce tableau en paramètre de ta fonction, plutôt que d'en définir un autre

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define TAILLE_PIECE 5
#define NB_PIECE_MAX 30
#define ZERO 48
#define UN 49
#define X_MAJ 88
#define ESPACE 32
 
void piece_fichier (char piece[NB_PIECE_MAX][TAILLE_PIECE])
{
    FILE *ptr_piece;
    int i , j , num_piece = 0;
    char ligne_piece [TAILLE_PIECE];
    ptr_piece = fopen ("piece.txt", "r");
    while (!feof (ptr_piece))
	{
	    fgets (ligne_piece, sizeof (ligne_piece), ptr_piece);
 
	    for (i=0 ; i<TAILLE_PIECE-1 ; i++)
	    {
    		if (ligne_piece [i] == ZERO)
    		{
    		    piece [num_piece] [i] = ESPACE;
    		}
    		else if (ligne_piece [i] == UN)
    		{
    		    piece [num_piece] [i] = X_MAJ;
    		}
 
	    }
	    num_piece ++;
	}
	for (j = 0 ; j < NB_PIECE_MAX-1 ; j++)
	{
		piece [j] [TAILLE_PIECE-1] = 0;
	}
}
 
int main ()
{
    char piece_retour [NB_PIECE_MAX] [TAILLE_PIECE];
    int j;
 
    piece_fichier (piece_retour);
    for (j = 0 ; j < NB_PIECE_MAX-1 ; j++)
    {
        printf ("%s\n" , piece_retour [j]);
    }
    return 0;
}