_getMessage should be inlined and/or renamed
还没有人认领这个 Issue。
评估
- 难度
- 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 分钟
- 30 天内合并 PR
- 2
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 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 一周以上 新手友好度 20/100
NativeScript/ios-device-lib#66 · 2 个 reaction ·
查看 NativeScript/ios-device-lib 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 70/100
shadps4-emu/shadps4-qtlauncher#453 ·
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
google/differential-privacy#516 ·
-
难度 1/5 1 小时以内 新手友好度 92/100
plengauer/DXGIOutputDuplication#76 ·
维护者通常 1 天内回复
-
Feature
难度 1/5 1 小时以内 新手友好度 65/100
Narezzurri/OpenVPN-Config-Manager#95 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
rhalbersma/xstd-bits#314 ·
维护者通常 1 天内回复