将两个类合而为一：设计错误？

Question

我编写了这两个类来简单地加密和解密字符串：

Encode.php

    class Encode {
        protected funcion do_encode($string) { .. }
    }


Decode.php

    class Decode {
        protected funcion do_decode($string) { .. }
    }

我想要想做的是：

Encrypt.php

    class Encrypt extends Encode, Decode {
        protected $stuff_for_parents;

        function __construct($configs) {
            $this->stuff_for_parents = $configs['SomeConf'];
        }

        public function encode($string) { $this->do_encode($string); }
        public function decode($string) { $this->do_decode($string); }
    }

但是我们不能包含多个类，所以：失败。
现在我的问题是：

1. 这是一个设计问题吗？因为这个场景对我来说并不奇怪，不是吗？
2. 是否有另一种方法可以让一个对象使用不同类中的两个函数？排序 $encrypt->encode($str); $encrypt->decode($str);

Answer 1

如果您希望 Encode 和 Decode 是单独的类，则可以在 Encrypt 中创建它们的实例。例如：

<?
class Encrypt {
  private $encoder;
  private $decoder;
  public function __construct() {
    $this->encoder = new Encode();
    $this->decoder = new Decode();
  }
  public function encode($string) {
    return $this->encoder->encode($string);
  }
  public function decode($string) {
    return $this->decoder->decode($string);
  }
}
?>

在该示例中，Encode 和 Decode 必须是具体类。但您可能想考虑使用接口。如果您认为在不同情况下可能需要使用不同类型的 Encode 和 Decode 对象，则接口非常有用。例如，也许您有 TripleDESEncode 类和 BlowfishEncode 类。它们都可以实现一个公共接口，如下所示：

<?
interface IEncode {
  public function encode($string);
}
class TripleDESEncode implements IEncode {
  public function encode($string) {...}
}
class BlowfishEncode implements IEncode {
  public function encode($string) {...}
}
?>

在这种情况下，您可能希望首先创建要使用的特定实例，然后将它们传递到 Encrypt 的构造函数中。这称为依赖注入：

<?
class Encrypt {
  public function __construct(IEncode $encoder, IDecode $decoder) {
    $this->encoder = $encoder;
    $this->decoder = $decoder;
  }
  ...
}

$myEncrypt = new Encrypt(new BlowfishEncode(), new BlowfishDecode());
echo $myEncrypt->encode('test');
?>

Answer 2

将算法封装在单独的类中并没有什么问题（事实上，如果您希望灵活地更改这些算法，尤其是在运行时），那么这是一个很好的做法，但是，您可能想要的是组合而不是继承：

class Encrypt {
    private $encoder;
    private $decoder;
    protected $stuff_for_parents;

    function __construct($configs, $encoder, $decoder) {
        $this->stuff_for_parents = $configs['SomeConf'];
        $this->encoder = $encoder;
        $this->decoder = $decoder;
    }

    public function encode($string) { $this->encoder->do_encode($string); }
    public function decode($string) { $this->decoder->do_decode($string); }
}

Answer 3

我认为你应该在加密类中创建两个方法/函数。这是非常合乎逻辑的。

它确实有道理，但可以通过编程语言允许的方式来完成。编码解码是加密类应该执行的不同功能，因此这些应该是方法。

Answer 4

我会写的是这样的：

class Encrypt
{
    protected $stuff_for_parents;

    function __construct($configs) {
        $this->stuff_for_parents = $configs['SomeConf'];
    }

    protected funcion encode($string)
    {
    }

    protected funcion decode($string)
    {
    }
}