2010-02-15 64 views
2

我已经临到了几个类似这样的代码行的下面一行,但我不能确定我应该怎么打破它:如何打破蟒蛇

blueprint = Blueprint(self.blueprint_map[str(self.ui.blueprint_combo.currentText())], runs=self.ui.runs_spin.text(), me=self.ui.me_spin.text(), pe=self.ui.pe_skill_combo.currentIndex()) 

在此先感谢

+0

我觉得留出一个括号(或添加额外的)是一个很好的方式。 – outis 2010-02-15 15:12:57

+0

谢谢大家的好答案,第一眼看上去“最漂亮”,但感谢迅速的反应! – FrederikNS 2010-02-15 15:15:56

回答

14
blueprint = Blueprint(
    self.blueprint_map[str(self.ui.blueprint_combo.currentText())], 
    runs=self.ui.runs_spin.text(), 
    me=self.ui.me_spin.text(), 
    pe=self.ui.pe_skill_combo.currentIndex(), 
) 
0

Anywhere的括号内应该工作,如:

blueprint = Blueprint(self.blueprint_map[str(self.ui.blueprint_combo.currentText())], 
     runs=self.ui.runs_spin.text(), me=self.ui.me_spin.text(), 
     pe=self.ui.pe_skill_combo.currentIndex()) 
+1

-1:这违反了PEP 8的惯例。挂起的缩进很好,但“第一行应该没有任何争论”。最好在左括号处打破,并将连续行上的零部件内容与一致的8(或4)列缩进对齐。 – bignose 2013-06-24 06:14:10

0
blueprint = Blueprint(self.blueprint_map[str(self.ui.blueprint_combo.currentText())], 
         runs=self.ui.runs_spin.text(), me=self.ui.me_spin.text(), 
         pe=self.ui.pe_skill_combo.currentIndex()) 
+1

-1:如果更改第一行的长度,则延续线会错误对齐。最好做一个标准的缩进级别。 – bignose 2011-04-15 04:15:42

4

我会做这种方式:

blueprint = Blueprint(
       self.blueprint_map[str(self.ui.blueprint_combo.currentText())], 
       runs=self.ui.runs_spin.text(), 
       me=self.ui.me_spin.text(), 
       pe=self.ui.pe_skill_combo.currentIndex()) 
+0

+1,但请使用标准的8(或4)列缩进。 – bignose 2013-06-24 06:16:42

+0

-1为大和任意大小的缩进。如果第一行的布局发生变化(例如,通过更改“蓝图”的名称),则需要更改以下所有行的缩进。最好使用正常的单个4字符缩进。 – 2014-07-03 14:17:42

5

这个怎么样

blueprint_item = self.blueprint_map[str(self.ui.blueprint_combo.currentText())] 
blueprint = Blueprint(blueprint_item, 
         runs=self.ui.runs_spin.text(), 
         me=self.ui.me_spin.text(), 
         pe=self.ui.pe_skill_combo.currentIndex()) 
+0

中间变量名称不会添加任何额外的信息,所以在这种情况下,我会赞成删除它。如果你没有缩小Blueprint()的参数,这很容易做到。使这些参数与这样的大括号保持一致是很常见的,但是却被误导了,因为这意味着如果第一行的布局改变了(例如通过改变'blueprint'的名字),那么你需要改变下面所有行的缩进。最好使用正常的单个4字符缩进。 – 2014-07-03 14:17:15