Fehler beim random module

[nb5code]

Hallo ich arbeite gerade an einem Spiel. Dabei möchte ich mir vier Farben( rot, gruen, blau, gelb ) mischen und ausgeben lassen

Code: Alles auswählen

from random import *
def getRandomColors():
    for i in range(4):
        print random.choice(["green", "blue", "yellow", "red"])

nur komischer Weise kommt dabei ein Fehler zustande :

Code: Alles auswählen

File "C:\IT\python\Game\src\Game.py", line 5
    print random.choice(["green", "blue", "yellow", "red"])
               ^
SyntaxError: invalid syntax

könnt ihr mir da vielleicht helfen ?

Danke!

[cofi]

Du hast zwei Fehler: Du stern-importierst `random` und damit gibt es kein `random.choice`. Bevor das aber passiert: Du benutzt Python 3 als waere es Python 2. Da fehlen Klammern beim `print` Aufruf.

[/me]

Dann gibt es auch noch einen Schönheitsfehler und einen semantischen Fehler. getRandomColors sollte laut PEP-8 get_random_colors heißen. Zudem sollte eine Methode mit get im Namen auch etwas zurückgeben, diese Methode gibt aber nur Werte aus.

[nb5code]

ich Versuche eszu verbessern danke

[nb5code]

ich dachte das sternchen importiert alles aus der Bibliothek

[nezzcarth]

ich dachte das sternchen importiert alles aus der Bibliothek

Genau das tut es auch: es importiert alles aus der Bibliothek in den öffentlichen Namensraum. 'import random' importiert die Bibliothek als solches, d.h. als eigenen Namensraum. Du machst einen Sternchenimport und versuchst 'choice' aus dem Namensraum random, der gar nicht existiert, aufzurufen. Allerdings sind Sternchenimport schlechter Stil. Entweder du importierst die gesamte Bibliothek, oder die spezifischen Namen, die du brauchst, z.B. 'from random import choice'.

[Hyperion]

Hallo ich arbeite gerade an einem Spiel. Dabei möchte ich mir vier Farben( rot, gruen, blau, gelb ) mischen und ausgeben lassen

Dann ist Dein Ansatz auch an sich verkehrt und Du suchst ``random.shuffle``!

Unabhängig davon noch kurz eine Anmerkung zum Stil: Wenn man einen Rückgabewert in Python nicht braucht, so sollte man diesen per Konvention als Unterstrich benennen, bei Dir also:

Code: Alles auswählen

for _ in range(4):
    pass

Das zeigt dem Leser sofort an, dass der Name nicht mehr gebraucht wird und lenkt den Fokus nicht in falsche Richtungen. Bei Dir würde man erst einmal nach dem ``i`` im Schleifenrumpf suchen, bevor man kapiert, dass es auf das ``i`` gar nicht ankommt.