1

我的代码遇到了一个我根本无法弄清楚的问题。

根据以下代码部分：

(worldmap)在类中创建第二个字典后Generate_Game_World，总部位置存储在

self.worldmap[HQLOCATIONX,HQLOCATIONY]["Region_Type"] = "HQ"

但是，这样做之后，它似乎用测试窗口中看到的“HQ”值填充了整个数组。我根本无法弄清楚为什么会这样。

import pygame
from tkinter import *

test = Tk ()
test.geometry = ("640x480")

pygame.init()

WORLDSIZE = 499
HQLOCATIONX = int(WORLDSIZE/2)
HQLOCATIONY = int(WORLDSIZE/2)

class Generate_Game_World ():

    regionData = {"Region_Type" : "None", 
                  "Region_Level" : 0}

    def __init__(self, mapSize):
        self.mapSize = mapSize

    def main (self):
        # creates 2d map
        self.worldmap = {(x,y) : self.regionData for x in range(self.mapSize) for y in range (self.mapSize)}

        # Sets the HQ to worldmap(249,249)
        self.worldmap[HQLOCATIONX,HQLOCATIONY]["Region_Type"] = "HQ"

        # checks worldmap(0,0) --> (10,10) for its Region Type
        for x in range (10):
            for y in range (10):
                label = Label (text=self.worldmap[x,y]["Region_Type"]).grid(row = x, column=y)

class Game (object):
    def main (self, screen):
        gameworld = Generate_Game_World(WORLDSIZE)     
        gameworld.main()

        while 1:

            for event in pygame.event.get():
                if event.type == pygame.QUIT:
                    return

                if event.type == pygame.KEYDOWN and event.key == pygame.K_ESCAPE:
                    return

            test.mainloop()
            pygame.display.flip()

if __name__ == "__main__":
    screen = pygame.display.set_mode((1024, 768))
    Game().main(screen)
4

1 回答 1

2

您所有的字典值都是对一个class-attribute的引用regionData。

您想创建副本：

self.worldmap = {(x,y): self.regionData.copy() for x in range(self.mapSize) for y in range (self.mapSize)}

这使用该dict.copy()方法来创建浅拷贝；这已经足够了，因为 的值Generate_Game_World.regionData只是字符串，而且它们是不可变的，因此可以安全地共享。

您还可以使用字典文字从头开始创建新字典；您似乎没有self.regionData在其他任何地方使用，因此可以进行内联：

self.worldmap = {(x,y): {"Region_Type" : "None", "Region_Level" : 0} 
                 for x in range(self.mapSize) for y in range (self.mapSize)}

最后但并非最不重要的一点是，您的意思是使用None单例而不是字符串"None"吗？

于 2013-08-17T21:59:57.570 回答