写了一个区间段修改的逻辑,这个注释能打几分!

2018-08-31 17:20:23 +08:00
 DavidNineRoc

在写一个日期不同而价格不同的数据库设计,用每日一条记录会简单一点。 现在用的是时间段的。最小单位是一天。如果当天临时变,采用日记记录表。


代码有点多,换成贴图,格式好看一点。

因为图片太长,微博会压缩,文末是超长原图

原图连接 微博的图片会被压缩 https://i.loli.net/2018/08/31/5b88fbf9b9f8e.png

4187 次点击
所在节点    PHP
39 条回复
b821025551b
2018-08-31 17:24:57 +08:00
0 分,10 行代码能解决掉掉问题写了 600 多行。
DavidNineRoc
2018-08-31 17:27:10 +08:00
@b821025551b 请指教。
iblislsy
2018-08-31 17:28:33 +08:00
贴代码的正确姿势捏
jrient
2018-08-31 17:29:28 +08:00
return 0;
php01
2018-08-31 17:29:47 +08:00
一句话,我喜欢别人这样写注释。但是我不喜欢这样写注释
ipwx
2018-08-31 17:40:16 +08:00
注释太多辣,本来一眼扫过去能看清楚的逻辑愣是被拆开来了
mcfog
2018-08-31 17:40:57 +08:00
大概 40%注释是自动生成了,另外 40%左右类似

//下面我要装逼了
装逼();
Rizio
2018-08-31 17:42:02 +08:00
@php01 +1
DavidNineRoc
2018-08-31 17:43:25 +08:00
@iblislsy 直接贴,换行不好控制
@jrient 不应该是 exit(0); >_<
@php01 有些地方不写多一点注释,害怕自己一个月后再回来看也懵逼
@ipwx 可能是自己写的,然后觉得正好把区分的地方写逻辑。
ljspython
2018-08-31 17:43:43 +08:00
喜欢看有这样注释的代码,但是别指望我写这样的注释
DavidNineRoc
2018-08-31 17:44:50 +08:00
@mcfog IDE 自动生成不讨论,主要是用来代码提示。 我说的注释是画图 >_<
lxy42
2018-08-31 17:54:48 +08:00
注释太多了,有些注释略显多余。
这些注释看起来都是在介绍业务逻辑,你不觉得维护起来很麻烦吗?我个人倾向于在文档中描述清楚业务逻辑,在注释中稍微注明一些“坑”即可。
iblislsy
2018-08-31 17:55:43 +08:00
@DavidNineRoc 有很多贴代码的链接...找一个帖进去,会生成一个链接
lxy42
2018-08-31 17:58:18 +08:00
还有一点就像#6 说的,有些简单的操作其实一眼就能通过名字猜出含义,但是被大量注释分割开了。
DavidNineRoc
2018-08-31 17:59:51 +08:00
@ljspython 哈哈,果然是 PHP 老手
@lxy42 因为以前接手过一个老旧项目,看一点代码去找文档,看一点去找文档维护才是真的心累。其他地方的代码基本就写一点坑,就这个控制器多一点。
@iblislsy gist?
iblislsy
2018-08-31 18:01:35 +08:00
cexy
2018-08-31 18:06:41 +08:00
php 天下第一,不做 php 好多年。。。手动滑稽
sagaxu
2018-08-31 18:07:54 +08:00
零分,又啰嗦又没用
UIXX
2018-08-31 18:21:30 +08:00
一些复杂的图形算法是这样写注释的。但是这个...我觉得不是很适合这样写。有点过度注释,效果可能并不是太好。
说到底,你说的是业务逻辑,条件分支多。最适合的改代码方式就是理解了业务逻辑后再看代码,而不是边看代码边捋业务逻辑。
如果是这样更好:
1、业务逻辑单独描述,不会给人概念断层的感觉。
2、函数内变量名称与业务逻辑描述有所对应
3、有意识地格式化一下代码,用代码结构来唤起人对于其中业务的建模
eluotao
2018-08-31 19:37:43 +08:00
这个长图用什么插件截的.能否告知.

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

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

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

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

© 2021 V2EX