NativeScript / NativeScript/ios-device-lib
_getMessage should be inlined and/or renamed
オープン
まだ誰も着手していません。
- 主要言語
- C++
- スター
- 16
- フォーク
- 8
- 平均マージ
- 19時間 29分
- マージ済み PR(30日)
- 2
説明
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
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
index.js の 126 行目付近、_getMessage が this._iosDeviceLibStdioHandler.writeData 用のデータを準備するために使われている箇所から始めます。これが唯一の使用箇所であることを確認し、そのうえでシリアライズ処理をインライン化するか、ネイティブメソッドの呼び出しを表すように helper の名前を変更するかを判断します。完了条件は、唯一の使用箇所が引き続き明確であり、不要になった、または名前を変更した helper が解決されていることです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript
- 領域
- mobile-dev
- issue の種類
- リファクタリング
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100