Das deutsche Python-Forum

Ich hab ein paar Zeilen Code, bei denen ich das Gefuehl habe man koennte es einfacher machen, aber ich finde nicht den richtigen Weg.
Im Prinzip steht ja in beiden Zweigen das gleiche, aber wenn ich xrange von einer groesseren zu einer kleineren Zahl laufen lasse, braucht es ja noch den dritten Parameter. Kann man hier irgendwie sinnvoll mit einer LC arbeiten?

Und: ist es schoener (und effizienter) wenn ich vorher cubes[-2] und cubes[-1] 'raushole' aus der Liste?

Code: Alles auswählen

if len(cubes[-2]) > len(cubes[-1]): 
    for i in xrange(len(cubes[-1]), len(cubes[-2])):
        name = str(i)
        out_cube.copy(cubes[-2][name].name, name)
else:
    for i in xrange(len(cubes[-2]), len(cubes[-1])):
        name = str(i)
        out_cube.copy(cubes[-1][name].name, name)

EDIT:: Code richtig eingerueckt.

Ungetestet:

Code: Alles auswählen

(len_a, cube_a), (len_b, cube_b) = sorted((len(c), c) for c in cubes[-2:])
for i in xrange(len_a, len_b):
    name = str(i)
    out_cube.copy(cube_b[name].name, name)

LC sind dazu da Listen zu erzeugen, da sehe ich nicht wie eine hier nützlich sein sollte.

Ungetestet:

Code: Alles auswählen

a, b = cubes[-2:]
if len(a) > len(b):
    a, b = b, a
for i in xrange(len(a), len(b)):
    name = str(i)
    out_cube.copy(a[name].name, name)

pillmuncher hat geschrieben:Ungetestet:

Code: Alles auswählen

a, b = cubes[-2:]
if len(a) > len(b):
    a, b = b, a
    for i in xrange(len(a), len(b)):
        name = str(i)
        out_cube.copy(a[name].name, name)

Die for Schleife muesste wieder 'ausgerueckt' sein, oder?

EDIT:: @pillmuncher hat sich ja durch dein blitzschnelles Editieren erledigt.

@BlackJack: funktioniert, wirkt schon sauberer

Das deutsche Python-Forum

Das muss irgendwie einfacher gehen. LC?

Das muss irgendwie einfacher gehen. LC?

Re: Das muss irgendwie einfacher gehen. LC?

Re: Das muss irgendwie einfacher gehen. LC?

Re: Das muss irgendwie einfacher gehen. LC?