0

可能重复:
使用“,”内爆数组并在最后一项之前添加“和”

我正在自定义一个 WordPress 主题,并且我在管理员的用户个人资料页面中创建了一些复选框,确切地说是 3 个。当用户选中每个框时,该框的值会保存在一个名为 $goals 的数组中,当他们保存他们的个人资料时。这一切都很好。

在模板页面上,我正在对如何根据用户选择的目标显示某些内容进行一些测试,因此我创建了以下代码来检查数组是否包含它自己的每个值,然后将目标组合在一起,然后根据这些选择回显特定内容。它工作得很好,但我觉得代码臃肿并且可以简化。这是我一直希望用我的代码做的事情。我非常感谢您对如何用更少的代码实现相同结果的想法。谢谢。

$goals = get_user_meta( $userID, 'goals', $single );

if (in_array('Weight Loss', $goals, true) && !in_array('Improve Fitness', $goals, true) && !in_array('Improve Health', $goals, true)) {
    echo 'Weight Loss';
} elseif (in_array('Improve Fitness', $goals, true) && !in_array('Weight Loss', $goals, true) && !in_array('Improve Health', $goals, true)) {
    echo 'Improve Fitness';
} elseif (in_array('Improve Health', $goals, true) && !in_array('Improve Fitness', $goals, true) && !in_array('Weight Loss', $goals, true)) {
    echo 'Improve Health';
} elseif (in_array('Weight Loss', $goals, true) && in_array('Improve Fitness', $goals, true) && !in_array('Improve Health', $goals, true)) {
    echo 'Weight Loss and Improve Fitness';
} elseif (in_array('Weight Loss', $goals, true) && !in_array('Improve Fitness', $goals, true) && in_array('Improve Health', $goals, true)) {
    echo 'Weight Loss and Improve Health';
} elseif (!in_array('Weight Loss', $goals, true) && in_array('Improve Fitness', $goals, true) && in_array('Improve Health', $goals, true)) {
    echo 'Improve Fitness and Improve Health';
} elseif (in_array('Weight Loss', $goals, true) && in_array('Improve Fitness', $goals, true) && in_array('Improve Health', $goals, true)) {
    echo 'Weight Loss, Improve Fitness and Improve Health';
} else {
    echo 'Nothing set';
}
4

2 回答 2

2

以人类可读的形式表示目标列表:

if (count($goals) > 1) {
    echo join(', ', array_slice($goals, 0, -1)) . ' and ' . end($goals);
} elseif ($goals) {
    echo $goals[0];
} else {
    echo "Nothing set";
}

根据您拥有的项目数量,代码将返回"X"/ "X and Y"/ "X, Y and Z"

如果有多个项目,它会执行以下操作:

  1. 将项目添加到但不包括最后一个项目 - usingarray_slice()并在它们之间放置一个逗号 - using join()
  2. 添加最后一项的单词" and "和值。

要将数组缩小$goals到您提到的那些项目:

$goals = array_intersect(array('Weight Loss', 'Improve Fitness', 'Improve Health'), $goals);

它会删除除您感兴趣的值之外的所有值,在您的情况下只有三个值。根据您的情况,您可能不需要此代码。

于 2012-10-03T08:42:58.033 回答
0

如果您不想像示例所建议的那样简单地连接字符串,则该代码的更简化版本将如下所示:

sort($goals);

switch ($goals) {
    case array('Weight Loss') :
        ...
        break;

    case array('Improve Fitness', 'Weight Loss') :
        ...
        break;

    ...
}

这里重要的是两个数组中的元素顺序相同,因此初始sort元素和比较元素按字母顺序排列。

话虽如此,必须有一种比处理所有可能的排列更好的方法来处理这个问题,因为随着您添加更多选项,它会呈指数增长。您通常应该遍历目标集合,为每个目标做一些事情,然后输出一些汇总值:

$output = array();

foreach ($goals as $goal) {
    switch ($goal) {
        case 'Weight Loss' :
            $output[] = 'I want to lose weight.';
            break;

        case 'Improve Fitness' :
            $output[] = ...
            break;

        ...
    }
}

echo join("\n", $output);

使用一个非常明智的基于 OOP 的结构,其中每个目标都被定义为一个单独的对象,这看起来更像:

$output = array();

foreach ($goals as $goal) {
    $output[] = $goal->objectiveDescription();
}

echo join("\n", $output);

换句话说,你解决问题的方式是不够的。在不知道您实际尝试做什么的情况下,很难提出建议。

于 2012-10-04T01:56:16.923 回答