データホルダーとしてあなたのオブジェクトを見るならば、あなたは彼らのカプセル化に違反します、しかし、あなたは実生活でのように、あなたは常に同意を求めなければなりません.

TL;DR: Don't mess with other object's data.

問題

情報隠蔽違反

カプセル化違反

カップリング

解決策

カップルのインターフェイスと行動、決してデータ.

サンプルコード

間違い

<?

final class Point {
    public $x;
    public $y;
}

final class DistanceCalculator {
    function distanceBetween(Point $origin, Point $destination) {
        return sqrt((($destination->x - $origin->x) ^ 2) + (($destination->y - $origin->y) ^ 2));
    }
}

右

<?

final class Point {
    private $rho;
    private $theta;

    public function x() {
        return $this->rho * cos($this->theta);
    }

    public function y() {
        return $this->rho * sin($this->theta);
    }
}

final class DistanceCalculator {
    function distanceBetween(Point $origin, Point $destination) {

        return sqrt((($destination->x() - $origin->x() ^ 2) + (($destination->y() - $origin->y()) ^ 2)));
    }

}

検出

あなたは、パブリック属性、セッターとGetters使用のためにあなたを警告して、彼らを落胆させるために、あなたのプリンターをセットすることができます.

タグ

カップリング

結論

あなたのクラスがセッター、ゲッターと公共の方法で汚染されるならば、あなたは確かに彼らの偶然の実施を結合する方法を持っています.

別名

不適切な親密さ

関係

コード匂い01 -貧血モデル

マキシ・コンティ・ 月20日・ 2分読む

#codenewbie #oop #beginners #computerscience

コード匂い28 -セッター

マキシ・コンティ・ Nov 19 ' 20・ 2分読む

#oop #codenewbie #programming #webdev

詳しい情報

Wikipedia

Refactoring.Guru

C2 Wiki

クレジット

絵Nicolas Poussin

A data structure is just a stupid programming language.

ビル・ゴスペル

ソフトウェア工学大引用

マキシ・コンティ・ Dec 28 ' 20・ 13分読む

#codenewbie #programming #quotes #software

この記事はCodesmellシリーズの一部です.

あなたのコードの臭い部分を見つける方法

マキシ・コンティ・ May 21 ' 21・ 4分読む

#codenewbie #tutorial #codequality #beginners

最終更新日: 2021/06/14