我是用对象还是 OOP 编写过程代码？

Question

所以基本上我正在从过程编码飞跃到面向对象编程。 我正在尝试实现 OOP 的原则，但我有一种挥之不去的感觉，我实际上只是用对象编写过程风格。

假设我有一个管道/椅子/打印机/任何东西的列表，它们都作为产品列在我的单表数据库中。我需要构建一个 Web 应用程序，根据其类型显示整个列表和项目，重点是“正确”使用 OOP 及其范例。

这样做有什么问题吗：

 类展示
     {

      公共函数 showALL(){
      $prep = "从我的产品中选择 *";
      $q = $this->db->;准备（$准备）；     
      $q->execute();
      while ($row = $q->fetch()) 
          {
            echo "bla bla bla 一些排列的显示".$row['something']       
          }
      }

然后

$sth = new show();
$sth->showAll();

我还会实现更具体的显示方法，例如：

showSpecificProduct($id)->(当用户点击其中一个链接时，$id 将通过 $_GET 传递，我们将拥有单独的product.php 文件，该文件将基本上只包含

include('show.class.php');
$sth = new show();
$sth->showSpecificProduct($id);

showSpecificProduct() 会同时执行选择查询和输出 html 以进行显示。

所以简而言之，我是在做这件事还是只是用类和对象进行程序编码还有任何想法/提示。如果我做错了，如何解决它？

Answer 1

除了@Phil 和@Drew 描述的模型实践之外，我还敦促您将业务层、数据层和视图层分开。

我已经包含了一个非常简单的版本，需要在您的实现中对其进行扩展，但其想法是使您的 Db 选择与输出分开，并几乎在控制器中将两者“连接”在一起。

class ProductController
{
    public $view;

    public function __construct() {
        $this->view = new View;
    }

    public function indexAction() {
        $model = new DbProductRepository;
        $products = $model->fetchAll();

        $this->view->products = $products;
        $this->view->render('index', 'product');
    }
}

class View
{
    protected $_variables = array();        

    public function __get($name) {
        return isset($this->_variables['get']) ? $this->_variables['get'] : null;
    }

    public function __set($name, $value) {
        $this->_variables[$name] = $value;
    }

    public function render($action, $controller) {
        require_once '/path/to/views/' . $controller . '/' . $action . '.php';
    }
}

// in /path/to/views/product/index.php
foreach ($this->products as $product) {
    echo "Product ID {$product['id']} - {$product['name']} - {$product['cost']}<br />\n";
}

Answer 2

更好的选择是实施存储库模式。一个示例接口可能是

interface ProductRepository
{
    public function find($id);

    public function fetchAll();
}

您然后创建该接口的具体实现

class DbProductRepository implements ProductRepsoitory
{
    private $db;

    public function __construct(PDO $db)
    {
        $this->db = $db;
    }

    public function find($id)
    {
        // prepare execute SQL statement
        // Fetch result
        // return result
    }

    public function fetchAll()
    {
        // etc
    }
}

直接从方法或函数echo通常是一个坏主意。让您的方法返回适当的对象/数组/任何内容并使用这些结果。

Answer 3

您上面描述的场景似乎是 MVC 的良好候选者。

在您的情况下，我将创建一个严格用于访问数据的类（选择产品类别或特定产品），然后让另一个文件（您的视图）获取输出并显示它。

它可能看起来像这样：

class Product_Model {
    public function find($prodId) { ... }
    public function fetchAll($category = '') { ... }
    public function search($string) { ... }
}

然后您可以在其他地方执行以下操作：

$products = new Product_Model();
$list = $products->fetchAll(37); // get all from category 37

// in true MVC, you would have a view that you would assign the list to
// $view->list = $list;

foreach($ilst as $product) {
    echo "Product ID {$product['id']} - {$product['name']} - {$product['cost']}<br />\n";
}

MVC 的基本原则是您拥有模型类，这些模型类只是表示来自某些数据源（例如数据库）的数据的对象。您可能有一个映射器，用于将数据库中的数据映射到数据对象或从数据对象中映射数据。然后控制器将从模型类中获取数据，并将信息发送到视图，在视图中处理实际的呈现。在控制器中拥有视图逻辑（html/javascript）是不可取的，并且直接与控制器中的数据进行交互是相同的。

Answer 4

首先，您需要研究类自动加载。这样，您不必包含您使用的每个类，您只需使用它，自动加载器就会找到正确的文件来为您包含。

http://php.net/manual/en/language.oop5.autoload.php

每个类都应该有一个单一的职责。您不会有一个连接到数据库并更改某些用户数据的类。相反，您将拥有一个数据库类，您将其传递给用户类，并且用户类将使用数据库类来访问数据库。每个职能也应该有单一的职责。你永远不应该在函数名称中添加“and”。

您不希望一个对象知道另一对象的属性。这会导致在一个类中进行更改迫使您在另一类中进行更改，并且最终会变得很难进行更改。属性应该供对象内部使用。

在开始编写类之前，您应该首先考虑您希望如何使用它（请参阅测试驱动开发）。您希望代码在使用时看起来如何？

$user = new User($db_object);
$user->load($id);
$user->setName($new_name);
$user->save();

既然您知道如何使用它，那么以正确的方式对其进行编码就容易得多。

有机会时研究敏捷原则。

Answer 5

一条经验法则是类名通常应该是名词，因为 OOP 是关于拥有与真实概念对象相对应的软件对象。类成员函数通常是动词，即可以对对象执行的操作。

在您的示例中， show 是一个奇怪的类名。更典型的方法是使用一个名为 ProductViewer 的类，并带有一个名为 show() 或 list() 的成员函数。此外，您还可以使用子类来获取专门的功能，例如特定产品类型的自定义视图。