Ignore:
Timestamp:
10/17/09 02:05:35 (10 years ago)
Author:
Kaydeth_parpg
Message:

Ticket #2: Patch by Kaydeth. Updated the classes in local_loaders and utilities directories to correct some obvious discrepancies with our coding standard. comment[s:trac, t:2] fixes[s:trac, t:2]

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/game/local_loaders/xmlmap.py

    r325 r328  
    5959        self.time_to_load = 0 
    6060 
    61         self.nspace = None 
     61        self.n_space = None 
    6262 
    6363    def _err(self, msg): 
     
    7878    def parseMap(self, map_elt): 
    7979        if not map_elt: 
    80             self._err('No <map> element found at top level of map file definition.') 
     80            self._err(\ 
     81                'No <map> element found at top level of map file definition.') 
    8182        id,format = map_elt.get('id'),map_elt.get('format') 
    8283 
    83         if not format == FORMAT: self._err(''.join(['This file has format ', format, ' but this loader has format ', FORMAT])) 
     84        if not format == FORMAT: self._err(''.join(['This file has format ', \ 
     85                                            format, \ 
     86                                            ' but this loader has format ', \ 
     87                                            FORMAT])) 
    8488        if not id: self._err('Map declared without an identifier.') 
    8589 
     
    8892            self.map = self.model.createMap(str(id)) 
    8993            self.map.setResourceFile(self.source) 
    90         except fife.Exception, e: # NameClash appears as general fife.Exception; any ideas? 
     94        # NameClash appears as general fife.Exception; any ideas? 
     95        except fife.Exception, e:  
    9196            print e.getMessage() 
    92             print ''.join(['File: ', self.source, '. The map ', str(id), ' already exists! Ignoring map definition.']) 
     97            print ''.join(['File: ', self.source, '. The map ', str(id), \ 
     98                           ' already exists! Ignoring map definition.']) 
    9399            return None 
    94100 
     
    111117 
    112118        if self.callback:         
    113             tmplist = map_elt.findall('import') 
     119            tmp_list = map_elt.findall('import') 
    114120            i = float(0) 
    115121         
     
    140146            if self.callback: 
    141147                i += 1                 
    142                 self.callback('loaded imports', float( i / float(len(tmplist)) * 0.25 + 0.25 ) ) 
     148                self.callback('loaded imports', \ 
     149                              float( i / float(len(tmp_list)) * 0.25 + 0.25 )) 
    143150 
    144151 
    145152    def parseLayers(self, map_elt, map): 
    146153        if self.callback is not None:         
    147             tmplist = map_elt.findall('layer') 
     154            tmp_list = map_elt.findall('layer') 
    148155            i = float(0) 
    149156 
     
    157164            y_offset = layer.get('y_offset') 
    158165            pathing = layer.get('pathing') 
    159             transparency = layer.get('transparency') 
     166            transparency = layer.get('transparency') 
    160167 
    161168            if not x_scale: x_scale = 1.0 
     
    172179 
    173180            if not id: self._err('<layer> declared with no id attribute.') 
    174             if not grid_type: self._err(''.join(['Layer ', str(id), ' has no grid_type attribute.'])) 
     181            if not grid_type: self._err(''.join(['Layer ', str(id), \ 
     182                                            ' has no grid_type attribute.'])) 
    175183 
    176184            allow_diagonals = pathing == "cell_edges_and_diagonals" 
    177             cellgrid = self.model.getCellGrid(grid_type) 
    178             if not cellgrid: self._err('<layer> declared with invalid cellgrid type. (%s)' % grid_type) 
    179  
    180             cellgrid.setRotation(float(rotation)) 
    181             cellgrid.setXScale(float(x_scale)) 
    182             cellgrid.setYScale(float(y_scale)) 
    183             cellgrid.setXShift(float(x_offset)) 
    184             cellgrid.setYShift(float(y_offset)) 
     185            cell_grid = self.model.getCellGrid(grid_type) 
     186            if not cell_grid: self._err('<layer> declared with invalid '\ 
     187                                       'cell grid type. (%s)' % grid_type) 
     188 
     189            cell_grid.setRotation(float(rotation)) 
     190            cell_grid.setXScale(float(x_scale)) 
     191            cell_grid.setYScale(float(y_scale)) 
     192            cell_grid.setXShift(float(x_offset)) 
     193            cell_grid.setYShift(float(y_offset)) 
    185194 
    186195            layer_obj = None 
    187196            try: 
    188                 layer_obj = map.createLayer(str(id), cellgrid) 
     197                layer_obj = map.createLayer(str(id), cell_grid) 
    189198            except fife.Exception, e: 
    190199                print e.getMessage() 
    191                 print 'The layer ' + str(id) + ' already exists! Ignoring this layer.' 
     200                print 'The layer ' + str(id) + \ 
     201                        ' already exists! Ignoring this layer.' 
    192202                continue 
    193203 
     
    205215            if self.callback is not None: 
    206216                i += 1 
    207                 self.callback('loaded layer :' + str(id), float( i / float(len(tmplist)) * 0.25 + 0.5 ) ) 
     217                self.callback('loaded layer :' + \ 
     218                              str(id), float( i / float(len(tmp_list)) * \ 
     219                                              0.25 + 0.5 ) ) 
    208220 
    209221        # cleanup 
    210222        if self.callback is not None: 
    211             del tmplist 
     223            del tmp_list 
    212224            del i 
    213225 
    214     def parseInstances(self, layerelt, layer): 
    215         instelt = layerelt.find('instances') 
    216  
    217         instances = instelt.findall('i') 
    218         instances.extend(instelt.findall('inst')) 
    219         instances.extend(instelt.findall('instance')) 
     226    def parseInstances(self, layer_elt, layer): 
     227        inst_elt = layer_elt.find('instances') 
     228 
     229        instances = inst_elt.findall('i') 
     230        instances.extend(inst_elt.findall('inst')) 
     231        instances.extend(inst_elt.findall('instance')) 
    220232        for instance in instances: 
    221233 
    222             objectID = instance.get('object') 
    223             if not objectID: 
    224                 objectID = instance.get('obj') 
    225             if not objectID: 
    226                 objectID = instance.get('o') 
    227  
    228             if not objectID: self._err('<instance> does not specify an object attribute.') 
    229  
    230             nspace = instance.get('namespace') 
    231             if not nspace: 
    232                 nspace = instance.get('ns') 
    233             if not nspace: 
    234                 nspace = self.nspace 
    235  
    236             if not nspace: self._err('<instance> %s does not specify an object namespace, and no default is available.' % str(objectID)) 
    237  
    238             self.nspace = nspace 
    239  
    240             object = self.model.getObject(str(objectID), str(nspace)) 
     234            object_id = instance.get('object') 
     235            if not object_id: 
     236                object_id = instance.get('obj') 
     237            if not object_id: 
     238                object_id = instance.get('o') 
     239 
     240            if not object_id: self._err('<instance> does not specify an '\ 
     241                                        'object attribute.') 
     242 
     243            n_space = instance.get('namespace') 
     244            if not n_space: 
     245                n_space = instance.get('ns') 
     246            if not n_space: 
     247                n_space = self.n_space 
     248 
     249            if not n_space: self._err('<instance> %s does not specify an '\ 
     250                                      'object namespace, and no default is '\ 
     251                                      'available.' % str(object_id)) 
     252 
     253            self.n_space = n_space 
     254 
     255            object = self.model.getObject(str(object_id), str(n_space)) 
    241256            if not object: 
    242                 print ''.join(['Object with id=', str(objectID), ' ns=', str(nspace), ' could not be found. Omitting...']) 
     257                print ''.join(['Object with id=', str(object_id), ' ns=', \ 
     258                               str(n_space), \ 
     259                               ' could not be found. Omitting...']) 
    243260                continue 
    244261 
     
    246263            y = instance.get('y') 
    247264            z = instance.get('z') 
    248             stackpos = instance.get('stackpos') 
     265            stack_pos = instance.get('stackpos') 
    249266            id = instance.get('id') 
    250267 
     
    272289                id = str(id) 
    273290 
    274             inst = layer.createInstance(object, fife.ExactModelCoordinate(x,y,z), str(id)) 
     291            inst = layer.createInstance(object, \ 
     292                                        fife.ExactModelCoordinate(x,y,z), \ 
     293                                        str(id)) 
    275294 
    276295            rotation = instance.get('r') 
     
    288307 
    289308            fife.InstanceVisual.create(inst) 
    290             if (stackpos): 
    291                 inst.get2dGfxVisual().setStackPosition(int(stackpos)) 
     309            if (stack_pos): 
     310                inst.get2dGfxVisual().setStackPosition(int(stack_pos)) 
    292311 
    293312            if (object.getAction('default')): 
     
    303322                inst_dict["xpos"] = x 
    304323                inst_dict["ypos"] = y 
    305                 inst_dict["gfx"] = objectID 
     324                inst_dict["gfx"] = object_id 
    306325                inst_dict["is_open"] = instance.get('is_open') 
    307326                inst_dict["locked"] = instance.get('locked') 
     
    312331                inst_dict["target_map_name"] = instance.get('target_map_name') 
    313332                inst_dict["target_map"] = instance.get('target_map') 
    314                 inst_dict["target_pos"] = (instance.get('target_x'), instance.get('target_y')) 
     333                inst_dict["target_pos"] = (instance.get('target_x'), \ 
     334                                           instance.get('target_y')) 
    315335                self.data.createObject(layer, inst_dict, inst) 
    316336                 
    317337    def parseCameras(self, map_elt, map): 
    318338        if self.callback:         
    319             tmplist = map_elt.findall('camera') 
     339            tmp_list = map_elt.findall('camera') 
    320340            i = float(0) 
    321341 
     
    328348            ref_cell_width = camera.get('ref_cell_width') 
    329349            ref_cell_height = camera.get('ref_cell_height') 
    330             viewport = camera.get('viewport') 
     350            view_port = camera.get('viewport') 
    331351 
    332352            if not zoom: zoom = 1 
     
    335355 
    336356            if not id: self._err('Camera declared without an id.') 
    337             if not ref_layer_id: self._err(''.join(['Camera ', str(id), ' declared with no reference layer.'])) 
    338             if not (ref_cell_width and ref_cell_height): self._err(''.join(['Camera ', str(id), ' declared without reference cell dimensions.'])) 
     357            if not ref_layer_id: self._err(''.join(['Camera ', str(id), \ 
     358                                                    ' declared with no '\ 
     359                                                    'reference layer.'])) 
     360            if not (ref_cell_width and ref_cell_height): 
     361                self._err(''.join(['Camera ', str(id), \ 
     362                                   ' declared without reference cell '\ 
     363                                   'dimensions.'])) 
    339364 
    340365            try: 
    341                 if viewport: 
    342                     cam = self.engine.getView().addCamera(str(id), map.getLayer(str(ref_layer_id)),fife.Rect(*[int(c) for c in viewport.split(',')]),fife.ExactModelCoordinate(0,0,0)) 
     366                if view_port: 
     367                    cam = self.engine.getView().addCamera(str(id), \ 
     368                                    map.getLayer(str(ref_layer_id)), \ 
     369                                    fife.Rect(\ 
     370                                    *[int(c) for c in view_port.split(',')]),\ 
     371                                    fife.ExactModelCoordinate(0,0,0)) 
    343372                else: 
    344373                    screen = self.engine.getRenderBackend() 
    345                     cam = self.engine.getView().addCamera(str(id), map.getLayer(str(ref_layer_id)),fife.Rect(0,0,screen.getScreenWidth(),screen.getScreenHeight()),fife.ExactModelCoordinate(0,0,0)) 
    346  
    347                 cam.setCellImageDimensions(int(ref_cell_width), int(ref_cell_height)) 
     374                    cam = self.engine.getView().addCamera(str(id), \ 
     375                                    map.getLayer(str(ref_layer_id)), \ 
     376                                    fife.Rect(0, 0, screen.getScreenWidth(), \ 
     377                                              screen.getScreenHeight()), \ 
     378                                              fife.ExactModelCoordinate(0, 0,\ 
     379                                                                        0)) 
     380 
     381                cam.setCellImageDimensions(int(ref_cell_width), \ 
     382                                           int(ref_cell_height)) 
    348383                cam.setRotation(float(rotation)) 
    349384                cam.setTilt(float(tilt)) 
     
    354389            if self.callback: 
    355390                i += 1 
    356                 self.callback('loaded camera: ' +  str(id), float( i / len(tmplist) * 0.25 + 0.75 ) )     
     391                self.callback('loaded camera: ' +  str(id), \ 
     392                              float( i / len(tmp_list) * 0.25 + 0.75 ) )     
Note: See TracChangeset for help on using the changeset viewer.