Pythonic ใช้ความเข้าใจรายการเพื่อผลข้างเคียงหรือไม่?

ลองนึกถึงฟังก์ชันที่ฉันเรียกหาผลข้างเคียงไม่ใช่คืนค่า (เช่นการพิมพ์ไปที่หน้าจอการอัปเดต GUI การพิมพ์ไปยังไฟล์ ฯลฯ )

def fun_with_side_effects(x):
    ...side effects...
    return y

ตอนนี้Pythonicใช้ความเข้าใจในรายการเพื่อเรียกสิ่งนี้ว่า func หรือไม่:

[fun_with_side_effects(x) for x in y if (...conditions...)]

โปรดทราบว่าฉันไม่ได้บันทึกรายการไว้ที่ใดเลย

หรือฉันควรเรียกสิ่งนี้ว่า func แบบนี้:

for x in y:
    if (...conditions...):
        fun_with_side_effects(x)

ไหนดีกว่ากันและทำไม?

มันต่อต้าน Pythonic มากที่จะทำเช่นนั้นและ Pythonista ที่เก๋า ๆ จะทำให้คุณรู้สึกแย่ รายการกลางจะถูกทิ้งไปหลังจากสร้างแล้วและอาจมีขนาดใหญ่มากและมีราคาแพงในการสร้าง

คุณไม่ควรใช้ความเข้าใจในรายการเพราะอย่างที่มีคนบอกว่าจะสร้างรายการชั่วคราวขนาดใหญ่ที่คุณไม่ต้องการ สองวิธีต่อไปนี้เทียบเท่า:

consume(side_effects(x) for x in xs)

for x in xs:
    side_effects(x)

ด้วยคำจำกัดความของconsumeจากitertoolsหน้าคน:

def consume(iterator, n=None):
    "Advance the iterator n-steps ahead. If n is none, consume entirely."
    # Use functions that consume iterators at C speed.
    if n is None:
        # feed the entire iterator into a zero-length deque
        collections.deque(iterator, maxlen=0)
    else:
        # advance to the empty slice starting at position n
        next(islice(iterator, n, n), None)

แน่นอนว่าอย่างหลังชัดเจนและเข้าใจง่ายกว่า

ความเข้าใจในรายการมีไว้สำหรับสร้างรายการ และถ้าคุณไม่ได้สร้างรายการคุณไม่ควรทำใช้ความเข้าใจในรายการ

ดังนั้นฉันจะได้ตัวเลือกที่สองเพียงแค่วนซ้ำรายการแล้วเรียกใช้ฟังก์ชันเมื่อเงื่อนไขมีผล

ประการที่สองจะดีกว่า

นึกถึงคนที่ต้องเข้าใจรหัสของคุณ คุณสามารถรับผลกรรมไม่ดีได้ง่ายๆด้วยประการแรก :)

คุณสามารถอยู่ตรงกลางระหว่างทั้งสองโดยใช้ตัวกรอง () ลองพิจารณาตัวอย่าง:

y=[1,2,3,4,5,6]
def func(x):
    print "call with %r"%x

for x in filter(lambda x: x>3, y):
    func(x)

ขึ้นอยู่กับเป้าหมายของคุณ

หากคุณกำลังพยายามดำเนินการบางอย่างกับแต่ละออบเจ็กต์ในรายการควรใช้แนวทางที่สอง

หากคุณกำลังพยายามสร้างรายการจากรายการอื่นคุณอาจใช้ความเข้าใจในรายการ

Explicit ดีกว่าโดยปริยาย ง่ายดีกว่าซับซ้อน (Python Zen)

คุณทำได้

for z in (fun_with_side_effects(x) for x in y if (...conditions...)): pass

แต่มันไม่ค่อยสวย

การใช้ความเข้าใจในรายการสำหรับผลข้างเคียงนั้นน่าเกลียดไม่ใช่ Pythonic ไม่มีประสิทธิภาพและฉันจะไม่ทำ ฉันจะใช้forลูปแทนเพราะกforลูปส่งสัญญาณถึงรูปแบบขั้นตอนซึ่งผลข้างเคียงมีความสำคัญ

แต่ถ้าคุณยืนยันอย่างเต็มที่ที่จะใช้ความเข้าใจในรายการสำหรับผลข้างเคียงคุณควรหลีกเลี่ยงความไม่มีประสิทธิภาพโดยใช้นิพจน์ตัวสร้างแทน หากคุณยืนยันในสไตล์นี้อย่างเต็มที่ให้เลือกทำอย่างใดอย่างหนึ่งต่อไปนี้:

any(fun_with_side_effects(x) and False for x in y if (...conditions...))

หรือ:

all(fun_with_side_effects(x) or True for x in y if (...conditions...))

นี่คือนิพจน์ตัวสร้างและไม่สร้างรายการแบบสุ่มที่ถูกโยนออกไป ฉันคิดว่าallแบบฟอร์มอาจจะชัดเจนกว่าเล็กน้อยแม้ว่าฉันคิดว่าทั้งคู่สับสนและไม่ควรใช้

ฉันคิดว่ามันน่าเกลียดและฉันจะไม่ทำในรหัส แต่ถ้าคุณยืนยันที่จะใช้ลูปของคุณในรูปแบบนี้ฉันจะทำมัน

ฉันมักจะรู้สึกว่าความเข้าใจในรายการและความสำคัญของพวกเขาควรส่งสัญญาณถึงความพยายามที่จะใช้บางสิ่งบางอย่างที่คล้ายกับรูปแบบการทำงานเล็กน้อย การใส่สิ่งที่มีผลข้างเคียงที่ทำลายสมมติฐานนั้นจะทำให้ผู้คนต้องอ่านโค้ดของคุณอย่างรอบคอบมากขึ้นและฉันคิดว่านั่นเป็นสิ่งที่ไม่ดี