Allgemeine Funktionen in Klasse oder außerhalb?

[Hellstorm]

Hallo,

ich habe hier folgende Klassen und Funktionen:

Code: Alles auswählen

class Uploader()
    __init__()
    fortschrittsanzeige()
    hochladen()
    groesse_lesbar()

zippen()
zwischenablage()

Ich bin mir jetzt unsicher, ob ich einige Funktionen wie groesse_lesbar(), die relativ allgemein sind und z.B. nur eine Zahl wie 239341434 in lesbare Mebibyte/Gibibyte umrechnet, in die Klasse oder außerhalb tun soll. Im Grunde haben die ja mit der eigentlichen Funktion in der Klasse nichts zu tun.

Oder sollen die gerade in die Klasse rein?

Danke!

[snafu]

Die sind besser außerhalb der Klasse aufgehoben. Dennn, wie du schon geschrieben hast: Das sind ja anscheinend Hilfsfunktionen, die auch in anderen Zusammenhängen als mit dem Upload-Vorgang verwendet werden können.

[Hyperion]

Alles was *nicht* unmittelbar zur Aufgabe einer Klasse ghört, sollte da auch nicht rein. Damit würdest Du das SRP verletzen.

Kleine Anmerkung noch zu Deinen Namen: ``fortschrittsanzeige`` klingt nicht nach Methode / Funktion! Wo ist da die Tätigkeit zu sehen? (Ähnliches gilt für ``zwischenablage``). Was *tut* die Methode denn? Versuche genau das im Namen auszudrücken. Das ist oftmals schwer, aber das A und O der Programmierung

[Hellstorm]

Danke für die Antworten

@Hyperion:
Fortschrittsanzeige ist eine Callback-Funktion für den SFTP-Client von Paramiko. Wie könnte ich das denn nennen?

Naja, zwischenablage könnte ich wahrscheinlich in „in_zwischenablage_einfuegen“ umbennennen.

[snafu]

Danke für die Antworten

@Hyperion:
Fortschrittsanzeige ist eine Callback-Funktion für den SFTP-Client von Paramiko. Wie könnte ich das denn nennen?

Naja, zwischenablage könnte ich wahrscheinlich in „in_zwischenablage_einfuegen“ umbennennen.

Nimm besser den Imperativ: `fuege_in_zwischenablage_ein()` oder auf englisch (sinngemäß): `copy_to_clipboard()`.