Klasse - Verbesserungen?

[Crazed]

Hey,

Ich habe mal probiert mit meinem spärlichen Wissen eine Klasse zu schreiben (habe bereits eine geschrieben) und wollte wissen was ihr vom Aufbau haltet. Verbesserungen? Aufmerkungen auf uneffektiven Code? Völlig "kacke? - Weißt mich bitte darauf hin damit ich es das nächstemal besser machen kann.

Und ja - mir ist durchaus bewusst das man das auch ohne Klasse hätte machen können, ich wollts aber trotzdem:

http://paste.pocoo.org/show/53212/

Ich bin mir sicher das es viel geordneter geht, würde mich sehr über Verbesserugen freuen.

Der Zweck der klasse:

Das Script liest aus einer textdatei im selben Verzeichniss Proxy Adressen im Format IP:Port aus wechselt jeden Proxy bei einem neuen Schleifendurchlauf und klickt dann auf einen Link. Gut um bei Diebesspielen zu schummeln/proxies zu testen etc.

MfG,
CracKPod

[lunar]

Darf man vorsichtig fragen, wo denn der Sinn von ``_create_file`` liegt?

Zudem ist dein Exception-Handling verbesserungsbedürftig. Datei-Objekte sollten in ``finally``-Blöcken geschlossen werden.

Ausnahmen in Bibliotheksklassen sollten an den Aufrufer weitergegeben werden.

Auch könntest du ``click`` eventuell noch entschalten und weiter aufteilen.

[Leonidas]

Du überschreibst das Builtin ``file``, deine Klasse ist Old-Style (erbt nicht von ``object``), außedem sind die Namen ``URL`` & ``Timeout`` nicht PEP8-konform (Hint: Kleinschreibung). Zeilen 29-32 würde ich in eine Zeile packen, so kompliziert ist das nicht, dass es da zur übersicht viel beiträgt.

Außerdem, statt Strings mit Kommas zusammenzusetzen würde ich eher String-Formatting nutzen.

[EyDu]

Und in der init-Methode kannst du "readlines" verwenden, an Stelle des splits.

[Crazed]

Du überschreibst das Builtin ``file``, deine Klasse ist Old-Style (erbt nicht von ``object``), außedem sind die Namen ``URL`` & ``Timeout`` nicht PEP8-konform (Hint: Kleinschreibung). Zeilen 29-32 würde ich in eine Zeile packen, so kompliziert ist das nicht, dass es da zur übersicht viel beiträgt.

Außerdem, statt Strings mit Kommas zusammenzusetzen würde ich eher String-Formatting nutzen.

Meinst du mit erbt nicht von object das ich es so aufrufen sollte?

class Klasse(mechanize.Browser) ?

[EyDu]

Nein, er meint:

Code: Alles auswählen

class ProxyClicker(object):
    ...

[Y0Gi]

Und in der init-Methode kannst du "readlines" verwenden, an Stelle des splits.

Oder sogar `list()`:

Code: Alles auswählen

with open('proxylist.txt') as f:
    proxies = list(f)