2010-10-28 39 views
6

¿Hay una mejor manera de hacer esto? Siento que estoy haciendo algo mal al ser demasiado repetitivo.¿Hay alguna forma mejor de escribir esto?

O = viz.pick(1, viz.WORLD) 

BackSetts = ["set_b1b", "set_b2a", "set_b1a", "set_b2b"] 
LeftSetts = ["set_l1a", "set_l1b", "set_l2a", "set_l1b"] 
NormSetts = ["set_nr_a", "set_nr_b"] 
Maps = ["MapA","MapB"] 

if O.name in BackSetts: 
    for i in set(BackSetts)|set(Maps): 
     WORLD[i].alpha(abs(WORLD[i].getAlpha()-1)) 

elif O.name in LeftSetts: 
    for i in set(LeftSetts)|set(Maps): 
     WORLD[i].alpha(abs(WORLD[i].getAlpha()-1)) 

elif O.name in NormSetts: 
    for i in NormSetts: 
     WORLD[i].alpha(abs(WORLD[i].getAlpha()-1)) 
+2

¿Qué estás tratando de hacer? – katrielalex

+0

Este fragmento de código le dice a mi motor 3D (similar a panda3d) que compruebe qué "mapas" son visibles (alfa) como texturas en mis polígonos cuando hace clic en un objeto de cambio con un mouse (viz.pick); invierte el estado de transparencia de la textura aplicada. – relima

Respuesta

3

La transformación es trivial:

O = viz.pick(1, viz.WORLD) 

BackSetts = ["set_b1b", "set_b2a", "set_b1a", "set_b2b"] 
LeftSetts = ["set_l1a", "set_l1b", "set_l2a", "set_l1b"] 
NormSetts = ["set_nr_a", "set_nr_b"] 
Maps = ["MapA","MapB"] 
anyset = [] 

if O.name in BackSetts: 
    anyset = set(BackSetts)|set(Maps) 

elif O.name in LeftSetts: 
    anyset = set(LeftSetts)|set(Maps) 

elif O.name in NormSetts: 
    anyset = NormSetts 

for i in anyset: 
    WORLD[i].alpha(abs(WORLD[i].getAlpha()-1)) 

Este se encarga de tal manera que no se NormSetts union'd con mapas, como en el código original.

Cuestiones relacionadas