11

我有这个 php 脚本：

function hoeveelzijner ($jaar, $id)
{
            function hoeveelhoeveel($beginstamp, $endstamp, $id)
            {
                $dates = mysql_query('SELECT v_date FROM visitors WHERE id="'.$id.'" AND v_date<"'.$endstamp.'" AND v_date>"'.$beginstamp.'"');
                return mysql_num_rows($dates);
            }
$i = 1;
while ($i < 13)
{
    $hoeveel[$i-1] = hoeveelhoeveel(mktime(0, 0, 0, $i, 1, $jaar),mktime(0, 0, 0, $i, cal_days_in_month(CAL_GREGORIAN,$i,$jaar),$jaar),$id);
    $i = $i+1;
}
return $hoeveel;
}

当我把它放在它下面时，它工作得很好：

$values = hoeveelzijner(2005, 1);

但是，当我这样做两次时，例如：

$values = hoeveelzijner(2005, 1);
$test = hoeveelzijner(2000, 4);

我收到此错误：致命错误：无法在第 69 行的 ... 中重新声明 hoeveelhoeveel()（之前在 ...:69 中声明）。

任何人都知道我做错了什么？如果我只能使用一次，它有点破坏使用函数的目的......

额外信息：我没有包含任何其他文件，也没有在脚本的其他地方重新声明该函数。

非常感谢！

4

3 回答 3

17

你不能只使用一次；你只能声明一次。每次hoeveelzijner使用该函数时，都会声明该函数hoeveelhoeveel。如果你不止一次调用它，你会重新声明它，这是被禁止的。

您有两种选择：第一种是将函数定义置于包含函数之外。该函数将在第一次解析文件时声明，然后重复使用。如果要将函数定义限制在特定范围内（原则上是个好主意），可以使用 PHP 5.3 中引入的匿名函数语法：

function hoeveelzigner($jaar, $id)
{
    $hoeveelhoeveel = function($beginstamp, $endstamp, $id)
    {
            $dates = mysql_query('SELECT v_date FROM visitors WHERE id="'.$id.'" AND v_date<"'.$endstamp.'" AND v_date>"'.$beginstamp.'"');
            return mysql_num_rows($dates);
    };

    // etc
}

然后，您可以将该函数用作$hoeveelhoeveel(...).

于 2011-09-25T11:55:30.323 回答
0

尽管 PHP 让您在任何地方声明一个函数，但我认为您所做的并不是最佳实践。

对于大多数人来说，它看起来是错误的。

您应该只在父函数之外声明该函数。

或者，您可以function_exists() 在内部函数周围添加一个，尽管我再次将内部函数声明在父函数之外。

甚至可能为它上课。

于 2011-09-25T11:55:48.753 回答
0

您不能再次声明它，特别是如果该函数多次调用它。这是一个应该有效的简单更改。

function hoeveelhoeveel($beginstamp, $endstamp, $id)

    $dates = mysql_query('SELECT v_date FROM visitors WHERE id="'.$id.'" AND v_date<"'.$endstamp.'" AND v_date>"'.$beginstamp.'"');
    return mysql_num_rows($dates);
}

function hoeveelzijner ($jaar, $id)
{
    $i = 1;
    while ($i < 13)
    {
        $hoeveel[$i-1] = hoeveelhoeveel(mktime(0, 0, 0, $i, 1, $jaar),mktime(0, 0, 0, $i, cal_days_in_month(CAL_GREGORIAN,$i,$jaar),$jaar),$id);
        $i = $i+1;
    }
    return $hoeveel;
}

然后，如果需要，您也可以在其他函数中以类似的方式调用任一函数。

于 2011-09-25T12:43:47.650 回答