これは条件式の有効な使用ですか？

Question

は、私がこれを行うための最善の方法が何であるかを把握しようとしている：

resource['contents'][media_type] = [] 
resource['contents'][media_type].append(row[0].toPython()) if row[0] is not None else None 
resource['contents'][media_type].append(row[2].toPython()) if row[2] is not None else None 

私はコードは非常に簡単だと思います。行に値がある場合は、それらをリストに追加します。このアプローチはOKと考えられていますか？他にも良い方法がありますか？​​メソッドは、含まれているオブジェクトの文字列の説明を返します。

Answer 1

副作用のために「3値」conditional expression（x if C else y）を使用することはまったくPythonicではありません。ここで私はそれを行うだろう方法は次のとおりです。

resource['contents'][media_type] = [] 
for index in (0, 2): 
    item = row[i] 
    if item is not None: 
     resource['contents'][media_type].append(item.toPython()) 

または冗長性を減らすために、リストの内包表記を使用して：

resource['contents'][media_type] = [row[i].toPython() for i in (0, 2) 
            if row[i] is not None] 

これらのアプローチは、はるかに読みやすいですし、重複を減らします。

Answer 2

いいえ、条件式を有効に使用することはできません。あなたのコードを読もうとする人は混乱します。

ifステートメントを使用してください。

lst = resource['contents'][media_type] = [] 
if row[0] is not None: lst.append(row[0].toPython()) 
if row[2] is not None: lst.append(row[2].toPython()) 

が、地元の参考のために、より良い名前を使う（contents多分？）、またはリストの内包表記を使用します：あなたは、リストへの別の参照を作成することによって、いくつかのスペースを節約することができ

resource['contents'][media_type] = [ 
    col.toPython() for col in (row[0], row[2]) if col is not None] 

Answer 3

Iドンそれを行うのが良い習慣と考えられるとは思わない。あなたが代わりに行うことができますすることです：

resource['contents'][media_type] = [] 

for irow in [0, 2]: 
    if row[irow] is not None: 
     resource['contents'][media_type].append(row[irow].toPython()) 

これは、あなたも（range(5)でirowのための）範囲を使用して、またはあなたが直接（for row in rows:）にアクセスできるかどうかの行を使用しての柔軟性を可能にします。