Wie kann man eine Reihe verketteter Methoden in Python durchbrechen?

Ich habe eine Zeile mit dem folgenden Code (keine Schuld an Namenskonventionen, sie gehören nicht mir):

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

Ich mag es nicht, wie es aussieht (nicht zu lesbar), aber ich habe keine bessere Idee, Zeilen in dieser Situation auf 79 Zeichen zu beschränken. Gibt es eine bessere Möglichkeit, es zu brechen (vorzugsweise ohne Backslashes)?

Sie könnten zusätzliche Klammern verwenden:

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

Dies ist ein Fall, in dem ein Zeilenfortsetzungszeichen dem Öffnen von Klammern vorgezogen wird. Die Notwendigkeit dieses Stils wird offensichtlicher, wenn Methodennamen länger werden und Methoden Argumente annehmen:

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 soll mit einem Maß an gesundem Menschenverstand und einem Auge für das Praktische und das Schöne interpretiert werden. Verstößt glücklich gegen eine PEP 8-Richtlinie, die zu hässlichem oder schwer lesbarem Code führt.

Wenn Sie jedoch häufig mit PEP 8 nicht einverstanden sind, kann dies ein Zeichen dafür sein, dass es Lesbarkeitsprobleme gibt, die über Ihre Wahl des Leerzeichens hinausgehen :-)

Meine persönliche Wahl wäre:

subkeyword = Session.query (
    Subkeyword.subkeyword_id,
    Subkeyword.subkeyword_word,
).Filtern nach(
    subkeyword_company_id = self.e_company_id,
    subkeyword_word = subkeyword_word,
    subkeyword_active = True,
).einer()

Speichern Sie einfach das Zwischenergebnis / Objekt und rufen Sie die nächste Methode auf, z

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

Laut Python-Sprachreferenz
können Sie einen Backslash verwenden.
Oder einfach brechen. Wenn eine Klammer nicht gepaart ist, behandelt Python dies nicht als Zeile. Und unter solchen Umständen spielt das Einrücken der folgenden Zeilen keine Rolle.

Es ist eine etwas andere Lösung als die von anderen, aber ein Favorit von mir, da es manchmal zu raffinierten Metaprogrammierungen führt.

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

Dies ist eine gute Technik zum Erstellen von Suchvorgängen. Gehen Sie eine Liste der Bedingungen durch, die Sie aus Ihrem komplexen Abfrageformular abrufen möchten (oder auf Zeichenfolgen basierende Abzüge darüber, wonach der Benutzer sucht), und lösen Sie dann einfach das Wörterbuch in den Filter auf.

Sie scheinen SQLAlchemy zu verwenden. Wenn dies zutrifft, verwendet die sqlalchemy.orm.query.Query.filter_by()Methode mehrere Schlüsselwortargumente, sodass Sie wie folgt schreiben können:

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

Aber es wäre besser:

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

Ich möchte die Argumente um zwei Blöcke und die Anweisung um einen Block wie folgt einrücken:

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)