Comment rompre une ligne de enchaînés méthodes en Python?

J'ai une ligne de code suivant (ne les blâme pas pour les conventions de nommage, ils ne sont pas la mienne):

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 il ressemble (pas trop lisible) mais je n'ai pas de meilleure idée de limiter le nombre de lignes à 79 caractères dans cette situation. Est-il une meilleure façon de le casser (de préférence sans barre oblique inverse)?

InformationsquelleAutor Juliusz Gonera | 2011-01-22
  1. 224

    Vous pouvez utiliser parenthèse supplémentaire:

    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()
    )
    • J'aime aussi les meilleurs. Ne pas ajouter plus de code et c'est sans barres obliques inverses.
    • Pas sûr de ce que justifie le supplément de l'indentation ici; je pense que cette solution se lit tout aussi bien avec la pendaison de lignes en retrait, juste une fois, la fuite parenthèse pas du tout.
    • À mon avis le double de l'indentation est utile ici, car c'est visuellement distincts à partir d'un bloc indenté. Quand ils sont entourés par d'autres codes de ce fait, il est plus évident que c'est un enveloppé seule ligne.
    • Meilleure réponse, en termes d'utilisation des parenthèses. Comme mentionné dans un commentaire de Shanimal dans une autre réponse, à l'aide de la continuation de ligne implicite via parenthèses est en fait PEP 8 préféré vs le caractère de continuation `
    • Je préfère les barres obliques inverses. La parenthèse n'est pas un indice pour l'ensemble de la situation. Par exemple, il ne fonctionne pas avec l'opérateur d'affectation. Imaginez que vous voulez des lignes de rupture de cette chaîne: foo.set_default('bar', {}).set_default('spam', {}).set_default('eggs', {})['lol'] = 'yeah'
    InformationsquelleAutor sth
  2. 45

    C'est un cas où un caractère de continuation de ligne est préféré ouvrir des parenthèses. La nécessité de ce style devient de plus en plus évident que les noms de méthode plus longtemps et que les méthodes de commencer à 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()

    PEP 8 est conçu pour être interprété avec une mesure de bon sens et un oeil pour la pratique et le beau. Heureusement violer toute la PEP 8 ligne directrice que les résultats de laid ou dur à lire le code.

    Cela étant dit, si vous avez souvent de vous trouver en désaccord avec PEP 8, il peut être un signe qu'il y a des raisons de lisibilité des enjeux qui transcendent votre choix d'espaces 🙂

    • +1 sur les barres obliques inverses et en alignant les filtres chaînés dans ce cas particulier. Cette situation se présente dans Django en tant que bien et il est plus lisible de cette façon-mais dans aucun situation je me sens comme entre parenthèses phrases sont supérieures (ne souffrent pas de la "il est des espaces après ma barre oblique inverse?" problème). Cela dit, parenthesizing l'expression peut être utilisée pour obtenir le même effet-mais il vous met en Lisp mode de lecture dans le milieu de la lecture Python, que je trouve choquante.
    • Je ne vois pas en quoi cette solution est mieux en mesure de faire face ", comme les noms de méthode plus longtemps et que les méthodes de commencer à prendre des arguments" que ce soit le "wrap à l'extérieur de la parens" ou "saut de ligne après chaque parenthèse ouverte et avant chaque fermer la parenthèse" des solutions. En fait c'est pire à la manipulation qui, depuis (au moins comme illustré ici), il nécessite beaucoup plus tiret pour chaque accrochage de la ligne.
    • Beaucoup trop de tiret pour le filtre les appels. Un onglet ou 4 places aurait été assez ici. Aussi l'alignement de l' ` ... Combien de secondes avez-vous maintenez cette touche espace? En général, je suis contre toutes les façons, qui vous obligent à marteau qui touche espace comme il n'y a pas de demain.
    • fwiw, PEP8 lit "Le moyen privilégié de l'habillage de longues lignes est par l'utilisation de Python est la continuation de ligne implicite à l'intérieur des parenthèses, crochets et accolades. De longues lignes peut être réparti sur plusieurs lignes en enveloppant les expressions entre parenthèses. Ceux-ci devraient être utilisés de préférence à l'aide d'une barre oblique inverse pour la continuation de ligne." —Python.org Il va à discuter lors de barres obliques inverses peut être approprié
    • Grande référence pour PEP8! Un ennuyeux problème ici avec le regroupement de toutes les .filter appels est que si vous changez subkeyword à sub_keyword, vous avez maintenant de fixer l'indentation de chaque ligne tout simplement parce que vous avez changé le nom de la variable. Pas bon quand le style fait entrave la maintenabilité...
    • L'alignement dans ce formulaire échoue également si le membre de GAUCHE de l'expression est trop long, comme my_obj["long_keyword_name"].attrs["another_long_name"] = my_obj["long_keyword_name"].attrs["another_long_name"].replace("aaa", "bbb").replace("ccc", "ddd").replace("eee", "fff").

    InformationsquelleAutor Raymond Hettinger
  3. 12

    Mon choix personnel serait:

    sous-mot-clé = Session.query( 
Sous-mot-clé.subkeyword_id, 
Sous-mot-clé.subkeyword_word, 
).filter_by( 
subkeyword_company_id=auto.e_company_id, 
subkeyword_word=subkeyword_word, 
subkeyword_active=True, 
).un()
    InformationsquelleAutor pkoch
  4. 9

    Simplement pour stocker les résultats intermédiaires/objet et d'invoquer la méthode,
    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()
    • Cela fonctionne bien pour quelque chose comme une requête, mais comme une tendance générale, je ne suis pas si sûr. Par exemple, lorsque le chaînage dans la Belle Soupe comme team_members = soup.find(class_='section team').find_all('ul').find_all('li'), la valeur de retour de chaque .find(...) appel ne rentre pas dans le sens de team_members encore.
    InformationsquelleAutor Ivo van der Wijk
  5. 4

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

    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 bonne technique pour la construction de recherches. Passer par une liste de conditions à la mine de la complexité de votre formulaire de requête (ou une chaîne basée sur des déductions au sujet de ce que l'utilisateur est à la recherche d'), puis il suffit de faire exploser le dictionnaire dans le filtre.

    InformationsquelleAutor Árni St. Sigurðsson
  6. 3

    Selon Référence Au Langage Python

    Vous pouvez utiliser une barre oblique inverse.

    Ou tout simplement de le casser. Si un support n'est pas connecté, python ne sera pas traiter que comme une ligne. Et dans de telles circonstances, l'indentation des lignes suivantes n'a pas d'importance.

    InformationsquelleAutor Haozhun
  7. 1

    Vous semble à l'aide de SQLAlchemy, si c'est vrai, sqlalchemy.orm.de la requête.De la requête.filter_by() méthode prend plusieurs arguments mot-clé, de sorte que vous pouvez l'é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 il 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()
    • +1 pour SQLAlchemy filter_by() de l'indice. C'est bon pour cet exemple, mais j'ai souvent l'utilisation de filtre() à la place qui accepte seulement 1 condition.
    InformationsquelleAutor minhee
  8. 1

    J'aime tiret, les arguments avancés par les deux blocs, et la déclaration d'un bloc, comme celles-ci:

    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)
    InformationsquelleAutor acgtyrant