如何重构与业务对象紧密耦合的“函数库”类？

Question

我有一个 Calc 类，我认为它被不正确地放置在我正在使用的代码库中。对于类结构，请参见下面的代码。

目前

• Spec类充当数据的存储，类似于C struct
• Calc 类是一个计算函数类库，当这些类需要进行一些计算时，它被实例化为其他类的一部分。 Calc 类还包含Spec 类，因此它可以使用Spec 的变量进行计算。
• Plot 类是表示图形的业务对象的示例。
• Controller 类是一个业务对象，代表我的应用程序中的各种控制器，其中进行了一些计算并以文本形式显示在用户页面“视图”中，并且还创建了一个图形，该图形也显示给用户“查看”页面。

class Spec
{
    public $a;
    public $b;
    public $c;
}

class Calc
{
    public $spec;

    function __construct()
    {
        $this->spec = new Spec();
    }

    function calcParameter()
    {
        $this->spec->c = $this->spec->a + $this->spec->b;
    }
}

class Plot
{
    public $calc;

    function __construct()
    {
        $this->calc = new Calc();
    }

    function calcPlot()
    {
        $this->calc->spec->c = $this->calc->spec->a * $this->calc->spec->b;
    }
}


class Controller
{
    public $calc;
    public $plot;

    function __construct()
    {
        $this->calc = new Calc();
        $this->plot = new Plot();
    }

    function doBusinessLogic()
    {

        //calc for local
        $this->calc->spec->a = 5;
        $this->calc->spec->b = 5;
        $this->calc->calcParameter();
        print "total is {$this->calc->spec->c}<br>\n";

        //later this format is used by JS to display computational results
        $plotJSON = json_encode($this->calc); 


        //calc for plot
        $this->plot->calc->spec->a = 7;
        $this->plot->calc->spec->b = 143;
        $this->plot->calcPlot();
        print "total is {$this->plot->calc->spec->c}<br>\n";

        //later this format is used by JS to display a plot
        $plotJSON = json_encode($this->plot); 
        print "
        <div id='plot' style='display:none'>$plotJSON</div>
        <script>
            var plot = JSON.parse(document.getElementById('plot').innerHTML);
            document.write('JS says - there are ' + plot.calc.spec.c + ' plot points<br>');
        </script>
        ";
    }  
}

//runs the above
(new Controller())->doBusinessLogic();

JS 是这样使用的：

var plot = JSON.parse(document.getElementById('plot').innerHTML);
document.write('JS says - there are ' + plot.calc.spec.c + ' plot points<br>');

问题

我的印象是 Calc 类没有正确放置（设计不正确），因此，它被注入到各个地方只是为了进行计算。我在想Calc 应该是一个没有参数的类，Spec 不应该是Calc 的一部分。而Calc 必须只包含进行计算的函数，Calc 的调用者将提供自己的数据并接收结果。因此Calc 将成为一个静态函数库，供其他人使用。但这是最好的方法吗？

如何重构它以最小化耦合？请记住，要么保留 JSON 格式，要么注意代码中的任何更改都可能需要更改 JS 代码。

在我深入重构之前，我现在的设计可以工作。有没有必要重构？只是检查。

Answer 1

好的，根据我目前看到的情况，Calc 类根本不应该存在。 calcParameter 方法应该是 Spec 类的成员。

Plot 的 calcPlot 方法也应该是 Spec 类的成员（它完全处理当前编写的 Spec 字段。）您仍然可能希望保留 Plot，尽管它可以做其他有趣的事情到它的Spec 对象。

像这样：

class Spec {
    public $a;
    public $b;
    public $c;

    function calcParameter() {
        $this->c = $this->a + $this->b;
    }

    function calcPlot() {
        $this->c = $this->a * $this->b;
    }
}

class Plot {
    public $spec;

    function __construct() {
        $this->spec = new Spec();
    }

    function calcPlot() {
        $this->spec->calcPlot()
    }
}


class Controller {
    public $spec;
    public $plot;

    function __construct() {
        $this->spec = new Spec();
        $this->plot = new Plot();
    }

    function doBusinessLogic() {
        //spec for local
        $this->spec->a = 5;
        $this->spec->b = 5;
        $this->spec->calcParameter();
        print "total is {$this->spec->c}<br>\n";

        //later this format is used by JS to display computational results
        $plotJSON = json_encode($this->spec); 

        //spec for plot
        $this->plot->spec->a = 7;
        $this->plot->spec->b = 143;
        $this->plot->calcPlot();
        print "total is {$this->plot->spec->c}<br>\n";

        //later this format is used by JS to display a plot
        $plotJSON = json_encode($this->plot); 
        print "
        <div id='plot' style='display:none'>$plotJSON</div>
        <script>
            var plot = JSON.parse(document.getElementById('plot').innerHTML);
            document.write('JS says - there are ' + plot.spec.c + ' plot points<br>');
        </script>
        ";
    }  
}

//runs the above
(new Controller())->doBusinessLogic();