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