NativeScript / NativeScript/ios-device-lib
_getMessage should be inlined and/or renamed
Offen
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C++
- Sterne
- 16
- Forks
- 8
- Ø Merge
- 19 Std. 29 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
I think _getMessage should be:
- Inlined as we are using it on only one place. I would assign the one line to a variable and pass it to this._iosDeviceLibStdioHandler.writeData
- Named something like _serializeNativeMethodCall... something that will ring the right bell when you read it.
https://github.com/telerik/ios-device-lib/blob/382c9c63dd1999ef247def16a3787b5987d63649/index.js#L126
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
Beginne in index.js bei etwa Zeile 126, wo _getMessage verwendet wird, um Daten für this._iosDeviceLibStdioHandler.writeData vorzubereiten. Bestätige, dass dies die einzige Verwendung ist, und entscheide dann, ob die Serialisierung inline eingefügt oder der Helper umbenannt werden soll, damit er den Aufruf der nativen Methode beschreibt. Als erledigt gilt, dass die einzige Verwendung weiterhin klar erkennbar ist und der veraltete oder umbenannte Helper aufgelöst ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- mobile-dev
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100