Development #32880
sur l'inspecteur, afficher les chaines de caractères entourées de guillemets
0%
Description
Actuellement on a des lignes de ce genre :
category_description None (type None) form_role_receiver_details form_role_receiver_emails form_role_receiver_name Agent form_slug x1-test-action-resoumission-en-mode-guichet-formulaire-2 form_status Nouveau form_status_changed True (type bool)
Il apparait qu'il serait plus facile, pour la compréhension de la rédaction des conditions notamment, d'afficher les chaînes de caractères avec des guillemets :
category_description None (type None) form_role_receiver_details "" form_role_receiver_emails "" form_role_receiver_name "Agent" form_slug "x1-test-action-resoumission-en-mode-guichet-formulaire-2" form_status "Nouveau" form_status_changed True (type bool)
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
Pas fan, ça amène derrière à se poser la question de comment alors représenter les guillemets à l'intérieur d'une chaine, et on se met à parler d'échappement de chaine, etc.
On a pris le principe d'afficher uniquement le type de donnée quand ce n'était pas une chaine de caractère, je trouve que ça va encombrer d'ajouter "(type chaine)" partout, mais si ça aide, bof mais plutôt ça qu'inventer un truc avec des guillemets.
Mis à jour par Thomas Noël il y a presque 5 ans
- Statut changé de Nouveau à Rejeté
Je comprends. Je n'ai pas d'autre idée pour faire comprendre que « True (type bool) » et « True » sont deux choses vraiment différentes, et qu'il faut ajouter des guillemets quand on veut utiliser le second dans une condition (et donc, implicitement, pas dans le premier).
(et pour l'échappement des guillemets c'est quand même pas compliqué, hein, et ça permettrait justement d'expliquer comment gérer ça dans une condition, aussi)
Bref, allez, on trouvera bien une autre idée un jour.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Rejeté à Nouveau
On pourrait pas juste poser une classe avec le type sur la cellule ? Et genre mettre un fond gris pour les chaînes.
Mis à jour par Frédéric Péters il y a presque 5 ans
On pourrait pas juste poser une classe avec le type sur la cellule ? Et genre mettre un fond gris pour les chaînes.
Je n'ai pas l'impression qu'un usager pigera ainsi que c'est "différent". Mais peut-être pas dans cette idée, que None, True, False, soient tapés en italique ?
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Frédéric Péters a écrit :
On pourrait pas juste poser une classe avec le type sur la cellule ? Et genre mettre un fond gris pour les chaînes.
Je n'ai pas l'impression qu'un usager pigera ainsi que c'est "différent". Mais peut-être pas dans cette idée, que None, True, False, soient tapés en italique ?
Le plus explicite pour moi ce serait d'avoir une troisième colonne "type" alors, c'est un inspecteur c'est parfait pour des informations techniques (ou alors en title, mais ce n'est pas explicite). Quand aux caractère d'échappement c'est à vous de voir mais si une personne copie/colle la chaîne dans une expression python ou Django il faudra bien échapper les guillemets.
Je vois un constat au départ de ce ticket mais pas d'objectifs: est-ce juste que les gens comprennent le type de la donnée ? Ça demande déjà qu'ils aient conscience des types, avoir cet état d'esprit fait 80% du boulot et à part la formation je ne vois pas comment le donner. Permettre de réutiliser une valeur dans une expression facilement ? Le plus simple ce serait un bouton pour copier la bonne expression dans le presse-papier (avec quote pour une chaîne, etc..).
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Frédéric Péters a écrit :
On pourrait pas juste poser une classe avec le type sur la cellule ? Et genre mettre un fond gris pour les chaînes.
Je n'ai pas l'impression qu'un usager pigera ainsi que c'est "différent". Mais peut-être pas dans cette idée, que None, True, False, soient tapés en italique ?
Je répond quand même à ta remarque : oui gris, italique, whatever, mais jouons sur le style, néanmoins ça me plairait de savoir exactement le problème qu'on essaie de résoudre en fait, car Thomas parle de guillemets et de conditions donc je pense qu'il sous-entend que les gens veulent copier/coller le contenu de l'inspecteur dans une condition et que ça ne marche pas "telle quelle". Dans ce cas ma proposition ou la tienne ne servent un peu à rien, et à voir un bouton explicite copier aurait une sens.
Mis à jour par Frédéric Péters il y a presque 5 ans
Je réponds un peu pour Thomas, qui lui-même est déjà un peu messager, mais le problème c'est "juste" une incompréhension du concept de type. (ce qui ne se résoudra pas par des couleurs/italiques/colonnes supplémentaires/etc.)
Mis à jour par Thomas Noël il y a presque 5 ans
Et, personne n'y comprends rien, parfois on voit « True (bool) » et parfois « True », on pourrait effectivement ajouter un code qui dit que le second est en fait une chaine de caractère, pour "insister". Et je proposais d'utiliser simplement la notation informatique usuelle pour ça, entourer de guillemets, coup de bol c'est ce qu'on utilise dans les conditions, je trouvais ça intelligent et en fait je continue de le pense secrètement.
Mis à jour par Frédéric Péters il y a presque 5 ans
parfois on voit « True (bool) » et parfois « True »,
Le bug étant alors que parfois des chaines contiennent littéralement None, ou True, ou False.
~~
(je trouve vraiment que les guillemets dégradent (un peu) la lisibilité générale de la page)
Mis à jour par Thomas Noël il y a presque 5 ans
Frédéric Péters a écrit :
Le bug étant alors que parfois des chaines contiennent littéralement None, ou True, ou False.
Bien sûr, mais ça ne pourra pas être facilement corrigé, parce que c'est pas mal historique (la valeur de form_var_case_a_chocher c'est "True" ou "False"). Sans compter aussi les résultats de calculs dans des variables de traitement, où la personne fait 1+2, mais c'est "3" qui sera le résultat, et pas 3.
Mais : je suis vraiment très d'accord que les guillemets c'est pas joli. Je n'avais juste pas d'autre idée.
Et peut-être/certainement que tout cela mériterait une documentation, mais cette documentation finira en explication de la notion du concept de type. Elle sera comprise un jour, oubliée 2 semaines plus tard.
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Nouveau à Fermé
C'était rejeté, le réouvrir n'a pas amené de changement, je referme.