公司代码理念不一样咋办

2019-01-09 08:18:09 +08:00
 livingwolf
前端,因以前公司业务不好,清闲了一年多,觉得这样下去不行,跳槽到现在公司,觉得现在公司的前端代码实在是太乱了,于是提出了重构版本,郁闷了。本人菜鸟,还是想提高自己水平的。
1、觉得以前文件结构目录不规范,所有页面都放在 components,提出页面放在 views 或 pages,组件放在 components,就这,说就好看点,所有提出来的组件应该要封装插件。
2、以前代码写的太乱,var、let、const 乱用,很多都没规范,提出用 eslint 进行代码质量检测,被拒了,说照 eslint 规范写太累了
3、公共方法杂乱,光是时间戳处理页面就好几个,每个人都写各自一个,但是有人说代码能跑就行,里面不用关心
4、同事固步自封,都 2019 了,es6 都还不熟悉,跟他们说 es6 语法,有个人竟然说会用以前写法,这样也不用编译,特么我无语了
5、有个后端同事觉得自己全栈,写出代码 if for 一堆,一直用 var,我用 array 的 some、every 竟然被说成最优写法,唉。
6、所有页面耦合性太高,比如主页面,侧边栏、头部、尾部等操作全都写在一个页面里,提出提取出来,被说只是好看
是我太过轻狂了吗?我觉得如果是一个项目你写完可以不管,但是是一个一直维护的产品的话,代码质量是非常重要的。
唉,辞职是不可能了,怎么端正自己心态了?
4626 次点击
所在节点    职场话题
36 条回复
xiaohuamao
2019-01-09 08:43:43 +08:00
某种意义上程序员就两种人:全天下老子第一 全天下老子最菜,你属哪种
ThomasZ
2019-01-09 08:44:02 +08:00
做好自己的,在你没有相关职务的时候这些想法不会被认可还可能造成矛盾,只有你有一定的话语权的时候才好使,至于自我充电那更是个人事情了。所以做好自己的工作,充实自己,在合适的时候跳槽还是靠谱
serge001
2019-01-09 08:45:06 +08:00
没办法, 这种问题已经讨论过很多次了, 除非用行政的力量干预, 否则同级没人会听你的....毕竟很多人不仅仅对代码, 对很多东西都没追求的
livingwolf
2019-01-09 08:50:06 +08:00
@xiaohuamao 那我不属于程序员,哈哈哈,我觉得比我厉害的人很多,比我菜的人不少
smilingsun
2019-01-09 08:53:42 +08:00
我以前也有这样的想法,不过我觉得

比写代码更难的是读代码
你以为你写的代码容易懂别人不一定觉得
livingwolf
2019-01-09 08:57:46 +08:00
@smilingsun 我以为我说的都是基本的,只是代码规范问题,都不涉及到读懂读不懂的事
whileFalse
2019-01-09 08:59:01 +08:00
建议在不影响用户和其他同事的情况下重构。

先把运行正常、不需要维护的页面重构掉。重构的时候用代码对比工具仔细查看想提取成模块的重复代码是否有细微的不一致。别大大咧咧重构完了功能被改掉了。重构完成后多测试。确认没问题了可以邀邀功,“之前有 5k 行代码,经过整理减少到 2k 行,目录结构拆解清晰,逻辑更加易读”

如果要推 es6,尽量做到让同事无感。别人家早上一来一更新代码卧槽编译不通过了。建议首先在你自己的组件中使用 es6,把编译后的代码和编译之前的都 commit 到版本控制里,保证同事们即使不安装 babel 也能正常使用。先让大家看到 es6 的爽点,再来谈论要不要上。

好好积攒经验值然后找到逼格更高的公司是上策;升为主管然后整理技术团队是中策,逼着别人用你刚学的雕虫小技是下下策。很多人就是来混口饭吃,复制粘贴多爽,“你算什么玩意”。
13662439966
2019-01-09 08:59:09 +08:00
某种意义上来讲,这还是回归到六字箴言当中
BingoXuan
2019-01-09 09:05:06 +08:00
六字真言送给你,要么忍要么滚
qiutianaimeili
2019-01-09 09:07:24 +08:00
还是太年轻。。。喜欢纠结这种小问题。
livingwolf
2019-01-09 09:15:03 +08:00
@whileFalse 现在项目用的是 vue 全家桶,不存在说 es6 编译不过,他们也用了点 es6,不过不规范。vue 也是,有些真的只是代码规范问题。
推广 es6,我现在是不指望了,从我一进去就问我 var、let 区别,到现在还问这个,就算我说不明白,真有心,自己去看书也行。
看大家说的,觉得我还是做只咸鱼吧,自己研究自己的,公司叫我做啥就做啥吧
因为已婚未孕,跳槽有点麻烦,现在
ooppstef
2019-01-09 09:35:52 +08:00
只是角度不同而已.

从公司的角度,你重构一需要资源(工时等),二来本身稳定的程序可能造成 bug.这种只有坏处没有好处的事情,公司怎么想?

更好的做法是,通过优化自己的模块+利用空闲时间一步步优化其他模块.在团队中证明你自己,拿到权限就好了
nfroot
2019-01-09 09:36:27 +08:00
重构是大工程啊。。。。
solaro
2019-01-09 09:40:21 +08:00
公司花钱聘请你来干活得,不是来提意见得,你还年轻
TomVista
2019-01-09 09:56:48 +08:00
代码写的乱,这点我同意你的想法,但是让同事用 es6 是你不对.

固步自封 if for array some var let???

不要想着推 es6 了,着手规范代码结构把.
otakustay
2019-01-09 10:00:21 +08:00
现在大概不分 page 和 view,全是 component 是主流
tabris17
2019-01-09 10:04:59 +08:00
你知道什么叫 DDD 么? deadline driven development
jmc891205
2019-01-09 10:22:12 +08:00
这问题大公司小公司都有
你就算去了 Google/Microsoft,如果要维护一个有好几年甚至十几年历史的产品,你会更抓狂。
zr8657
2019-01-09 10:35:08 +08:00
没事别给自己填堵,有这功夫不如去造轮子。你想想为什么同一个文件里会有“ getUser、getUser1、getUserLsq ”?
liaoguiming
2019-01-09 10:42:40 +08:00
如有机会(新需求,新项目),不影响他人的情况下,自己按照自己的方式写就行了。

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

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

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

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

© 2021 V2EX