Discussion :

[Flo FR]

Bonjour,

J'ai un un bug vraiment bizarre :
- Lorsque je remplis mon premier tableau "tab" jusque la valeur 27, mon tableau "phrase" s'affiche correctement.
- Mais lorsque je remplis mon premier tableau "tab" jusque la valeur 27 incluse ou strictement inférieur à 28, alors dans mon tableau "phrase", la 1ère lettre en l'occurence le "b" ici disparait ...

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
#include <iostream>
 
using namespace std;
 
int fct_count(char* ph)
{
	int i = 0;
	while (ph[i] != '\0')
		i++;
	return (i);
}
 
int main(void)
{
	int tab[27];
	char phrase[] = "bonjour le monde";
 
	int nb = fct_count(phrase);
 
	for (int i = 0; i <= 27; i++)
		tab[i] = 0;
 
	for (int y = 0; y < nb; y++)
		cout << phrase[y];
 
	return (0);
}

Une fois compilé ce code affiche exactement : " onjour le monde" (un espace au lieu d'un b)

Merci d'avance pour votre aide

Cordialement, Flo FR

[Luc Hermitte]

Il n'y a rien de bizarre, tu itères 28 fois sur un tableau de 27 cases ...

[ternel]

Réponse simple:
Compte les nombres entre 0 et 27 inclus.

Réponse longue:
La première ligne du main, int tab[27], déclare un variable locale qui est un tableau de 27 int. C'est à dire que les 27 premiers ints de la pile seront tab.
La ligne suivante, char phrase[] = "bonjour le monde" déclare un tableau de caractères. Le compilateur les places sur la première case de la pile, donc au bout de tab.

Le for (int i = 0; i <= 27; i++) tab[i] = 0; mets 0 dans les éléments de tab[0] à tab[27], inclus.
C'est là que la réponse courte intervient.

tab[27], c'est 27 * sizeof(int) plus loin que tab[0].
C'est à dire que c'est le 28° int de la pile.
Manque de chance, ce n'est plus dans tab, mais le début de phrase.

C'est un "buffer overflow", fait manuellement. C'est une erreur pouvant mener, dans d'autres situations, à des failles de sécurité graves (raison pour laquelle gets() est interdite)
La solution, c'est de toujours utiliser < pour un tableau dans les boucles for. Ici, ca donnerait for (int i = 0; i < 27; i++) tab[i] = 0;;

Par contre, ta fonction auxiliaire n'a d'utilité que pour l'exercice, vu qu'il existe strlen pour ca, si on reste avec les char*.
Le mieux étant d'utiliser std::string (dans <string>, ou en général dans <iostream>)

Par ailleurs, le cout peut aussi s'écrire sans boucle en profitant qu'un char[] soit convertible en char*: cout << phrase << endl;.

[Flo FR]

En effet, j'étais sûr que c'était une erreur débile, je sais pas pourquoi mais je pensais que mon tableau était de taille 28 (dans ma tête j'avais machinalement compté de 0 à 27).

Merci pour ton aide

[Flo FR]

Merci leternel pour le complément d'information et l'astuce pour éviter les "buffer overflow".
Pour ce qui est de strlen, je n'ai même pas fait ça dans le cadre de mon exercice mais j'avais besoin d'une fonction pour compter et vu que ça fait un petit moment que je n'ai pas codé, je n'ai pas pensé à utiliser strlen.