Bugfixes, Statusausgabe, Argparse

0x0verflow commented 4 years ago

Hallo, meine Kerle! Nachdem der HusoBot von Twitter ermordet wurde, musste ich mir ein neues Hobby suchen. Und da wäre ich! Hier wäre mein Beitrag zu einer besseren Welt:

Argparse:

-threads (Anzahl der Threads bestimmen)
-url (URL ändern)
-proxylist(Proxylist aus Datei laden)
Manifestierung der Beiträger

Statusausgabe:

Status wird jetzt alle 5s lesbar ausgegeben
Bessere Performance (Ich bin Geschwindigkeit!)

Bugfixes:

Nichtverwendung des Proxys beim get(...)-Requests am Anfang der vote_thread-Methode. BKA lässt grüßen!
Umbenennung von main(...) in vote_thread(...). Lesbarkeit und so...

Ich hoffe, dass ich weiterhelfen konnte und ihr den PR annehmt. Ansonsten habe ich jetzt wohl meinen eigenen Fork ¯_(ツ)_/¯

lennardwalter commented 4 years ago

Klar wird der angenommen! Fehlt nur noch, ganz gemäß der deutschen Korrektheit, die bessere Ausstattung mit Kommentaren... Außerdem könntest du noch ein weiteres Kommandozeilenargument hinzufügen, mit dem man das Wort bestimmen kann, welches gespammt wird. Natürlich mit einem sehr großen Schriftzug, was Hurensohn doch für ein schönes Wort ist... Aber dennoch wollen einige (nachdem Langenscheidt verkündet hat, dass sie Hurensohn nicht in die Top 10 aufnehmen werden) andere Wörter spammen.

0x0verflow commented 4 years ago

So, damit wäre der ganze Spaß erledigt. Du kannst jetzt mergen.

lennardwalter commented 4 years ago

Jop. Die Kommentare sollten angelsächsisch sein. Die Informationsschreiben gerne auf Deutsch. Im Quellcode sollte dann eine angelsächsische Beschreibung, was da ausgegeben wird, zu finden sein.

0x0verflow commented 4 years ago

Wäre erledigt. Noch andere Einwände?

lennardwalter commented 4 years ago

Wundervoll! Wird gemerged:)

0x0verflow commented 4 years ago

Problem: der info_msg_thread darf nicht in der enumerate() Methode ignoriert werden, da das Programm sonst nicht wartet, bis er zum letzten Mal durchgelaufen ist. Ist keine große Sache, ist nur ärgerlich. Und - wenn das eh gefixt wird - bitte auch die proxie-typo oben fixen.

lennardwalter commented 4 years ago

Den Typo fix ich gerne. Aber verstehe nicht, was du mit dem info_msg_thread-Problem meinst. Unten, wenn es darum geht, zu warten, bis die Threads geschlossen sind, wir ja kein Thread ignoriert (außer der Mainthread). Nur oben wird ein Thread mehr gestartet, was nichts am warten auf die Schließung ändert. Kann aber auch sein, dass ich gerade nicht richtig denke, ist ja auch schon spät...

0x0verflow commented 4 years ago

Macht im Endeffekt keinen Unterschied, wäre nur schöner. Der info_msg_thread wird über einen boolean gestoppt - läuft also nicht von alleine aus. Sobald der bool true ist, kann der Thread auslaufen. Also müssen alle nicht-Haupt-Threads gleich behandelt werden, somit wird nur der Main-Thread abgezogen.

lennardwalter / jugendwort-spammer