gepackte Arumente in Funktionen

[Tombery]

Hallöchen

Ich bin nun auf folgendes Problem gestoßen:
Ich habe mir folgende Funktion gebastelt:

Code: Alles auswählen

def select_player(state1, state2, state3, state4, k):

Diese Funktion ist in einer eigenen Pythondatei genannt Functions.py. Dort sind alle meine Funktionen definiert.

Im Hauptprogramm habe ich die Funktion mit import Funktion importiert.

Das Programm läuft nun super, jedoch möchte ich das noch etwas verschönern.
Ich möchte also die Argumente der select_player Funktion zu einem Argument zusammenfassen.
Also etwa so:

Code: Alles auswählen

state = [state1, state2, state3, state4]

Wenn ich nun im Hauptprogramm select_player(state1, state2, state3, state4, k) zu def select_player(state, state4, k) ändere, funktioniert das Programm nicht mehr.

Hab schon viel gegoogelt und eigentlch sollte es mit def select_player(*state, k) funktionieren.
Aber das geht auch nicht und vorallem kann ich dies nur im Hauptprogramm so anpassen.

Wenn ich in der Fuktionsdatei die Definition auf def select_player(*state, k) anpasse läuft noch weniger als nix.
Dann kommt folgende Fehlermeldung
missing 1 required keyword-only argument: 'k'

Wie packt und entpackt man Funktionsargumente richtig, wenn diese zusätzlich importet werden müssen?

[snafu]

Das musst du so aufrufen:

Code: Alles auswählen

select_player(*states, k=123)

Also das k muss dabei explizit angesprochen werden.

Daher steht in der Fehlermeldung ja auch:
keyword-only argument: 'k'

[Tombery]

Sorry hab ich vergessen zu beschreiben.
Das k ist natürlich schon vorher definiert und soll einfach nur übernommen werden.
Also daran liegt es nicht.

[__blackjack__]

@Tombery: Doch daran liegt es. Und es nichts damit zu tun ob die Funktion importiert wurde oder im aktuellen Modul definiert.

Ich hoffe die Namen sind original nicht so besch…eiden. Also ein Modul das `Functions` heisst und Argumente die durchnummeriert sind, oder `k`. Wobei man bei durchnummerierten Argumenten vielleicht lieber die Signatur von der Funktion nach `select_player(states, k)` ändern sollte statt anzufangen *-”Magie” beim Aufruf zu verwenden.

[Tombery]

Ich kann das k nicht expilzit ansprechen, da es ein Laufindex ist.
Dies hat funktionier als noch def select_player(state1, state2, state3, state4, k): geschrieben war.
Heißt das man muss die restlichen Variablen immer explizit ansprechen, wenn *Magie benutzt wird?

[snafu]

@Tombery: Ich kann mich natürlich nur auf die Infos beziehen, die du hier angibst. Und damit funktioniert die Lösung. Wir kommen hier also nur weiter, wenn du ein Beispiel gibst, das näher am Original liegt...

[snafu]

Ich kann das k nicht expilzit ansprechen, da es ein Laufindex ist.

Aber es ist doch keine Variable k gemeint, sondern der Funktionsparameter k. Der kann kein Laufindex sein. Also der Wert dahinter natürlich schon, aber nicht der Bezeichner an sich.

Hast du eine Schleife und willst wiederholt die beschriebene Funktion mit einem anderen Wert aufrufen oder worum geht es? Selbst wenn dem so ist, ändert das nichts am Vorgehen. Die Fehlermeldung gibt da wie gesagt auch bereits den entsprechenden Hinweis. Taucht nun eine andere Fehlermeldung auf oder woran erkennst du, dass es immer noch falsch ist?

[Tombery]

Es schein Fehler scheint nicht an select_player(*state, k) in der Hauptdatei zu liegen.

Das Problem entsteht in meiner Function Datei.
Dort steht noch:
def selectPlayer(state1, state2, state3, state4, k):
Dadurch fällt das Programm in eine Endlosschleife

auf def selectPlayer(state, k): kann ich das in der Function Datei nicht ändern, da sonst:
NameError: name 'state' is not defined erscheint.

Also nehme ich an, dass dort def selectPlayer(state1, state2, state3, state4, k): stehen bleiben muss, dies stört mich nicht wirklich.
In der select Player Function wird überprüft if state >= 3: dies trifft nie ein, deshalb kommt das Programm in eine Dauerschleife.
Eine Änderung zu state1+ state2+state3+state4 >= 3 bringt keine besserung.

[Sirius3]

@Tombery: Programmieren durch Raten funktioniert nicht. *-Magie wird benutzt, um eine Variable Anzahl an Parametern entweder zu übergeben, oder in der Funktion zu bekommen.

Man kann also sowohl auf der aufrufenden Seite als auch bei der Definition etwa ändern und es funktioniert weiterhin, wenn man es richtig macht.

Da Du aber bisher noch keinen Code gezeigt hast, der irgendetwas tut und auch nicht die Fehlermeldungen, die Du bekommst, oder warum jetzt plötzlich eine Endlosschleife ensteht, kann man hier nur wild raten.

Das einzige was man definitiv sagen kann, ist, dass Du die ganze Sache falsch angehst. *-Magie ist wirklich nur etwas, was man sparsam einsetzen sollte. Bei dem, was Du hier vermutlich tust, ist es aber der falsche Weg. Benutze eine Liste `states`. Und dazu mußt Du natürlich sowohl die Definition als auch den Programmcode drumherum ändern.

[Tombery]

Das Problem scheint wirklich darin zu liegen, dass in der Functions Datei noch mit den Argumenten gerechnet werden. Diese werden über die Funktion wieder an Main übergeben wobei dort der Wert wieder manipuliert wird. Diese Änderung des Wertes die im Main geschieht, geht durch das Verwenden von *-Magie oder durch das einfache reduzieren auf eine Liste verloren. Das heißt die Function bekommt von der Änderung im Main nix mit.
Ich werde es nun mit state1 + state2 + state3 + state4 lassen. Da es sowieso nur als optische Verschönerung gedacht war.
Leider bin ich nun auf einen anderen Fehler gestoßen. Als ich ihn gestern Nacht erkannte habe ich, vor Verzweiflung, nur noch den Laptop geschlossen.

Vielleicht komme ich später nochmal auf dieses Problem mit einer genaueren Beschreibung zurück.

[sparrow]

Eine Funktion soll auch gar nichts mitbekommen, von irgendwelchen Dingen, die irgendwo außerhalb der Funktion passieren.
Wenn dem so ist, ist dein Code kaputt.

[snafu]

Fazit: Du hast irgendwas gemacht, was irgendwie nicht funktionierte. Dann hast du noch irgend etwas anderes gemacht, was auch nicht klappte und den Laptop zugemacht. Und alle Leser freuen sich über ihre verschwendete Lebenszeit. Hurra!