[Magicor-devel] rev 162 - sliding fire

[Claudio <cla...@ya...>]

rev 162: fire pyshics change: when a fire sits over an ice block and the block is pushed, the fire travels on top of the block ( prev was: fire retain x-pos and falls when block moves away ). See at pompei level 11 for demo, or pompei level 09.
  I think the change doenst break any existing level, if there where the feature can be made optional ( via fire parameter )
  ###
Peter, I have a question about the actor classes design: currently theres a number of parameters that must be pased to the constructor for things that exists as only one instance, like level, firegroup, blockgroup, lightgroup. I see several disadvantages with this:
.The constructor list grows big and less clear
.If you want to change one of the groups ( like we do for the fragment bug ) you need to change every invocation to the constructor, not simply the constructor
.If an instance of class A must spawn an instance of class B and class B needs a group, then you must add the group as a parameter in the constructor of A, and change every invocation to the A constructor.
  You see any disadvantage in making globals for this things, so as no need to pass as constructor params ? There's no hurry with this.
  In the meantime, I used a proxy ( a dict and a function, the function transfers the info from the PlayState to the dict ). For the 'sliding fire' modification, It lets me not modify the blocks constructor nor each block instantiation, and it will be easy to change to the 'globals' or the 'all parameters' form. 
  ###
working in a base class for balls, lava lumps, etc ( flying non living things ).  with a little luck I can show some code next week.
   
cya

 		
---------------------------------
 Preguntá. Respondé. Descubrí.
 Todo lo que querías saber, y lo que ni imaginabas,
 está en Yahoo! Respuestas (Beta).
 Probalo ya!