V2EX › fyxtc 的所有回复 › 第 72 页 / 共 80 页

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

1 ... 64 65 66 67 68 69 70 71 72 73 ... 80

❮

❯

2018-06-29 11:35:21 +08:00

回复了 fyxtc 创建的主题 › 程序员 › 新公司，代码审查的时候 leader 修改了一些我个人觉得真的没必要的地方

@smilenceX 那个是常量，我打错了，不好意思
@yadgen 老哥你说的对，很受用，谢谢

2018-06-29 11:33:25 +08:00

回复了 fyxtc 创建的主题 › 程序员 › 新公司，代码审查的时候 leader 修改了一些我个人觉得真的没必要的地方

@smilenceX 不是改名字，是增加一个选择，增加了可以链式调用的方式（ ps:我的例子里不需要链式

function Node:show()
self:setVisible(true)
return self
end

2018-06-29 11:28:00 +08:00

回复了 fyxtc 创建的主题 › 程序员 › 新公司，代码审查的时候 leader 修改了一些我个人觉得真的没必要的地方

@fcten 嗯，这个是我写的不好，按理说缩写也应该是 resp

2018-06-29 11:27:17 +08:00

回复了 fyxtc 创建的主题 › 程序员 › 新公司，代码审查的时候 leader 修改了一些我个人觉得真的没必要的地方

@liyer 你说的点我理解。首先一些函数的命名本来就存在主观的理解，initSkinImage => initSkin 和 randSelectLabel => randomOptions 能说后者的命名风格和前者存在什么区别吗？而且首先代码里就存在的大量的同义调用，比如 setVisble(true)而不是 show，也存在大量的可链式调用的非链式调用。我个人觉得这两个点完全不存在风格问题，不属于风格定义的范畴，而是特意去注意：诶，这个可以这样。

2018-06-29 11:19:38 +08:00

回复了 fyxtc 创建的主题 › 程序员 › 新公司，代码审查的时候 leader 修改了一些我个人觉得真的没必要的地方

@widewing 我可能对自己的代码还是比较看重的吧，也比较在意别人随意动我的代码，如果是写得不好，或者有问题，那当然没话说。关键是，这种类似的问题，原代码就存在，原代码里我还看到了一个冗余的 temp 交换值，直接 a,b = b, a 不就好了。唉。。。算了

2018-06-29 11:10:00 +08:00

回复了 fyxtc 创建的主题 › 程序员 › 新公司，代码审查的时候 leader 修改了一些我个人觉得真的没必要的地方

@yippees leader 人还是不错的，就是在代码命名上有点过于较真了，缩写也要被说一顿：为什么要缩写
我：...... 而且命名风格上不可能做到完全统一的，我个人觉得只要做到表达出了方法的目的，然后阅读代码的人没有太多障碍就可以了。。。唉而且这还是在一周之内提了三次新需求之后我自己已经重构了几次了，还要这样改我的代码命名，心态真的有点崩

2018-06-29 09:23:46 +08:00

回复了 OMGZui 创建的主题 › 程序员 › 我被女同事吐槽不会穿衣搭配

@LeungV2 是我了。。。。哦不。。。我上班也穿拖鞋。。。。🤣

2018-06-29 09:23:12 +08:00

回复了 OMGZui 创建的主题 › 程序员 › 我被女同事吐槽不会穿衣搭配

@LeungV2 是我了。。。。

2018-06-28 09:38:44 +08:00

回复了 jakewww 创建的主题 › Go 编程语言 › golang ide vs vim

@sulinehk vsc 的 vi 插件真的不如 sublime 的。。。。用着总感觉哪里不舒服

2018-06-27 11:41:16 +08:00

回复了 unstop 创建的主题 › 酷工作 › [远程] [苏州] 每天工作四小时，睡到自然醒，深度参与有挑战的项目，体面地挣钱，来加入这家很新很小的公司吧！

@brucezhou 朋友。。。你的链接多了一个逗号哦最后。。。