Discussion :

[chimer51]

Bonjour, Bonsoir,
Je reproduit strlcat et strlcpy et j'aimerais savoir qu'elle sont mes erreurs sur c'est deux fonctions

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
 unsigned int    my_strlcat(char *dest, char *src, unsigned int size)
{
    int i = 0;
    int i2 = 0;
    while(dest[i] != '\0')
        i++;
    while((src[i2] && size - i + 1 > i2)
    {
        dest[i++] = src[i2];
        i2++;
    }
    dest[i] = '\0';
    return (i);
}
 
unsigned int    my_strlcpy(char *dest, char *src, unsigned int size)
{
    int i = 0;
    while(src[i] != '\0' && size > i+1 )
    {
        dest[i] = src[i];
        i++;
    }
    if (src[i] == '\0')
        return(i-1);
    else
    {
        while(size > i)
        {
            dest[i] = '\0';
            i++;
        }
        return (i -1);
    }
}

Je n'ai pas reelement compris ce que la fonction devait retourner et les man en generale mais comparer a strncat et strncpy ce sont les seuls interpretation que j'ai pu faire, merci d'avance.

[ternel]

première erreur, les pointeurs vers la source doivent être poineur vers constant.
les noms de variables sont mauvais.
J'aimerai bien voir src_size et dest_size plutot que i et i2


Essaie ce genre de code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
unsigned int my_strlcat(char *dest, char const *src, unsigned int buffer_size)
{
	unsigned int dest_base_size = strlen(dest);
	unsigned int i = 0;
	unsigned int const i_max = (buffer_size-1) - dest_base_size;
	while (src[i]!='\0' && i < i_max) {
		dest[dest_base_size+i] = src[++i];
	}
	dest[dest_base_size+i] = '\0';
	return dest_base_size+i;
}

Je ne corrige pas la seconde fonction, je te la laisse pour l'exercice.

D'après les mans, ces fonctions doivent retourner la taille de la chaine créée (ce que retournerait strlen(dest) après appel de la fonction)
en l'occurence, l'index du '\0' final

[chimer51]

Merci pour la reponse :
Mauvais nom de variable et type a par ca une petite erreur que je n'avais pas vérifier.
Par contre attention

Code :

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

		dest[dest_base_size+i] = src[++i];

enlève le premier caractère a concaténer mais j'ai régler le problème en reprenant mon ancien code a par ca c'est juste encore merci.
Ps : grâce a ça je vais pouvoir aussi compléter strlcpy .
Résolu

[phi1981]

D'après les mans, ces fonctions doivent retourner la taille de la chaine créée (ce que retournerait strlen(dest) après appel de la fonction)
en l'occurence, l'index du '\0' final

C'est bizarre, je n'interprète pas le man de la même façon que toi. Voici ce que j'ai trouvé ici, sur la page man de OpenBSD :

Like snprintf(3), the strlcpy() and strlcat() functions return the total length of the string they tried to create. For strlcat() that means the initial length of dst plus the length of src.

Traduction libre en français :

Comme snprintf(), strlcpy et strlcat() retournent la longueur totale de la chaîne qu'elles ont ESSAYE de créer. Pour strlcat(), ceci signifie la taille initiale de dst plus la longueur de src.

Donc pas nécessairement le nombre de caractères effectivement écrits.

[gangsoleil]

Donc pas nécessairement le nombre de caractères effectivement écrits.

C'est bien ce que retournent strlcat et strlcpy : cela permet de détecter plus facilement les chaines tronquées -- du moins c'est ce qu'explique le man :-)