dongwen6743 2013-01-30 15:52
浏览 26
已采纳

OOP PHP和实例化类

I am relatively new to OOP approach to PHP, but just wanted something clarified:

if(isset($_POST['insertProduct']) && !empty($_POST['productName'])){
    $newProduct = new product();
    $newProduct->productName = $_POST['productName'];
    $newProduct->servingSize = $_POST['servingSize'];
    $productID = $newProduct->insertProduct();   
}

So the above code is executed when a form is posted to the page to enter a new product into the DB.

$selectProduct = new product();
if($selectProduct->getProducts()){
    foreach($selectProduct->getProducts() as $product){ 
        echo '<option value="' .
            $product['productID'] . '">' . 
            $product['productName'] . '</option>';
    }
}

The above piece is on the same page, and obviously now there are two instances of the product class that exist (assuming the form has been posted to add a new product). Does this show a benefit of the OO approach? As in using two instances to access different methods? Or is it a bad way of doing what I want to do?

  • 写回答

1条回答 默认 最新

  • douhe5092 2013-02-17 14:47
    关注

    I think getProducts should be at least a static method, since it doesn't have nothing to do with the instance of a product. A product class should represent only the single data of a product, therefore generic non-instances methods should be set to static or moved to another class.

    I'd suggest you to create a Factory class that will create the product instances for you:

    $array = FactoryProducts::getProducts();
    

    But you surely need to move that method outside the product class. For the rest, everything seems fine.

    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论

报告相同问题?

悬赏问题

  • ¥15 matlab数字图像处理频率域滤波
  • ¥15 在abaqus做了二维正交切削模型,给刀具添加了超声振动条件后输出切削力为什么比普通切削增大这么多
  • ¥15 ELGamal和paillier计算效率谁快?
  • ¥15 file converter 转换格式失败 报错 Error marking filters as finished,如何解决?
  • ¥15 ubuntu系统下挂载磁盘上执行./提示权限不够
  • ¥15 Arcgis相交分析无法绘制一个或多个图形
  • ¥15 关于#r语言#的问题:差异分析前数据准备,报错Error in data[, sampleName1] : subscript out of bounds请问怎么解决呀以下是全部代码:
  • ¥15 seatunnel-web使用SQL组件时候后台报错,无法找到表格
  • ¥15 fpga自动售货机数码管(相关搜索:数字时钟)
  • ¥15 用前端向数据库插入数据,通过debug发现数据能走到后端,但是放行之后就会提示错误