V2EX › freefcw 的所有回复 › 第 25 页 / 共 114 页

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

1 ... 21 22 23 24 25 26 27 28 29 30 ... 114

❮

❯

2014-12-17 10:30:37 +08:00

回复了 niuer 创建的主题 › 程序员 › 使用 12-factor 的理念快速创建高可维护性的应用—叶峰（ 1213 开发者实践日）

这是机器翻译的吧。。怎么这么生硬

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-02 22:50:52 +08:00

回复了 goofansu 创建的主题 › 问与答 › 升级到 Yosemite 的同学，请问你们修改/etc/hosts 以后有效果吗？

一直都很正常，不知道楼主什么地方出问题了

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里面

最后那个正则，你自己去读一下看看一次能读懂意思么

2014-11-20 11:44:59 +08:00

回复了 atao 创建的主题 › macOS › Mac Appstore 也开始有 1 元应用了，不过数量极少，发现新的请来此分享

我也买了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……北京联通没反应