遇到朋友提交的这种代码该怎么办

2015-08-19 16:55:27 +08:00
 aprikyblue

受朋友委托,然后跟另一个朋友一起开发的,
一起开发的这个,之前 2 年没联系,

结果 ........
看完我一句也不想说




12591 次点击
所在节点    程序员
104 条回复
tchekai704
2015-08-20 12:35:56 +08:00
@mzer0 你说的使用结构体进行因果优化, 是指使用 hash/map/dict 之类的数据结构,从而隐藏 if/else 的逻辑吗??
branchzero
2015-08-20 12:36:34 +08:00
撕逼预订。
这是吐槽命名有些凑了拼音么?
而且貌似判断变量的值的代码风格有好多种。
感觉不是一个人写的,或者说他精分了?
O21
2015-08-20 12:52:03 +08:00
矫情。
Clarencep
2015-08-20 13:06:35 +08:00
不能一起愉快玩耍就分吧
ifconfig
2015-08-20 13:19:42 +08:00
楼主你太年轻了,这样的队友算真心好队友了,我维护过几千行无注释数十个 if else 硬塞成 mvc 的项目,那叫一个想死
Jaylee
2015-08-20 13:55:23 +08:00
把你自己的代码贴上来看看
billwang
2015-08-20 14:12:36 +08:00
楼主是在踩着别人秀自己高大上的吗?
yeqiu
2015-08-20 14:49:18 +08:00
楼主要是觉得你的队友不行,请联系我。

@前面说只是编码风格不同的
我们排除易读性和健壮性,单去看看 CLR 就能知道效率差多少了吧。
yeqiu
2015-08-20 14:52:13 +08:00
@yeqi CLR=>CIL
karlakte
2015-08-20 16:11:44 +08:00
首先说 C#应该是驼峰命名法 方法名 /属性名首字母要大写, 然后这么多魔法值 维护起来有点麻烦。代码整洁就看个人了
tuimaochang
2015-08-20 17:32:52 +08:00
要不是你朋友,连注释都不写。
hitmanx
2015-08-20 18:37:09 +08:00
@harry890829 是不是还有效率上的区别,记得 switch case 有些编译器会优化出一个 jump table 啥的,只要计算一次, O (1 )的时间,不排除记错了。。

不过话说,这个注释真是详细,已经很幸福了。。
bk201
2015-08-20 20:01:59 +08:00
一句都不想说你还发帖子说一下。。
代码风格这东西没啥说头,只问你能不能看懂,功能实现没有,代码不要做无用功简洁就 ok 拉,要不楼主 t 贴个自己的代码对比下?
w88975
2015-08-20 20:48:21 +08:00
@init 看注释你就应该知道是游戏了,但是如果是游戏的代码,不可能判断颜色区域的,所以就是外挂了啊
harry890829
2015-08-21 09:38:49 +08:00
@fo2w 你这样就不能好好聊天了……你这么说循环的话,就已经到达质变的情况了,要是只是说 switch case 和 if else 的话,在字符上并没有很大的差别,如果像 @hitmanx 说的那样, switch case 会被编译器优化的话后,那么就确实会有差别了(并没有检查)
kaedea
2015-08-21 10:57:12 +08:00
@theoractice 交出头像
halfcrazy
2015-08-21 11:04:26 +08:00
这种代码风格,让我想起了一个小伙伴
soundofu
2015-08-21 11:20:59 +08:00
@hitmanx 不同平台的编译上确实有区别,例如在 TI DSP 上,如果 CCS 编译,会有一个.switch 区域,执行的时候会接近 O (1 ),此时通过人工修改更高命中率的 case 的次序应该是没有什么效果的。但是 51 平台上用伟福或者 Keil uV2 会出现 switch 执行时间的陷阱,不知还有没有人遇到过...

@fo2w 在嵌入式 MCU 里面确实会有这种情况。如果用 for ,每次赋值之间起码增加了一次 ACC 和一次判断跳转,对于带有流水线并且没有分支预测或乱序执行的 CPU ,执行效率大大降低。况且遇到这种需要批量生成的代码,也是直接 excel 里拖一下,然后 Ctrl+C , Ctrl+V 过去的。
theoractice
2015-08-21 16:59:37 +08:00
@kaedea 我的头像?只是一张手机拍的风景照啊。
tjxjj
2015-08-26 11:18:31 +08:00
都是程序员,何苦难为程序员

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

https://ex.noerr.eu.org/t/214336

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

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

© 2021 V2EX