新公司，代码审查的时候 leader 修改了一些我个人觉得真的没必要的地方

[fyxtc]

for k, v in pairs(icons) do （一些调用） end

for 里几行代码，然后把 v 改成了 item。。。。改成 icon 我也服气一点啊。。。（这里的 kv 真的是习惯了，如果是 python 的话我就会写 for icon in icons:）

一个初始化图片 uri 的方法名：initSkinImage => initSkin.......

随机显示一些按钮的文本方法名：randSelectLabel => randomOptions

一些变量名： GROUP_INDEX = "group_index" => GROUP_INDEX = "gi" USER_STATUS_DATA = ” user_status_data" => USER_STATUS_DATA = "u_sd"

一个调用我是拆开的 a.do1() a.do2() => a.do1().do2()

更新用户状态方法名 updateUserRes => updateUserResponse

最不能忍的是这个。 image.setVisible(true) => image.show()

show 的内部实现就是调用 setVisible,两个方法都是框架自带的 照这种改法，现有代码里的所有 setVisible 调用处都该改了

其他有些地方是命名是修改得好的，比如 title => titles, enterStartView => enterReadyView

但是像上面那些的真的有点不太能接受。。。唉

[acidsweet]

代码是给机器运行，但是却是给人读的

[likuku]

@fyxtc 缩写...容易歧义。“随机”，你用了缩写 rand，但这个也是 “渲染” 的意思，被改为 random 很合理。

[felinx]

是他直接改你代码还是要求按这个改？

如果是前者，那么他不会做 leader 得在学几年怎么当 leader，如果是后者，只不过是统一内部习惯而已，没有很严格意义的对错好坏，适应下就好。

[blankme]

@likuku
渲染是 render

[epkT6QJ3RSaz6AnJ]

5 楼+1

[fyxtc]

@acidsweet 除了 res 那个确实是我的问题，其他的我并不觉得会让人觉得不好读

[githubhaoliu]

updateUserRes 我一般简写为 updateUserRes 可能还是会被你 leader 改掉。。[逃]。。

[githubhaoliu]

updateUserResp

[moshao6]

5 楼+1
适应过程，慢慢就好。

[cppgohan]

感觉楼主提到的一些改动 kv, do1().do2() 有点儿矫枉过正的嫌疑.

Bob 大叔 Clean Code 书中说的, "single letter names should only be used as local variables inside small methods – length of the name should correspond to the size of its scope" 我觉得挺合理的.. 另外 temp 交换值这个应该很不 pythonic 吧, 确实更应该改.

不过我觉得 leader 能重视统一风格是好事, 我工作中遇到的代码, 我倒真希望能有人把风格和命名统一一下, 真的没辙.. 代码基有快 10 年历史吧, 连基础的 tab 和 space 都没统一过.. 个别文件的编码格式也不统一, 更别说代码组织风格了..

[cunkouwdy007]

我觉得吧，更重要的不是来 V 站讨论，而是直接跟你的 leader 讨论，不同公司有不同的处理办法，交流下各自的想法不更好

[yopming]

GROUP_INDEX = "group_index" => GROUP_INDEX = "gi"
updateUserRes => updateUserResponse

这就有点互相矛盾了，到底要缩写还是不要呢

[kzzhr]

送他一本代码大全

[lance6716]

你可能是对“风格问题”的认识太狭义了 比如 SkinImage 词语重复（ skin 当然是图片啊），常量一般用简短的字符串或者数字

[huskar]

觉得不妥的地方和 leader 讨论，提前做好功课：列举出你的写法有什么好处，哪些开源项目使用你的写法等。

leader 能这么细致的 code review，应该也会乐意和你讨论。

如果你只会说“我习惯这样写”或者“这样写也没错”，那就乖乖按 leader 的风格来，为团队改变自己的习惯。

[vincenttone]

1. 改回来
2. 无视
3. 换组
4. 离职

实际上工作中要求修改代码的情况还是有一些的，特别是我放飞自我脑洞大开起名字的时候，一般这种我也懒得管了，这东西见仁见智；而且我自己的代码过几天我自己都不认识，也很可能会被我自己重构掉，所以现在代码的质量肯定不是以后的你认为比较好的；还有就是我一般不会让 leader 帮忙 review，不然会有不必要的调整，或者要分享设计，作为一个打工者，可以选择高调也可以选择 低调，哪些符合你自己的性格又符合你的利益呢？作为工程师所谓的骄傲或者自我意识之类的东西，那些东西实际上都是为了写出更好的代码？还是为了平添对别人的不满或者让自己陷入不利的境地呢？

个人认为如果你的代码写的好，设计也不错，除非 leader 想拿你刷存在感，不然不会改你代码的，这也适用于其他人（或者代码审查和修改程序）。

[imn1]

这个看是什么情况
我以前在公司写 PHP 的时候都是用一些通用的变量名，而不是指代意义的变量名，也不用短变量名
但在家就刚好相反，越短越好
原因就是在公司我写的，八成都是模块，class，function，基本上要给别人调用，或者别人要加入模式，抽象化使用
别人原封不动使用的也有，修改重用的也有。所以……

你这个情况，我感觉公司有一套标准，你不妨问问 leader 是否如此

[nanau2016]

你的 leader 一定是个日本人

[zn]

@yopming
我感觉，GROUP_INDEX = "group_index" => GROUP_INDEX = "gi"
这种改法，很可能是这些常量是做 key 用的，所以短一点能省内存。

[xiaket]

我觉得你的 leader 改得都挺不错的啊... 只有我一个人是这个感觉吗?