mapbox / mapbox/MapboxGeocoder.swift
MBPlacemark should store property values parsed from GeoJSON
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
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Ö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