php - 哪些变量应该设置为 php 中类的属性？

Question

<?php

class oopClass{

    function __construct($editingtext, $searchfor, $replacewith){

        if(!empty($editingtext) && !empty($searchfor) && !empty($replacewith)){

           $editingtext = str_replace($searchfor,$replacewith,$editingtext);

           echo $editingtext;

        }else{

          echo 'All Fields Are Required.';

        }
    }
}

//closing php

该代码正在运行，但是由于没有设置类的属性，这是一种不好的做法，因此应将此代码的哪些变量设置为类属性，为什么？

Answer 1

如果上面的代码是您打算使用此代码执行的全部操作，则不一定是不好的做法。如果你需要扩展它的功能，我可以想象它$editingtext可能是一个属性。

class oopClass{

    private $editingtext;        

    function __construct($editingtext, $searchfor, $replacewith){

        $this->editingtext = $editingtext;                

        if(!empty($this->editingtext) && !empty($searchfor) && !empty($replacewith)){

           $this->editingtext = str_replace($searchfor,$replacewith,$this->editingtext);

           echo $this->editingtext;

        }else{

          echo 'All Fields Are Required.';

        }
    }
}

//closing php

Answer 2

您的代码还有其他问题，而不是缺少属性。您正在构造一个对象，并在构造函数中输出结果。这是不好的做法。

我会像这样修复它：

class TextReplacer {
    var $search;
    var $replace;

    function __construct($s, $r) {
         $this->search = $s;
         $this->replace = $r;
    }

    function replace($text) {
        // your code, using the properties for search and replace, RETURNING the result
        return $ret;
    }
}

然后像这样调用：

$oo = new TextReplacer("bar", "baz");
echo $oo->replace("let's replace some bars in here");

简而言之：

1. 如果你的类是这样设计的，那么不使用属性没有错。
2. 请使用有用的类、方法和变量名。
3. 不要在一个方法中做超过一件事（“副作用”）。
4. 不输出结果，而是返回它。由类的用户决定结果会发生什么。
5. （最重要的）：在编码之前思考。