对于这些嵌套的 if/elseif 语句，更优雅的解决方案是什么？

Question

我正在构建一个包含具有用户配置文件的用户的网站。配置文件中的许多字段都是可选的。

存在大量用户生成内容的机会，因此我需要在网站的许多不同位置（评论、帖子等）显示该内容的作者。在用户的个人资料中，他能够（可选）填写他的“名字”、“姓氏”和“显示名称”。

为了显示作者，我编写了一个辅助方法，该方法查看提供的这些字段的数组，并按照以下优先顺序返回最适合用户的名称：

1. 如果用户填写了 display_name，则此将显示。
2. 如果用户填写了first_name和last_name，但没有填写display_name，则会显示两个名字
3. 如果用户只填写了first_name< /code>，它将显示first_name。
4. 如果用户只填写last_name，则会显示last_name。
5. 如果所有其他方法都失败，将显示用户 ID，即 user123
6. 如果没有任何数组键存在，或者参数为 NULL，则名称将显示为 NULL

该方法效果很好，但很丑。必须有一种方法可以用嵌套 if/else 语句的替代方案来美化它。

public function nameify($names = NULL) {
    $name = '';
    if (!empty($names)) {
        if (!empty($names['display_name'])) {
            $name = $names['display_name'];
        } elseif (!empty($names['first_name'])) {
            $name = $names['first_name'];
            if (!empty($names['last_name'])) {
                $name .= ' ' . $names['last_name'];
            }
        } elseif (!empty($names['last_name'])) {
            $name = $names['last_name'];
        }

        if (empty($name) && !empty($names['id'])) {
            $name = 'user' . $names['id'];
        } else {
            $name = 'NULL';
        }
    } else {
        $name = 'NULL';
    }
    return $name;
}

Answer 1

public function nameify($names = NULL) {
    if ($names) {
        if (!empty($names['display_name'])) {
            return $names['display_name'];
        }
        if (!empty($names['first_name'])) {
            $name = $names['first_name'];
        } 
        if (!empty($names['last_name'])) {
            $name .= ' ' . $names['last_name'];
        }
        if (empty($name) && !empty($names['id'])) {
            $name = 'user' . $names['id'];
        }
    }
    return $name ? ltrim($name) : 'NULL';
}

首先设置默认值，如果没有其他匹配，则返回默认值。然后，因为我们总是希望返回显示名称（如果我们这样做的话）。

编辑：调整以防止返回“NULL”

Answer 2

使用三元条件我们可以缩短和美化代码：

public function nameify($names = NULL) {
    $name = 'NULL';

    if (!empty($names)) {

        $name = ($names['display_name']) ? $names['display_name'] : trim($names['first_name']." ".$names['last_name']);

        if(!$name) $name = ($names['id'] > 0) ? 'user'.$names['id'] : 'NULL';
    }

    return $name;
}

Answer 3

我建议这样：

public function nameify($names = null) {
    if(empty($names))
        return null;

    if(!empty($names['display_name']))
        return $names['display_name'];

    if(!empty($names['first_name'])) {
        $name = $names['first_name'];
        if (!empty($names['last_name'])) {
            $name .= ' ' . $names['last_name'];
        }
        return $name;
    }

    if(!empty($names['id]))
        return 'user' . $names['id'];

    return null;
}

Answer 4

虽然不多，但是因为 $name 它至少是 NULL：

public function nameify($names = NULL) {
    $name = 'NULL';
    if (!empty($names)) {
        if (!empty($names['display_name'])) {
            $name = $names['display_name'];
        } elseif (!empty($names['first_name'])) {
            $name = $names['first_name'];
            if (!empty($names['last_name'])) {
                $name .= ' ' . $names['last_name'];
            }
        } elseif (!empty($names['last_name'])) {
            $name = $names['last_name'];
        }

        if ($name=='NULL' && !empty($names['id'])) {
            $name = 'user' . $names['id'];
        } 
    } 
    return $name;
}

Answer 5

//pointers to functions
$arrayOfSulutions{"display_name_strategy", "full_name_strategy" ..., "null_strategy" } 
function display_name_strategy{
     return $names['display_name'];
}
$i = 0;
while($res == null){
     $res = call($arrayOfSulutions[$i++]);
}

Answer 6

可读性稍差，但有效）：

list($idx,$name) = array_shift(array_filter(array(
    $names['display_name'],
    implode(' ',array_filter(array($names['first_name'],$names['last_name']))),
    'user'.$names['id'];
    )));

Answer 7

状态机非常适合这样的复杂逻辑。实现起来也非常简单（使用 switch 语句）。

Answer 8

我不确定我的版本会更简单，但这里是：

public function nameify($names = null) {
    $result = array();

    if( !empty($names['display_name']) ) {
        array_push($result,$names['display_name']);
    } else {
        if( !empty($names['first_name']) ) {
            array_push($result, $names['first_name']);
        }
        if( !empty($names['last_name']) ) {
            array_push($result, $names['last_name']);
        }
    }

    if( empty($result) && !empty($names['id']) ) {
        array_push($result, 'user'.$names['id']);
    }

    return (empty($result) ? 'NULL' : implode(' ', $result));
}

Answer 9

我会同意：

if( empty($names['display_name']) ) {
    $name = $names['first_name'] . ' ' $names['last_name'];
else
    $name = $names['display_name'];

$name = trim($name);
if( empty($name) ) 
    $name = 'user'.$names['id'];
if( empty($name) ) 
    $name = 'NULL';

这将是“核心逻辑”...需要进行其他检查，例如 $names != NULL 或其他检查。