Discussion :

[xavcemoi]

Bonjour à tous,

Je suis débutant en Python, j'essaye de coder un script pour un casseur de mot de passe. Tout se passe bien jusqu'à se que j'incorpore une liste de mots dans mon script , j'obtiens un message d'erreur du type :

AttributeError: 'NoneType' object has no attribute 'readlines'

voilà un bout de mon script:

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
 
class Exception_as_Error:
    pass
 
 
def crack_dict(md5, file, ofiles=None):
    try:
        trouve = False
        ofile = open(file, "r")
        for mot in ofiles.readlines():
            mot = mot.strip("\n")
            hashmd5 = hashlib.md5(mot.encode("utf8")).hexdigest()
            if hashmd5 == md5:
                print("Mot de passe trouvé : " + str(mot) + " (" + hashmd5 + " )")
                trouve = True
        if not trouve:
            print("Mot de passe non trouvé :(")
            ofile.close()

J'ai du mal à voir d'où vient le problème, serait-il possible que l'on m'éclaire un peu.

Cordialement

[fred1599]

Bonjour,

Le problème est que ofiles peut valoir None, il faut prendre en compte cette possibilité pour éviter le type d'erreur que tu reçois.

[papajoker]

bonjour,

Il faudrait voir dans un premier temps si la variable ofiles existe après un ofile = open(file, "r")EDIT: grillé

--

je suppose que le fichier est volumineux ... donc il faut lire ligne par ligne et non tout fourrer dans une variable en ram

Code :

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

mot = mot.strip("\n")

je suppose que tu as 100 milions de lignes (sinon aucun intérêt), il faut donc gagner la moindre nano-seconde

Code :

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

mot = mot.rstrip("\n")

pour le fichier.close(), puisque tu utilises des "try:", tu peux éventuellement le mettre dans un "finally" ou, utiliser un "with open()"

[wiztricks]

Il faudrait voir dans un premier temps si la variable ofiles existe après un ofile = open(file, "r")

C'est tout vu puisque tous les paramètres de la fonction seront variables locales...

- W

[xavcemoi]

Bonjour à tous,

Merci pour vos retours, j'ai réussi en incorporant vos remarques, je n'avais pas déclaré la variable.

Il s'agit d'un petit casseur de mot de passe puisqu'il n'a à trouver que des md5 et des mots ne comportant que 5 lettres c'est juste pur le fun, il n'y a rien de malveillant derrière.

Merci,

Cordialement

[Sve@r]

je n'avais pas déclaré la variable.

En Python on ne déclare pas de variables, on les crée en leur donnant une valeur. Montre ton nouveau code et on te dira s'il y a des trucs à améliorer.

c'est juste pur le fun, il n'y a rien de malveillant derrière.

Oui on s'en doute. Surtout en Python (ce n'est pas le plus adapté pour ce travail, question rapidité il est à la ramasse face à la foultitude de ceux qu'on trouve sur le net)

[Sve@r]

Bonjour

j'essaye de coder un script pour un casseur de mot de passe.

Hum... interdit par les règles. Enfin si on considère que c'est pour s'exercer et que de toute façon ce genre de script mettra des milliards d'années pour trouver un résultat ça peut passer...

Code :

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

1
2
class Exception_as_Error:
    pass

A quoi sert cette instruction?

print("Mot de passe trouvé : " + str(mot) + " (" + hashmd5 + " )")

print("Mot de passe trouvé: %s (%s)" % (mot, hashmd5)).

Code python :

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

1
2
3
if not trouve:
	print("Mot de passe non trouvé :(")
	ofile.close()

Ah d'accord. Tu ne fermes le fichier que si tu n'as pas trouvé. Oui, moi aussi chez-moi j'ouvre des dictionnaires et je cherche un mot dedans et si le mot n'y est pas je laisse le dictionnaire ouvert sur la table. Total j'ai des milliards de dictionnaires ouverts sur ma table.
Etre débutant n'est pas une excuse pour ne pas réfléchir à la logique de son code...

Code python :

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

1
2
3
4
5
6
7
trouve=False
for mot in ...
	if mot ...
		print(...)
		trouve = True
if not trouve:
	print(...)

Ok, tu ne connais pas le else qui suit une boucle, c'est excusable. Ca permet justement d'éviter ce schéma "je mets un flag et à la fin je teste le flag". Accessoirement quand on a trouvé inutile de tester les autres mots (et c'est ce qui permet au else de fonctionner).

Code python :

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

1
2
3
4
5
6
for mot in ...
	if mot ...
		print(...)
		break
else:
	print(...)

Si on est sorti de la boucle par break (parce qu'on a trouvé) alors le "else" n'est pas pris en compte. Sinon (parce qu'on n'a pas trouvé) alors il l'est.

Et puis éviter les noms de variables identiques à une lettre près (ofile/ofiles) c'est pas top question lisibilité/compréhension. Te devrais quand-même être capable de plus d'originalité et surtout trouver des noms de variables en rapport avec leur rôle...

[wiztricks]

Salut,

J'ai du mal à voir d'où vient le problème, serait-il possible que l'on m'éclaire un peu.

Normalement Python vous donne la ligne et l'instruction qui pèche... mais comme il n'y a qu'un appel a .readlines dans le code que vous montrez, on peut supposer qu'il s'agit de la ligne 10: for mot in ofiles.readlines():

Ce qui suggère que la variable ofiles est à None (ce qui est la valeur par défaut donnée par les paramètres de la fonction) ou que vous avez ajouté un "s" à la variable associé au fichier ouvert à la ligne précédente (et on évite de s'endormir sur son clavier).

Pour le reste, vous avez des tutos qui proposent des exercices corrigés pour vous entraîner à voir comment Python réagit.
C'est là qu'on se familiarise avec les messages d'erreurs et la relecture attentive du code qu'on a écrit: si on n'apprend pas à le faire sur ces exercices "simples", on est vite perdu lors qu'on part à coder sans avoir acquis ces réflexes.

- W

[fred1599]

Hello,

Le code "fonctionnel" a été créé par tes soins ?

Plusieurs remarques et questions,

1. Pourquoi utiliser typing d'un seul coup ? Quel intérêt à part ajouter une couche de complexité supplémentaire ? Qui puis est mal utilisé...
2. Atexit ? Pourquoi ?
3. La fonction crack_dict devrait retourner un objet selon ta déclaration de type, mais ça ne retourne rien
4. J'ai pas compris la cohérence entre mot_fr ligne 24 et 26, et surtout encore une confusion avec les types (là c'est Any, sauf que c'est mal déclaré et ce n'est pas vrai !)
5. Où est définie la variable lettres ?
6. Ligne 38, n'hérite pas d'Exception, donc ça marche pas !
7. Ligne 55, la variable haslib peut être définie à la valeur None, et ça risque de poser problème.

Conclusion, fait simple ! Vire les types, tu sais pas, ou celui qui t'as aidé ne sait pas les utiliser !

Bref si ça fonctionne, c'est parce-que tu n'as pas testé toutes les éventualités !

[xavcemoi]

Bonjour à tous,

Non, le code fonctionnel n'est pas de moi, c'était codé en python 3.8 et avec pycharm 2020 et le code fonctionne.

Ne me reste plus qu'à bosser et je reprendrai ce code un peu plus tard.

Merci à vous pour tous ces retours,

@plus