Styling kondisi multi-baris dalam pernyataan 'jika'? [Tutup]

Kadang-kadang saya memecah kondisi panjang ifmenjadi beberapa baris. Cara paling jelas untuk melakukan ini adalah:

  if (cond1 == 'val1' and cond2 == 'val2' and
      cond3 == 'val3' and cond4 == 'val4'):
      do_something

Tidak terlalu menarik secara visual, karena aksi menyatu dengan kondisi. Namun, ini adalah cara alami menggunakan indentasi Python yang benar dari 4 spasi.

Untuk saat ini saya menggunakan:

  if (    cond1 == 'val1' and cond2 == 'val2' and
          cond3 == 'val3' and cond4 == 'val4'):
      do_something

Tapi ini tidak terlalu cantik. :-)

Bisakah Anda merekomendasikan cara alternatif?

Anda tidak perlu menggunakan 4 spasi pada saluran kondisional kedua Anda. Mungkin menggunakan:

if (cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'):
    do_something

Juga, jangan lupa spasi putih lebih fleksibel daripada yang Anda kira:

if (   
       cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'
   ):
    do_something
if    (cond1 == 'val1' and cond2 == 'val2' and 
       cond3 == 'val3' and cond4 == 'val4'):
    do_something

Keduanya cukup jelek.

Mungkin kehilangan tanda kurung ( Panduan Gaya mencegah ini)?

if cond1 == 'val1' and cond2 == 'val2' and \
   cond3 == 'val3' and cond4 == 'val4':
    do_something

Setidaknya ini memberi Anda beberapa perbedaan.

Atau bahkan:

if cond1 == 'val1' and cond2 == 'val2' and \
                       cond3 == 'val3' and \
                       cond4 == 'val4':
    do_something

Saya rasa saya lebih suka:

if cond1 == 'val1' and \
   cond2 == 'val2' and \
   cond3 == 'val3' and \
   cond4 == 'val4':
    do_something

Berikut Panduan Gaya , yang (sejak 2010) merekomendasikan menggunakan tanda kurung.

Saya telah menggunakan yang berikut ini dalam kasus degenerasi di mana itu hanya AND atau OR.

if all( [cond1 == 'val1', cond2 == 'val2', cond3 == 'val3', cond4 == 'val4'] ):

if any( [cond1 == 'val1', cond2 == 'val2', cond3 == 'val3', cond4 == 'val4'] ):

Ini mencukur beberapa karakter dan membuatnya jelas bahwa tidak ada kehalusan kondisi.

Seseorang harus memperjuangkan penggunaan spasi putih vertikal di sini! :)

if (     cond1 == val1
     and cond2 == val2
     and cond3 == val3
   ):
    do_stuff()

Ini membuat setiap kondisi terlihat jelas. Ini juga memungkinkan ekspresi yang lebih bersih dari kondisi yang lebih kompleks:

if (    cond1 == val1
     or 
        (     cond2_1 == val2_1
          and cond2_2 >= val2_2
          and cond2_3 != bad2_3
        )
   ):
    do_more_stuff()

Ya, kami memperdagangkan sedikit real estat vertikal untuk kejelasan. Layak IMO.

Saya lebih suka gaya ini ketika saya memiliki kondisi jika sangat besar:

if (
    expr1
    and (expr2 or expr3)
    and hasattr(thingy1, '__eq__')
    or status=="HappyTimes"
):
    do_stuff()
else:
    do_other_stuff()

Inilah pendapat saya yang sangat pribadi: kondisi lama (dalam pandangan saya) adalah bau kode yang menyarankan refactoring ke fungsi / metode pengembalian boolean. Sebagai contoh:

def is_action__required(...):
    return (cond1 == 'val1' and cond2 == 'val2'
            and cond3 == 'val3' and cond4 == 'val4')

Sekarang, jika saya menemukan cara untuk membuat kondisi multi-line terlihat bagus, saya mungkin akan menemukan diri saya puas dengan memilikinya dan melewatkan refactoring.

Di sisi lain, membiarkan mereka mengganggu indera estetika saya bertindak sebagai insentif untuk refactoring.

Kesimpulan saya, oleh karena itu, adalah bahwa beberapa kondisi garis harus terlihat jelek dan ini merupakan insentif untuk menghindarinya.

Ini tidak banyak membaik tetapi ...

allCondsAreOK = (cond1 == 'val1' and cond2 == 'val2' and
                 cond3 == 'val3' and cond4 == 'val4')

if allCondsAreOK:
   do_something

Saya sarankan memindahkan andkata kunci ke baris kedua dan membuat indentasi semua baris yang berisi kondisi dengan dua spasi alih-alih empat:

if (cond1 == 'val1' and cond2 == 'val2'
  and cond3 == 'val3' and cond4 == 'val4'):
    do_something

Inilah cara saya memecahkan masalah ini dalam kode saya. Memiliki kata kunci sebagai kata pertama dalam garis membuat kondisi jauh lebih mudah dibaca, dan mengurangi jumlah ruang semakin membedakan kondisi dari tindakan.

Tampaknya layak mengutip PEP 0008 (panduan gaya resmi Python), karena berkomentar tentang masalah ini panjang lebar:

Ketika bagian kondisional dari suatu- ifpernyataan cukup panjang untuk mengharuskan itu dituliskan di beberapa baris, perlu dicatat bahwa kombinasi dari kata kunci dua karakter (yaitu if), ditambah spasi tunggal, ditambah tanda kurung pembukaan menciptakan sebuah 4- alami. ruang indent untuk baris-baris berikutnya dari conditional multiline. Ini dapat menghasilkan konflik visual dengan deretan kode indentasi yang bersarang di dalam- ifpernyataan, yang juga secara alami akan menjorok ke 4 spasi. PEP ini tidak mengambil posisi eksplisit tentang bagaimana (atau apakah) untuk lebih jauh membedakan garis kondisional dari suite bersarang di dalam ifpernyataan. Opsi yang dapat diterima dalam situasi ini termasuk, tetapi tidak terbatas pada:

# No extra indentation.
if (this_is_one_thing and
    that_is_another_thing):
    do_something()

# Add a comment, which will provide some distinction in editors
# supporting syntax highlighting.
if (this_is_one_thing and
    that_is_another_thing):
    # Since both conditions are true, we can frobnicate.
    do_something()

# Add some extra indentation on the conditional continuation line.
if (this_is_one_thing
        and that_is_another_thing):
    do_something()

Perhatikan "tidak terbatas pada" dalam kutipan di atas; Selain pendekatan yang disarankan dalam panduan gaya, beberapa yang disarankan dalam jawaban lain untuk pertanyaan ini juga dapat diterima.

Inilah yang saya lakukan, ingat bahwa "semua" dan "apa saja" menerima iterable, jadi saya hanya memasukkan syarat yang panjang dalam daftar dan membiarkan "semua" melakukan pekerjaan.

condition = [cond1 == 'val1', cond2 == 'val2', cond3 == 'val3', cond4 == 'val4']

if all(condition):
   do_something

Saya terkejut tidak melihat solusi pilihan saya,

if (cond1 == 'val1' and cond2 == 'val2'
    and cond3 == 'val3' and cond4 == 'val4'):
    do_something

Karena andmerupakan kata kunci, itu akan disorot oleh editor saya, dan terlihat cukup berbeda dari do_something di bawahnya.

Menambah apa yang dikatakan @krawyoti ... Kondisi lama berbau karena sulit dibaca dan sulit dimengerti. Menggunakan fungsi atau variabel membuat kode lebih jelas. Dalam Python, saya lebih suka menggunakan ruang vertikal, melampirkan tanda kurung, dan menempatkan operator logis di awal setiap baris sehingga ekspresi tidak terlihat seperti "mengambang".

conditions_met = (
    cond1 == 'val1' 
    and cond2 == 'val2' 
    and cond3 == 'val3' 
    and cond4 == 'val4'
    )
if conditions_met:
    do_something

Jika kondisi perlu dievaluasi lebih dari sekali, seperti dalam satu whilelingkaran, maka menggunakan fungsi lokal adalah yang terbaik.

Secara pribadi, saya ingin menambahkan makna pada pernyataan if panjang. Saya harus mencari melalui kode untuk menemukan contoh yang sesuai, tetapi inilah contoh pertama yang terlintas dalam pikiran: katakanlah saya kebetulan menemukan beberapa logika unik di mana saya ingin menampilkan halaman tertentu tergantung pada banyak variabel.

Bahasa Inggris: "Jika pengguna yang masuk BUKAN seorang guru administrator, tetapi hanya guru biasa, dan bukan siswa itu sendiri ..."

if not user.isAdmin() and user.isTeacher() and not user.isStudent():
    doSomething()

Tentu ini mungkin terlihat baik-baik saja, tetapi membaca itu jika pernyataan itu banyak pekerjaan. Bagaimana kalau kita menetapkan logika pada label yang masuk akal. "Label" sebenarnya adalah nama variabel:

displayTeacherPanel = not user.isAdmin() and user.isTeacher() and not user.isStudent()
if displayTeacherPanel:
    showTeacherPanel()

Ini mungkin tampak konyol, tetapi Anda mungkin memiliki kondisi lain di mana Anda HANYA ingin menampilkan item lain jika, dan hanya jika, Anda menampilkan panel guru ATAU jika pengguna memiliki akses ke panel tertentu lainnya secara default:

if displayTeacherPanel or user.canSeeSpecialPanel():
    showSpecialPanel()

Cobalah menulis kondisi di atas tanpa menggunakan variabel untuk menyimpan dan memberi label logika Anda, dan Anda tidak hanya berakhir dengan pernyataan logis yang sangat sulit dibaca, tetapi Anda juga baru saja mengulangi sendiri. Meskipun ada pengecualian yang masuk akal, ingat: Jangan Ulangi Diri Sendiri (KERING).

"semua" dan "apa saja" bagus untuk banyak kondisi dengan tipe huruf yang sama. TAPI mereka selalu mengevaluasi semua kondisi. Seperti yang ditunjukkan dalam contoh ini:

def c1():
    print " Executed c1"
    return False
def c2():
    print " Executed c2"
    return False


print "simple and (aborts early!)"
if c1() and c2():
    pass

print

print "all (executes all :( )"
if all((c1(),c2())):
    pass

print

(Saya telah sedikit mengubah pengidentifikasi sebagai nama dengan lebar tetap tidak mewakili kode nyata - setidaknya bukan kode nyata yang saya temui - dan akan percaya pada keterbacaan contoh.)

if (cond1 == "val1" and cond22 == "val2"
and cond333 == "val3" and cond4444 == "val4"):
    do_something

Ini bekerja dengan baik untuk "dan" dan "atau" (penting bahwa mereka pertama kali berada di baris kedua), tetapi lebih sedikit untuk kondisi panjang lainnya. Untungnya, yang pertama tampaknya menjadi kasus yang lebih umum sementara yang terakhir sering mudah ditulis ulang dengan variabel sementara. (Biasanya tidak sulit, tetapi bisa sulit atau jauh lebih jelas / mudah dibaca untuk menjaga hubungan arus pendek "dan" / "atau" ketika menulis ulang.)

Karena saya menemukan pertanyaan ini dari posting blog Anda tentang C ++ , saya akan menyertakan bahwa gaya C ++ saya identik:

if (cond1 == "val1" and cond22 == "val2"
and cond333 == "val3" and cond4444 == "val4") {
    do_something
}

Polos dan sederhana, juga lolos pemeriksaan pep8:

if (
    cond1 and
    cond2
):
    print("Hello World!")

Baru-baru ini saya lebih suka fungsi alldan anyfungsinya, karena saya jarang mencampurkan perbandingan Dan dan Atau ini berfungsi dengan baik, dan memiliki keuntungan tambahan Gagal Dini dengan pemahaman generator:

if all([
    cond1,
    cond2,
]):
    print("Hello World!")

Hanya ingat untuk lulus dalam satu iterable! Melewati argumen-N tidak benar.

Catatan: anyseperti banyak orperbandingan, allseperti banyak andperbandingan.

Ini menggabungkan dengan baik dengan pemahaman generator, misalnya:

# Check if every string in a list contains a substring:
my_list = [
    'a substring is like a string', 
    'another substring'
]

if all('substring' in item for item in my_list):
   print("Hello World!")

# or

if all(
    'substring' in item
    for item in my_list
):
    print("Hello World!")

Lebih lanjut tentang: pemahaman generator

Bagaimana jika kita hanya memasukkan garis kosong tambahan antara kondisi dan tubuh dan melakukan sisanya dengan cara kanonik?

if (cond1 == 'val1' and cond2 == 'val2' and
    cond3 == 'val3' and cond4 == 'val4'):

    do_something

ps Saya selalu menggunakan tab, bukan spasi; Saya tidak bisa memperbaiki ...

Apa yang biasanya saya lakukan adalah:

if (cond1 == 'val1' and cond2 == 'val2' and
    cond3 == 'val3' and cond4 == 'val4'
   ):
    do_something

dengan cara ini kurung kurawal dan usus besar secara visual menandai akhir dari kondisi kita.

Semua responden yang juga menyediakan multi-kondisional untuk pernyataan if sama jeleknya dengan masalah yang disajikan. Anda tidak menyelesaikan masalah ini dengan melakukan hal yang sama ..

Bahkan jawaban PEP 0008 menjijikkan.

Inilah pendekatan yang jauh lebih mudah dibaca

condition = random.randint(0, 100) # to demonstrate
anti_conditions = [42, 67, 12]
if condition not in anti_conditions:
    pass

Ingin aku memakan kata-kataku? Yakinkan saya bahwa Anda membutuhkan multi-kondisi dan saya akan mencetak ini dan memakannya untuk hiburan Anda.

Saya pikir solusi @kanda akan baik dengan twist kecil. Jika Anda memiliki kondisi dan nilai di daftar masing-masing, Anda dapat menggunakan pemahaman daftar untuk melakukan perbandingan, yang akan membuat hal-hal sedikit lebih umum untuk menambahkan pasangan kondisi / nilai.

conditions = [1, 2, 3, 4]
values = [1, 2, 3, 4]
if all([c==v for c, v in zip(conditions, values)]):
    # do something

Jika saya benar-benar ingin membuat kode pernyataan seperti ini, saya akan menulisnya seperti ini untuk keterbacaan:

if (condition1==value1) and (condition2==value2) and \
   (condition3==value3) and (condition4==value4):

Dan hanya untuk melempar solusi lain di luar sana dengan seorang iandoperator :

proceed = True
for c, v in zip(conditions, values):
    proceed &= c==v

if proceed:
    # do something

Hanya beberapa ide acak lainnya demi kelengkapan. Jika mereka bekerja untuk Anda, gunakan itu. Kalau tidak, Anda mungkin lebih baik mencoba sesuatu yang lain.

Anda juga bisa melakukan ini dengan kamus:

>>> x = {'cond1' : 'val1', 'cond2' : 'val2'}
>>> y = {'cond1' : 'val1', 'cond2' : 'val2'}
>>> x == y
True

Opsi ini lebih rumit, tetapi Anda mungkin juga menganggapnya berguna:

class Klass(object):
    def __init__(self, some_vars):
        #initialize conditions here
    def __nonzero__(self):
        return (self.cond1 == 'val1' and self.cond2 == 'val2' and
                self.cond3 == 'val3' and self.cond4 == 'val4')

foo = Klass()
if foo:
    print "foo is true!"
else:
    print "foo is false!"

Entah apakah itu bekerja untuk Anda, tetapi itu pilihan lain untuk dipertimbangkan. Ini satu cara lagi:

class Klass(object):
    def __init__(self):
        #initialize conditions here
    def __eq__(self):
        return (self.cond1 == 'val1' and self.cond2 == 'val2' and
               self.cond3 == 'val3' and self.cond4 == 'val4')

x = Klass(some_values)
y = Klass(some_other_values)
if x == y:
    print 'x == y'
else:
    print 'x!=y'

Dua yang terakhir saya belum diuji, tetapi konsep harus cukup untuk membuat Anda pergi jika itu yang Anda inginkan.

(Dan sebagai catatan, jika ini hanya satu kali, Anda mungkin lebih baik menggunakan metode yang Anda sajikan pada awalnya. Jika Anda melakukan perbandingan di banyak tempat, metode ini dapat meningkatkan keterbacaan yang cukup untuk membuat Anda tidak merasa begitu buruk tentang fakta bahwa mereka agak berantakan.)

Saya telah berjuang untuk menemukan cara yang baik untuk melakukan ini juga, jadi saya baru saja datang dengan ide (bukan peluru perak, karena ini terutama masalah selera).

if bool(condition1 and
        condition2 and
        ...
        conditionN):
    foo()
    bar()

Saya menemukan beberapa kelebihan dalam solusi ini dibandingkan dengan yang lain yang pernah saya lihat, yaitu, Anda mendapatkan persis 4 ruang lekukan tambahan (bool), yang memungkinkan semua kondisi untuk berbaris secara vertikal, dan badan pernyataan if dapat dimasukkan ke dalam cara yang jelas (ish). Ini juga menyimpan manfaat evaluasi hubung singkat operator boolean, tetapi tentu saja menambahkan overhead panggilan fungsi yang pada dasarnya tidak melakukan apa-apa. Anda dapat berdebat (secara sah) bahwa fungsi apa pun yang mengembalikan argumennya dapat digunakan di sini alih-alih bool, tetapi seperti yang saya katakan, itu hanya sebuah gagasan dan pada akhirnya masalah selera.

Cukup lucu, ketika saya menulis ini dan berpikir tentang "masalah", saya datang dengan ide lain , yang menghilangkan overhead panggilan fungsi. Mengapa tidak menunjukkan bahwa kita akan memasuki kondisi kompleks dengan menggunakan pasangan kurung tambahan? Katakan, 2 lagi, untuk memberikan 2 spasi indentasi bagus dari sub-kondisi relatif terhadap tubuh pernyataan if. Contoh:

if (((foo and
      bar and
      frob and
      ninja_bear))):
    do_stuff()

Saya agak suka ini karena ketika Anda melihatnya, sebuah lonceng dengan segera berbunyi di kepala Anda mengatakan "hei, ada hal rumit yang terjadi di sini!" . Ya, saya tahu bahwa tanda kurung tidak membantu keterbacaan, tetapi kondisi ini seharusnya jarang muncul, dan ketika itu muncul, Anda harus berhenti dan membacanya dengan cermat (karena kompleks ).

Pokoknya, hanya dua proposal lagi yang belum saya lihat di sini. Semoga ini bisa membantu seseorang :)

Anda dapat membaginya menjadi dua baris

total = cond1 == 'val' and cond2 == 'val2' and cond3 == 'val3' and cond4 == val4
if total:
    do_something()

Atau bahkan menambahkan satu syarat pada satu waktu. Dengan begitu, setidaknya itu memisahkan kekacauan dari if.

Saya tahu utas ini sudah lama, tetapi saya memiliki beberapa kode Python 2.7 dan PyCharm (4.5) masih mengeluh tentang hal ini:

if foo is not None:
    if (cond1 == 'val1' and cond2 == 'val2' and
        cond3 == 'val3' and cond4 == 'val4'):
            # some comment about do_something
            do_something

Bahkan dengan peringatan PEP8 "garis indentasi visual dengan indentasi yang sama dengan baris logis berikutnya", kode sebenarnya sepenuhnya OK? Itu bukan "indentasi berlebihan?"

... ada saatnya saya berharap Python akan menggigit peluru dan hanya pergi dengan kurung kurawal. Saya bertanya-tanya berapa banyak bug yang secara tidak sengaja diperkenalkan selama bertahun-tahun karena kesalahan indentasi ...

Kemas kondisi Anda ke dalam daftar, lalu lakukan sesuatu Suka:

if False not in Conditions:
    do_something

Saya menemukan bahwa ketika saya memiliki kondisi yang panjang, saya sering memiliki kode pendek. Dalam hal ini, saya hanya menggandakan indentasi tubuh, dengan demikian:

if (cond1 == 'val1' and cond2 == 'val2' and
    cond3 == 'val3' and cond4 == 'val4'):
        do_something

  if cond1 == 'val1' and \
     cond2 == 'val2' and \
     cond3 == 'val3' and \
     cond4 == 'val4':
      do_something

atau jika ini lebih jelas:

  if cond1 == 'val1'\
     and cond2 == 'val2'\
     and cond3 == 'val3'\
     and cond4 == 'val4':
      do_something

Tidak ada alasan indentasi harus berupa kelipatan 4 dalam kasus ini, mis. Lihat "Disejajarkan dengan pembatas pembuka":

http://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Indentation#Indentation

Inilah pendekatan lain:

cond_list = ['cond1 == "val1"','cond2=="val2"','cond3=="val3"','cond4=="val4"']
if all([eval(i) for i in cond_list]):
 do something

Ini juga memudahkan untuk menambahkan kondisi lain dengan mudah tanpa mengubah pernyataan if dengan hanya menambahkan kondisi lain ke daftar:

cond_list.append('cond5=="val5"')

Saya biasanya menggunakan:

if ((cond1 == 'val1' and cond2 == 'val2' and
     cond3 == 'val3' and cond4 == 'val4')):
    do_something()

jika kondisi if & an else kita harus menjalankan beberapa pernyataan di dalamnya, maka kita dapat menulis seperti di bawah ini. Setiap saat kita memiliki contoh lain dengan satu pernyataan di dalamnya.

Terima kasih, ini berhasil untuk saya.

#!/usr/bin/python
import sys
numberOfArgument =len(sys.argv)
weblogic_username =''
weblogic_password = ''
weblogic_admin_server_host =''
weblogic_admin_server_port =''


if numberOfArgument == 5:
        weblogic_username = sys.argv[1]
        weblogic_password = sys.argv[2]
        weblogic_admin_server_host =sys.argv[3]
        weblogic_admin_server_port=sys.argv[4]
elif numberOfArgument <5:
        print " weblogic UserName, weblogic Password and weblogic host details are Mandatory like, defalutUser, passwordForDefaultUser, t3s://server.domainname:7001 ."
        weblogic_username = raw_input("Enter Weblogic user Name")
        weblogic_password = raw_input('Enter Weblogic user Password')
        weblogic_admin_server_host = raw_input('Enter Weblogic admin host ')
        weblogic_admin_server_port = raw_input('Enter Weblogic admin port')
#enfelif
#endIf

Maaf noobness saya, tetapi kebetulan saya tidak memiliki pengetahuan tentang #Python seperti kalian di sini, tetapi kebetulan saya menemukan sesuatu yang mirip ketika membuat skrip objek saya sendiri dalam pemodelan BIM 3D, jadi saya akan menyesuaikan algoritma saya dengan itu dari python.

Masalah yang saya temukan di sini, adalah dua sisi:

1. Nilai saya terasa asing bagi seseorang yang mungkin mencoba menguraikan skrip.
2. Pemeliharaan kode akan dikenakan biaya tinggi, jika nilai-nilai itu diubah (kemungkinan besar), atau jika kondisi baru harus ditambahkan (skema rusak)

Lakukan untuk melewati semua masalah ini, skrip Anda harus seperti ini

param_Val01 = Value 01   #give a meaningful name for param_Val(i) preferable an integer
param_Val02 = Value 02
param_Val03 = Value 03
param_Val04 = Value 04   # and ... etc

conditions = 0           # this is a value placeholder

########
Add script that if true will make:

conditions = conditions + param_Val01   #value of placeholder is updated
########

### repeat as needed


if conditions = param_Val01 + param_Val02 + param_Val03 + param_Val04:
    do something

Kelebihan metode ini:

1. Skrip dapat dibaca.

2. Script bisa dipelihara dengan mudah.

3. kondisi adalah operasi perbandingan 1 dengan jumlah nilai yang mewakili kondisi yang diinginkan.
4. Tidak perlu untuk kondisi bertingkat

Semoga ini bisa membantu kalian semua