Les variables

Version imprimable

Les variables

Salut tout le monde!

Voilà voilà, j'ai débuté la programmation en c++ le week-end dernier. J'ai appris avec les tutos vidéos de titotuto.fr.
J'ai décidé d'essayer des programmes simples et de demander de l'aide en cas de soucis pour m'aider à développer mes compétences.

Actuellement, je suis sur un programme de test de mot de passe. Tout simplement, on demande à l'utilisateur un mot de passe puis on vérifie sa validité.

Voilà le code source:
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 #include <stdio.h> #include <stdlib.h> int main(void) { int vraipass(1234),userpass; // initialisation du vrai mot de passe, et de celui entré par l'utilisateur printf("Veuillez entrer votre mot de passe puis appuyez sur Enter :\n"); // demande du mot de passe scanf("%f",&userpass); // enregistrement du mot de passe entré par l'utilisateur if(userpass=vraipass) // vérification de la validité du mot de passe { printf("\nLe mot de passe est correct.\n\n"); // le mot de passe est valide } else { printf ("\n Le mot de passe entré est invalide.\n\n"); // le mot de passe n'est pas valide } system ("pause"); return 0; }
J'ai un petit problème : peu importe le mot de passe rentré, le programme me dit qu'il est correct. Donc il considère que userpass=vraipass. Comment cela se fait...?

Ensuite, j'ai essayé de mettre une variable char (char vraipass(BlaBla),userpass;) pour avoir un mot de passe avec des lettres, mais le programme m'indique un problème sur cette ligne au moment de la compilation... Comment créer une variable en caractères? La ligne "if(userpass=vraipass)" fonctionnera-t-elle toujours?

Et enfin, mais ça ce n'est qu'accessoire, y a-t-il un moyen pour que le programme revienne à la ligne 9 en cas de mot de passe éroné, donc après avoir indiqué la ligne 18.

Voilà tout, merci d'avance pour votre aide! ;)

12/09/2011, 22h07
ram-0000

Salut,

Ici, c'est un forum C++ et en plus, tu débutes en C++ donc autant commencer par les bonnes habitudes.

scanf() et printf(), c'est du C. En C++, il y a std::cin et std::cout.

Maintenant, les erreurs :

Code:

scanf("%f",&userpass);

Le format %f attend l'adresse d'une variable de type float, tu lui donnes l'adresse d'une variable de type int. ==> Erreur, crash potentiel.

Code:

if(userpass=vraipass)

L'affectation se fait un le signe "=", le test d'égalité se fait avec le signe "==". Cette erreur aurait dû se voir avec un compilateur bien réglé au niveau de la génération des warnings/erreurs.

Au fait, une question, que se passe t'il (une fois que tu auras corrigé tous les bugs) si lors de la saisie du mot de passe (tu attends un nombre), l'utilisateur saisit "toto". Probablement pas ce que tu as prévu.

En règle générale, il ne faut pas faire confiance à l'utilisateur et tout ce qui est saisi (au moins au clavier) devrait être validé et pour cela reçu dans un buffer de caractères pour analyse et validation par le programme.

Salut,

Citation:

Envoyé par Over703

Et enfin, mais ça ce n'est qu'accessoire, y a-t-il un moyen pour que le programme revienne à la ligne 9 en cas de mot de passe éroné, donc après avoir indiqué la ligne 18.

Oui, cela s'appelle les boucles
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 bool correct =false // pour pouvoir utiliser le résultat de la comparaison // ailleurs que dans le text ailleurs que dans le texte while( ! correct ) // tant que ce n'est pas correct (on aurait pu écrire //while( correct == false ) { printf("Veuillez entrer votre mot de passe puis appuyez sur Enter :\n"); scanf("%d",&userpass); correct = (userpass == vraipass) if(correct) { printf("\nLe mot de passe est correct.\n\n"); } else { printf ("\n Le mot de passe entré est invalide.\n\n"); } }
(je n'ai pas corrigé les erreurs de logique !!!)

Deux remarques complémentaires :

1- Il faut indenter son code, c'est une question de lisibilité, mais il faut essayer de le faire de manière intelligente !

Le fait d'aligner les blocs sur la fin du test (comme tu l'as fait) risque de t'emmener très loin et de provoquer l'apparition d'une barre de défilement horizontale, ce qui nuira autant à la lisibilité du code que le fait de ne pas indenter du tout ;)

Utilise "simplement" un nombre fixe d'espaces (entre quatre et huit, c'est pas mal :D ) "suffisant" pour que l'on puisse clairement identifier les différents blocs ;)

2- Les commentaires c'est bien, à condition qu'ils aient un sens et ne se contentent pas de répéter ce que le code fait !

On se doute que le if vérifie la validité du mot de passe (tu as eu l'intelligence d'utiliser des variables suffisamment explicite :D ) ou que printf("mot de passe incorrect") va afficher le message en question !

Par contre, le fait d'indiquer le but d'une variable ou d'une instruction, pouvant être mal comprise rendra tes commentaires des plus utiles ;)

13/09/2011, 09h31
Over703

Citation:

Envoyé par ram-0000

Ici, c'est un forum C++ et en plus, tu débutes en C++ donc autant commencer par les bonnes habitudes.

scanf() et printf(), c'est du C. En C++, il y a std::cin et std::cout.

Ah bon... oO

Citation:

Envoyé par ram-0000

Au fait, une question, que se passe t'il (une fois que tu auras corrigé tous les bugs) si lors de la saisie du mot de passe (tu attends un nombre), l'utilisateur saisit "toto". Probablement pas ce que tu as prévu.

En règle générale, il ne faut pas faire confiance à l'utilisateur et tout ce qui est saisi (au moins au clavier) devrait être validé et pour cela reçu dans un buffer de caractères pour analyse et validation par le programme.

Bah... Je n'y suis pas encore. x')

@Koala01 : Merci pour le code, je testerais à partir de ça ce soir! Et merci aussi pour le cours sur les commentaires! :mrgreen:

Re bonjour tout le monde, j'ai un nouveau problème.

En ayant marre des icônes en bazar sur le bureau, j'ai décidé de tout simplifier avec un programme permettant de charger une application, ou jeu, au choix.
Pour le moment, j'en suis au choix entre lancer un jeu ou une application.

Voilà 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 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 #include <iostream> using namespace std; int main() { char choixType(0), choixJeu(0), ChoixApp(0); int choixTypeCorrect(0), choixJeuCorrect(0), ChoixAppCorrect(0); while (choixTypeCorrect == 0 ) { cout << "1. Lancer un jeu." << endl << "2. Lancer une application" << endl << endl; cout << "Rentre ton choix (1 ou 2) puis appuie sur Enter : "; cin >> choixType; cout << endl; if (choixType == 1) { choixTypeCorrect = '1'; } else { if (choixType == 2) { choixTypeCorrect = '1'; } else { choixType = '0'; cout << "Le choix n'est pas valide. Tu dois choisir 1 ou 2." << endl << endl << endl; } } } cout << endl << "Le choix est valide!" << endl << endl; /* maintenant que le choix est validé, le programme va afficher la liste correspondante C'est à dire celle des jeux si choixType==1, si non celle des applications. */ return 0; }
Pour ce qui est de mon problème, c'est qu'il considère toujours que mon choix n'est pas valide, donc ni que choixType == 1 ni que choixType == 2.

Voilà un screen en essayant avec 1 ou 2:
http://img31.imageshack.us/img31/984/1ou2.jpg
Comme vous pouvez le voir, on reste dans la boucle et il ne considère pas mes choix comme corrects, alors qu'ils le sont.

Si j'essaie un autre chiffre, une lettre ou un caractère spécial:
http://img850.imageshack.us/img850/8963/autresc.jpg
Là non plus le choix n'est pas validé, mais c'est normal car c'est ce que je veux.

Et enfin, un problème un peu spécial:
http://img692.imageshack.us/img692/8...caracteres.jpg
Comme vous pouvez le voir, si je mets deux caractères il me remet deux lignes. Si j'en mets 3 il me met trois lignes, ect...

Quelqu'un a une idée de pourquoi ça ne fonctionne pas pour 1 et pour 2? Et pourquoi mettre plusieurs lignes quand le choix comporte plusieurs caractères?

Et aussi, mais ça n'est qu'accessoire, est-ce possible de lancer un .exe depuis un programme en C++? Juste histoire de savoir si je pourrais concrétiser le projet ou s' il ne restera qu'un entraînement.

Vous savez tout. Merci d'avance! :)

15/09/2011, 16h03
oodini

Pour commencer, tes variables choix ont le mauvais type.
Etant en char, quand tu rentres sur la console le caractère 1, la valeur choixType == 41.

Il faut que tu déclares tes variables comme des entiers.

Je ne sais pa avec quoi tu code, mais un debugger t'aurais aidé.

Pense également à utiliser un switch plutôt que des if/else.
15/09/2011, 17h40
Over703

Ah. Ok pour la variable entier. En fait j'avais mis char pour que les lettres soient refusées, je craignais qu'avec unsinged int les lettres fassent bugger le truc.

J'utilise CodeBlocks.

Je ne connais pas encore les switch. Je vais chercher sur google, merci de l'astuce!