apollographql / apollographql/fullstack-tutorial
Confusing naming of function passed to Array.map
- Vorherrschende Sprache
- TypeScript
- Sterne
- 1.2k
- Forks
- 808
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Hello,
not a real issue but just a naming of a method that confused me for a bit.
In [datasource/launch](https://github.com/apollographql/fullstack-tutorial/blob/master/final/server/src/datasources/launch.js#L10) there is method called `launchReducer` passed to `Array.map`. This method is responsible to take each launch returned by the Rest API and **mapping** it to another object.
So I guess this function should be called `lauchMapper`, because that's what it does. It doesn't reduce multiple element into a single one, isn't it?
Just highlighting this because it is a bit confusing
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Öffne final/server/src/datasources/launch.js bei der Definition von launchReducer und prüfe, wo sie an Array.map übergeben wird. Überprüfe alle weiteren Referenzen auf die Funktion und aktualisiere anschließend die Benennung konsistent, sodass der Name der Funktion ihre Mapping-Rolle widerspiegelt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 55/100