googlemaps / googlemaps/android-maps-compose
Replace most GoogleMap() rememberUpdatedState() usages with single State object and encapsulate Subcomposition functionality
- Vorherrschende Sprache
- Kotlin
- Sterne
- 1.3k
- Forks
- 181
- Ø Merge
- 2 T. 23 Std.
- Gemergte PRs (30 T.)
- 18
Beschreibung
The GoogleMap() implementation uses a gazillion individual rememberUpdatedState() calls. This is unwieldy and likely contributed to #481.
Instead, a single State object encapsulating the individual state properties can be used and passed as a parameter. This pattern is in compliance with Google Compose API guidelines.
The subcomposition should be made a separate composable function to prevent accidental access to local variables and better encapsulate subcomposition functionality.
Beitragsleitfaden
Rechercherichtung
Beginne damit, die Implementierung von GoogleMap() zu lokalisieren und ihre Verwendungen von rememberUpdatedState() sowie den Subcomposition-Code zu erfassen. Prüfe anschließend das Muster der Google Compose API-Richtlinie, trenne dann die Subcomposition und ersetze die individuelle Zustandsverwaltung durch ein einzelnes, kapselndes State-Objekt; abgeschlossen ist die Arbeit, wenn die Implementierung gekapselt ist, ohne das Verhalten zu ändern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- android, kotlin
- Bereich
- mobile
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100