Discussion :

[bowow]

Bonjour, je suis amené à refaire du C, mais j'aimerais acquérir les bonnes méthodes de programmation:

Admettons 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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
void print_str(const char *pstr)
{
	printf("%s", pstr);
}
 
int main()
{
 
	char strz = 0;
	char *strn = NULL;
	char *dest = NULL;
	char strc[10] = {0};
 
	// Cas N°1 : Affichage d'une chaine constante
	print_str("bordeleuhdemerdeuh");
 
	// Cas N°2 : Allocation dynamique
	strn = (char *)malloc(sizeof(char) * 2);
	if (strn == NULL) perror("bad allocation!\n");
	strn = "bordeleuh";
	printf("%s", strn);
 
	// Cas N°3 : Allocation statique
	int r =	strcpy_s(dest, 10, "biduletruc");
	if (r!=0)
	{
		perror("Bad copy\n");
		return -1;
	}
 
	getchar();
	return 0;
}

1/ Les initialisations sont bonnes ?
2/ Le cas 2 fonctionne et ne plante pas, à ma grande surprise, comment se fait-il? Compte-tenu de l'allocation dynamique de strn, elle n'est pas assez grande? Il ne fait pas de realloc auto, si?
3/ Le cas 3 crash sans surprise,strcpy_s n'a pas de safe method pour éviter de copier dans dest une chaîne trop grande ? Le seul choix possible est de faire un strlen("chaine") avant de faire le strcpy ?


Le but était de provoquer des crashs pour ajouter des safe method de contrôle.

Merci.

[foetus]

Tu dis n'importe quoi

Le cas 2 tu as une fuite mémoire puisque tu écrases le pointeur retourné par malloc sans le libérer. Tu connais free?

Et si le cas 2 ne plante pas c'est parce que tu mets une chaîne statique dans ta variable.
Mais comme je n'ai pas le souvenir d'avoir créer une variable pour stocker une chaîne statique (ce qui est inutile ) j'ai un doute


Pour le cas 3 c'est normal que cela plante ou ne fasse rien: tu passes en paramètre NULL. Tu croyais quoi? Que strcpy_s alloue à ta place
Et bien perdu

these functions return EINVAL and set errno to EINVAL when strDestination or strSource is a null pointer, and they return ERANGE and set errno to ERANGE when the destination string is too small.

[stendhal666]

le cas n° 2:

Code :

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

1
2
3
4
5
char* strn = malloc(strlen("Bordeleux")+1); // pas la peine de sizeof(char), c'est toujours = 1, pas la peine de caster sauf compilateur c++
if (!strn) return -1; // on teste au cas où
strcpy(strn, "Bordeleux");
printf("%s", strn);
free(strn);

le cas n° 3:
dest = NULL donc tu ne pourras rien copier dedans. L'allocation statique c'est les tableaux ou les chaînes constantes connues à la compilation.
Pour te défaire de toute la gestion mémoire, ce qui est bien sympa, tu as la fonction asprintf, mais elle n'est pas dans le standard C; tu l'as en GNU C et CLANG aussi
http://www.google.fr/url?sa=t&rct=j&...EvhRvuOn-rc5HA
D'ailleurs quand tu auras bien maîtrisé toute cette affaire d'allocation, implémenter asprint est un bon exercice :-)

Ce que tu peux faire aussi, c'est copier dans la chaîne destination n caractères de la chaîne d'entrée avec strncpy. Du coup tu copies au max le nombre de caractères alloués dans la chaîne de destination -1, mais cela peut tronquer la chaîne de départ. Et il ne faut pas oublier de mettre le caractère nul à la fin