V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX  ›  freefcw  ›  全部回复第 25 页 / 共 113 页
回复总数  2260
1 ... 21  22  23  24  25  26  27  28  29  30 ... 113  
2014-12-09 18:08:59 +08:00
回复了 BInaryTree111 创建的主题 程序员 可能是最好吃的一场马拉松
五花肉你好,五花肉再见

替我向sunny问好,顺带求卫衣
2014-12-05 13:56:53 +08:00
回复了 yangguang31466 创建的主题 分享创造 [公测] 一款不出名体验还行的 chrome 扩展 Infinity new tab
装了一个,不错,非常的赞,特别是把扩展也加入管理了
2014-12-05 13:53:32 +08:00
回复了 DearTanker 创建的主题 问与答 这个社区采用什么程序?是开源的吗? http://club.fir.im/
应该是自己写的,nodejs + express + mongodb实现的
一直都很正常,不知道楼主什么地方出问题了
2014-12-01 17:57:59 +08:00
回复了 Sunyanzi 创建的主题 PHP 好像不自觉就写了难以维护的代码 ...
@Sunyanzi


最近在读《代码整洁之道》,其实正好符合你的这个问题,建议对照着核对一下
2014-12-01 17:23:46 +08:00
回复了 Sunyanzi 创建的主题 PHP 好像不自觉就写了难以维护的代码 ...
@Sunyanzi

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


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

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

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

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

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

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

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

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

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


最后在说一下

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

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

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

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

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

最后那个正则,你自己去读一下看看一次能读懂意思么
我也买了1元的manico,用不到,纯粹是支持作者……
2014-11-10 14:34:32 +08:00
回复了 touzi 创建的主题 4G 联通网上营业厅被挤爆了...
i cannot say any more
2014-11-09 00:40:48 +08:00
回复了 passluo 创建的主题 分享发现 皮皮书屋(ppurl)将在 2014 年 11 月 12 日停止服务
wish~~谢谢pp
2014-11-08 15:45:20 +08:00
回复了 xoxo 创建的主题 分享发现 闲得无聊, 再来败败家, 送 700MB 流量, 留下手机号即送
omg……北京联通没反应
2014-11-08 14:34:06 +08:00
回复了 c4pt0r 创建的主题 程序员 开源了一个分布式 Redis 解决方案, 求 Star & Fork
@c4pt0r 我可是晚到的113号会员……
2014-11-08 13:56:41 +08:00
回复了 c4pt0r 创建的主题 程序员 开源了一个分布式 Redis 解决方案, 求 Star & Fork
赞黄大仙
终于找到解决问题的办法了,看来12款的显卡性能还是弱了
2014-11-02 00:18:38 +08:00
回复了 baozijun 创建的主题 Android 为什么某宝上的 N5 如此便宜,但看评论又不太像假货,求解惑.
你真的下单的时候还只是1200么。。。至少也是1600+了
1 ... 21  22  23  24  25  26  27  28  29  30 ... 113  
关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2542 人在线   最高记录 6679   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 40ms · UTC 15:34 · PVG 23:34 · LAX 08:34 · JFK 11:34
Developed with CodeLauncher
♥ Do have faith in what you're doing.