Discussion :

[djamiirr]

Salut à tous

j'ai creé un code C pour calculer la position d'un caractere (c) dans une chaîne (ch) mais le problème que l'application s'exécute mais il paraît qu'il y a une boucle infinie avec message Windows "pos.exe a cessé de fonctionner" et au dessous de l'IDE il y a les erreurs suivantes :

Code :

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

1
2
3
 C:\test\SansNom1.c In function `pos': 
10 C:\test\SansNom1.c [Warning] comparison between pointer and integer 
11 C:\test\SansNom1.c [Warning] comparison between pointer and integer

et voici mon 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
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
int pos(char c[],char ch[])
{
    int i=0,n=strlen(ch);
          do
          {
          i++;
          }while ((ch[i]!=c)||(i!=n));
          if (ch[i]!=c){i=0;} ;
          return i;
}
int main (int argc,char *argv[])
{
    char c[1],ch[9999];
    scanf("%s %s",ch,c);
    printf("%d",pos(c,ch));
    getchar();
}

aidez-moi SVP

[Médinoc]

Pourquoi avoir déclaré c comme un tableau de un caractère, au lieu de simplement un caractère?
(note: le code scanf() d'un caractère seul est %c)

[alyma]

Alors en effet, petit problème avec char c[1]:
- Soit tu mets char c[2] car il faut inclure le caractère de fin de chaîne et dans ce cas dans ta fonction du compare non pas avec c mais c[0];
- soit tu utilises simplement char c = 'c';

Sinon, ton code pose problème sur deux autres points:
- tu ne testes pas le premier élément de la chaîne (et oui attention aux do ... while), après est-ce volontaire?
- la condition de la boucle n'est pas bonne : que ce passe-il si tu as atteint la fin de la chaîne ? Que vaut alors : "(ch[i]!=c)||(i!=n)" ?
et oui ça fait 0||1 = 1; donc tu continues... Aurais-tu fais une confusion entre || et && ?

Bon courage à toi!

[djamiirr]

Pourquoi avoir déclaré c comme un tableau de un caractère, au lieu de simplement un caractère?
(note: le code scanf() d'un caractère seul est %c)

Merci mais "projet.exe a cessé de fonctionner"

[djamiirr]

Alors en effet, petit problème avec char c[1]:
- Soit tu mets char c[2] car il faut inclure le caractère de fin de chaîne et dans ce cas dans ta fonction du compare non pas avec c mais c[0];
- soit tu utilises simplement char c = 'c';

Sinon, ton code pose problème sur deux autres points:
- tu ne testes pas le premier élément de la chaîne (et oui attention aux do ... while), après est-ce volontaire?
- la condition de la boucle n'est pas bonne : que ce passe-il si tu as atteint la fin de la chaîne ? Que vaut alors : "(ch[i]!=c)||(i!=n)" ?
et oui ça fait 0||1 = 1; donc tu continues... Aurais-tu fais une confusion entre || et && ?

Bon courage à toi!

le but de cette condition quand il trouve ch[i]=c la boucle s'arrete sinon s'il ne trouve pas la boucle s'arret aussi mais a la fin de cette foncion pos si "ch[i]!=c" la pos retour avec "0" sinon elle retour avec valeur "i" laposotion ce c dans ch
en plus j'ai change c a c[2] mais meme erreur

[Médinoc]

C'est pourtant pas compliqué avec un seul caractère:

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
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
int pos(char c, char const ch[]) /*const parce que la fonction ne modifie pas la chaîne*/
{
	int i=0, n=strlen(ch);
	do
	{
		i++;
	} while ( (ch[i]!=c) && (i!=n) ); /*Corrigé || en &&*/
	if (ch[i]!=c) {i=0;}
	return i;
}
int main (int argc,char *argv[])
{
    char c, ch[9999];
    scanf("%9999s %c", ch, &c); /*Attention, scanf("%c") nécessite passage par pointeur*/
    printf("%d", pos(c,ch));
    getchar();
}

[djamiirr]

merci le programme est fonctionel mais pourquoi il faut changer de || a && en effet si une seule condition est vraie (s'il trouve le caractere ou atteint la fin de la chaine) la boucle s'arret

C'est pourtant pas compliqué avec un seul caractère:

Code :

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

1
2
 
	} while ( (ch[i]!=c) && (i!=n) ); /*Corrigé || en &&*/

[deletme]

Bonjour,

L'opération logique && entre les deux tests est nécessaire pour que dès lors qu'une condition est atteinte, la boucle s'arrête. En effet, si tu rencontres ton caractère, tu sors de ta boucle, peu importe s'il reste des caractères dans ta chaine. La condition est vraie tant que mon caractère lu est différent du caractère recherché et tant qu'il reste des caractères à lire.

En laissant ||, si tu rencontres ton caractère mais qu'il te reste des caractères dans la chaine, tu vas poursuivre ta boucle. La condition est vraie tant que le caractère lu est différent du caractère recherché OU s'il reste des caractères à lire.

[djamiirr]

Bonjour,

L'opération logique && entre les deux tests est nécessaire pour que dès lors qu'une condition est atteinte, la boucle s'arrête. En effet, si tu rencontres ton caractère, tu sors de ta boucle, peu importe s'il reste des caractères dans ta chaine. La condition est vraie tant que mon caractère lu est différent du caractère recherché et tant qu'il reste des caractères à lire.

En laissant ||, si tu rencontres ton caractère mais qu'il te reste des caractères dans la chaine, tu vas poursuivre ta boucle. La condition est vraie tant que le caractère lu est différent du caractère recherché OU s'il reste des caractères à lire.

je comprend que avec while && sera ou et || sera et.
c'est juste??

[deletme]

Il ne faut pas le comprendre comme ça.

J'ai deux conditions testées à chaque tour de boucle de mon while :

• est-ce que je veux quitter ma boucle dès que l'une des deux est fausse (je continue tant que la première ET la deuxièmes sont vraies) = &&

• ou alors je veux quitter ma boucle uniquement lorsque les deux sont fausses (je continue ma boucle tant que l'une OU l'autre est vraie) = ||

[Médinoc]

Tu veux "sortir de la boucle si on a trouvé le caractère ou si on arrive au bout".

Mais le while() dit "rester dans la boucle si..." il faut donc inverser la condition en disant "on reste dans la boucle tant qu'on n'a ni trouvé le bon caractère ni arrivé au bout".

Soit comme ceci:

Code C :

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

	} while ( (ch[i]!=c) && (i!=n) ); /*Corrigé || en &&*/

Soit comme cela:

Code C :

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

	} while ( !( (ch[i]==c) || (i==n) ) );

Ces deux formulations sont équivalentes.

[djamiirr]

merci pour vos reponces maintenant j'ai compris le traitement