Discussion :

[shvan]

Bonjour,

Je suis débutant en c et il m'arrive souvent de croiser des 'bus error' à tel point que je préfère les segfaults
J'ai pour exercise de créer une fonction qui transforme en maj les caractères minuscule d'une chaîne.

je n'attends pas de solution à l'exercise (plus amusant de trouver soi meme) juste une explication à l'error bus au lancement du prog.

Code :

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

1
2
3
4
5
6
7
8
9
10
char	*ft_strupcase(char *str)
{
	while (*str != '\0')
	{
		if (*str <= 122 && *str >= 97)
			*str = *str - 32; // ligne en cause du bus error  :( why ?
		str++;
	}
	return (str);
}

merci d'avance

[Pyramidev]

Bonjour,

Je parie que tu essaies de modifier une chaîne non modifiable, par exemple avec un appel du genre ft_strupcase("test").
En langage C, bien que les string litterals comme "test" ne soient pas modifiables, elles sont convertibles en pointeurs vers caractère non constant, ce qui est aberrant, mais c'est ainsi.

« String literals are not modifiable (and in fact may be placed in read-only memory such as .rodata). If a program attempts to modify the static array formed by a string literal, the behavior is undefined. »
Source : http://en.cppreference.com/w/c/language/string_literal

Il faut que tu passes en argument de ta fonction un tableau de char que tu as alloué dans la pile, par exemple ainsi :

Code :

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

1
2
char tab[] = "test";
ft_strupcase(tab);

A part ça, le code de ta fonction aurait été plus clair en remplaçant 97 par 'a', 122 par 'z' et 32 par ('a' - 'A').

[shvan]

Franchement un grand MERCI !

[Sve@r]

Code c :

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

1
2
if (*str <= 122 && *str >= 97)
	*str = *str - 32;

Bonjour

Sais-tu que les constantes écrites sous forme "lettre" ont la même valeur numérique que leur valeur ascii et peuvent s'utiliser comme des nombres ?

Autrement dit, 'b' - 'a' vaut 1 (et ce, dans tous les sens que permettent les équations algébriques). Ainsi, tu peux réécrire ta conversion de façon plus lisible de cette façon

Code c :

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

1
2
if (*str >= 'a' && *str <= 'z')
	*str = *str - ('a' - 'A');   // Ou encore mieux *str = *str - 'a' + 'A'