Ignore:
Timestamp:
10/10/09 17:26:08 (10 years ago)
Author:
Kaydeth_parpg
Message:

Ticket #2: Patch by Kaydeth. Updated the classes in console.py through engine.py (went through files alphabetically) to correct some obvious discrepancies with our coding standard. comment[s:trac, t:2]

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/game/scripts/engine.py

    r310 r312  
    1717 
    1818# there should be NO references to FIFE here! 
    19 import pickle, sys 
     19import pickle 
     20import sys 
    2021from gamestate import GameState 
    2122from objects import * 
     
    3839        # a World object (the fife stuff, essentially) 
    3940        self.view = view 
    40         self.mapchange = False 
    41         self.gameState = GameState() 
     41        self.map_change = False 
     42        self.game_state = GameState() 
    4243        self.pc_run = 1 
    43         self.targetPosition = None 
     44        self.target_position = None 
    4445    def reset(self): 
    4546        """Clears the data on a map reload so we don't have objects/npcs from 
     
    6162        # can't pickle SwigPyObjects 
    6263        behaviours = {} 
    63         behaviours[self.gameState.PC.ID] = self.gameState.PC.behaviour; 
    64         self.gameState.PC.behaviour = None; 
    65          
    66         npcs = [npc for npc in self.gameState.objects.values() if npc.trueAttr("NPC")] 
     64        behaviours[self.game_state.PC.ID] = self.game_state.PC.behaviour; 
     65        self.game_state.PC.behaviour = None; 
     66         
     67        npcs = [npc for npc in self.game_state.objects.values() \ 
     68                if npc.trueAttr("NPC")] 
    6769        for npc in npcs: 
    6870            behaviours[npc.ID] = npc.behaviour; 
    6971            npc.behaviour = None; 
    7072         
    71         pickle.dump(self.gameState, f) 
     73        pickle.dump(self.game_state, f) 
    7274        f.close() 
    7375         
     
    7577        for npc in npcs: 
    7678            npc.behaviour = behaviours[npc.ID]; 
    77         self.gameState.PC.behaviour = behaviours[self.gameState.PC.ID] 
     79        self.game_state.PC.behaviour = behaviours[self.game_state.PC.ID] 
    7880 
    7981    def load(self, path, filename): 
     
    9092            sys.stderr.write("Error: Can't find save game file\n") 
    9193            return 
    92         self.gameState = pickle.load(f) 
     94        self.game_state = pickle.load(f) 
    9395        f.close() 
    94         if self.gameState.currentMap: 
    95             self.loadMap(self.gameState.currentMapName, self.gameState.currentMap)  
     96        if self.game_state.currentMap: 
     97            self.loadMap(self.game_state.currentMapName, \ 
     98                         self.game_state.currentMap)  
    9699 
    97100    def createObject (self, layer, attributes, instance): 
     
    113116         
    114117        if obj.trueAttr("PC"): 
    115             self.addPC( layer, obj, instance) 
     118            self.addPC(layer, obj, instance) 
    116119        else: 
    117             self.addObject( layer, obj, instance) 
     120            self.addObject(layer, obj, instance) 
    118121 
    119122         
     
    133136         
    134137        # sync with game data 
    135         if not self.gameState.PC: 
    136             self.gameState.PC = pc 
     138        if self.game_state.PC is None: 
     139            self.game_state.PC = pc 
    137140             
    138         self.gameState.PC.setup() 
     141        self.game_state.PC.setup() 
    139142 
    140143 
     
    150153        """ 
    151154         
    152         ref = self.gameState.getObjectById(obj.ID)  
     155        ref = self.game_state.getObjectById(obj.ID)  
    153156        if ref is None: 
    154157            # no, add it to the game state 
    155             obj.map_id = self.gameState.currentMap 
    156             self.gameState.objects[obj.ID] = obj 
     158            obj.map_id = self.game_state.currentMap 
     159            self.game_state.objects[obj.ID] = obj 
    157160        else: 
    158161            # yes, use the current game state data 
     
    178181           @rtype: boolean 
    179182           @return: Status of result (True/False)""" 
    180         for i in self.gameState.getObjectsFromMap(self.gameState.currentMap): 
     183        for i in self.game_state.getObjectsFromMap(self.game_state.currentMap): 
    181184            if (i.ID == ident): 
    182185                # we found a match 
     
    193196        actions=[] 
    194197        # note: ALWAYS check NPC's first! 
    195         obj = self.gameState.getObjectById(obj_id) 
    196          
    197         if obj: 
     198        obj = self.game_state.getObjectById(obj_id) 
     199         
     200        if obj is not None: 
    198201            if obj.trueAttr("NPC"): 
    199202                # keep it simple for now, None to be replaced by callbacks 
     
    201204                actions.append(["Attack", "Attack", self.nullFunc, obj]) 
    202205            else: 
    203                 actions.append(["Examine", "Examine", self.gameState.PC.approach, \ 
    204                                 [obj.X, obj.Y], ExamineBoxAction(self, obj.name, obj.text)]) 
     206                actions.append(["Examine", "Examine", \ 
     207                                self.game_state.PC.approach, [obj.X, obj.Y], \ 
     208                                ExamineBoxAction(self, obj.name, obj.text)]) 
    205209                # is it a Door? 
    206210                if obj.trueAttr("door"): 
    207211                    actions.append(["Change Map", "Change Map", \ 
    208                        self.gameState.PC.approach, [obj.X, obj.Y], \ 
     212                       self.game_state.PC.approach, [obj.X, obj.Y], \ 
    209213                            ChangeMapAction(self, obj.target_map_name, \ 
    210214                                obj.target_map, obj.target_pos)]) 
    211215                # is it a container? 
    212216                if obj.trueAttr("container"): 
    213                     actions.append(["Open", "Open", self.gameState.PC.approach, [obj.X, obj.Y], OpenBoxAction(self, "Box")]) 
     217                    actions.append(["Open", "Open",  
     218                                    self.game_state.PC.approach, \ 
     219                                    [obj.X, obj.Y], \ 
     220                                    OpenBoxAction(self, "Box")]) 
    214221                # can you pick it up? 
    215222                if obj.trueAttr("carryable"): 
     
    226233        # TODO: work more on this when we get NPCData and HeroData straightened 
    227234        # out 
    228         npc = self.gameState.getObjectById(npcInfo.ID) 
    229         self.gameState.PC.approach([npc.getLocation().getLayerCoordinates().x, npc.getLocation().getLayerCoordinates().y], TalkAction(self, npc)) 
     235        npc = self.game_state.getObjectById(npcInfo.ID) 
     236        self.game_state.PC.approach([npc.getLocation().\ 
     237                                     getLayerCoordinates().x, \ 
     238                                     npc.getLocation().\ 
     239                                     getLayerCoordinates().y], \ 
     240                                     TalkAction(self, npc)) 
    230241 
    231242    def loadMap(self, map_name, map_file): 
     
    237248           @param map_file: Filename of map file to load 
    238249           @return: None""" 
    239         self.gameState.currentMap = map_file 
    240         self.gameState.currentMapName= map_name 
     250        self.game_state.currentMap = map_file 
     251        self.game_state.currentMapName= map_name 
    241252        self.view.loadMap(map_name, str(map_file)) 
    242253        self.view.setActiveMap(map_name) 
     
    245256 
    246257        # create the PC agent 
    247         self.view.activeMap.addPC(self.gameState.PC.behaviour.agent) 
    248         self.gameState.PC.start() 
     258        self.view.activeMap.addPC(self.game_state.PC.behaviour.agent) 
     259        self.game_state.PC.start() 
    249260 
    250261 
     
    254265           @param position: Screen position of click 
    255266           @return: None""" 
    256         if(self.pc_run==1): 
    257             self.gameState.PC.run(position) 
     267        if(self.pc_run == 1): 
     268            self.game_state.PC.run(position) 
    258269        else: 
    259             self.gameState.PC.walk(position) 
    260  
    261     def changeMap(self, mapName, mapFile, targetPosition): 
    262         """Registers for a mapchange on the next pump(). 
     270            self.game_state.PC.walk(position) 
     271 
     272    def changeMap(self, mapName, mapFile, target_position): 
     273        """Registers for a map change on the next pump(). 
    263274           @type nameName: String 
    264275           @param mapName: Id of the map to teleport to 
    265276           @type mapFile: String 
    266277           @param mapFile: Filename of the map to teleport to 
    267            @type targetPosition: Tuple 
    268            @param targetPosition: Position of PC on target map. 
     278           @type target_position: Tuple 
     279           @param target_position: Position of PC on target map. 
    269280           @return None""" 
    270         # set the parameters for the mapchange if moving to a new map 
    271         print self.gameState.currentMapName 
    272         if mapName != self.gameState.currentMapName: 
    273             self.gameState.currentMapName = mapName 
    274             self.gameState.currentMap = mapFile 
    275             self.targetPosition = targetPosition 
    276             # issue the mapchange 
    277             self.mapchange = True 
     281        # set the parameters for the map change if moving to a new map 
     282        print self.game_state.currentMapName 
     283        if mapName != self.game_state.currentMapName: 
     284            self.game_state.currentMapName = mapName 
     285            self.game_state.currentMap = mapFile 
     286            self.target_position = target_position 
     287            # issue the map change 
     288            self.map_change = True 
    278289        else: 
    279290            #set the player position on the current map 
    280             self.view.teleport(targetPosition) 
     291            self.view.teleport(target_position) 
    281292 
    282293    def handleCommands(self): 
    283         if self.mapchange: 
    284             self.loadMap(self.gameState.currentMapName, self.gameState.currentMap) 
    285             self.view.teleport(self.targetPosition) 
    286             self.mapchange = False 
     294        if self.map_change: 
     295            self.loadMap(self.game_state.currentMapName, \ 
     296                         self.game_state.currentMap) 
     297            self.view.teleport(self.target_position) 
     298            self.map_change = False 
    287299 
    288300    def pump(self): 
Note: See TracChangeset for help on using the changeset viewer.