V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
V2EX  ›  freefcw  ›  全部回复第 34 页 / 共 122 页
回复总数  2435
1 ... 30  31  32  33  34  35  36  37  38  39 ... 122  
2014 年 12 月 1 日
回复了 Sunyanzi 创建的主题 PHP 好像不自觉就写了难以维护的代码 ...
@Sunyanzi

我真的不知道怎么说了,很有耐心的给你分析了很多


对于两种返回方式分割成两个函数调用,这样代码会更可读(感觉上)。数据结构类似不是放一块的原因。不过是两个不同的方式而已,你就要弄的这么复杂,让人要停下来前后看才能看懂

payinfo提取为一个类,这样可以把很多相关的功能放到一起,不是更清晰明了?
$notify_id这一行拆分,难道不应该放到payinfo这个类里面去?明显是和payinfo有关的信息

unset这是蛋疼事情,如果后续变动怎么办?

ksort的功能应该和md5这个都放到payinfo的一个方法里面么

return里面到底执行了多少个功能呢?典型应该拆分的地方,一句话做了这么多事情
那个http 请求,不应该是一个单独的函数的功能么
那个签名,如果后续变了怎么办?

唉,不能不感慨下,能写出这样的代码的人,我只能说脑容量一定很大,或者是我等智力有所不及……不然怎么会花这么多人绞尽脑汁分析这段代码呢
2014 年 12 月 1 日
回复了 vitozhang 创建的主题 程序员 遇到整个文件就一行的代码怎么处理增强可读性
@vitozhang 等别人的答复比你直接搜索的快?
2014 年 11 月 30 日
回复了 sutking 创建的主题 NGINX sites-available/default 不起作用了什么情况?怎么办?
你确定sites-enabled目录下有这个文件么?如果有你确定是链接到sites-available目录下的这个文件么
2014 年 11 月 29 日
回复了 vitozhang 创建的主题 程序员 遇到整个文件就一行的代码怎么处理增强可读性
很简单直接联想到代码格式化的问题,我觉得楼主应该更多求助搜索引擎

要问也要换一个值得讨论的方式,比如说有什么值得推荐的PHP代码格式化工具
2014 年 11 月 29 日
回复了 Sunyanzi 创建的主题 PHP 好像不自觉就写了难以维护的代码 ...
p.s.

我不知道为什么你要把两个功能放一起。。notify和return,应该分开才清晰

总而言之,你这个函数里面做了太多的事情,直接意义上会认为你是在验证回调,但是还分了两种回调,还去请求了?反正$this->$method[$type]('sign')这种代码。。请问怎么读懂这行代码?还不如一个if else来得清晰易懂
获取notify_id这种低层级的代码。。。
还有一个ssl_request的调用,加上一大堆的字符串拼接。。。这些和verfiy_callback有什么直接关系么??


最后在说一下

"谁如果觉得我这段代码晦涩只能说明他自己 php 水平不过关 ..."

楼主是在玩代码混乱大赛么。。代码除了运行外,更重要也是读的,据说统计结果是90%的时间是在读代码,不知道楼主写这段代码的时候反复读了多少次,想了多少道,纠结了多久代码要怎么改,不过我想既然发出来问,就意味着知道里面肯定有问题
2014 年 11 月 29 日
回复了 Sunyanzi 创建的主题 PHP 好像不自觉就写了难以维护的代码 ...
一行代码之有一个功能,三秒钟能看懂这一行做了什么
一个函数只有一个功能,不混杂要处理更低一层逻辑的代码
2014 年 11 月 29 日
回复了 Sunyanzi 创建的主题 PHP 好像不自觉就写了难以维护的代码 ...
代码太长了,你确定一个函数的功能就是一个?

三目运算符是我极不愿意用的,至少三目运算符只在短小的时候我才会考虑,这么长一个完全就不具备可读性

目测这个函数至少要拆分到3个函数才具备一目了然性

1. $payinfo提取为类A应该是明智的做法
2. notify_id应该放到类A里面
3. 签名payinfo应该也要放到类A里面

最后那个正则,你自己去读一下看看一次能读懂意思么
我也买了1元的manico,用不到,纯粹是支持作者……
2014 年 11 月 10 日
回复了 touzi 创建的主题 4G 联通网上营业厅被挤爆了...
i cannot say any more
2014 年 11 月 9 日
回复了 passluo 创建的主题 分享发现 皮皮书屋(ppurl)将在 2014 年 11 月 12 日停止服务
wish~~谢谢pp
2014 年 11 月 8 日
回复了 xoxo 创建的主题 分享发现 闲得无聊, 再来败败家, 送 700MB 流量, 留下手机号即送
omg……北京联通没反应
2014 年 11 月 8 日
回复了 c4pt0r 创建的主题 程序员 开源了一个分布式 Redis 解决方案, 求 Star & Fork
@c4pt0r 我可是晚到的113号会员……
2014 年 11 月 8 日
回复了 c4pt0r 创建的主题 程序员 开源了一个分布式 Redis 解决方案, 求 Star & Fork
赞黄大仙
终于找到解决问题的办法了,看来12款的显卡性能还是弱了
你真的下单的时候还只是1200么。。。至少也是1600+了
2014 年 10 月 31 日
回复了 zer 创建的主题 Apple 10.10.1 啥时候更新啊
貌似我没遇到这个问题,虽然最开始的时候重新连了一次鼠标
2014 年 10 月 21 日
回复了 P9 创建的主题 macOS 没人觉得升级到 10.10 后 sublime 敲中文一秒蹦一个出来么?
@P9 好忧伤,不输入中文怎么哦日子呢。。。输入法完全跟不上手速
2014 年 10 月 21 日
回复了 P9 创建的主题 macOS 没人觉得升级到 10.10 后 sublime 敲中文一秒蹦一个出来么?
我也遇到了卡顿,不知道为什么,搜狗输入法
2014 年 10 月 18 日
回复了 Bakemono 创建的主题 天黑以后 20141018 午夜俱樂部
10.10下面跑

brew install php55-mcrypt --without-homebrew-php

出错
1 ... 30  31  32  33  34  35  36  37  38  39 ... 122  
About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Solana   ·   4179 Online   Highest 6679   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 37ms · UTC 10:15 · PVG 18:15 · LAX 03:15 · JFK 06:15
♥ Do have faith in what you're doing.