14

我见过一些项目，其中类具有获取和设置方法来操作插入数据。让我在这里举个例子：

    class Student extends dbClass
{
    private $TableID;
    private $FullName;
    private $Gender;
    private $Address;




    function setTableID($Value)
    {
        $this->TableID = $Value;
    }

    function getTableID()
    {
        return $this->TableID;
    }

    function setFullName($Value)
    {
        $this->FullName = $Value;
    }

    function getFullName()
    {
        return $this->FullName;
    }

    function setGender($Value)
    {
        $this->Gender = $Value;
    }

    function getGender()
    {
        return $this->Gender;
    }

    function setAddress($Value)
    {
        $this->Address = $Value;
    }

    function getAddress()
    {
        return $this->Address;
    }


    function UpdateStudent()
    {
        $sql = "UPDATE INTO usertable SET
        FullName = '".$this->getFullName()."',
        Gender = '".$this->getGender()."',
        Address = '".$this->getAddress()."'
        where TableID='".$this->getTableID()."'";
        $this->query($sql);
    }
}

以上是我见过的示例类。以下是他们如何使用它的过程：

$student = new Student;
$student->setTableID = 1;
$student->setFullName('My Name');
$student->setGender('Male');
$student->setAddress('this is my address');

$studen->UpdateStudent();

这样做值得吗？我个人认为设置字段然后获取和更新其中的记录是没有用的。为每个模块制作它确实需要很多时间。处理这种事情的最佳方法是什么？这样做是否有任何安全问题？

4

4 回答 4

14

这样做值得吗？

这取决于。

通过公开一个“智能”属性（即 getter 和/或 setter）从用户那里抽象出一个字段有两个缺点：

  1. 你需要写更多的代码；如果该属性并没有真正做任何聪明的事情，那么这是没有任何用处的代码。
  2. 该属性的用户有些不便，因为他们还必须输入更多内容。

它有一个优点：

  1. 将来，即使之前没有任何逻辑，您也可以向属性添加逻辑，而不会破坏用户的代码。

如果这个优势是有意义的（例如，您正在编写一个可重用的软件库），那么编写属性而不是裸字段就很有意义。如果没有，你就是在做没有任何好处的工作。

处理这种事情的最佳方法是什么？

您可以覆盖魔术__get和__set函数（可能在基类中，因此您也可以继承覆盖）以自动将属性访问转发给您的 getter 和 setter。简化代码：

public function __get($name) {
    $getter = 'get'.$name;
    if (method_exists($this, $getter)) {
        return $this->$getter();
    }

    $message = sprintf('Class "%1$s" does not have a property named "%2$s" or a method named "%3$s".', get_class($this), $name, $getter);
    throw new \OutOfRangeException($message);
}

public function __set($name, $value) {
    $setter = 'set'.$name;
    if (method_exists($this, $setter)) {
        return $this->$setter($value);
    }

    $getter = 'get'.$name;
    if (method_exists($this, $getter)) {
        $message = sprintf('Implicit property "%2$s" of class "%1$s" cannot be set because it is read-only.', get_class($this), $name);
    }
    else {
        $message = sprintf('Class "%1$s" does not have a property named "%2$s" or a method named "%3$s".', get_class($this), $name, $setter);
    }
    throw new \OutOfRangeException($message);
}

警告购买者：因为__get和__set被覆盖，__isset也__unset应该被覆盖！

这样做是否有任何安全问题？

不，根本没有（假设您没有意外插入错误）。

于 2012-05-16T10:48:19.053 回答
6

在没有属性（实际上导致函数调用的公共成员“变量”）的语言中，通常建议使用 getter/setter 而不是公共变量。否则，如果人们已经在使用您的普通字段，您将无法稍后添加逻辑（例如，在设置变量时）。

由于 PHP 是这样一种语言（不幸的是），答案是肯定的，使用它们。

于 2012-05-16T10:48:22.043 回答
2

制作 setter 和 getter 有助于强制执行 OOP 封装。我不确定 PHP，但对于许多其他语言（Java、C++），一个好的 IDE（eclipse/netbeans）会为你自动生成这些 setter 和 getter。

对于简单的类型，它可能不会立即显而易见，但如果必须执行任何更复杂的处理，那么它就会变得更加明显。

于 2012-05-16T10:46:07.363 回答
1

关于为什么有时使用 getter 和 setter 的示例：我有在 200 个不同文件中设置值的代码：

$cat->name = 'Fery'          // in file one
$favoriteCat->name = 'Tery'  // in another file
$fatCat->name = 'jery'             // in another file
// 200 more calls in alot of places

想象一下客户突然有一个新要求：“所有猫的名字都必须在前面加上‘Sweet’

现在我们必须找到所有猫的声明并替换它们的赋值：

$cat->name = 'Sweet '.'Fery'
// times 200

另一方面，如果我们使用 setter 方法，我们需要做的就是在一个地方添加代码以添加“Sweet”：

public function setCatname($catname)
{
  $this->catName = 'Sweet ' . $catname;
}
于 2018-07-05T08:44:30.480 回答