Problem mit update() bei Dict's

[jtschoch]

Hallo,
Ich möchte in einen Dictionary zwei Werte updaten.
die normale Ausgabe ist so:

Code: Alles auswählen

0, 0
0, 1
0, 2
0, 3
0, 4
1, 1
.
.
.

Wenn ich es ausprinten lasse ist das so, aber ich habe in meinen Dict sind sie nicht so, die Ausgabe ist so:

Code: Alles auswählen

{'action': None, 'y': 4, 'tile': '/gametiles/baum1.png', 'x': 2}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/baum1.png', 'x': 2}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/baum2.png', 'x': 4}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/baum3.png', 'x': 0}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/baum1.png', 'x': 2}
{'action': None, 'y': 4, 'tile': '/gametiles/grass.png', 'x': 3}
{'action': None, 'y': 4, 'tile': '/gametiles/baum2.png', 'x': 4}

Warum? Ich sehe da keine 1, 1 und so.

Hier ist der Code:

Code: Alles auswählen

#!/usr/bin/python
# -*- coding: utf-8 -*-

"""
Generate the playingfield
"""
#~Imports

import cPickle as pickle
import random

#~Define arena

height_rows=5
width_cols=5

#~Define images

grass = {"tile" : "/gametiles/grass.png", "action" : None, "x" : None, "y": None}
baum1 = {"tile" : "/gametiles/baum1.png", "action" : None, "x" : None, "y": None}
baum2 = {"tile" : "/gametiles/baum2.png", "action" : None, "x" : None, "y": None}
baum3 = {"tile" : "/gametiles/baum3.png", "action" : None, "x" : None, "y": None}

#~Variables

row=[]
for y in range(height_rows):
    col=[]
    for x in range(width_cols):
        number = random.randint(1, 10)
        if number == 5:
            baum1.update({"x" : x, "y" : y})
            col.append(baum1)      
        elif number == 1:
            baum2.update({"x" : x, "y" : y})
            col.append(baum2) 
        elif number == 7:
            baum3.update({"x" : x, "y" : y})
            col.append(baum3)
        else:
            grass.update({"x" : x, "y" : y})
            col.append(grass)
    row.append(col)
for i in row:
    for i in i:
        print i
name = raw_input("mapname: ")
pickle.dump(row, open("../maps/" + name + ".sav", "wb" ) )
print "fertig"

[BlackJack]

@jtschoch: Du hast da egal wie gross das Spielfeld ist *vier* Wörterbücher und mit den `update()`-Aufrufen veränderst Du immer die selben vier Objekte. Das heisst wenn Du einem `grass.update(…)` zum Beispiel mit den Werten für Feld 1,1 ausführst und später zum Beispiel noch einmal für Feld 2,3 dann ersetzt Du im *selben* Objekt die Werte von vorher.

Du musst vor dem Ändern eine Kopie von dem jeweiligen Wörterbuch erstellen und die dann aktualisieren und dem Spielfeld hinzufügen.

Der Vergleich mit den Zahlen 5, 1, und 7 ist total willkürlich. Da würde ich 1, 2, 3 nehmen, das ist weniger verwirrend und kommt auf das gleiche Ergebnis hinaus.

[jbs]

Wenn es etwas meta sein darf, dann kannst du deine if-Abfrage so zusammen fassen:

Code: Alles auswählen

trees = [ {...}, {...}, {...} ]
...
number = random.randint(0, 9)
if number <= 3:
    to_update = trees[number]
else:
    to_update = grass

to_update.update(...)

[BlackJack]

Das ``if``/``else`` könnte man sich noch sparen wenn man `max()` verwendet.

Code: Alles auswählen

field = field_prototypes[max(randint(9), 4)].copy().update(...)

[jbs]

Meintest du nicht min?

[BlackJack]

@jbs: Ja, meinte ich natürlich.

[jtschoch]

Da kommt aber am ende None raus!

Code: Alles auswählen

#!/usr/bin/python
# -*- coding: utf-8 -*-

"""
Generate the playingfield
"""
#~Imports

import cPickle as pickle
import random

#~Define arena
#5*5 = 25

height_rows=5
width_cols=5

#~Define images
objects =  \
[
{"tile" : "/gametiles/grass.png", "action" : None, "x" : None, "y": None},
{"tile" : "/gametiles/baum1.png", "action" : None, "x" : None, "y": None},
{"tile" : "/gametiles/baum2.png", "action" : None, "x" : None, "y": None},
{"tile" : "/gametiles/baum3.png", "action" : None, "x" : None, "y": None}
]
tiles = 3
#~Variables

row=[]
for y in range(height_rows):
    col=[]
    for x in range(width_cols):
        col.append(objects[min(random.randint(1, 9), tiles)].copy().update({"x" : x, "y" : y}))
    row.append(col)
for i in row:
    for i in i:
        print i
name = raw_input("mapname: ")
pickle.dump(row, open("../maps/" + name + ".sav", "wb" ) )
print "fertig"

[BlackJack]

@jtschoch: Stimmt. Man muss es geringfügig ändern.

Die Namen sind übrigens teilweise schlecht gewählt. `row` steht nicht für *eine* Zeile sondern viele, und `col` steht nicht für eine Spalte, sondern für eine Zeile. Das ist verwirrend.

Von dem `i` in der zweiten ``for``-Schleife auf Modulebene mal ganz zu schweigen. Das ist ziemlich grausam.

[jtschoch]

Oh ja, die schleife soll eigentlich gar nicht ausgeprintet werden hab es nur schnell hingeschrieben,
aber das ändert trotzdem nix!

Achso wie könnte ich es sonnst zum Beispiel nennen?

[EyDu]

Achso wie könnte ich es sonnst zum Beispiel nennen?

"row" als "rows" und "col" als "row".

[jtschoch]

Aber das ändert an den None auch nix!

[EyDu]

Du packst doch auch jede Menge Nones rein. Schau dir die Stelle an, bei der die Listen befüllt werden und zerlege Ausdrücke in Teilausdrücke.

[jtschoch]

hä wie, verstehe gerade Bahnhof sorry!

[EyDu]

Du sollst dir einfach anschauen was du in cols packst.

[jtschoch]

Hä ich pack das rein,

Code: Alles auswählen

col.append(objects[min(random.randint(1, 9), tiles)].copy().update({"x" : x, "y" : y}))

was ist daran falsch?

[cofi]

Der Rueckgabewert von `dict.update`

[jtschoch]

Verstehe das trotzdem noch nicht

[jbs]

Rufe mal im interaktiven Modus ``print({}.update({}))`` auf.

[jtschoch]

None, weil der Return None ist, muss ich dann objects adden?
Könnt ihr mir bitte mal eine Lösung oder den richtigen Code schicken habe überhaupt keinen Plan.

[jbs]

Ich denke, es ist besser die Schritte einzeln durchzuführen. BlackJacks Beispiel ist eher akademisch interessant und nicht unbedingt im Produktiveinsatz zu empfehlen. Code sollte möglichst klar und verständlich für alle sein, auch wenn jeder etwas anderes darunter versteht.