Discussion :

[bj303931]

Bonjour, ça marche pas.

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
#include <stdio.h>
#include <stdlib.h>
 
int c(char *s, char *c){
    while(*s){
 
        if (s==c){return 1;}
        else {return 0;}
 
    }
}
 
int main()
{
    printf("%d",c("pikachu", 'i'));
    return 0;
}

[nnovic]

Hello,
Ce serait sympa que tu prennes deux minutes pour expliquer le problème: qu'est-ce que tu veux faire, qu'est-ce que tu as obtenu. Et c'est pareil pour tes autres posts, d'ailleurs. Sinon, tout ce que tu obtiendras seront des réponses aussi brèves que tes questions. On n'est pas là pour faire tes exos à ta place.

[Daïmanu]

Bonjour.

Je plussoie nnovic, tu ne fournis pas assez de détails.

Néanmoins, il y a plusieurs soucis avec ton code :
La fonction et un de ses paramètres s'appelle c. Il y a risque de les confondre, et le compilateur devrait d’ailleurs t'en avertir. Tu devrais les nommer de façon plus explicite.
La fonction cherche un caractère dans une chaîne, sa signature devrait être int search(char const *string, char c).
La fonction retourne un booleen (vrai/faux) ou une position ? Dans le premier cas tu devrais utiliser #include <stdbool.h> et utiliser les constantes true / false pour plus de clarté.

[stendhal666]

Il y a trois problèmes majeurs:
1) tu n'incrémentes pas s: si tu veux parcourir la chaîne 's', il faudrait un ++s à la fin de ta boucle
2) tu compares l'adresse des pointeurs plutôt que de comparer le caractère sur lequel ils pointent: *s == *c plutôt s == c
3) tu demandes un pointeur sur char pour le deuxième argument mais tu appelles la fonction avec un char, pas un pointeur sur char

Mais je suis d'accord qu'il vaudrait mieux changer la signature et prendre un char en valeur plutôt que son adresse...

[ternel]

Et surtout, on ne retourne pas de manière systématique dès le premier pas de boucle.
le return faux est à mettre après la boucle.