如何在Python中打破一行链式方法?

问题描述 投票:0回答:9

我有一行以下代码(不要责怪命名约定,它们不是我的):

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 个字符。有没有更好的方法来打破它(最好没有反斜杠)?

python coding-style pep8
9个回答
323
投票

您可以使用额外的括号:

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()
    )

72
投票

在这种情况下,行继续字符优先于左括号。随着方法名称变得更长并且方法开始接受参数,对这种样式的需求变得更加明显:

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 不一致,这可能表明存在超出您选择的空白的可读性问题:-)


21
投票

我个人的选择是:

子关键字 = Session.query(
    子关键字.subkeyword_id,
    子关键字.subkeyword_word,
)。过滤(
    subkeyword_company_id=self.e_company_id,
    子关键字_字=子关键字_字,
    subkeyword_active=真,
)。一()

12
投票

只需存储中间结果/对象并调用其上的下一个方法, 例如

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()

9
投票

这与其他人提供的解决方案有点不同,但这是我最喜欢的,因为它有时会带来漂亮的元编程。

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()

这是构建搜索的好技术。通过条件列表从复杂的查询表单中挖掘(或基于字符串的关于用户正在查找的内容的推论),然后将字典分解到过滤器中。


4
投票

根据Python语言参考
您可以使用反斜杠。
或者干脆打破它。如果括号未配对,Python 不会将其视为一行。在这种情况下,下面几行的缩进并不重要。


2
投票

你似乎使用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()

1
投票

我喜欢将参数缩进两块,将语句缩进一块,如下所示:

for image_pathname in image_directory.iterdir():
    image = cv2.imread(str(image_pathname))
    input_image = np.resize(
            image, (height, width, 3)
        ).transpose((2,0,1)).reshape(1, 3, height, width)
    net.forward_all(data=input_image)
    segmentation_index = net.blobs[
            'argmax'
        ].data.squeeze().transpose(1,2,0).astype(np.uint8)
    segmentation = np.empty(segmentation_index.shape, dtype=np.uint8)
    cv2.LUT(segmentation_index, label_colours, segmentation)
    prediction_pathname = prediction_directory / image_pathname.name
    cv2.imwrite(str(prediction_pathname), segmentation)

1
投票

顶部答案略有不同:主要对象(

Session
)保留在顶行,并使用单个缩进。这可以快速识别主对象和所有后续的链式方法调用..

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()
)
© www.soinside.com 2019 - 2024. All rights reserved.