[python] list comprehension: goede praktijk?

[Salvatron]

Ik gebruik list comprehensions maar dit vind ik er vreemd uitzien omdat regels beginnen en eindigen met "[" en "]" . Is dit een gangbare/normale praktijk?

Dit is een normale list comprehension:

Python:

modes=[1,2,3,4,5]
multiplied_numbers=[i*2 for i in modes]
for i in multiplied_numbers:
    print(i)

Hier zie je een list comprehension waarbij de regel code begint met "[" en eindigt met "]":

Python:

modes=[1,2,3,4,5]
[print(i*2) for i in modes]

Vraag: is dit een acceptabele, normale manier van het gebruik van list comprehensions in python? Ik vind het namelijk vreemd om een regel code met "[" en "]" te beginnen en eindigen.

[ Voor 8% gewijzigd door Salvatron op 14-09-2022 02:00 ]

[DataGhost]

De laatste manier is zeker niet acceptabel. Het werkt wel, maar is erg lelijk. Je maakt eigenlijk een lijst met het resultaat van je print-calls. Die returnen elk None dus je krijgt een lijst [None, None, ...] als resultaat, naast output op je scherm. De eerste manier is prima en daar zijn meerdere variaties op mogelijk. De meest gangbare oplossing daarvoor ligt het dichtst bij je laatste stukje code, maar dan met de loop zelf ongeveer gedaan zoals in je eerste voorbeeld.

[DaFeliX]

Ik vermoed dat de auteur van codeblok twee heeft geleerd dat "for" in python vaak onnodig is, en dat er een betere methode is. Alleen vind ik die oplossing echt lelijk.

Ik zou het zelf als volgt doen:

Python:

modes=[1,2,3,4,5]
multiplied_numbers=[i*2 for i in modes]
print(*modes,sep="")

Nog steeds geen "for", maar wel leesbaar

[edit]Of wellicht zo, hoewel iets minder leesbaar als je met ints werkt:

Python:

modes=[1,2,3,4,5]
multiplied_numbers=[i*2 for i in modes]
print(''.join(map(str, modes)))

[ Voor 21% gewijzigd door DaFeliX op 14-09-2022 07:27 ]

[Woy]

Ik zou toch gewoon zoiets doen

Python:

for item in [i*2 for i in modes]:
    print(item)

#of gewoon
for item in modes:
    print(item*2)

List comprehensions zijn erg handig, maar control flow statements zijn ook prima.

In het geval dat je de lijst wel delimeted wil hebben zou ik weer meer de oplossing van @DaFeliX gebruiken

Python:

print(', '.join(map(str, [i*2 for i in modes])))

Maar het gebruik van een non-pure function in een list-comprehension zou ik niet doen.

[ Voor 36% gewijzigd door Woy op 14-09-2022 08:21 ]

[RayNbow]

Als kanttekening: Python heeft naast list comprehensions ook generator expressions die efficiënter kunnen zijn. Bij een generator expression wordt niet een volledig gevuld lijst-object aangemaakt.

[DataGhost]

Woy schreef op woensdag 14 september 2022 @ 08:18:

In het geval dat je de lijst wel delimeted wil hebben zou ik weer meer de oplossing van @DaFeliX gebruiken

Python:

print(', '.join(map(str, [i*2 for i in modes])))

Maar het gebruik van een non-pure function in een list-comprehension zou ik niet doen.

Het gebruik van map wordt doorgaans niet als Pythonic gezien, zeker niet als je het combineert met een list comprehension. Dan is dit gebruikelijker:

Python:

print(', '.join([str(i*2) for i in modes]))
# of impliciet als generator
print(', '.join(str(i*2) for i in modes))

[Woy]

DataGhost schreef op woensdag 14 september 2022 @ 09:37:
[...]

Het gebruik van map wordt doorgaans niet als Pythonic gezien, zeker niet als je het combineert met een list comprehension. Dan is dit gebruikelijker:

Python:

print(', '.join([str(i*2) for i in modes]))
# of impliciet als generator
print(', '.join(str(i*2) for i in modes))

Ja helemaal gelijk, ik had vooral gewoon code gekopieerd, maar niet helemaal juist gereplaced

[ Voor 3% gewijzigd door Woy op 14-09-2022 10:33 ]

[Ben(V)]

Als het om grote aantallen gaat (dus modes is een lange lijst) dan zou ik een generator gebruiken.
Dat scheelt enorm in geheugen gebruik en snelheid.
Met een list comprehension maakt je een hele nieuwe lijst aan terwijl je alleen het resultaat van elke berekening wilt printen.

Zoiets dus:

code:

modes=[1,2,3,4,5]

for Result in (i*2 for i in modes):
    print(Result)

[ Voor 19% gewijzigd door Ben(V) op 15-09-2022 11:40 ]

[DataGhost]

Ben(V) schreef op donderdag 15 september 2022 @ 11:38:
Als het om grote aantallen gaat (dus modes is een lange lijst) dan zou ik een generator gebruiken.
Dat scheelt enorm in geheugen gebruik en snelheid.
Met een list comprehension maakt je een hele nieuwe lijst aan terwijl je alleen het resultaat van elke berekening wilt printen.

Zoiets dus:

code:

modes=[1,2,3,4,5]

for Result in (i*2 for i in modes):
    print(Result)

Alleen heb je in dit voorbeeld helemaal geen intermediate lijst of generator nodig, sterker, dat is behoorlijk lelijk. Alleen al tweemaal "for" op dezelfde regel zou een hint moeten zijn. De i*2 doe je dus lekker in de body en dat kan direct bij het printen.

[Ben(V)]

Uiteraard, maar dit was uiteraard een voorbeeld.
Wat ik wilde zeggen is dat heel vaak een generator een veel betere oplossing is dan een list comprehension.

[Salvatron]

Ik moet dit allemaal nog even onderzoeken want ik ben niet bekend met o.a. generators.

[Ben(V)]

https://towardsdatascienc...n-generators-a47b3cab1a23