NativeScript / NativeScript/ios-device-lib
_getMessage should be inlined and/or renamed
Aperta
Nessuno ha ancora preso questa issue.
- Lingua principale
- C++
- Stelle
- 16
- Fork
- 8
- Merge medio
- 19h 29m
- PR unite (30g)
- 2
Descrizione
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
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in index.js intorno alla riga 126, dove _getMessage viene usato per preparare i dati per this._iosDeviceLibStdioHandler.writeData. Conferma che questo sia il suo unico utilizzo, quindi decidi se incorporare la serializzazione direttamente o rinominare l’helper in modo che descriva la chiamata al metodo nativo. Il lavoro è completato quando l’unico utilizzo rimane chiaro e l’helper obsoleto o rinominato è risolto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript
- Ambito
- mobile-dev
- Tipo di issue
- Refactoring
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100