PHP：用算法简化？

Question

这是一段我认为不是很“漂亮”的 PHP 代码，我确信可以用 for 或其他东西来简化它。我正在尝试找到适用于此的算法，但我无法弄清楚，请帮助我。

这是代码：

if(isset($four))
{
 if(isset($navi[$one][$two][$three][$four])) echo "/content/" . $one . "/" . $two . "/" . $three . "/" .$four . ".php";
 else echo "error";
}
else if(isset($three))
{
 if(isset($navi[$one][$two][$three]))  echo "/content/" . $one . "/" . $two . "/" . $three . ".php";
 else echo "error";
}
else if(isset($two))
{
 if(isset($navi[$one][$two])) echo "/content/" . $one . "/" . $two . ".php";
 else echo "error";
}
else if(isset($one))
{
 if(isset($navi[$one]))echo "/content/" . $one . ".php";
 else echo "error";
}
else
{
 echo "error";
}

谢谢！

Answer 1

更新、测试：

$parts = array($one, $two, $three, $four);
$reversed = array_reverse($parts);

function getPath($ret, $n) {
    global $parts;
    foreach (range(0, $n) as $i) {
        $ret = $ret[$parts[$i]];
    }
    return $ret;
}

$error = false;
foreach (range(0, count($reversed)) as $i) {
    if (!$reversed[$i]) {
        unset($reversed[$i]);
        continue;
    }

    if (!getPath($navi, count($parts) - $i - 1)) {
        $error = true;
        break;
    }
}

if ($error) {
    echo "error!";
} else {
    echo "/content/" . implode("/", array_reverse($reversed)) . ".php";
}

Answer 2

问题是您正在使用 isset(foo)，这使得很难将变量放入数组中。如果测试变量的长度就可以了，那么使用：

$parts = array_reverse(array($one,$two,$three,$four));
foreach ($parts as $i => $value) 
    if(strlen($value)==0) 
        unset($array[$i]);
$final = join('/',parts);
if(isset($navi[$final])) echo "/content/" . $final . ".php";
else echo "\nerror\n\n\n";

但这需要您将 $navi 构造更改为看起来像

$navi['foo/bar/baz'] = "someval";

而不是

$navi['foo']['bar']['baz'] = "someval";

因为我们将使用连接字符串 $final 来查找$isset。否则唯一的路线是动态生成的 php，这是可怕的。

您能否以不同的方式构建数据？循环获取数组维度并不好，而上面的平面列表更容易完成。

Answer 3

我认为您的代码存在更深层次的问题。但为了解决这个问题——我的猜测是：

$urls = array();
$tempNavi = $navi;
foreach (array('one', 'two', 'three', 'four') as $var) {
    if (!isset($var) || !isset($tempNavi[$var]))
        break;
    $tempNavi = $tempNavi[$var];
    $urls[] = $var;
}

if ($urls) {
    echo '/content/' . implode('/', $urls);
} else {
    echo 'error';
}

Answer 4

为了完整起见，递归解决方案：

function navi_recurse(&$navi, &$steps, $i = 0) {
  if ($i < count($steps) - 1) {
    $step = $steps[$i];
    if ( isset($navi[$step]) )
      return navi_recurse($navi[$step], $steps, $i+1);
    else
      return "error\n";
  } 
  return '/content/'.implode('/', $steps).'.php';
}

像这样调用：

$steps = array($one, $two, $three, $four);
echo navi_recurse($navi, $steps);