Générateur de mot de passe en C#

Version imprimable

Bonjour bonjour tout le monde !

Je code actuellement un mini programme en C# pour générer un ou plusieurs mots de passe avec une certaine complexité (minuscules, maj, caractères spéciaux, chiffres) et d'une longueur choisie par l'utilisateur. Rien de bien folichon me direz-vous mais voilà, je suis pas très futte-futte niveau progra, c'est juste pour me faciliter les choses par la suite pour le boulot (admin système)

Donc concrètement, j'ai une fonction "generate" qui se charge de construire un mot de passe.
j'ai un bouton, qui quand il est cliqué, lance la génération du mot de passe seul et l'affiche dans une textbox.
J'ai un second bouton qui lui, lorsqu'il est appuyé, génère un nombre (choisi par l'utilisateur) de mots de passe et les affiche dans une listbox.

pour le second bouton, j'ai un problème : il m'affiche 2 fois le même mdp
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 private void genMultiple_Click(object sender, EventArgs e) { int nbPasswd = int.Parse(domainPasswd.Text); passwdlist.Clear(); for (int i = 0; i < nbPasswd; i++) { password = ""; password = generate(); passwdlist.Add(password); } for (int i = 0; i < passwdlist.Count; i++) { list.Items.Add(passwdlist[i]); } }
Je sais que l'erreur est toute bête mais je n'arrive pas à la trouver...

Toute aide serait la bienvenue :-) Merci d'avance !

02/09/2016, 10h29
LittleWhite

Bonjour,

Surement la fonction Generate() qui retourne deux fois le même mot de passe. Il suffit de vérifier avec un débogueur.
Hypothèse aveugle : probablement car la graine est définit dans la fonction de génération.
02/09/2016, 12h21
WaterTwelve21

Bonjour ,

En effet le problème doit venir de ta fonction generate(), nous l'expliciter pourrait être pratique .
Moi aussi j'y vais de mon Hypothèse aveugle : Si tu utilise Random.Next méfie toi ! il est calé au temps système , essaye juste avec un thread.Sleep(500) pour voir ;)

Si les résultats sont alors différents , le problème vient de la.

salut,

ce n'est pas en lien avec ton erreur, mais pourquoi 2 boucles ?

Citation:

Envoyé par theITguyInTheBasmnt

Code:

1
2
3
4
5
6
7
8
9
10
11
 
            for (int i = 0; i < nbPasswd; i++)
            {
                password = "";
                password = generate();
                passwdlist.Add(password);
            }
            for (int i = 0; i < passwdlist.Count; i++)
            {
                list.Items.Add(passwdlist[i]);
            }

celle-ci suffit, non ?

Code:

1
2
3
4
5
6
7
 
            for (int i = 0; i < nbPasswd; i++)
            {
                password = "";
                password = generate();
                list.Items.Add(password);
            }

Bien le bonjour !

Tout d'abord merci de vos réponses :-)

Voici le code de ma fonction generate()

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
42
43
44
45
 
private string generate()
        {
                Random rnd = new Random();
                int length = int.Parse(comboLen.Text);
 
                string passwd = "";
 
                //choix du caractère special
                int iSpecialChar = rnd.Next(0, 5);
                string chosenSpecChar = specialchars[iSpecialChar];
 
                //choix aléatoire de la position du caractère spécial
                int specialCharPosition = rnd.Next(1, length);
 
                //MessageBox.Show(chosenSpecChar + " à la position : " + specialCharPosition);
 
                //DEBUT : generation du mot de passe
                for (int i = 1; i < length; i++)
                {
                    if (i == specialCharPosition)
                        passwd += chosenSpecChar;
 
                    int jNewchar = rnd.Next(0, 3);
                    switch (chartype[jNewchar])
                    {
                        case "minuscule":
                            int iNewMinLetter = rnd.Next(0, 26);
                            string NewMinLetter = tabalphabetmin[iNewMinLetter];
                            passwd += NewMinLetter;
                            break;
                        case "majuscule":
                            int iNewCapLetter = rnd.Next(26);
                            string NewCapLetter = tabalphabetMAJ[iNewCapLetter];
                            passwd += NewCapLetter;
                            break;
                        case "nombre":
                            int iNewNumber = rnd.Next(0, 10);
                            string NewNumber = numbers[iNewNumber];
                            passwd += NewNumber;
                            break;
                    }
                }
                return passwd;            
        }

encore merci du coup de main !

PS : Pour répondre à Benoit, c'est ce que j'avais à la base mais je voulais vérifier avec une méthode différente :D

02/09/2016, 13h40
WaterTwelve21

Tu devrais sortir ton Random de ta méthode pour qu’il s’initialise une seul fois , le phénomène vient que les nombres aléatoires en programmation ne sont pas réellement aléatoires . Je suis incapable de t'expliquer précisément , quelqu'un le fera sans aucun doute mieux que moi , en tous cas , surtout lors de boucles rapprochées le random va retourner les memes resultats, de toute manière tu pourrai le vérifier par débug .

Donc sort le random en dehors de ta méthode ou test avec Thread.Sleep()

Bonjour,

Je rejoins la précédente intervention. :plusser:

Donc ceci devrait palier au problème :

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
42
43
44
45
46
 
Random rnd = new Random();
 
private string generate()
        {
                int length = int.Parse(comboLen.Text);
 
                string passwd = "";
 
                //choix du caractère special
                int iSpecialChar = rnd.Next(0, 5);
                string chosenSpecChar = specialchars[iSpecialChar];
 
                //choix aléatoire de la position du caractère spécial
                int specialCharPosition = rnd.Next(1, length);
 
                //MessageBox.Show(chosenSpecChar + " à la position : " + specialCharPosition);
 
                //DEBUT : generation du mot de passe
                for (int i = 1; i < length; i++)
                {
                    if (i == specialCharPosition)
                        passwd += chosenSpecChar;
 
                    int jNewchar = rnd.Next(0, 3);
                    switch (chartype[jNewchar])
                    {
                        case "minuscule":
                            int iNewMinLetter = rnd.Next(0, 26);
                            string NewMinLetter = tabalphabetmin[iNewMinLetter];
                            passwd += NewMinLetter;
                            break;
                        case "majuscule":
                            int iNewCapLetter = rnd.Next(26);
                            string NewCapLetter = tabalphabetMAJ[iNewCapLetter];
                            passwd += NewCapLetter;
                            break;
                        case "nombre":
                            int iNewNumber = rnd.Next(0, 10);
                            string NewNumber = numbers[iNewNumber];
                            passwd += NewNumber;
                            break;
                    }
                }
                return passwd;            
        }

IMPORTANT : Si tu ouvres plusieurs fois ton programme pour générer un ensemble de mots de passe il coule de source que tu devras stocker les mots de passes générés dans une sorte de fichier de configuration. Ainsi lorsque ton programme sera exécuté ultérieurement il faudra prévoir de comparer si le mot de passe généré ne corresponds pas aux mots de passe présents dans le dit fichier de conf, sinon l'utilisateur pourrait se retrouver (après plusieurs exécution de ton programme) avec des mots de passe identiques.

A+

02/09/2016, 14h11
Invité

Merci beaucoup ! Ca fonctionne à merveille maintenant :D

Merci pour le conseil au sujet des mots de passe qui auraient déjà été générés, je vais certainement faire comme vous me dites, un fichier de configuration.

A terme, pourquoi pas une base de données ;)

Encore merci !

Citation:

Envoyé par wallace1

IMPORTANT : Si tu ouvres plusieurs fois ton programme pour générer un ensemble de mots de passe il coule de source que tu devras stocker les mots de passes générés dans une sorte de fichier de configuration. Ainsi lorsque ton programme sera exécuté ultérieurement il faudra prévoir de comparer si le mot de passe généré ne corresponds pas aux mots de passe présents dans le dit fichier de conf, sinon l'utilisateur pourrait se retrouver (après plusieurs exécution de ton programme) avec des mots de passe identiques.

C'est bien compliqué comme approche. Le plus simple ne serait-il pas d'instancier Random avec une graine ?

La MSDN donne un exemple simple sur la manière de procéder, que je me permets de copier/coller ci-dessous :
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 using System; using System.Threading; public class Example { public static void Main() { Random rand1 = new Random((int) DateTime.Now.Ticks & 0x0000FFFF); Random rand2 = new Random((int) DateTime.Now.Ticks & 0x0000FFFF); Thread.Sleep(20); Random rand3 = new Random((int) DateTime.Now.Ticks & 0x0000FFFF); ShowRandomNumbers(rand1); ShowRandomNumbers(rand2); ShowRandomNumbers(rand3); } private static void ShowRandomNumbers(Random rand) { Console.WriteLine(); byte[] values = new byte[4]; rand.NextBytes(values); foreach (var value in values) Console.Write("{0, 5}", value); Console.WriteLine(); } } // The example displays output similar to the following: // 145 214 177 134 173 // // 145 214 177 134 173 // // 126 185 175 249 157
Pour l'usage que tu comptes faire, c'est amplement suffisant.