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

V2EX = way to explore

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

现在注册

已注册用户请登录

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

❮

❯

2018-07-04 09:47:08 +08:00

回复了 yuhuan66666 创建的主题 › 生活 › 感觉自己租房住特别的寂寞

@mengdisheng 老哥近水楼台先得月

2018-07-02 13:56:32 +08:00

回复了 xianfeng09 创建的主题 › 程序员 › 所有伟大的创新，本质上都是从一小撮年轻人肆无忌惮打破规则开始的

这两年，python 的“传销”和你们是同一拨人吧。。。。

2018-07-02 09:31:29 +08:00

回复了 lanmingyan 创建的主题 › 程序员 › 大家有什么简单的办法同步两台电脑间的代码呢

emmmm 我用的是 dukto，跨平台的传输工具，很方便

2018-06-29 18:44:17 +08:00

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

@ofooo 没有硬撑啊哈哈，res 是不好，已认识到
@zhea55
@akira
@fuxiaohei

反驳 show 的观点，请见最后一条附言

2018-06-29 15:19:49 +08:00

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

@islandev 我觉得老大人也很好，是很 nice 的人，有点地方改的也很好。因为我自己也是重视命名的，所以 review 的话我还是更偏向希望能看到一些建设性的修改，比如性能优化或者隐藏 bug 这样的

2018-06-29 14:48:10 +08:00

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

@yizheshiyang emmm，声明一下，这里的 show 也是要调用到 c++，可以看#18，而且你说的这句话说服力不是很大，首先 show 并不明显更好，按照你的语言表述“设置可见”个人觉得明显是前者更符合这一唯一目的，而且还能形成对比，暗示之前是 false 的。这么说吧，等你需要碰到类似 image.setVislble(isFin)这样的情况，你应该就能理解我了（写过 cocos 的可能更容易理解我的表述）。再考虑到的是团队中已经存在大量的 setVisible(true/false)了，所以。。。。我说这个是见仁见智的问题，真的完全属于无关痛痒且没有任何实质差别

2018-06-29 11:51:04 +08:00

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

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

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 朋友。。。你的链接多了一个逗号哦最后。。。