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...".