这段代码要怎么简写呢? PHPMD 提示需要那个 else 的

2015-12-11 12:13:25 +08:00
 Nixus

1936 次点击
所在节点    PHP
15 条回复
qiayue
2015-12-11 12:22:52 +08:00
提示说不需要 else ,你可以简化代码去掉 else
Nixus
2015-12-11 12:23:54 +08:00
我知道,就是没想到简化怎么写
Nixus
2015-12-11 12:24:07 +08:00
@qiayue 没想到简化怎么写
shiye515
2015-12-11 12:26:27 +08:00
在 if 里 return 就不需要 else 了
Nixus
2015-12-11 12:36:33 +08:00
@shiye515 这里是在 foreach 中的,而且$links['img']也是一个数组,如果 return 了,数据就不完整了,所里,这里是无论如何都不能 return 的
同时, else 也无法提到 if 之前,毕竟是$links['img']和$links['txt']是两个完全不一样的数组!
Nixus
2015-12-11 12:41:03 +08:00
if 中加入 continue
然后删除 else ,直接把 else 中得代码暴露出来
Nixus
2015-12-11 12:41:43 +08:00
@shiye515 感谢提醒!
500miles
2015-12-11 12:53:46 +08:00
这个场景 真没必要 = =.
Nixus
2015-12-11 13:11:24 +08:00
@500miles 我只是想知道,简化的写法是什么样子的,因为我看到这个提示的时候,想了好久,都没有想到该如何简化
qiayue
2015-12-11 13:28:01 +08:00
@Nixus 你 6 楼自己说出来了,就是 continue
500miles
2015-12-11 13:51:07 +08:00
@Nixus 用的 sublime 么? PHPMD 的插件?
Nixus
2015-12-11 14:05:34 +08:00
@qiayue 真的呀?哈哈,谢谢
@shiye515 感谢提醒!
@500miles 编辑器是 Atom ,用的也是 Atom 的插件
that24
2015-12-12 09:00:53 +08:00
楼主, ATOM 写 PHP 好使不?感觉界面挺漂亮,于是自己尝试了一下感觉很不好用啊,命名空间、类这些东西都不提示,跳转也不好使
Nixus
2015-12-12 09:56:55 +08:00
@that24 需要插件!我用了 4 年 vim 了,感觉 vim 很折腾。后来发现了 Atom ,就一直使用 Atom 了。 Atom 上的插件很丰富,比 vim 安装,方便很多,而且没有 vim 那么折腾!
你需要提示,可以安装插件来实现, autocomplete-php 之类的,插件很丰富,应该有你需要的功能
klgd
2015-12-12 16:48:30 +08:00
@Nixus 我觉得没什么可简化的了,你使用 continue 和 else 有什么区别吗?简化哪里了? 没必要尽信那个什么 phpmd (我没用过)

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

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

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

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

© 2021 V2EX