Comparer deux chaines de caractere: une fixe et une variable

J'ai un petit souci pour comparer 2 chaines de caractères:
voici le code:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

char login1[50]="admin"; char mdp1[50]="azerty";   char login[50]=""; char mdp[50]="";   int main() {   if(recv(ns, login, 50, 0) != SOCKET_ERROR) { printf("Recu : %s\n", login); }   if (strcmp(login1, login) == 0) // Si strcmp renvoie 0 (chaînes identiques) { printf("Les logins sont identiques\n"); } else { printf("login differents\n"); }   }

---

Le code est un peu simplifie ici !!

Je voudrai mettre la valeur que je reçoit dans la chaine de caractère "login" pour ensuite la comparer avec la chaine "login1"

Merci de votre aide.

Bonjour,

Pouvez vous expliciter votre "petit soucis" en voyant le code simplifier, je me dis que tout ce qui est manquant ( pour moi ) et dans le code qui n'a pas été mis :aie:.

J'imagine que le test ne marche pas après réception des données. Avez vous essayer d'afficher login, juste pour savoir ce qu'il y avait dedans, pour être bien sur de dire que le strcmp est bon ou pas.

Salut,

Citation:

---

Envoyé par cobra85

J'ai un petit souci pour comparer 2 chaines de caractères:

---

Ton probleme est ailleurs, l'echec de la comparaison est une consequence.

Code:

---

1 2 3

char login[50]=""; char mdp[50]="";

---

login et mdp sont des chaines de caracteres constantes, donc impossible d'y ecrire.
Il faut les declarer comme suit:

Code:

---

1 2	char login[50]; char mdp[50];

---

Si tu veux les initialiser par le string vide (c'est la plupart du temps une bonne idee de toute maniere) il faut utiliser memset():

Code:

---

1 2 3

memset(login, 0, 50); memset(mdp, 0, 50);

---

Deux petites remarques:
tu devrais utiliser

Code:

---

recv(ns, login, 50 - 1, 0)

---

pour etre sur d'avoir un string correct (c'est a dire termine par \0) et donc eviter les problemes lors du strcmp() si tu recois un string qui fait plus de 50 caracteres.
Il te serait plus simple a long terme d'utiliser

Code:

---

1 2 3

#define TAILLE_BUFFER 50 char login[TAILLE_BUFFER]; recv(ns, login, TAILLE_BUFFER - 1, 0)

---

Ca evite beaucoup de probleme par la suite et c'est plus lisible.

@tonton fred :

Citation:

---

login et mdp sont des chaines de caracteres constantes, donc impossible d'y ecrire.
Il faut les declarer comme suit:.............

---

Il est parfaitement correct d'initialiser des tableaux de cette façon :

Code:

---

1 2	char login[50]=""; char mdp[50]="";

---

Salut,

Merci, pour votre aide, en faite, j'ai trouver assez tard dans la nuit, mais le problème venait du nombre de caractères transmis:

Quand je transmettais "admin", il y avait un '\n' à la fin, donc avec un bout de code, j'ai supprimer ce caractère. Et cela fonctionne.

Pour ceux que cela intéresse, voici le code:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

static void search(char *login) { char *p = strchr(login, '\n');   if (p) { *p = 0; } }   int main { if (recv(ns, login, 50, 0) != SOCKET_ERROR) { search(login); printf("\nRecu :%s", login); }   }

---

Et ensuite je compare.

Merci

Citation:

---

Envoyé par diogene

@tonton fred :
Il est parfaitement correct d'initialiser des tableaux de cette façon :

Code:

---

1 2	char login[50]=""; char mdp[50]="";

---

---

Ha oui :oops: