Question

我有一行以下代码（不要责怪命名约定，它们不是我的）：

subkeyword = Session.query(
    Subkeyword.subkeyword_id, Subkeyword.subkeyword_word
).filter_by(
    subkeyword_company_id=self.e_company_id
).filter_by(
    subkeyword_word=subkeyword_word
).filter_by(
    subkeyword_active=True
).one()

我不喜欢它的样子（不太可读）但我没有更好的想法在这种情况下将行限制为79个字符。是否有更好的方法来打破它（最好没有反斜杠）？

Answer 1

您可以使用其他括号：

subkeyword = (
        Session.query(Subkeyword.subkeyword_id, Subkeyword.subkeyword_word)
        .filter_by(subkeyword_company_id=self.e_company_id)
        .filter_by(subkeyword_word=subkeyword_word)
        .filter_by(subkeyword_active=True)
        .one()
    )

Answer 2

这种情况下，行连续字符优先打开括号。当方法名称变得更长并且方法开始采用参数时，对这种风格的需求变得更加明显：

subkeyword = Session.query(Subkeyword.subkeyword_id, Subkeyword.subkeyword_word) \
                    .filter_by(subkeyword_company_id=self.e_company_id)          \
                    .filter_by(subkeyword_word=subkeyword_word)                  \
                    .filter_by(subkeyword_active=True)                           \
                    .one()

PEP 8旨在通过常识和对实际和美丽的关注来解释。高兴地违反任何导致丑陋或难以阅读的代码的PEP 8指南。

话虽如此，如果您经常发现自己与PEP 8不一致，这可能表明存在超出您选择的空白的可读性问题： - ）

Answer 3

我个人的选择是：

subkeyword = Session.query(
    Subkeyword.subkeyword_id,
    Subkeyword.subkeyword_word,
).filter_by(
    subkeyword_company_id=self.e_company_id,
    subkeyword_word=subkeyword_word,
    subkeyword_active=True,
).one()

Answer 4

只需存储中间结果/对象并在其上调用下一个方法， e.g。

q = Session.query(Subkeyword.subkeyword_id, Subkeyword.subkeyword_word)
q = q.filter_by(subkeyword_company_id=self.e_company_id)
q = q.filter_by(subkeyword_word=subkeyword_word)
q = q.filter_by(subkeyword_active=True)
subkeyword = q.one()

Answer 5

根据Python Language Reference
你可以使用反斜杠或者简单地打破它。如果括号未配对，python不会将其视为一行。在这种情况下，跟随行的缩进并不重要。

Answer 6

这是一个与其他人提供的不同的解决方案，但却是我的最爱，因为它有时会导致漂亮的元编程。

base = [Subkeyword.subkeyword_id, Subkeyword_word]
search = {
    'subkeyword_company_id':self.e_company_id,
    'subkeyword_word':subkeyword_word,
    'subkeyword_active':True,
    }
subkeyword = Session.query(*base).filter_by(**search).one()

这是一种很好的构建搜索的技巧。从复杂的查询表单（或基于字符串的关于用户要查找的内容的推断）中查看要查找的条件列表，然后将字典分解为过滤器。

Answer 7

你好像使用SQLAlchemy，如果是，sqlalchemy.orm.query.Query.filter_by()方法需要多个关键字参数，所以你可以这样写：

subkeyword = Session.query(Subkeyword.subkeyword_id,
                           Subkeyword.subkeyword_word) \
                    .filter_by(subkeyword_company_id=self.e_company_id,
                               subkeyword_word=subkeyword_word,
                               subkeyword_active=True) \
                    .one()

但它会更好：

subkeyword = Session.query(Subkeyword.subkeyword_id,
                           Subkeyword.subkeyword_word)
subkeyword = subkeyword.filter_by(subkeyword_company_id=self.e_company_id,
                                  subkeyword_word=subkeyword_word,
                                  subkeyword_active=True)
subkeuword = subkeyword.one()

Answer 8

我喜欢将参数缩进两个块，并将语句缩进一个块，如下所示：

true

如何在Python中打破一系列链式方法？

8 个答案: