Discussion :

[Over703]

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 :

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
 
#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! ;)

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

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

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 :

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

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.

[koala01]

Salut,

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 :

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

[Over703]

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

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!

[Over703]

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 :

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

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:

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:

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!

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

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