发一个认为自己代码很差很差的

2018-01-12 01:30:36 +08:00
 Annual

RT,主题 https://mi90.com/20180112012844.png
是不是乱的无可救药?

11607 次点击
所在节点    PHP
88 条回复
oaix
2018-01-12 09:07:45 +08:00
不觉得太乱, 挺直白的.
格式有点乱, 行内少了很多空格(比如`=`左右, if 后面 等等), 不是大问题, 用 IDE 的格式化功能格式化一下.
另外 if 只有一个子语句时, 也建议加上大括号.
sql 那段有点, 可能有注入的风险.
另外行间不同功能逻辑其实用空行隔开了, 可以用 IDE 提供的重构功能, 提取成一个函数.
q409195961
2018-01-12 09:10:06 +08:00
金额是不是可以传 0.0091
whx20202
2018-01-12 09:22:00 +08:00
突然觉得 python 真特么好,看代码就跟看文章一样
flyshu
2018-01-12 09:22:59 +08:00
挺好的 原生不都这样写吗
yichinzhu
2018-01-12 09:27:51 +08:00
一眼就看到了 SQL 注入,,,
DavidNineRoc
2018-01-12 09:41:08 +08:00

这样子优雅很多 (逃
建议使用 StyleCI 格式化一下代码,有多种标准格式可以选
v2xe2v
2018-01-12 09:42:43 +08:00
没格式化,难受
xhinliang
2018-01-12 09:45:56 +08:00
前鹅肠员工路过。
在鹅肠见过比这恶心一百倍的代码。
ZXCDFGTYU
2018-01-12 10:17:10 +08:00
这代码真的是能写成这样也是一种奇迹,哪怕用个 tp 也不至于写成这样啊。。
13036101641
2018-01-12 10:18:24 +08:00
一般都有代码规范的插件或者工具吧
carlclone
2018-01-12 10:22:58 +08:00
远古 PHPer 面向过程面条式代码
lbp0200
2018-01-12 10:36:52 +08:00
@yichinzhu +1 SQL 注入是原罪,初级水平。
yujieyu7
2018-01-12 10:39:37 +08:00
在 header 函数前加错误控制符是什么鬼?
killall
2018-01-12 10:44:45 +08:00
这就是传说中的面条代码…不分结构不分分层,所有的东西全都烩到一起,出现问题从头读到尾,数据结构变了得改一堆的文件…业务增长在文件里不停地打补丁,最后知道维护到整个人崩溃。
justfindu
2018-01-12 10:45:14 +08:00
原生也就只能这样写了 格式化一下也许好看一些
lepig
2018-01-12 10:45:30 +08:00
还行 没到吐槽的地步
b821025551b
2018-01-12 10:50:48 +08:00
挺好的了,能看懂,格式化一下逻辑很清晰。
bramblex
2018-01-12 11:16:48 +08:00
@killall 把整个网站写到一个 php 文件里面
3than
2018-01-12 11:24:28 +08:00
@yichinzhu +1 不管程序写的怎么样 但是 sql 注入..
hei1000
2018-01-12 11:26:13 +08:00
我不是稿 php 的不平论,但是最后那两行长度上真实毁了前面

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

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

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

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

© 2021 V2EX