Discussion :

[cobra85]

J'ai un petit souci pour comparer 2 chaines de caractères:
voici 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
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.

[LittleWhite]

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 .

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.

[tonton fred]

Salut,

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 :

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

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 :

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

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 :

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

1
2
3
 
memset(login, 0, 50);
memset(mdp, 0, 50);

Deux petites remarques:
tu devrais utiliser

Code :

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

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 :

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

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.

[diogene]

@tonton fred :

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 :

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

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

[cobra85]

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 :

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
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

[tonton fred]

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

Code :

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

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

Ha oui