2016-08-15 3 views
0

私は、この行が100個の文字の下に適合させる、そして可能な限り(80文字の制限を除く)PEP8準拠したとして、それを行う必要があります。次の行を100文字以下になるようにどのようにフォーマットしますか?

date = dateparser.parse(parsed_response["creation_time"]) + datetime.timedelta(minutes=parsed_response["time"]) 

あなたがそれを行うだろうか?括弧を付けて2行に分けると、私の意見では見た目が悪くなるようです。

+1

2つの引数を2つの一時変数割り当てに分割します。 – Mephy

+0

変数への抽出? – ospahiu

+0

PEP8に加えて読む価値のある「Google Python Style Guide」もあります。 – VPfB

答えて

2

まあ、明白なアプローチは、dateparser.parse(parsed_response["creation_time"])datetime.timedelta(minutes=parsed_response["time"])の変数を導入することです。これは、特に、変数に "日付"よりもわかりやすい名前を付ける機会を得た場合、コードが何をしているのかをより明確にするという面白い副作用があります。

1

次のようにあなたは自分のラインを破ることができます:

date = dateparser.parse(parsed_response["creation_time"]) + \ 
     datetime.timedelta(minutes=parsed_response["time"]) 

しかし、前の変数を準備し、ちょうどそれらを追加すると読みやすくなります。それに

creation_time = dateparser.parse(parsed_response["creation_time"]) 
parsed_response_time = datetime.timedelta(minutes=parsed_response["time"]) 

date = creation_time + parsed_response_time 
+0

下の行を右揃えにすると、よりよく見えるようになります –

+0

確かに、pep8もそれでいいです(編集済み)。 – fpietka

0

私のソリューションは、このようなものになるだろう:あなたはそれがPEP8に準拠しました

creation_time = dateparser.parse(parsed_response["creation_time"]) 
time_delta = datetime.timedelta(minutes=parsed_response["time"]) 
date = creation_time + time_delta 

そのように、あなたはフェッチして再度解析することなく、あなたには、いくつかの他の目的に再利用することができ、これらの2つの変数を得ました。

関連する問題