V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
Nixus
V2EX  ›  PHP

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

  •  
  •   Nixus · 2015-12-11 12:13:25 +08:00 · 2001 次点击
    这是一个创建于 3266 天前的主题,其中的信息可能已经有所发展或是发生改变。

    E83B11B7-6C13-42F3-98AA-92FDEFA9A62A.png

    15 条回复    2015-12-12 16:48:30 +08:00
    qiayue
        1
    qiayue  
       2015-12-11 12:22:52 +08:00
    提示说不需要 else ,你可以简化代码去掉 else
    Nixus
        2
    Nixus  
    OP
       2015-12-11 12:23:54 +08:00
    我知道,就是没想到简化怎么写
    Nixus
        3
    Nixus  
    OP
       2015-12-11 12:24:07 +08:00
    @qiayue 没想到简化怎么写
    shiye515
        4
    shiye515  
       2015-12-11 12:26:27 +08:00 via Android   ❤️ 1
    在 if 里 return 就不需要 else 了
    Nixus
        5
    Nixus  
    OP
       2015-12-11 12:36:33 +08:00
    @shiye515 这里是在 foreach 中的,而且$links['img']也是一个数组,如果 return 了,数据就不完整了,所里,这里是无论如何都不能 return 的
    同时, else 也无法提到 if 之前,毕竟是$links['img']和$links['txt']是两个完全不一样的数组!
    Nixus
        6
    Nixus  
    OP
       2015-12-11 12:41:03 +08:00
    if 中加入 continue
    然后删除 else ,直接把 else 中得代码暴露出来
    Nixus
        7
    Nixus  
    OP
       2015-12-11 12:41:43 +08:00
    @shiye515 感谢提醒!
    500miles
        8
    500miles  
       2015-12-11 12:53:46 +08:00
    这个场景 真没必要 = =.
    Nixus
        9
    Nixus  
    OP
       2015-12-11 13:11:24 +08:00
    @500miles 我只是想知道,简化的写法是什么样子的,因为我看到这个提示的时候,想了好久,都没有想到该如何简化
    qiayue
        10
    qiayue  
       2015-12-11 13:28:01 +08:00
    @Nixus 你 6 楼自己说出来了,就是 continue
    500miles
        11
    500miles  
       2015-12-11 13:51:07 +08:00
    @Nixus 用的 sublime 么? PHPMD 的插件?
    Nixus
        12
    Nixus  
    OP
       2015-12-11 14:05:34 +08:00
    @qiayue 真的呀?哈哈,谢谢
    @shiye515 感谢提醒!
    @500miles 编辑器是 Atom ,用的也是 Atom 的插件
    that24
        13
    that24  
       2015-12-12 09:00:53 +08:00
    楼主, ATOM 写 PHP 好使不?感觉界面挺漂亮,于是自己尝试了一下感觉很不好用啊,命名空间、类这些东西都不提示,跳转也不好使
    Nixus
        14
    Nixus  
    OP
       2015-12-12 09:56:55 +08:00
    @that24 需要插件!我用了 4 年 vim 了,感觉 vim 很折腾。后来发现了 Atom ,就一直使用 Atom 了。 Atom 上的插件很丰富,比 vim 安装,方便很多,而且没有 vim 那么折腾!
    你需要提示,可以安装插件来实现, autocomplete-php 之类的,插件很丰富,应该有你需要的功能
    klgd
        15
    klgd  
       2015-12-12 16:48:30 +08:00
    @Nixus 我觉得没什么可简化的了,你使用 continue 和 else 有什么区别吗?简化哪里了? 没必要尽信那个什么 phpmd (我没用过)
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1016 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 20:23 · PVG 04:23 · LAX 12:23 · JFK 15:23
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.