mapbox / mapbox/MapboxGeocoder.swift

MBPlacemark should store property values parsed from GeoJSON

Offen
#24 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Swift
Sterne
133
Forks
46
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Finde MBPlacemark und untersuche, wie das zugrunde liegende GeoJSON Eigenschaften wie addressDictionary bereitstellt. Bestätige, welche Werte bei wiederholtem Zugriff neu berechnet werden, und füge anschließend Caching für die Lebensdauer hinzu, wobei das bestehende Verhalten der Eigenschaften erhalten bleibt. Erledigt ist die Aufgabe, wenn wiederholter Eigenschaftszugriff das unveränderliche GeoJSON nicht mehr erneut parst.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
swift
Bereich
mobile-dev
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.