Das deutsche Python-Forum

kbr hat geschrieben: Sonntag 9. Januar 2022, 14:26 Die Musterlösung sollte aus meiner Sicht einen besseren Funktionsnamen haben, "shelf" als Parameter erhalten und statt "range(0, len(shelf))" "enumerate" verwenden. Noch eleganter geht es mit der List-Methode "index" sowie einem Rückgabewert, der die Position des neuen Elements angibt oder einen anderen Wert hat, falls im Regal kein Platz mehr frei war.

@andie39: Wenn Du magst, versuche zur Übung die Aufgabe mit "enumerate" zu lösen und dann später auch noch den oben genannten Rückgabewert zu liefern.

shelf = ["Zaubersäge", "leer", "Wunderkekse", "Trickkarten", "leer"]

def add_shelf(article):
    for i in shelf:
        if i == "leer":
            i in shelf = article
            
        
    
add_shelf("Hase“)
print(shelf)

andie39 hat geschrieben: Sonntag 9. Januar 2022, 15:38 Der Fehler tritt bei i in shelf = article auf.

Sirius3 hat geschrieben: Sonntag 9. Januar 2022, 17:47 Was als erstes auffällt, ist die Funktionsdefinition mitten im direkt ausgeführten Code. Auch wenn man keine main-Funktion hat, sollte doch Funktionsdefinitionen am Anfang des Skripts stehen.
Dann ist natürlich die globale Variable `shelf` ein NoGo. Das macht den Kurs noch untauglicher.
`add_shelf` ist der falsche Funktionsname, der müßte `add_to_shelf` heißen.
Die Schleife über einen Index sollte man Anfängern auch erst gar nicht zeigen.
Im Fehlerfall einfach weitermachen, als ob nichts wäre, ist das nächste NoGo.

Code: Alles auswählen

def add_to_shelf(shelf, article):
    for index, element in enumerate(shelf):
        if element == "leer":
            shelf[index] = article
            break
    else:
        raise ValueError("kein leerer Platz")
    
shelf = ["Zaubersäge", "leer", "Wunderkekse", "Trickkarten", "leer"]
add_to_shelf(shelf, "Rubik's Cube")
print(shelf)

Ich hoffe, das wird alles im Anschluß noch behandelt.

__blackjack__ hat geschrieben: Sonntag 9. Januar 2022, 20:42 Und noch was: Falls es ``for`` nicht gäbe und man sich das selbst basteln müsste, hier das was bei jeder ``for``-Schleife abläuft:

Code: Alles auswählen

for item in items:
    do_something(item)


# =>


iterator = iter(items)
while True:
    try:
        item = next(iterator)
    except StopIteration:
        break
    
    do_something(item)

Das erstellen des Iterators und abfragen der einzelnen Elemente in einer IPython-Shell manuell:

Code: Alles auswählen

In [84]: shelf = ["Zaubersäge", "leer", "Wunderkekse", "Trickkarten", "leer"]   

In [85]: iterator = iter(shelf)                                                 

In [86]: iterator                                                               
Out[86]: <list_iterator at 0x7ffa6fd72a20>

In [87]: next(iterator)                                                         
Out[87]: 'Zaubersäge'

In [88]: next(iterator)                                                         
Out[88]: 'leer'

In [89]: next(iterator)                                                         
Out[89]: 'Wunderkekse'

In [90]: next(iterator)                                                         
Out[90]: 'Trickkarten'

In [91]: next(iterator)                                                         
Out[91]: 'leer'

In [92]: next(iterator)                                                         
---------------------------------------------------------------------------
StopIteration                             Traceback (most recent call last)
<ipython-input-92-4ce711c44abc> in <module>
----> 1 next(iterator)

StopIteration:

Sirius3 hat geschrieben: Donnerstag 13. Januar 2022, 17:37 Zähle mal alle Klammern.

Dateien öffnet man immer mit with. `eingabe` ist ein zu generischer Name, `filmtitel` wäre doch viel sprechender. `input` kann schon einen Prompt mit ausgeben, da braucht es kein extra `print`:

Code: Alles auswählen

def add_movie():
    film_titel = input("Geben Sie einen Filmtitel ein:")
    with open("movies.csv", "w") as file:
        file.write(film_titel + "\n")
[/quote]



*FACEPALM*

Und ich habe geschaut und mich gefragt, warum das nicht geht obwohl doch der code eigentlich stimmt.........
Danke dir.

Ja stimmt deiner ist besser und kürzer aber ich sehe mit der Klammer funktioniert mein Code dennoch.
Es macht mehr und mehr Spass etwas zu programmieren!  :)