Initialisation d'un char*, malloc ou pas ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Initialisation d'un char*, malloc ou pas ?

cycojesus

Mèo Lười

Quelle solution vous semble la plus adaptée, sachant que les deux semblent fonctionner ?

Code :

#include <stdlib.h> // getenv
#include <stdio.h> // fprintf
#include <string.h> // strncpy
int main( void )
{
char* file = "/.ngstar/scores";
/* SOLUTION 1 */
char* scorefile = getenv( "HOME" );
int size = strlen( scorefile );
/* FIN SOLUTION 1 */
/* SOLUTION 2 */
char* scorefile;
int size = ( strlen( getenv( "HOME" ) ) + strlen( file ) ) * sizeof( char );
scorefile = (char*)malloc( size );
strncpy( scorefile, getenv( "HOME" ), size );
/* FIN SOLUTION 2 */
strncat( scorefile, file, strlen( file ) );
fprintf( stdout, "%i : %s\n", size, scorefile );
}

---------------
Chết rồi ! ✍ ⌥⌘ http://github.com/gwenhael-le-moine/slackbuilds/

Publicité

matafan

La solution 2, puisque dans la 1 tu écris après la fin de la chaine renvoyée par getenv. Par contre, si c'est pour utiliser strncat comme tu le fais, autant utiliser strcat (par contre c'est bien pour strncpy). Et puis sizeof(char) == 1. Et puis faut pas oublier le '\0' final quand tu malloc scorefile. Et puis si tu stockais le retour de getenv dans un pointeur, ça t'eviterais de l'appeler deux fois.

Message édité par matafan le 20-10-2004 à 15:48:13

Taz

bisounours-codeur

strncpy( scorefile, getenv( "HOME" ), size );
-->
boom

char*)malloc( size );
-->
magnifique ce cast

* sizeof( char )
-->
* 1

char* file = "/.ngstar/scores";
-->
manque un const

-->manque des free

cycojesus

Mèo Lười

matafan a écrit :

Ce qui donne

Code :

#include <stdlib.h> // getenv
#include <stdio.h> // fprintf
#include <string.h> // strncpy
int main( void )
{
char* scorefile;
char* file = "/.ngstar/scores";
char* home = getenv( "HOME" );
int size = ( strlen( home ) + strlen( file ) );// * sizeof( char );
scorefile = (char*)malloc( size + 1 );
strncpy( scorefile, home, size );
strcat( scorefile, file );
fprintf( stdout, "%i : %s\n", size, scorefile );
free( scorefile );
}

---------------
Chết rồi ! ✍ ⌥⌘ http://github.com/gwenhael-le-moine/slackbuilds/

cycojesus

Mèo Lười

Taz a écrit :

strncpy( scorefile, getenv( "HOME" ), size );
-->
boom

char*)malloc( size );
-->
magnifique ce cast

* sizeof( char )
-->
* 1

char* file = "/.ngstar/scores";
-->
manque un const

-->manque des free

ok pour le reste, mais pourquoi boom (ça compile et ça marche) ?

---------------
Chết rồi ! ✍ ⌥⌘ http://github.com/gwenhael-le-moine/slackbuilds/

Taz

bisounours-codeur

parce strncpy ça pue, ça fout pas bien l'\0'

cris56

on ta aussi dit de pas caster le malloc et size_t ca mange pas de pain

cycojesus

Mèo Lười

Code :

#include <stdlib.h> // getenv
#include <stdio.h> // fprintf
#include <string.h> // strcpy
int main( void )
{
char* scorefile;
const char* file = "/.ngstar/scores";
char* home = getenv( "HOME" );
size_t size = ( strlen( home ) + strlen( file ) );// * sizeof( char );
scorefile = malloc( size + 1 );
// strncpy( scorefile, home, size );
strcpy( scorefile, home );
strcat( scorefile, file );
fprintf( stdout, "%i : %s\n", size, scorefile );
free( scorefile );
}

DESCRIPTION
The strcpy() function copies the string pointed to by src
(including the terminating `\0' character) to the array
pointed to by dest. The strings may not overlap, and the
destination string dest must be large enough to receive
the copy.

The strncpy() function is similar, except that not more
than n bytes of src are copied. [b]Thus, if there is no null
byte among the first n bytes of src, the result will not
be null-terminated.
[/b]

Taz > Ta remarque sur strncpy correspond à ce que j'ai mis en gras ?

EDIT : pourquoi ne pas caster un malloc ? réponse : http://www.stanford.edu/~blp/writi [...] -cast.html

EDIT2 : et sur le strncpy : http://www.stanford.edu/~blp/writings/clc/strncpy.html

Message édité par cycojesus le 20-10-2004 à 16:22:55

---------------
Chết rồi ! ✍ ⌥⌘ http://github.com/gwenhael-le-moine/slackbuilds/

cris56

je sais psa ce que ca dis ton truc, mais regarde donc le prototype de malloc et tu veras que ca retourne (bien entendu) un void * qui sert de type générique en c, la conversion est implicite

cycojesus

Mèo Lười

cris56 a écrit :

je sais psa ce que ca dis ton truc, mais regarde donc le prototype de malloc et tu veras que ca retourne (bien entendu) un void * qui sert de type générique en c, la conversion est implicite

ça dis, en gros, qu'en C il ne faut pas caster, qu'en C++ il faut caster sachant que le malloc en C++ c'est mal.

---------------
Chết rồi ! ✍ ⌥⌘ http://github.com/gwenhael-le-moine/slackbuilds/

Publicité

cris56

voila, et en c++ quand on interface avec des modules en c, il me semble qu'il y a extern "C", donc aucune excuse pour utiliser des malloc/free dans du code c++

cycojesus

Mèo Lười

cris56 a écrit :

voila, et en c++ quand on interface avec des modules en c, il me semble qu'il y a extern "C", donc aucune excuse pour utiliser des malloc/free dans du code c++

j'ai mis du temps à le trouver celui-là [:ddr555]

How should malloc()'s return value be used?

In general, the return value of malloc() should be used directly in an assignment or initialization. I don't recommend casting its return value:

* The cast is not required in ANSI C.
* Casting its return value can mask a failure to #include <stdlib.h>, which leads to undefined behavior. As C99 slows becomes more popular, this will become less of an issue, because C99 requires all functions to be declared before they are called.
* If you cast to the wrong type by accident, odd failures can result. This is especially true if <stdlib.h> is not #included, as above, but alignment can still cause trouble on particularly odd systems.

Some people do disagree, often because such casts are required in C++. But good C++ code should generally avoid malloc() entirely, using C++'s new instead. Most of the time, it doesn't make sense to compile code as both C and C++, outside of specialized circumstances such as those described by P.J. Plauger in article 9sFIb.9066$nK2.4505@nwrddc01.gnilink.net.

Quelques trucs en C très pratique

Message édité par cycojesus le 20-10-2004 à 17:02:13

---------------
Chết rồi ! ✍ ⌥⌘ http://github.com/gwenhael-le-moine/slackbuilds/

cris56

oui, en c99 ya plus de déclarations implicites

cris56

cycojesus a écrit :

Quelques trucs en C très pratique

cherches faq + c sur google pages francophones et tu trouveras la meme chose en francais et en plus complet (except les liens *.developpez.com)

FORUM HardWare.fr

Programmation

Initialisation d'un char*, malloc ou pas ?

Sujets relatifs
malloc et piles	[php] initialisation d un array ?
Fonction et malloc local	Pourquoi caster un malloc
Initialisation des labels à partir d'un fichier excel	initialisation et réception d'informations pas usb
initialisation communication port serie	initialisation d'une base de données Oracle
[MFC] Initialisation des radio dans une boite de dialogue	[C++] std::vector initialisation / constantes
Plus de sujets relatifs à : Initialisation d'un char*, malloc ou pas ?

Page générée en 0.071 secondes