Discussion :

[math25]

Voila le code :

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
#include <stdio.h>
 
int stringLen(char *str);
int getStr(char str[], int strLen);
 
int main(void)
{
	char sentence[80];
	int nbrechar, nbrechar_no_space, i;
	printf("veuillez saisir une phrase: ");
	nbrechar = getStr(sentence, 80);
	nbrechar_no_space = stringLen(sentence);
	printf("votre phrase contient %d caracteres. sans compter les espaces elle contient %d caracteres\n", nbrechar, nbrechar_no_space);
 
	printf("\"");
	for(i = 0; sentence[i] != '\0'; i++)
	{
		if(sentence[i] == ' ')
			sentence[i] = '*';
		printf("%c", sentence[i]);
	}
	printf("\"");
	return 0;
}
 
int stringLen(char *str)
{
	int i = 0, count = 0;
	while(str[i++])
	{
		if(str[i] == ' ')
			continue;
		count++;
	}
	return count;
}
 
int getStr(char str[], int strLen)
{
	int i = 0;
	while((str[i] = getchar()) != '\n')
		if(++i > strLen - 1)
		{
			--i;
			while(getchar() != '\n');
			break;
		}
	str[i] = '\0';
	return i;
}

Ce code se compile et s’exécute sans problème.
Mais malheureusement, la fonction stringLen ne donne pas le nombre de caractères exact (sans compter les espaces) lorque la phrase commence, au moins, par une espace.

[math25]

j'aimerais seulement comprendre
pourquoi cette fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
int stringLen(char *str)
{
	int i = 0, count = 0;
	while(str[i++])
	{
		if(str[i] == ' ')
			continue;
		count++;
	}
	return count;
}

se comporte d'une manière différente lorsqu'il y'a un espace (ou des espaces au début de la phrase). elle donne le nombre de caractères saisis (sans compter les espaces) + 1.
lorsqu'il y'a des espaces au milieu ou à la fin de la phrase (ou les 2) la fonction donne le résultat attendu.

[edgarjacobs]

Hello,

Très mauvais emploi de l'opérateur ++ !

Ligne 5, tu incrémentes i, et donc ligne 7 tu testes en fait str[i+1]. Un code:

Code :

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

1
2
3
for(int i=0; str[i]; i++)
    if(str[i]!=' ')
        count++;

[math25]

Merci beaucoup edgarjacobs
vous m'avez aidé à trouver le bug

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
 
int stringLen(char *str)
{
	int i = 0, count = 0;
	while(str[i])
	{
		if(str[i] == ' ')
		{
			i++;
			continue;
		}
		i++;	
		count++;
	}
	return count;
}

[edgarjacobs]

De rien, mais ton code est inutilement compliqué.

[Sve@r]

Bonjour

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
int stringLen(char *str)
{
	int i = 0, count = 0;
	while(str[i])
	{
		if(str[i] == ' ')
		{
			i++;
			continue;
		}
		i++;	
		count++;
	}
	return count;
}

Bien compliqué en effet...

Code c :

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

1
2
3
4
5
6
7
8
9
10
int stringLen(char *str)
{
	int i = 0, count = 0;
	while (str[i])
	{
		if (str[i] != ' ') count++;
		i++;	
	}
	return count;
}

Et quand tu apprendras les pointeurs, on pourra se passer de "i" et de cette notation indicée qui engendre des calculs inutiles.

[henderson]

Code :

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

1
2
3
4
 
int cnt=0;
while(str[cnt] != 0) cnt++;
return cnt;

C'est plus simple !

[Sve@r]

C'est plus simple !

Plus simple pour juste compter les caractères... mais le but du PO était de compter les caractères sans les espaces...

[henderson]

Au temps pour moi !!!
Comme tu l'as très bien dit,Sve@r, il vaut mieux travailler par pointeur et non par offset :

Code :

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

1
2
3
4
5
6
7
8
 
int cnt = 0;
while(*str) // tant que le pointeur ne pointe pas sur un 0
	{
	if(*str != ' ') cnt++;
	str++; // pointe sur le caractère suivant
	}
return cnt;

[Guesset]

Bonjour,

On peut aussi remplacer le if(*str != ' ') cnt++; par cnt += *str != ' ';.

Cela a deux avantages :

• cela devient pratiquement illisible .
• cela donne une chance au compilateur de ne pas avoir de saut donc de produire un code un peu plus rapide en utilisant par exemple setnz après le cmp [str], 32.

Capillotracté ?

Salutations

[edgarjacobs]

@Guesset: dans le mode illisible

Code :

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

1
2
3
4
int cnt = 0;
while(*str)
    cnt += *str++ != ' ';
return cnt;