was gutes/oder nicht zum importieren

[INFACT]

Hallo,
Ich habe ein kleines Programm zu importieren mit BruteForce programm, Cursor werten, ganze Pfäde löschen u.s.w.

Ihr könnts euch ja mal angucken: http://paste.pocoo.org/show/103088/
ob ihrs braucht oder nicht...

Werde es auch immer neu Patchen...

Edit: Wenn man windows hat sollte man SendKeys von hier http://www.rutherfurd.net/python/sendkeys/
Downloaden und dann das hier: http://paste.pocoo.org/show/103073/ im gleichen Ordner in dem SendKeys ist unter BetaSendKeys.py speichern

[Hyperion]

Ich habe ein kleines Programm zu importieren mit BruteForce programm, Cursor werten, ganze Pfäde löschen u.s.w.

Was genau macht denn das Programm? Daraus werd ich nicht schlau ...

Der Code ist leider unschön zu lesen, weil die Zeilen viel zu lang sind!

[INFACT]

Ja ich weiß.
Viel:

Es hat ein BruteForce Programm(easy_brute,Brute,Complex(2)), es kann verschiedene Sachen auf der Festplatte finden(Find),
Es kann einen zufälligen Pfad aus der Festplatte ausgeben, was leider ziemlich lange dauert, einen sound abspielen(play) und ko(s)mische töne ausgeben(noisy)

[Hyperion]

Hm ... ok, es ist also ein Mischmasch von vielen Sachen ...

Na auf jeden Fall solltest Du das Modul dann besser dokumentieren! So versteht man ja nicht, was genau eine Funktion macht, was man übergeben muss / kann und was man zurückbekommt.

[jonas]

Hi,
was mir aufgefallen ist :

Zeile 21 sollte ``now = GetCursor()`` heißen sonst kannst
du dir deine eigene Funktion auch sparen.

Allgemein:

> Zeilen viel zu lang (>80 Zeichen)
> keine Leerzeichen zwischen z.B.
`` sys.platform == "win32" ``, dass macht es unübersichtlich
> wie mein Vorredner schon sagte, zu wenig
dokumentiert

Hoffe das klingt nicht zu hart.
Verbesser es und stell es hier wieder rein...


MfG Jonas

[INFACT]

Entschuldigung, dass ich erst spät antworte!
Ja werde ich tun und das dann neu reinstellen.