-1

我正在尝试在函数中创建一个全局变量,但是当我尝试在函数外部回显时它没有传递。

function check_input($data)
{
        if ( preg_match("/http/i", $data)) {$GLOBALS['spam'] = 'yes'; 
}

check_input($data);
echo $spam;
echo $GLOBALS['spam'];
4

3 回答 3

2

正确的做法是return从函数中取出值,而不是依赖全局变量。

function check_input($data)
{
    //Note the use of true instead of "yes". 
    //You can do more stuff with true/false.
    if ( preg_match("/http/i", $data)) { return true; }
    else { return false; }
}

$is_spam = check_input($data);
echo $is_spam; //1 or 0, because that's how true and false display in echo.

另请参阅:为什么全局状态如此邪恶?

于 2013-07-21T18:51:40.737 回答
0

您的代码的问题是您有语法错误:

function check_input($data)
{
        if ( preg_match("/http/i", $data)) { $GLOBALS['spam'] = 'yes'; } //Note the } I've added.
}

check_input($data);
echo $spam;
echo $GLOBALS['spam'];

应该做得很好。

于 2013-07-21T18:52:54.193 回答
0

您可以为此使用 global 关键字

function check_input($data)
{
    global $spam;
    if ( preg_match("/http/i", $data)) $spam = 'yes';
}
$data = "http://www.example.com";
check_input($data);
echo $spam;
echo $GLOBALS['spam'];

但问题是您为什么要这样做,更好的方法是将 $spam 通过引用传递给函数或将其作为返回变量返回。

function check_input($data, &$spam)
{
    if ( preg_match("/http/i", $data)) $spam = 'yes';
}
$data = "http://www.example.com";
$spam = "no";
check_input($data, $spam);
echo $spam;
echo $GLOBALS['spam'];

或者

function check_input($data)
{

    return ( preg_match("/http/i", $data))?'yes':'no';
}
$data = "http://www.example.com";
$spam = check_input($data);
echo $spam;
echo $GLOBALS['spam'];
于 2013-07-21T18:57:32.940 回答