drsqpko5286 2016-01-27 19:35 采纳率: 0%
浏览 18
已采纳

在PHP类中使用实用程序函数的最佳实践是什么? [关闭]

I'm coming into object-oriented PHP from a more functional Javascript background, so I'm fairly new to both PHP and OOP.

What I'm trying to do is make a class that that is passed the contents of a text file, divides it up into manageable pieces, and then loops over those pieces (steps), and handles them accordingly, sometimes making HTTP requests, sometimes saving chunks of info into a $fileVariables array. Here's a basic outline of what I have:

class Script {
    public $file, $client, $fileVariables;

    function __construct($file) {
        $this->file = $file;
        $this->client = new \GuzzleHttp\Client();
        $this->fileVariables = array();
    }

    public function parseAndRun() {
        $client = $this->client;
        $file = $this->file;
        // this is the function that performs the bulk of the class's
        // work. It relies on some utility functions that are defined
        // after, and are utilized within this function like this:
        $steps = $this->divideFileIntoSteps($file);
        // or like this:
        $this->setVariable($index, $value);
    }

    public function divideFileIntoSteps($file) {
        // return array of file divided into steps
    }

    public function setVariable($index, $value) {
        // push $index => $value into $fileVariables array
    }
}

I think it's possible that I'm trying to do something that would be totally okay in functional Javascript, but just feels sloppy in object-oriented PHP. Also, when I run PHPUnit tests on this, I get an error that's something like Serialization of 'Closure' is not allowed, which I think from doing some googling on the error might have something to do with how I'm doing this. Also, the fact that I have to use $this-> over and over again on commonly used functions makes me feel like I'm doing something wrong. It just gives me that bad code feeling.

Is there a better way to do this, bearing in mind that a lot of the functionality has to do with getting and setting items in the $fileVariables array?

  • 写回答

1条回答 默认 最新

  • dsbgltg159136540 2016-01-27 20:19
    关注

    Your code is basically correct.

    You don't need $file = $this->file;

    $this prefix is standard for accessing member functions and variables in PHP.

    Your function could look like this:

    public function parseAndRun() {
        $steps = $this->divideFileIntoSteps($this->file);
        // or like this:
        $this->setVariable($index, $value);
    }
    

    Another best practice is to declare private all methods that you don'n need to access from outside the class. Member variables should all be private and you should have accessor functions for them.

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

报告相同问题?

悬赏问题

  • ¥15 生物医学数据分析。基础课程就v经常唱课程舅成牛逼
  • ¥15 云环境云开发云函数对接微信商户中的分账功能
  • ¥15 空间转录组CRAD遇到问题
  • ¥20 materialstudio计算氢键脚本问题
  • ¥15 有没有代做有偿主要做数据可视化部分即可(2023全国高考更省一本线理科类)
  • ¥15 配置FPT报错,该如何处理
  • ¥15 请大家看一下这个代码咋写,一点思路都没有,最好能做一下,不要伪代码,有偿
  • ¥15 有偿请人帮写个安卓系统下禁止装软件及禁止拷入文件的程序
  • ¥100 用 H.265 对音视频硬编码 (CUDA)
  • ¥20 mpich安装完成后出问题