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");

简而言之：

如果你的课程是这样设计的，那么不使用属性没有错。
请使用有用的类，方法和变量名称。
方法中不要做多于一件事（“副作用”）。
不输出结果，但返回结果。由班级的用户决定结果会发生什么。
（最重要的）：在编码之前先思考。

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

2 个答案: