Советы по рогалику

import player
import myMap
import tkinter
 
# массив цветов, возвращает цвет в зависимости
# от id предмета который находится в клетке    
def color(i):
    return ['black', 'red', 'green', 'blue'][i]
 
 
Height = 10# высота окна
Width  = 10# ширина ркна
Size   = 20# размер клетки
 
root = tkinter.Tk()
window = tkinter.Canvas(root, width=400, height=200)
window.pack()
 
p  = player.Player(1, x = 0, y = 0, health = 10, attack = 3) # герой
en = player.Player(2, x = 5, y = 5, health = 7,  attack = 1) # враг
map = myMap.Map(10, 10)
 
done = False
 
 
def moveRight(event):
    p.moveRight()
    draw(event)
 
def moveLeft(event):
    p.moveLeft()
    draw(event)
 
def moveUp(event):
    p.moveUp()
    draw(event)
 
def moveDown(event):
    p.moveDown()
    draw(event)
 
# устанавливает ячейки карты
# в нужные значения
def evaluate():
    map.clear()
    map.cell(en)
    map.cell(p)
 
# рисует окно
def draw(event):
    evaluate()
    for y in range(0, Height):
        for x in range(0, Width):
            current = map.getCell(x, y)
            window.create_rectangle(x * 20, y * 20, x * 20 + 20, y * 20 + 20, fill=color(current))
            window.pack()
 
root.bind('w', moveUp)
root.bind('a', moveLeft)
root.bind('s', moveDown)
root.bind('d', moveRight)
root.mainloop()

class Map:
    
    map_ = []
    content  = []
    def __init__(self, width, height):
        self.width  = width
        self.height = height
        self.clear()
 
    #заполняет карту нулями, очищает массив обьектов
    def clear(self):
        self.content.clear()
        self.map_.clear()
        for x in range(0, self.width):
            buff = []
            for y in range(0, self.height):
                buff.append(0)
                
            self.map_.append(buff)
 
    def getCell(self, x, y):
        if x>self.width:
            x=0
 
        if x<0:
            x=self.width-1
 
        if y>self.height:
            y=0
 
        if y<0:
            y=self.height-1
            
        return self.map_[x][y]
 
    # устанавливает ячейку с координатами
    # obj.x и obj.y и запоминает последний объект,
    # или если ячейка занята начинает бой между
    # obj  и содержимым ячейки    
    def cell(self, obj):
        for i in self.content:
            if i.x == obj.x and i.y == obj.y:
                   i.hit(obj)
                   obj.hit(i)
                   break
 
        else:
            self.content.append(obj)
            self.map_[obj.x][obj.y] = obj.symbol
 
    def print(self):
        for y in range(0, self.height):
            for x in range(0, self.width):
                print(self.map_[x][y], end = "")
 
            print()

class Player:
 
    symbol = ""
    
    def __init__(self, id_, x=0, y=0, attack = 0, health = 0):
        self.symbol = id_
        self.x = x
        self.y = y
        self.health = health
        self.attack = attack
 
    def hit(self, en):
        print("{} hits {}".format(en.symbol, self.symbol))
        en.health -= self.attack
        en.check()
        self.health -= en.attack
        print("{} - en.health, {} - self.health".format(en.health, self.health))
        self.check()
 
    def check(self):
        if self.health <= 0:
            self.symbol = 0
            self.attack = 0
            
    def moveRight(self):
        self.x += 1
 
    def moveLeft(self):
        self.x -= 1
 
    def moveUp(self):
        self.y -= 1
 
    def moveDown(self):
        self.y += 1