Das deutsche Python-Forum

Hallo

Ich habe für verschiedene Aufgaben verschiedene Klassen geschrieben. Oft benötige ich in einer Klasse zusätzliche Infos und importiere die Klasse aus einer anderen Datei, hier stoße ich jedoch auf Probleme, wenn sich die Parteien gegenseitig versuchen zu importieren. Wie kann ich dieses Problem umgehen?

Beispiel:
ich habe ein kleines Skript und importiere am Angang
Gerade wollte ich der Klasse Locations eine neue Funktion hinzufügen, "get_locations_product_infos", daher wollte ich die Klasse "Products" importieren. Die Klasse Products importiert jedoch schon die Klasse Locations, um die Orte der Produkte zu finden.

Ich wüsste gerne wie ich das schöner lösen kann, Freue mich sehr auf Hinweise, Danke!

Ja, gibt es schon viele Themen dazu, z.B. viewtopic.php?t=19333
Mit hat nun geholfen, die Klasse erst kurz vor Bedarf zu importieren

@martinjo: Was aber wenn ich mir die Importe anschaue der falsche Ansatz ist. Das sieht ja *sehr* nach einem Modul pro Klasse aus, was grundsätzlich schon mal komisch riecht, aber einfach mal falsch ist wenn Klassen so eng verbunden sind, dass die Module sich gegenseitig importieren. Dann gehören die offensichtlich ins selbe Modul oder die Funktionalität innerhalb der Klassen ist falsch aufgeteilt.

Edit: Selbst bei einem Modul pro Klasse ist da immer noch jeder Name einmal zu viel im voll qualifizierten Namen. WTF‽

Ein Modul pro Klasse ist voll OK, schließlich haben wir ja alle mal Java gemacht. Auch fürs Repository und für Eclipse wird es so besser.
Die gegenseitige Abhängigkeit lässt sich oftmals durch ein Drittes Modul auflösen.

@MagBen: Ironie versteht nicht jeder.

Das hat wenig mit Ironie verstehen oder nicht zu tun wenn man eine Frage zu einem Fachgebiet beantwortet muss man davon ausgehen, dass die der Fragesteller nicht so gut mit der Materie auskennt und daher die Ironie in der Antwort logischerweise nicht verstehen kann.

Letztendlich sind alles Helferklassen für meine Warenwirtschaft, also Buchhaltung, Bank- und Paypal Transaktionen auflisten und zuweisen. Statistiken erstellen, VAT Nummern prüfen, Erinnerungs-Emails senden, Ware in Webshop und eBay abgleichen, Inventurlisten mit Bildern erstellen, Produktinfos aus dem Internet suchen, Forumeinträge generieren, DHL Label erstellen, Senungsnummern hinzufügen und Versandkosten errechnen um nur mal ein paar wenige zu nennen, alles ist irgendwie ineinander verzahnt, wenn ich das alles in eine Datei schreibe wird das etwas unübersichtlich und hätte über eine Millionen Zeilen Code.

martinjo hat geschrieben: ↑Montag 17. Dezember 2018, 13:43 [...] alles ist irgendwie ineinander verzahnt [...]

Dann ist genau das dein Problem. Technisch gibt es Möglichkeiten, dass Klassen sich gegenseitig importieren. De facto spricht das aber in der weitaus größten Zahl der vorkommenden Fälle von einem falschen Anwendungsentwurf und daher sollte man erst einmal über die Struktur nachdenken.

@martinjo: die Auflistung Deiner Hilfsklassen hört sich auf den ersten Blick nicht sehr verzahnt an, das sind alles Aufgaben, die unabhängig voneinander laufen. Das meiste sind sogar Tätigkeiten, die gar keiner Klasse bedürfen (jedenfalls nicht direkt).
Kannst Du näher beschreiben, warum da was verzahnt ist?

martinjo hat geschrieben: ↑Freitag 23. November 2018, 17:05 Gerade wollte ich der Klasse Locations eine neue Funktion hinzufügen, "get_locations_product_infos", daher wollte ich die Klasse "Products" importieren. Die Klasse Products importiert jedoch schon die Klasse Locations, um die Orte der Produkte zu finden.

Ich würde das eher so aufbauen: Ein Ort ist die Sammelstelle für verschiedene Produkte, also sollte auch er seine derzeit vorhandenen Produkte kennen. Ich nehme ja mal an, dass die Sache mit dem Ort eine dynamische Geschichte ist, sich also ändern kann, d. h. das Produkt kann den Ort wechseln. Und selbst wenn mit dem Ort nur genau ein Produkt verknüpft werden kann (z. B. weil er ein bestimmtes Fach im Lager darstellt), passt dieses Vorgehen immer noch.

Das Produkt hingegen sollte gar nichts davon wissen müssen, wo es gerade liegt. Da der Ort der "Verwalter" ist, muss er gefragt werden. Und somit muss das Produkt keine Orte mehr importieren, sondern es erfolgt stattdessen eine Zuweisung via location.set_product(dein_produkt) oder etwas in der Art. Diese Zuweisung macht aber nicht das products-Modul, sondern der Prozess, der zur Bewegung des Produktes führt, also z. B. der Warenein- oder ausgang oder eine Bewegung innerhalb des Lagers.

Das locations-Modul hätte außerdem eine Funktion zum Abfragen des Ortes für ein Produkt: Und auch hier würde ich auf einen Import der Product-Klasse verzichten. Typprüfungen vermeidet man in Python bekanntlich so gut es geht. has_product() würde halt bei einem falschen Typen stumpf False liefern. Und schon ist man ein bißchen was von der Verzahnung beider Module losgeworden...