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