| ../ |
|---|
PHP7.4で継承を使っていると、アクセス修飾子でおかしな挙動がある。バグなのか、仕様なのか分からないが、注意が必要である。
Userクラス単独でのテスト
まず、Userというクラスを定義してみる。idとnameを属性に持たせる。
<?php
namespace samples;
class User {
private $id;
private $name; // ここをprotected,publicに変えてテスト
public function __construct(string $id, string $name) {
$this->setId($id);
$this->setName($name);
}
public function getId(): string { return $this->id; }
public function getName(): string { return $this->name; }
public function setId(string $id): void { $this->id = $id; }
public function setName(string $name): void { $this->name = $name; }
}
以下のようなテストコードを実行してみる。name属性の変更を試みる。
$user = new User('U001', 'Aさん');
$user->name = '山本さん'; // private,protectedだと、ここでエラー
$user->setName('池田さん');
var_dump($user);
すると、privateな属性に代入しようとしているのでエラーになる。これは正しい動作である。
Fatal error: Uncaught Error: Cannot access private property
samples\User::$name in samples\User.php
ちなみに、宣言をprotectedに変えても同様なエラーが出る。そして、publicに変えると正常に動作して、以下のようなvar_dump()の結果が表示される。一旦、山本さんに変更され、その後で池田さんに上書きされる。正しい挙動である。
class samples\User#1 (2) {
public $id =>
string(4) "U001"
public $name =>
string(12) "池田さん"
}
ここまでは問題ない。
Identifierを継承したUserクラスでのテスト
次に、Identifierというクラスとそれを継承したUserクラスで試してみる。
まず、Identifierというクラスを作る。前述のUserをリネームして、abstractを宣言したものである。
<?php
namespace samples;
abstract class Identifier {
private $id;
private $name;
public function __construct(string $id, string $name) {
$this->setId($id);
$this->setName($name);
}
public function getId(): string { return $this->id; }
public function getName(): string { return $this->name; }
public function setId(string $id): void { $this->id = $id; }
public function setName(string $name): void { $this->name = $name; }
}
次に、Identifierを継承したUserクラスを作る。継承をイメージしやすいようにmail属性を追加してみる。
<?php
namespace samples;
require_once(__DIR__."/Identifier.php");
class User extends Identifier {
private $mail;
public function __construct(string $id, string $name, string $mail){
parent::__construct($id, $name);
$this->setMail($mail);
}
public function getMail(): string { return $this->mail; }
public function setMail(string $mail): void { $this->mail = $mail; }
}
この状況で、前述のテストコードと同等のことを行ってみる。
$user = new User('U001', 'Aさん', "aaa@xxx.com");
$user->name = '山本さん';
$user->setName('池田さん');
var_dump($user);
結果は、以下のようになる。エラーにはならない。
class samples\User#1 (4) {
private $mail =>
string(11) "aaa@xxx.com"
private $id =>
string(4) "U001"
private $name =>
string(12) "池田さん"
public $name =>
string(12) "山本さん"
}
「$user->name = '山本さん';」の実行では、privateな属性なのでアクセスできないかと思えば、勝手に新たなpublicな$nameという属性を追加しているのである。継承元を探索できていない。また、おそらく、PHPでのクラスの実装はarrayを使っていて、アクセス修飾子＋属性名で識別しているのだろう。PHPのバグなのか、仕様なのかは分からない。
PHPでクラスを扱うときは、アクセス修飾子を信用せずに、setter/getterをきちんと定義して使用しないと危険である。
| ../ |
|---|