Discussion :

[alatox]

Salut

J'essaye d'utiliser un Set dans une boucle for d'une méthode et lors de l'appel à la méthode add la boucle s'arrête après une itération.

Code :
------

Code :

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

1
2
3
4
5
6
7
8
9
10
 
mySet = Set[]
 
def maMethode(chaine):
   mList = chaine.split(' ')
   for m in mList:
       mySet.add(m)
 
maMethode('un deux deux trois')
print mySet

-----

Le print donne "un"et on sort on ne peut rien executer après add dans la boucle for.


Pouvez-vous m'éclairer ?

Merci

[rambc]

Bonsoir,
chez moi j'obtiens Set(['un', 'trois', 'deux']). Je suis sous Python 2.6.

[josmiley]

c'est quoi ?

Code :

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

mySet = Set[]

s'pas mieux ça ?

Code :

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

1
2
3
4
setmoissa = lambda phrase: set(phrase.split())
 
monset = setmoissa("un deux deux trois")
print monset

[PauseKawa]

Bonsoir,
chez moi j'obtiens Set(['un', 'trois', 'deux']). Je suis sous Python 2.6.

Sans modifier la première ligne ?
Mais bon, moi aussi j'ai ce résultat

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Python 2.6.6 (r266:84292, Sep 15 2010, 15:52:39) 
[GCC 4.4.5] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> mySet = set()
>>> def maMethode(chaine):
...    mList = chaine.split(' ')
...    for m in mList:
...        mySet.add(m)
... 
>>> maMethode('un deux deux trois')
>>> print mySet
set(['un', 'trois', 'deux'])

Y compris comme cela

Code :

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

1
2
3
4
5
6
7
8
mySet = set()
 
def maMethode(chaine):
    for m in chaine.split():
        mySet.add(m)
 
maMethode('un deux deux trois')
print mySet

Je trouve que c'est plus lisible que lambda

@+

[Pierre Maurette]

Je ne vois pas trop comment le code initial, même légèrement corrigé, a pu afficher "un".
Je ne trouve pas le lamda particulièrement illisible, mais en revanche plutôt à éviter quand il est gratuit. Moins maintenable, et il peut casser des outils (vue outline, refactoring, etc.). Sans lamdda, c'est tout aussi simple:

Code :

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

1
2
3
4
5
6
7
8
9
def buildSet(chaine):
    return set(chaine.split(' '))
 
mySet = buildSet('un deux deux trois')
print mySet
mySet = mySet.union(buildSet('one two three two'))
print mySet
mySet = buildSet('one two three two')
print mySet

Notez que le comportement n'est pas le même que celui de la fonction du message initial. L'utilisation immodérée des foo, bar et autres maMethode peut masquer des erreurs de conception. Ce n'est pas plus cher d'utiliser des noms significatifs dès le brouillon:

Code :

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

1
2
3
4
5
6
7
8
9
def mySetAddElems(chaine):
    for m in chaine.split():
        mySet.add(m)
 
mySet = set()
mySetAddElems('un deux deux trois')
print mySet
mySetAddElems('one two three two')
print mySet

Là, on peut rendre la fonction un peu plus réutilisable et un peu moins piégée:

Code :

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

1
2
3
4
5
6
7
8
9
10
def addElems(chaine, set):
    for m in chaine.split():
        set.add(m)
 
mySet = set()
print mySet
addElems('un deux deux trois', mySet)
print mySet
addElems('one two three two', mySet)
print mySet

[rambc]

Bonjour.

Sans modifier la première ligne ?

Si je te rassure.

[PauseKawa]

Code :

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

1
2
3
4
5
6
>>> mySet = Set[]
  File "<stdin>", line 1
    mySet = Set[]
                ^
SyntaxError: invalid syntax
>>>

[rambc]

Je voulais dire : "Si je te rassure, j'ai bien modifié la ligne...".

[dividee]

Une autre version, pour la route:

Code :

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

1
2
def addElems(chaine):
    mySet.update(chaine.split(' '))

Mais ma préférence va tout de même à la première fonction de Pierre Maurette:

Code :

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

1
2
def buildSet(chaine):
    return set(chaine.split(' '))

Compréhensible en isolation (pas besoin de savoir ce qu'est mySet), fonctionnelle (la valeur de retour ne dépend que de celles des arguments), et sans effets de bord (ne modifie rien en dehors de la fonction).

[PauseKawa]

Une question: Puisque c'est l'espace le caractère de séparation pour split pourquoi le mettre au lieu d'un .split() ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
>>> def buildSet(chaine):
...     return set(chaine.split())
... 
>>> mySet = buildSet('un deux deux trois')
>>> print mySet
set(['un', 'trois', 'deux'])
>>> mySet = mySet.union(buildSet('one two three two'))
>>> print mySet
set(['un', 'trois', 'deux', 'one', 'three', 'two'])
>>> mySet = buildSet('one two three two')
>>> print mySet
set(['three', 'two', 'one'])
>>> def buildSet(chaine):
...     return set(chaine.split(' '))
... 
>>> mySet = buildSet('un deux deux trois')
>>> print mySet
set(['un', 'trois', 'deux'])
>>> mySet = mySet.union(buildSet('one two three two'))
>>> print mySet
set(['un', 'trois', 'deux', 'one', 'three', 'two'])
>>> mySet = buildSet('one two three two')
>>> print mySet
set(['three', 'two', 'one'])
>>>

[dividee]

Une question: Puisque c'est l'espace le caractère de séparation pour split pourquoi le mettre au lieu d'un .split() ?

Les caractères de séparation par défaut ne sont pas limités à des espaces, mais comprennent tous les caractères "blancs" (tabulation, retour de ligne, ...). Les deux formulations ne sont donc pas équivalentes.

[Pierre Maurette]

Me suis planté, le split() devait être vide. Bien sûr.

[PauseKawa]

Les caractères de séparation par défaut ne sont pas limités à des espaces, mais comprennent tous les caractères "blancs" (tabulation, retour de ligne, ...). Les deux formulations ne sont donc pas équivalentes.

Code :

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

1
2
3
4
5
>>> txt = '     a       b c    d'
>>> print txt.split()
['a', 'b', 'c', 'd']
>>> print txt.split(' ')
['\ta\tb', 'c', '', '', '', 'd']

Autant pour moi. Promis, la prochaine fois je teste.