Discussion :

[youchlure]

Bonjour à tous,

je cherche à déplacer 60 personnes sur un espace séparé en 3 parties.

Je fais cela :

i:= 1.
self popAgri timesRepeat:

theCells at: [((i-1) \\ (theCells size)) +1].
farmeri moveTo: theCells].

En sachant qu'ici popAgri représente les 60 personnes.
theCells correspond à mon espace. theCells size à la taille : 3.
farmeri correspond à mes agriculteurs qui sont donc 60.

Le probleme est que ça ne marche pas
Lors de l'exécution il me dit que (i-1) \\ (theCells size) : non integer index...

Merci pour votre aide, et bonne journée

[milco]

tu as fait un mauvais copier/coller de ton code.
j'ai du mal à comprendre du coup.
peux-tu vérifier ?

[maske]

Tu t'en es sorti ?

Le code qui était copié collé était-il bon finalement (pas d'erreur comme demandé par Milco ?) ?

Pour moi l'erreur c'est que tu sembles essayer d'accéder à une collection (theCells) en lui passant un bloc au lieu d'un entier - d'où l'erreur "Non Integer Index".

Il faut retirer les crochets et passer de [((i-1) \\ (theCells size)) +1] à (((i-1) \\ (theCells size)) +1).

Le fait de rajouter des [ ] transforme ton objet en BlockClosure (en bloc quoi) et tu le passe en paramètre au at: pour accéder aux éléments de ta collection (paramètre qui devrait être un entier).

Sauf que pour ça il faut être sûr que :

- le code copié collé est correct
- theCells est bien une collection - et donc le message at: doit prendre un entier en paramètre, ce qui serait cohérent avec ton erreur "Non Integer Index"

Tiens nous au courant

[milco]

en complément, voici mes autres remarques :

- il restera un crochet orphelin (c'est pourquoi je parle d'erreur de copier coller)

- tu veux utiliser l'opérateur // ou \\ ??

- d'expérience, timesRepeat: est rarement la solution à notre besoin

- si tu veux boucler sur ta collection, c'est plutot un do:
éventuellement un inject: into: (mais cette méthode récursive n'est pas à mettre entre toutes les mains)

- je n'ai pas moveTo: dans mon smalltalk (enfin que sur des objets graphiques)
donc je ne sais pas ce que tu veux faire à ce moment là

- je ne vois pas d'incrémentation à ton indice
de plus i-1 la première fois donnera 0.
contrairement à Java , les indices commencent à 1

Est-ce que ton besoin est d'éclater la collection de 60 éléments en trois "sous-collection" ?

[maske]

- je ne vois pas d'incrémentation à ton indice

Bien vu

de plus i-1 la première fois donnera 0.
contrairement à Java , les indices commencent à 1

Il rajoute un +1 : ((i-1) \\ (theCells size)) +1 évalue d'abord le résultat du \\.

Il va peut-être refaire surface nous dire si on l'a bien aidé ou pas, pour une fois que ça bouge sur ce forum =p

[youchlure]

Salut et merci de vos réponses.

Comme vous vous en doutez, le temps de réponse étant un peu long sur ce forum , j'ai avancé et trouvé la solution.
Celle ci se rapproche de ce que vous m'avez proposé

Merci !