Discussion :

[jam7575]

Bonjour,
j'ai une séquence de code qui me pose problème, peut-être une grosse bêtise de ma part :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
    selection1 = 0   
    if len(points_liste_l.curselection()) > 0:       # sélection de points
       selection = [points_liste_l.get(i) for i in points_liste_l.curselection()] 
       selection1 = 1
    elif num1_v.get() != '' and num2_v.get() != "":
       selection = [] 
       for i in range(int(num1_v.get()), int(num2_v.get())):
           selection.append(points_liste_l.get(i))
           print('liste', points_liste_l.get(i))
       selection1 = 1 
    print('sel', selection, '\n', points_liste_l.get(0), '\n', liste_points[0])

Si le 'if' est satisfait, tout se passe bien.
Par contre, dans le cas 'elif', selection ne contient que des chaînes vides. Je la remplis pourtant avec les mêmes points_liste_l.get(i) que dans le cas 'if'.
Il y a peut-être quelque chose qui m'échappe dans la forme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

[points_liste_l.get(i) for i in points_liste_l.curselection()]

En fait le second cas correspond à une sélection (dans une listbox donc) à partir de numéros donnés par ailleurs (num1_v et num2_v).
Le premier cas correspond à une sélection multiple par clic. Y a-t-il d'autres moyens de sélection ? Comme ce que j'essaye de faire dans le cas 'elif' ?

Merci pour votre aide.

[Sve@r]

Bonjour

Par contre, dans le cas 'elif', selection ne contient que des chaînes vides. Je la remplis pourtant avec les mêmes

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

points_liste_l.get(i)

que dans le cas 'if'.

Oui mais les valeurs du "i" ne sont peut-être pas les mêmes... points_liste_l.curselection() pour l'un, range(int(num1_v.get()), int(num2_v.get())) pour l'autre. Il faudrait vérifier ce détail.

Il y a peut-être quelque chose qui m'échappe dans la forme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

[points_liste_l.get(i) for i in points_liste_l.curselection()]

C'est une liste en intension. Je me demande d'ailleurs pourquoi tu n'as pas utilisé la même syntaxe dans l'autre cas: selection = [points_liste_l.get(i) for i in range(int(num1_v.get()), int(num2_v.get()))].

Et num1_v.get() != '' s'écrit directement num1_v.get() (et pareil pour num2_v.get() != "").

[binarygirl]

Il faudrait un exemple de code complet pour débugger.
Mais je demande même si le elif est traversé lorsque le if n'est pas satisfait, pour l'instant rien n'est prouvé.

Je suis un peu sceptique quant aux conventions de nommage (qui peuvent créer de la confusion), et j'ai l'impression que selection1 serait en fait un booléen.

[Sve@r]

Mais je demande même si le elif est traversé lorsque le if n'est pas satisfait,

Si le "if" n'est pas satisfait, alors le "elif" ne sera exécuté que si l'expression évaluée est vraie (num1_v.get() et num2_v.get() non vides).
Et il semble exécuté car le PO a dit "je récupère des chaines vides" donc le tableau est rempli de trucs. Trucs vides mais trucs quand-même (s'il n'était pas exécuté, le tableau ne serait pas rempli du tout).

et j'ai l'impression que selection1 serait en fait un booléen.

Mouais ça y ressemble. C'est vrai que nommer un tableau "selection" et un booléen "selection1" c'est pas super éclectique dans le choix des noms (et en plus inutile car le booléen peut se retrouver dans le fait que le tableau est ou n'est pas rempli)

[wiztricks]

Salut,

En fait le second cas correspond à une sélection (dans une listbox donc) à partir de numéros donnés par ailleurs (num1_v et num2_v).

listbox.get(n) retourne une chaine vide lorsque n correspond à un index inexistant.

Le premier cas correspond à une sélection multiple par clic. Y a-t-il d'autres moyens de sélection ? Comme ce que j'essaye de faire dans le cas 'elif' ?

Pour pouvoir avoir une selection multiple, il faut avoir créé la listbox avec l'option selectmode='extended'.
Dans ce cas, sous Windows, pour sélectionner plusieurs items "disjoints", on clique avec la touche CTRL enfoncée. Pour sélectionner plusieurs items consécutifs, on clique le premier et on clique le 2nd avec la touche TAB enfoncée.

Indépendamment de cela, la méthode .get prend un 2nd paramètre optionnel: .get(m, n) retourne les items entre m et n (pas besoin d'aller les chercher un par un).
De plus la condition len(points_liste_l.curselection()) > 0 s'écrit not points_liste_l.curselection(): un tuple "vide" est évalué à False.


- W

[jam7575]

Merci pour tous ces retours rapides et suggestions pertinentes.
Oui :
1. il y a bien passage par le 'elif' quand c'est le cas
2. effectivement, j'ai découvert que dans le cas de ce type de sélection, seuls les premier et dernier numéros étaient utilisés. C'est d'ailleurs une modification que je comptais faire.
3. les indices existent bien (cf. point 1.)
4. 'selection1' est bien utilisé comme booléen ; c'est un ajout quand j'ai voulu ajouter une sélection autre que par 'clic' (j'ai récemment ajouté un 'Button' supposé tout sélectionner, d'où cette modif inélégante dans le code, qui me permet cependant de récupérer aisément un code qui marche bien). J'avoue qu'en général j'utilise rarement des booléens purs

Autres suggestions :
a. je n'ai pas encore essayé "extended", mais a priori je ne vois pas ce que cela changerait

Je ne crois pas qu'un code plus complet serait plus utile. Je ne comprends toujours pas ces chaînes vides, j'ai fait quelques tests avec des programmes simplifiés où le même code ne donne pas de chaîne vide.
Pas d'idée sur une sélection autrement que par clic ?
Cordialement

[umfred]

Tu affiches le nombre attendu de 'liste' avec ton print ? ne pas oublier (si le cas est possible) où num1_v est supérieur à num2_v

[jam7575]

Merci,
oui, tout est cohérent, j'ai essayé avec des traces pour vérifier.
Cordialement

[wiztricks]

Je ne crois pas qu'un code plus complet serait plus utile. Je ne comprends toujours pas ces chaînes vides, j'ai fait quelques tests avec des programmes simplifiés où le même code ne donne pas de chaîne vide.

Les tests simples montrent que ça fonctionne comme documenté. Après, la cuisine ajoutée pour que ça ne marche plus, c'est juste le code que vous ne montrez pas!

Pas d'idée sur une sélection autrement que par clic ?

Votre question est difficile à comprendre et j'ai essayé de vous donnez quelques pistes.
Une dernière pourrait être de regarder les méthodes qui permettent de modifier la selection...

a. je n'ai pas encore essayé "extended", mais a priori je ne vois pas ce que cela changerait

Il faut avoir passé du temps à voir comment fonctionnent les différents widgets avant de vouloir les utiliser dans une application et savoir quelles options utiliser ou pas (et le justifier). Et "tester" est une lecture active de la documentation: on écrit de petits codes pour constater ce que ça fait et conforter avoir compris ce que la documentation raconte.
note: sans cette option, le défaut est "browse" et curselection contient au plus un item: pas la peine de faire une boucle pour les parcourir.

- W

[umfred]

Pas sûr que ça aide, mais je lit ici https://www.tutorialspoint.com/python/tk_listbox.htm que Listbox.get peut prendre 2 paramètres, l'indice de départ et l'indice de fin, et renvoie alors un tuple avec le texte des éléments entre l'indice de départ et l'indice de fin inclus; ça éviterai déjà une boucle.

A tout hasard, selection n'est pas utilisé pour autre chose, comme définir la liste des éléments de la listbox ?

[jam7575]

Bonsoir,
oui, du fait de ce problème, j'ai ajouté des traces et, dans le cas où ça marche (2 clics, en début et en fin), j'ai vu que ça fonctionnait avec ces deux seuls éléments copiés dans 'selection' via curselection().
Depuis, dans le cas qui ne fonctionne pas (j'essaye de récupérer le début et la fin depuis deux 'Entry'), j'ai supprimé la boucle et ne recopie dans 'selection" que les deux valeurs (et bien sûr, 'selection' reste avec des chaînes vides).
Cordialement

[wiztricks]

Depuis, dans le cas qui ne fonctionne pas (j'essaye de récupérer le début et la fin depuis deux 'Entry'), j'ai supprimé la boucle et ne recopie dans 'selection" que les deux valeurs (et bien sûr, 'selection' reste avec des chaînes vides).

Sans un code qu'on puisse lancer et reproduire le problème... on peut juste compatir.

- W

[umfred]

Après essais, je dirais que tes valeurs sont en dehors des index disponibles dans ta liste: si tu tapes en dehors, ça retourne des chaines vides

[wiztricks]

Après essais, je dirais que tes valeurs sont en dehors des index disponibles dans ta liste: si tu tapes en dehors, ça retourne des chaines vides

C'est ce que j'avais (déjà) mentionné dans cette réponse (ainsi que l'existence du paramètre optionnel du .get).
Par ailleurs, lorsque la documentation dit:

pathName get first ?last?
If last is omitted, returns the contents of the listbox element indicated by first, or an empty string if first refers to a non-existent element. If last is specified, the command returns a list whose elements are all of the listbox elements between first and last, inclusive. Both first and last may have any of the standard forms for indices.

tester c'est juste s'assurer qu'on sait bien lire l'anglais (ce qui n'est pas inutile mais ça n'apporte pas grand chose sur les causes du problème rencontré par le PO).

- W

[jam7575]

non, les indices sont bons, aucune erreur là-dessus.

[Sve@r]

A un moment donné il faut arrêter de contempler son écran d'un air inspiré en attendant l’annonciation de l'ange Gabriel et se prendre par la main pour faire de vrais checks. Puisque tes chaines vides sont apparemment stockées dans le "elif", probablement par cette instruction selection.append(points_liste_l.get(i)) ben il serait bon, ici, de rajouter un peu quelques print() de ci, de là voir ce qui se passe...

[jam7575]

ceci a été fait.
Pas la peine de prendre des allures déplaisantes pour dire des banalités. Mieux vaut s'abstenir.

[wiztricks]

non, les indices sont bons, aucune erreur là-dessus.

Comment en êtes vous si sûr?

- W

[Sve@r]

ceci a été fait.

Et donc ça donne quoi? Les chaines que tu affiches sont vides ou pas? Ou alors tu trouves plus fun de nous laisser deviner...

Pas la peine de prendre des allures déplaisantes pour dire des banalités.

Oui enfin ce n'est une banalité que parce que tu dis, après coup, que ça a été fait. Mais tu l'aurais dit avant ça nous aurait évité de chercher dans le vide. Pas grave, tant que c'est pas toi qui cherche hein...

Mieux vaut s'abstenir.

C'est noté.

[papajoker]

bonjour

Depuis, dans le cas qui ne fonctionne pas (j'essaye de récupérer le début et la fin depuis deux 'Entry'), j'ai supprimé la boucle et ne recopie dans 'selection" que les deux valeurs (et bien sûr, 'selection' reste avec des chaînes vides).

Tu es pourtant "Consultant informatique" ???
Comment veux-tu une aide avec un descriptif si vague et aucun code ? un problème uniquement avec des "Entry" qui sortent de nulle part ? Si tu as des clients comme toi, ton travail doit être chaud.

j'ai ajouté des traces ... j'ai supprimé la boucle

Donc on ignore tout de ce que tu as fait

On ne te demande pas le code de ton application mais un code d'un script minimal qui teste ce que tu désires (faut-il encore comprendre ce que tu désires ... )