在php函数中使用变量

Question

试图使我的网站更整洁和易于维护，所以我试图将大量重复的代码移入函数中。

我有一个函数，该函数接受ID的参数，在where子句中使用此ID运行数据库检查，然后为返回的rowCount设置新变量。

但是，它始终返回0 /不返回任何值。

我做了一个简单的例子，我正在做什么：

$buildID = 5;

function select_All_Comments_From_ID($buildID){
    $idNew = $buildID;
        global $idNew
}

select_All_Comments_From_ID($buildID);

echo $idNew;

知道为什么会这样吗？

实际上是这样的：

function select_All_Comments_From_ID($buildID){
    $query = " SELECT * FROM comments WHERE buildID = :buildID"; 
    $query_params = array(':buildID' => $buildID); 
    try { 
        $stmt = $db->prepare($query); 
        $result = $stmt->execute($query_params);
        $row = $stmt->fetch();
        $countComments = $stmt->rowCount();
        global $countComments;
    }
    catch(PDOException $ex) { die();
    }
}

然后，我试图使用

$countComments

但是没有运气。

代码本身不在功能中时起作用。

Answer 1

更改

$countComments = $stmt->rowCount();
global $countComments;

在

$GLOBALS['countComments'] = $stmt->rowCount();

顺便说一句，评论是正确的，这是一个糟糕的解决方案，您应该真正对其进行更改。

Answer 2

你有哪怕定义为全局在函数内部函数外定义$ countComments。

范例：

<?php
$a = 1;
$b = 2;

function Sum()
{
    global $a, $b;

    $b = $a + $b;
} 

Sum();
echo $b;
?>

Answer 3

您可以从函数中返回一个值，而不是使用Global，如下所示：

function select_All_Comments_From_ID($buildID){
    $query = " SELECT * FROM comments WHERE buildID = :buildID"; 
    $query_params = array(':buildID' => $buildID); 
    try { 
        $stmt = $db->prepare($query); 
        $stmt->execute($query_params);
        $row = $stmt->fetch();
        $countComments = $stmt->rowCount();
        return $countComments; //sends the value back outside the scope of the function
    }
    catch(PDOException $ex) { die(); }
}

调用函数时，需要一个变量来捕获返回值，如下所示：

$buildID = 5;
$countComments = select_All_Comments_From_ID($buildID);
echo $countComments;

Answer 4

global $idNew仅在执行变量时（而不是之前）影响变量查找。 例如看一下

<?php
$a = 9;
foo();

function foo() {
    $a = 5; // this sets the value for the variable in the local lookup table
    echo $a, "\r\n"; // scope of $a is still local -> 5
    global $a; // now the lookup for "a" is switched to the global scope
    echo $a, "\r\n";  // $a "points" to the global value -> 9
}

输出是

5
9

而你的情况

function select_All_Comments_From_ID($buildID){
    $idNew = $buildID;
        global $idNew
}

该函数将值写入其本地$ idNew，然后对idNew的查找切换到全局范围-但这不会将本地值复制到全局范围； 只是查找已更改。

---

无论如何，如前所述，您实际上应该返回值，而不是全局设置它们。 并且，如果您只关心表中（匹配）记录的数量而不是记录的有效载荷数据，请使用SELECT Count(...)查询。 否则，您的脚本会将所有数据从数据库服务器传输到php实例的内存空间中……除了加热cpu和内存外，别无其他。

function select_All_Comments_From_ID($buildID){
    $query = " SELECT Count(*) as c FROM comments WHERE buildID = :buildID"; 
    $query_params = array(':buildID' => $buildID); 
    try { 
       // what's $db here ?
       // neither has it been passed to the function nor is it fetched from the global scope
        $stmt = $db->prepare($query); 
        $result = $stmt->execute($query_params);
        $row = $stmt->fetch();
        return $row['c'];
    }
    catch(PDOException $ex) { 
      // using die(...) is crude, even more so without any notice/text
      die();
    }
}