Instructions for Zipper exercise are misleading.
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Documentation
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- javascript
- Domaine
- documentation
Piste de recherche
Commencez par les instructions et la suite de tests de l’exercice Zipper du track JavaScript, puis comparez leur description d’un Zipper avec les références liées de Wikipedia et Wikibooks. Décidez si l’exercice doit clarifier le modèle fonctionnel, ajouter des tests pour les arbres immuables et l’état de Zipper, ou être renommé, puis mettez à jour la documentation ou les tests pertinents afin que le comportement attendu soit sans ambiguïté.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hi,
Issue:
I don't know if this is the right place to report this but I found the instructions for Zipper on the Javascript track to be misleading.
The instructions say "Zippers are a purely functional way of navigating within a data structure and manipulating it. They essentially contain a data structure and a pointer into that data structure (called the focus)."
In a purely functional setting, the data structure is immutable. The pointer into the data structure could read from inside the structure, but it couldn't modify it in any way. The only way to set a new value is to build new structure with the modified value based on the old data.
The linked Wikipedia page says of list-zippers: "Consequently, a list-zipper is a pair consisting of both the location as a context or starting point, and a recording or path that permits reconstruction from that starting location." And the reference that Wikipedia uses says (in a mythological setting about exploring a labyrinth): "The zipper is a pair of Ariadne's thread and the current sub-labyrinth that the player stands on top."
The point being the Zipper does not contain the full data structure and a pointer. It is two things:
- The steps (thread) leading up to the focused node (Did I go left or right here? What was the value of the node here? What was the other subtree?)
- The rest of the structure, from the focused node down (In this case the subtree starting at the focused node).
From these two things, you can rebuild the full structure, including making modifications by creating new subtrees with modified values as you reverse the steps.
Now, I managed to implement a Zipper according to the description on Wikipedia but I am not sure it makes sense to do this in Javascript. As the Wikipedia says, "In a non-purely-functional programming language, it may be more convenient to simply traverse the original data structure and modify it directly (perhaps after deep cloning it, to avoid affecting other code that might hold a reference to it)." But doing this is not a Zipper.
Potential Solutions:
-
Update problem statement to better reflect what a Zipper is. (Recorded thread + rest of structure)
-
Modify the test suite: If we still want a Zipper in Javascript, then the test suite needs to be more detailed. First, the input tree should be frozen/immutable. It should check that all output is also immutable and all intermediate stages do not have side effects (i.e a new Zipper with a new thread and a new tree is created at every stage).
-
Add guiding tests: Have tests that check what the current thread is in the Zipper after running different methods and what the current subtree is. E.g. if zipper is
Zipper.fromTree(bt(1, bt(2, null, leaf(3)), leaf(4))), the Zipper returned by zipper.left() will have subtreebt(2, null, leaf(3))and thread will be something like[{direction: "left", value: 1, otherSubtree: leaf(4)}]This will make the exercise much easier, since the nature of what a Zipper is will be clear. You may not want to do this though since the problem is specified as Hard; a suitable description on the problem page may be sufficient. -
Change the problem to making a copy of the tree and maintaining a pointer into the tree. The problem would need a different name and couldn't refer to the Zipper data structure.
Anyway, I haven't contributed here before so I don't know what the procedure is for this sort of thing. Also I'm not particularly familiar with Javascript so there might be better solutions than my suggestions above. Please let me know your thoughts and if I should do anything.
Thanks,
Josh
- Langage dominant
- JavaScript
- Étoiles
- 652
- Forks
- 658
- Merge moyen
- 13 h 46 min
- PR mergées (30 j)
- 6
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de exercism/javascript
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
exercism/javascript#2865 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
exercism/javascript#2847 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
exercism/javascript#2832 ·
-
exercism/javascript#2310 · 22 commentaires · 1 personne assignée ·
-
pinned :pushpin:
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 5/100
exercism/javascript#2144 ·
Toutes les issues de exercism/javascript
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
mksglu/context-mode#1200 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
neondatabase/website#5944 ·
-
module: core
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
bigbluebutton/bigbluebutton#25849 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
jaegertracing/jaeger-ui#4506 ·