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

代码优雅度研究

  •  
  •   lovecy · 2020-11-26 19:38:31 +08:00 · 1411 次点击
    这是一个创建于 1453 天前的主题,其中的信息可能已经有所发展或是发生改变。

    示例的两种写法,哪种更优雅嘞,或者大佬们有更好的写法- -

    前置说明

    首先有一个非常大的关联数组

    $replace_data = array(
        1 => 11,
        3 => 33,
        // 后面还有很多
    );
    

    过滤并替换下面这个二维数组

    $input = array(
        array(1, 0),
        array(1, 2),
        array(2, 0),
        array(3, 1),
        array(3, 4),
    );
    

    对于每一个子数组按以下规则过滤

    1. index=0 的数据如果不在 replace_data 里,丢弃
    2. index=1 的数据如果不为 0,并且不在 replace_data 里,丢弃
    3. 根据 replace_data,修改 index=0 的数据,若 index=1 的数据不为 0,也得修改

    期待的过滤结果

    $output = array(
        array('11', '0'),
        array('33', '11'),
    );
    

    示例代码

    示例代码 1

    $output = array();
    foreach ($input as $v) {
        if (empty($replace_data[$v[0]])) {
            continue;
        }
        if (($v[1] !== 0) && empty($replace_data[$v[1]])) {
            continue;
        }
        $v[0] = $replace_data[$v[0]];
        if ($v[1] !== 0) {
            $v[1] = $replace_data[$v[1]];
        }
        $output[] = $v;
    }
    

    示例代码 2

    $output = array();
    foreach ($input as $v) {
        if (empty($replace_data[$v[0]])) {
            continue;
        }
        if (($v[1] !== 0) && empty($replace_data[$v[1]])) {
            continue;
        }
        $v[0] = $replace_data[$v[0]];
        if ($v[1] !== 0) {
            $v[1] = $replace_data[$v[1]];
        }
        $output[] = $v;
    }
    
    3 条回复    2021-01-15 14:38:05 +08:00
    lovecy
        1
    lovecy  
    OP
       2020-11-26 19:53:12 +08:00
    不好意思,第二个代码放错了,放成一样的了,但是无法 append,不知道为啥。
    两个我感觉都很不优雅,寻求一个更优雅的写法
    v11exboy
        2
    v11exboy  
       2021-01-14 17:52:47 +08:00
    优雅?粗暴就完了!

    $output = array();
    foreach ($input as $value) {
    if (!empty($replace_data[$value[0]]) && (!$value[1] || !empty($replace_data[$value[1]]))) {
    $output[] = array($replace_data[$value[0]], $value[1] ? $replace_data[$value[1]] : 0);
    }
    }
    lovecy
        3
    lovecy  
    OP
       2021-01-15 14:38:05 +08:00
    @v11exboy 见过很多 BUG 都是,if 写太长,后面的判断语句就算写错了有时候也检测不到。。。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   3463 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 11:41 · PVG 19:41 · LAX 03:41 · JFK 06:41
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.