probléme avec malloc

**tomasi** · 14/11/2006, 23h30

bonjour,

j'ai le code suivant :

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define N 20
 
int main(){
  char* tab[N];
  char chaine[30]="abc";
 
  tab[0] = malloc(strlen(chaine) + 1);
  strcpy(tab[0], chaine);
 
  printf("%s", tab[0]);
 
  system("pause");
 
  return EXIT_SUCCESS;
}

à la compilation, il me renvoie
invalid conversion from `void*' to `char*'
pour la ligne
tab[0] = malloc(strlen(chaine) + 1);

pourtant tab est un tableau de pointeurs vers des chaines de caractères...

j'avoue ne pas comprendre

**Emmanuel Delahaye** · 14/11/2006, 23h40

Envoyé par tomasi

à la compilation, il me renvoie
invalid conversion from `void*' to `char*'
pour la ligne
tab[0] = malloc(strlen(chaine) + 1);

pourtant tab est un tableau de pointeurs vers des chaines de caractères...

Ce code est correct. Vérifie que tu compiles bien en C (.c) et non en C++ (.cpp ou .C).

**tomasi** · 15/11/2006, 00h16

l'extension avait été mise automatiquement en .cpp

merci

ce code n'est pas valable en c++ ?

**Emmanuel Delahaye** · 15/11/2006, 00h34

Envoyé par tomasi

ce code n'est pas valable en c++ ?

Non. C et C++ sont 2 langages différents.

**salseropom** · 15/11/2006, 09h16

Salut, après le malloc vérifie que tab[0]!=NULL et libère la mémoire aussi (free(tab[0]))

Je pense que ton prgm est destiné à faire autre chose car sinon tu pourrais faire

Code :

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

1
2
 
tab[0]=strdup(chaine);

et ensuite

Code :

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

1
2
 
free(tab[0])

**Gruik** · 15/11/2006, 11h41

strdup est une extension gnu ,)

**Emmanuel Delahaye** · 15/11/2006, 11h49

Envoyé par Gruik

strdup est une extension gnu ,)

C'est surtout une fonction POSIX.1, donc très portable.

**Médinoc** · 15/11/2006, 11h49

Non, je crois bien que c'est POSIX.

Et supporté par Visual (même si la dernière version gueule "deprecated!!!!!" si on n'ajoute pas un underscore devant le nom

)

Edit: Ah, grillé par Emmanuel...

**Gruik** · 15/11/2006, 11h53

Ah désolé, enfin c'est pas marqué dans le man

**Emmanuel Delahaye** · 15/11/2006, 11h59

Envoyé par Médinoc

Et supporté par Visual (même si la dernière version gueule "deprecated!!!!!" si on n'ajoute pas un underscore devant le nom

)

#undef strdup
#define strdup(a) _strdup(a)

**Emmanuel Delahaye** · 15/11/2006, 12h14

Envoyé par Gruik

Ah désolé, enfin c'est pas marqué dans le man

Il est écrit que "strndup(), strdupa(), et strndupa() sont des extensions GNU."

**Gruik** · 15/11/2006, 12h17

Oui, j'ai vu, mais ils parlent pas de POSIX

**Emmanuel Delahaye** · 15/11/2006, 12h25

Envoyé par Gruik

Oui, j'ai vu, mais ils parlent pas de POSIX

Envoyé par POSIX.1

The Open Group Base Specifications Issue 6
IEEE Std 1003.1, 2004 Edition
Copyright © 2001-2004 The IEEE and The Open Group, All Rights reserved.
NAME

strdup - duplicate a string

SYNOPSIS

[XSI] [Option Start] #include <string.h>

char *strdup(const char *s1); [Option End]

DESCRIPTION

The strdup() function shall return a pointer to a new string, which is a duplicate of the string pointed to by s1. The returned pointer can be passed to free(). A null pointer is returned if the new string cannot be created.

RETURN VALUE

The strdup() function shall return a pointer to a new string on success. Otherwise, it shall return a null pointer and set errno to indicate the error.

ERRORS

The strdup() function may fail if:

[ENOMEM]
Storage space available is insufficient.

The following sections are informative.
EXAMPLES

None.

APPLICATION USAGE

None.

RATIONALE

None.

FUTURE DIRECTIONS

None.

SEE ALSO

free(), malloc(), the Base Definitions volume of IEEE Std 1003.1-2001, <string.h>

CHANGE HISTORY

First released in Issue 4, Version 2.

Issue 5

Moved from X/OPEN UNIX extension to BASE.

End of informative text.

La référence de ce site est donnée sur mon site

http://emmanuel-delahaye.developpez.com/notes.htm#posix

**Gruik** · 15/11/2006, 12h45

Okay mercix

**denoo** · 15/11/2006, 13h50

Envoyé par tomasi

bonjour,

j'ai le code suivant :

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define N 20
 
int main(){
  char* tab[N];
  char chaine[30]="abc";
 
  tab[0] = malloc(strlen(chaine) + 1);
  strcpy(tab[0], chaine);
 
  printf("%s", tab[0]);
 
  system("pause");
 
  return EXIT_SUCCESS;
}

à la compilation, il me renvoie
invalid conversion from `void*' to `char*'
pour la ligne
tab[0] = malloc(strlen(chaine) + 1);

pourtant tab est un tableau de pointeurs vers des chaines de caractères...

j'avoue ne pas comprendre

puisque tu programme en C++ (tu a un comppilateur C++) alors:

tu doit convertir le pointeur retourner par malloc au type de 'tab',c-a-d char* (faire un cast)

donc changer la ligne:

Code :

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

tab[0] = malloc(strlen(chaine) + 1);

par:

Code :

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

tab[0] = (char*) malloc(strlen(chaine) + 1);

**Médinoc** · 15/11/2006, 14h04

Envoyé par denoo

Puisque tu programmes en C++ (tu as un compilateur C++) alors:
Tu dois convertir le pointeur retourné par malloc au type de 'tab', c-a-d char* (faire un cast)
Donc changer la ligne:

Code :

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

tab[0] = malloc(strlen(chaine) + 1);

par:

Code :

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

tab[0] = (char*) malloc(strlen(chaine) + 1);

BLASPHÈME!!
Le posteur originel ne programme pas en C++, mais en C. Le seul problème était un problème de nom du fichier.

De plus, en C++, on n'utilise pas malloc(), mais new[].
Sans compter qu'en C++, le cast se fait avec l'opérateur static_cast<>(), et surtout pas avec les parenthèses seules, qui permettent toutes les dérives possibles et imaginables.

À poster des impiétés pareilles (et aussi mal écrites), tu vas encourager notre pauvre posteur égaré à caster le retour de malloc() en C, ô sacrilège.