mapbox / mapbox/MapboxGeocoder.swift

MBPlacemark should store property values parsed from GeoJSON

Ouverte
#24 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Swift
Étoiles
133
Forks
46
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Each property that’s parsed out of MBPlacemark’s backing GeoJSON is potentially expensive, especially when accessed frequently. It’s easy to come up with a pathological case that involves putting addressDictionary fields into a table view cell. Since the backing GeoJSON is immutable, these property values should be lazily computed then stored for the lifetime of the object.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Localisez MBPlacemark et examinez comment son GeoJSON sous-jacent fournit des propriétés telles que addressDictionary. Confirmez quelles valeurs sont recalculées lors d'accès répétés, puis ajoutez une mise en cache pendant toute la durée de vie, tout en préservant le comportement existant des propriétés. Le travail est terminé lorsque les accès répétés aux propriétés n'analysent plus à nouveau le GeoJSON immuable.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
swift
Domaine
mobile-dev
Type d'issue
Refactorisation
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.