NativeScript / NativeScript/ios-device-lib
_getMessage should be inlined and/or renamed
Ouverte
Personne n'a encore pris cette issue.
- Langage dominant
- C++
- Étoiles
- 16
- Forks
- 8
- Merge moyen
- 19 h 29 min
- PR mergées (30 j)
- 2
Description
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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez dans index.js autour de la ligne 126, où _getMessage est utilisé pour préparer les données destinées à this._iosDeviceLibStdioHandler.writeData. Vérifiez qu’il s’agit de son seul usage, puis décidez s’il faut intégrer la sérialisation directement ou renommer le helper afin de décrire l’appel de méthode native. Le travail est terminé lorsque l’usage unique reste clair et que le helper obsolète ou renommé est résolu.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- mobile-dev
- Type d'issue
- Refactorisation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100