Comment casser une ligne de méthodes chaînées en Python?

J'ai une ligne du code suivant (ne blâmez pas les conventions de nommage, elles ne sont pas les miennes):

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

Je n'aime pas à quoi ça ressemble (pas trop lisible) mais je n'ai pas de meilleure idée de limiter les lignes à 79 caractères dans cette situation. Existe-t-il un meilleur moyen de le casser (de préférence sans contre-obliques)?

Vous pouvez utiliser des parenthèses supplémentaires:

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

C'est un cas où un caractère de continuation de ligne est préférable aux parenthèses ouvertes. La nécessité de ce style devient plus évidente à mesure que les noms de méthodes s'allongent et que les méthodes commencent à prendre des arguments:

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

La PEP 8 est destinée à être interprétée avec une mesure de bon sens et un œil à la fois pratique et beau. Violez heureusement toute directive PEP 8 qui entraîne un code laid ou difficile à lire.

Cela étant dit, si vous vous trouvez fréquemment en désaccord avec PEP 8, cela peut être un signe qu'il y a des problèmes de lisibilité qui transcendent votre choix d'espaces :-)

Mon choix personnel serait:

subkeyword = Session.query (
    Subkeyword.subkeyword_id,
    Subkeyword.subkeyword_word,
).filtrer par(
    subkeyword_company_id = self.e_company_id,
    subkeyword_word = sous-mot_clé,
    subkeyword_active = Vrai,
).une()

Stockez simplement le résultat / objet intermédiaire et invoquez la méthode suivante dessus, par exemple

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

Selon la référence du langage Python,
vous pouvez utiliser une barre oblique inverse.
Ou tout simplement le casser. Si un crochet n'est pas apparié, python ne le traitera pas comme une ligne. Et dans de telles circonstances, l'indentation des lignes suivantes n'a pas d'importance.

C'est une solution un peu différente de celle fournie par d'autres, mais une de mes préférées car elle conduit parfois à une métaprogrammation astucieuse.

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

C'est une technique intéressante pour créer des recherches. Parcourez une liste de conditions à extraire à partir de votre formulaire de requête complexe (ou des déductions basées sur une chaîne sur ce que l'utilisateur recherche), puis éclatez simplement le dictionnaire dans le filtre.

Vous semblez utiliser SQLAlchemy, si c'est vrai, la sqlalchemy.orm.query.Query.filter_by()méthode prend plusieurs arguments de mot-clé, vous pouvez donc écrire comme:

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

Mais ce serait mieux:

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

J'aime mettre en retrait les arguments de deux blocs et l'instruction d'un bloc, comme ceci:

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)