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

V2EX = way to explore

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

现在注册

已注册用户请登录

这是一个创建于 3306 天前的主题，其中的信息可能已经有所发展或是发生改变。

phpmd

else

代码

提示

15 条回复 • 2015-12-12 16:48:30 +08:00

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 via Android

在 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 （我没用过）