2010-10-28 15 views
6

C'è un modo migliore per farlo? Mi sento come se stessi facendo qualcosa di sbagliato essendo troppo ripetitivo.C'è un modo migliore per scrivere questo?

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

Cosa stai cercando di fare? – katrielalex

+0

Questo pezzo di codice indica al mio motore 3D (simile a panda3d) di controllare quali "mappe" sono visibili (alfa) come trame sui miei poligoni quando un oggetto switch viene cliccato da un mouse (viz.pick); inversione dello stato di trasparenza della texture applicata. – relima

risposta

3

La trasformazione banale è:

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)) 

Questo si prende cura in modo tale che non sia NormSetts union'd con mappe delle città, come nel codice originale.

Problemi correlati