Problème avec realloc

Version imprimable

Voir 40 message(s) de cette discussion en une page

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
char *str_cpy( char *s1, char *s2 )
{
    s1=NULL;
 
    char *RetourRealloc=(char *)realloc( s1, (LongueurChaine(s2)+1)*sizeof( char ) );
 
    if ( RetourRealloc != NULL )
       {
           s1 = RetourRealloc;
           int i;
           for ( i=0 ; i<=LongueurChaine(s2) ; i++ )
               {
                   s1 = RetourRealloc;
                   s1[i] = s2[i];
               }
       }
 
    else printf("erreur");
 
    return s1;
}

Ceci fonctionne mais est-ce correct (mise à part la gestion de l'erreur, je cherche des informations sur perror pour cela ) ? Ce qui m'ennuie c'est que je ne peux pas faire de free sur RetourRealloc sinon je libère aussi s1 puisqu'il pointe dessus.

Merci

10/12/2009, 16h52
Médinoc

s1 ne devrait pas être un paramètre.

Oui cela me perturbe aussi, mais c'est ce le prof demande à moins que je n'ai pas compris l'énoncé ce qui doit être probable. Je vais essayé de le réécrire autrement. Le but étant de recopier s2 dans s1 sachant que s1 a pu déjà être alloué.

Grâce à votre FAQ j'ai trouvé les informations nécessaires sur perror. Voici ce que j'en ai fais.

J'ai mis

Code:

s1;

pour que ça plante et que je puisse faire fonctionner perror. Cela me dit invalid argument et le code erreur associé est 22.

Merci
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 char *str_cpy( char *s1, char *s2 ) { s1; char *RetourRealloc=(char *)realloc( s1, (LongueurChaine(s2)+1)*sizeof( char ) ); if ( RetourRealloc != NULL ) { s1 = RetourRealloc; int i; for ( i=0 ; i<=LongueurChaine(s2) ; i++ ) { s1 = RetourRealloc; s1[i] = s2[i]; } } else { perror("\n > Erreur d'allocation m\202moire dans la fonction str_cpy "); printf("\n > Code erreur associ\202 : %d", errno ); } return s1; }
Cela permet d'afficher l'erreur, maintenant si je veux pourvoir également reprendre à 0 l'opération sans interruption du programme jusqu'à la réussite, je pourrais faire un appel récursif ou c'est proscris ?

Excusez moi de toutes ces questions car je vous sollicite beaucoup, c'est que je n'ai pas toutes ces informations dans mes cours et je n'ai pas acheté de livre ce qui ne va pas tarder.

10/12/2009, 17h04
Médinoc

Es-tu donc incapable de déclarer une variable locale? :roll:

Je pense que oui. Je suppose que ceci est censé me mettre sur la voie pour résoudre le problème du free.

EDIT : non c'est pour le paramètre s1, je n'ai rien dis. C'est qu'à la base j'avais un void comme retour. s1 était bien modifé dans la fonction mais arrivé dans le main ça m'affichait la valeur de s1 déclaré dans le main et pas celle issue de la modification dans la fonction.

Voilà, j'espère que c'est mieux ainsi. Merci

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
char *str_cpy( char *s1 )
{
 
    char *ChaineTemporaire=NULL;
 
    char *RetourRealloc=(char *)realloc( ChaineTemporaire, (LongueurChaine(s1)+1)*sizeof( char ) );
 
    if ( RetourRealloc != NULL && ChaineTemporaire == NULL )
       {
           ChaineTemporaire = RetourRealloc;
           int i;
           for ( i=0 ; i<=LongueurChaine(s1) ; i++ )
               {
                   ChaineTemporaire[i] = s1[i];
               }
       }
 
  else {
            perror("\n > Erreur d'allocation m\202moire dans la fonction str_cpy ");
            printf("\n > Code erreur associ\202 : %d", errno );
       }
 
    free(RetourRealloc);
    return s1;
}

De toute façon, à cause du nom, je me suis mépris sur tes attentions.
Je conseille ceci:

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
char *my_strcpy(char *dest, char const *src)
{
	while( *src != '\0' )
	{
		*dest = *src;
		dest++;
		src++;
	}
	*dest = '\0';
}
 
char *str_realloc_cpy(
 char *s1,       /*[in/free/opt] Pointeur vers une chaîne allouée sur le tas, ou NULL.*/
 char const *s2  /*[in] Chaîne à écrire */
 )
{
	size_t cchNouvelleTaille = LongueurChaine(s2)+1;
	size_t cbNouvelleTaille = cchNouvelleTaille * sizeof *s1;
 
	/*Note: si ça ne compile pas sans le cast, c'est que tu compiles en C++.*/
	char *RetourRealloc = realloc( s1, cbNouvelleTaille );
 
	if ( RetourRealloc != NULL )
	{
		s1 = RetourRealloc;
		my_strcpy(s1, s2);
	}
else
	{
		int err = errno;
		perror("\n > Erreur d'allocation m\202moire dans la fonction str_realloc_cpy ");
		printf("\n > Code erreur associ\202 : %d\n", err );
	}
	return NULL; /*On suit le même comportement que realloc()*/
}

10/12/2009, 17h23
darkwall_37
Ah oui pardon errno est une variable globale, comme conseillé il vaut mieux la sauvegarder dans une autre variable si elle modifiée entre temps.

Merci encore.

Par contre il y a quelque chose que je ne comprends pas dans votre fonction
Code:

1 2 3 4 char *str_realloc_cpy( char *s1, /*[in/free/opt] Pointeur vers une chaîne allouée sur le tas, ou NULL.*/ char const *s2 /*[in] Chaîne à écrire */ )
Pourquoi renvoyez vous NULL, ce ne devrait pas plutôt être le retour de my_strcpy ?
10/12/2009, 17h25
Médinoc

Au passage, pour l'utilisation de realloc(), je conseille ceci:
http://www.developpez.net/forums/d49...e/#post3001764
(voir aussi le message juste après, avec les macros)
10/12/2009, 17h37
darkwall_37

Ok, je mets ça dans mes historiques pour une prochaine fois car il est clair que si je rends ça au prof, je vais me faire engueuler car je n'ai ni le recul nécessaire ni les compétences pour écrire de telles fonctions actuellement.

Voir 40 message(s) de cette discussion en une page