Question

这实际上是我的第一个问题。我正处于OOP PHP的学习过程中，并尝试设计我的第一堂课。这是我的代码

<?php
class category{

public  $catid;
public $datacol;

    public function __construct($catid=0)
    {
        $this->catid=0;
        $this->datacol=$this->load($catid);         
    }

    public function load($catid)
    {
        global $conn;
        $sql=' select * '
            .' FROM tbl_categories'
            .' WHERE catid='.$catid;
        if (!($res=mysqli_query($conn,$sql)) || !mysqli_num_rows($res)>0) 
            return false;
        $this->datacol = mysqli_fetch_array($res);
        $this->catid = $this->datacol['catid'];
        return true;
    }

    public function reload() {
        return $this->load($this->getId());
    }

    /* ------------------> Getter methods <--------------------- */
    public function getId() { return $this->catid; }
    public function getName() { return $this->datacol['category']; }
    public function getOneliner() { return $this->datacol['categoryoneliner']; }
    public function getBrief() { return $this->datacol['categorybrief']; }
    public function getThumb() { return $this->datacol['timg']; }
    public function getImage() { return $this->datacol['limg']; }
    public function getParentID() { return $this->datacol['parentcat']; }
    public function getPagetitle() { return  $this->datacol['catpagetitle']; }
    public function getKeywords() { return $this->datacol['catkeywords']; }
    public function getMetadesc() { return $this->datacol['categorymetadesc']; }
    public function getPriority() { return $this->datacol['priority']; }
    public function getRemarks() { return ($this->datacol['remarks']); }
    public function getRow() { return $this->datacol; }

}

?>

现在，当我使用以下代码从其他文件创建其对象时：

$cat=new category(10);
echo var_dump($cat);
echo var_dump($cat->getId());
echo var_dump($cat->getName());

当调用getID（）时，输出为我提供了正确的catid。但是为getName（）或getID（）以外的任何其他函数显示NULL。

我做错了什么？

提前致谢

Answer 1

正如@Arnold Gandarillas所说，你在构造函数中覆盖了$this->datacol。

您的构造函数调用$datacol，它设置load()：

$datacol

然后该函数返回public function load($catid) { ... $this->datacol = mysqli_fetch_array($res); return true; }。在构造函数中，此true已分配给true，覆盖以前的值：

$datacol

将构造函数更改为：

public function __construct($catid=0)
{
    $this->catid=0;
    $this->datacol=$this->load($catid);         
}

Answer 2

好的 - 仔细看看你的功能：

public function load($catid)
{
    global $conn;
    $sql=' select * '
        .' FROM tbl_categories'
        .' WHERE catid='.$catid;
    if (!($res=mysqli_query($conn,$sql)) || !mysqli_num_rows($res)>0) 
        return false;
    $this->datacol = mysqli_fetch_array($res);
    $this->catid = $this->datacol['catid'];
    return true;
}

因此，您的函数返回true的{{1}}。这个布尔值在这里被分配到false：

$this->datacol

当这项任务发生之后您$this->datacol = $this->load($catid);，$this->datacol = mysqli_fetch_array($res);成为 true 或 false ，没有来自mysql的真实数据。

所以，一个解决方案可以删除作业：

$this->datacol

在这种情况下，您的public function __construct($catid=0) { $this->catid=0; $this->load($catid); }不会被覆盖。

Answer 3

简单的$ this-＆gt; datacol = $ this-＆gt; load（$ catid）;不是必需的

您已分配$ this-＆gt; datacol两次

 public function __construct($catid=0)
    {
        $this->catid=0;
        $this->load($catid);         
    }
public function load($catid)
{
    global $conn;
    $sql=' select * '
        .' FROM tbl_categories'
        .' WHERE catid='.$catid;
    if (!($res=mysqli_query($conn,$sql)) || !mysqli_num_rows($res)>0) 
        return false;
    $this->datacol = mysqli_fetch_array($res);
    $this->catid = $this->datacol['catid'];
    return true;
}

如果遗漏任何内容，请添加

没有从PHP类获得所需的输出

3 个答案: