Web 3 项目前端 repo 代码疑问

2022-07-06 16:32:24 +08:00
 YadongZhang

可能是普遍现象:

楼下有补充吗

4394 次点击
所在节点    职场话题
65 条回复
rongchuan
2022-07-07 11:07:23 +08:00
@YadongZhang 我觉得各个项目组情况不一样,如果你们没有那么多条条框框的话,这个反倒是一个好机会。你可以跟组长讲一下你发现的这些问题,然后主动跟他要活,推动一些代码规范的制定,下次绩效评选你妥妥的 375 。不过以老油条的角度来说,最好是你推动规范制定,然后让他们各自去改自己不规范的代码,不要自己去帮他们改,因为一般这种类似的情况改代码容易改出 bug...
wobuhuicode
2022-07-07 11:09:02 +08:00
@YadongZhang 人家要求的是快速开发,稳定运行,就算你用 JQ 写一套跑起来能达到这个两个要求就够了。哪有那么多考量
murmur
2022-07-07 11:09:54 +08:00
山寨币考虑安全性不是搞笑么,银行全实名制又是风控又是断卡都防不住,山寨币私钥拿走啥都没有,追都追不回来
rongchuan
2022-07-07 11:16:53 +08:00
@murmur 那些项目俗称杀猪盘,土狗盘,现在市场里的确不少这种项目,这种是犯罪了,跟安全性也没啥关系...上面说的安全主要是考虑防黑客盗币还有一些极端情况,是技术问题。其实现在这种土狗盘盛行的情况下,像我一般都懒得辨别,我都不投,我目前就存了一堆 usdc 吃 8%的利息
YadongZhang
2022-07-07 11:53:52 +08:00
@wobuhuicode #42

所以赚不赚钱跟代码水平没啥关系
Bingchunmoli
2022-07-07 13:39:40 +08:00
目前培训机构和市场普遍中低端前端用的是 js+vue2 (甚至可以兼容 IE ),vue3 不熟,ts 都是 any 是常见情况
wobuhuicode
2022-07-07 13:53:30 +08:00
@YadongZhang 用上新技术就是高代码水平???还是说用上 写 JQ 就不是高水平???只有做到快速开发,稳定运行的代码才是高水平。
YadongZhang
2022-07-07 14:17:29 +08:00
@wobuhuicode #47

???
Orainsink
2022-07-07 15:56:13 +08:00
刚好手里的这个项目就是 scss 和 css-in@-js 混用,而且用 momentjs ,并且默认和具名导出混用的。不过这是个 monorepo 项目。
1.从安全性考虑,引入的第三方插件必须过安全审查,如果已经用了 momentjs ,那么换 dayjs 的话需要有审查的人负责。如果需要改代码,需要有新的测试覆盖。
2.css 那个大部分是项目维护时间太长导致的遗留问题。同样的问题,从 scss 迁移到 css-in-js 不是小工程,需要人推动工作并且回归测试。但就算不改,对最后打包出来的代码也没什么影响。
3.eslint 设置了 default export 导出的 warning ,新写的代码需要避免这个,但是老代码没必要动。选哪个导出方式对项目的影响几乎为 0 ,重要的是你们的意见能达成一致。
4.路径别名并不是必须的。
5.全 any 是人的问题。但是禁 any 更没必要。在不需要类型判断的地方用 any 可以减少开发时间,也减少 tsc 等待时间。
6.Vue3 和 Vue2 的差别还是大,可能只是你的同事不想学 Vue3

其实项目技术栈的选择和 Tech lead 个人偏好有很大关系,你只能提意见,怎么选还是一起开会做决定。不要自作主张提 PR 该这些东西,出了问题你负责不说,还会给同事留下刺头的印象
Pastsong
2022-07-07 16:00:44 +08:00
前端包多个几十 k 对用户的影响大,还是重构一个稳定长期运行的模块,重构出 bug 在线上炸掉对用户影响大?
mufeng
2022-07-07 16:03:12 +08:00
@anguiao Vue 2 + Pinia + TypeScript 配合很好
YadongZhang
2022-07-07 16:18:19 +08:00
@Orainsink #49

scss 迁移 css-in-js 做过,大工程,需要 QA 对着 Figma 测试

提个 PR 会出什么问题,不是还没合并吗,先提 PR 好讨论啊,PoC ?
YadongZhang
2022-07-07 16:20:24 +08:00
@Pastsong #50

这么说没有重构做出来的 v2 版本?

另外几十 kb 是按单个用户计算的,如果用户量足够多,那就不是几十 kb 了
YadongZhang
2022-07-07 16:27:13 +08:00
named export 和 default export 好像不只是写法不一样,好像和 babel 有啥关系

某次大厂面试问了,当时面试官怎么说的记不清了,我是小菜鸡不懂 babel 。
walpurgis
2022-07-07 17:20:24 +08:00
个人项目我可能会去压榨这几十 KB ,毕竟服务器费用是自己出的
团队项目在已经大量使用 moment 的前提下,换 dayjs 产生的收益你真的算过吗,节省下来的费用能超过你一天的工资吗
TomatoYuyuko
2022-07-07 17:57:31 +08:00
一个很简单的问题,我用 vanillaJS ,和 vue3 或者 React 比,做出你所说的 web3 项目,对你老板来说有什么差别吗?
如果是只讨论开发体验那就更简单了,个人开发,想怎么写就怎么写,团队开发,大家想怎么写就怎么写,没有什么标准答案
bobo2
2022-07-07 17:59:40 +08:00
CSS in JS 的优势在哪,我倒是觉得单独的 scss 或者 less 更易维护
YadongZhang
2022-07-07 18:43:29 +08:00
不讨论了,人各有志,唉
pieerepeng
2022-07-08 01:51:34 +08:00
一线的项目基本都是 react 的,虽然我们自己用 vue

感觉开发者们关注的是实现后的视觉 /交互效果,DAU 能上个万就是成功项目了,package size 差一点真不太重要。

同时,可能很多实际工程师不是专业的前端(比如我),不太关心例如“default export 不用 named export”这样的问题,path alias 我其实是不喜欢用的,但有时候前端配了,我也会用,就是这么随遇而安

以及,客观的说,大部分 web3 项目没啥迭代的机会,同时就一个前端主力开发,也没啥大团队配合的机会
Envov
2022-07-08 23:34:31 +08:00
你的的确更好,但是你要尊重别人的选择

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/864467

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX