Discussion :

[Benjamin Delespierre]

Bonjour à tous.

Considérons l'exemple suivant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
interface Model {
	public function __construct ($id = null);
}
 
abstract class BaseModel implements Model {
	public function __construct ($id = null) {
		var_dump($id);
	}
}
 
class MySQLModel extends BaseModel {
	public function __construct ($id = null) {
		throw new RuntimeException('Missing $tbl parameter');
	}
	public function __construct ($tbl, $id = null) {
		var_dump($tbl,$id);
	}
}

Inutile de préciser que ça ne fonctionne pas en PHP, la redéfinition n'étant pas possible. J'ai cependant ce besoin, il faut que ma classe concrête MySQLModel implémente l'interface Model mais elle doit recevoir un paramètre supplémentaire sur son constructeur.
Dès lors comment s'en sortir ? J'ai pensé à:

• utiliser func_get_args
• ajouter un argument arbitraire supplémentaire sur le prototype du constructeur dans l'interface (de type array) (que c'est moche)
• enlever tous les paramètres du constructeur dans l'interface et laisser les classes filles se débrouiller avec func_get_args (ce qui rends l'interface 'achement utile !)
• dégager l'interface
• vous demander votre avis

[Benjamin Delespierre]

Peut être que l'hypothèse la plus raisonnable est la suivante:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
interface Model {
	public function __construct ($id = null);
}
 
abstract class BaseModel {
	public function __construct ($id = null) {
		var_dump($id);
	}
}
 
class MySQLModel extends BaseModel {
 
	protected $_table;
 
	/**
	 * Default constructor
	 * @param string $tablename The MySQL tablename
	 * @param mixed $id [optional] [default `null`] The row ID to match
	 */
	public function __construct ($id = null) {
		$args = func_get_args();
		if (empty($args))
			throw new InvalidArgumentException("First parameters is missing");
 
		if (!is_string($args[0]))
			throw new InvalidArgumentException("Firt parameter is expected to be string, " . gettype($args[0]) . " given");
		list($table,$id) = $args + array('', null);
 
		if (empty($table))
			throw new InvalidArgumentException("Invalid tablename");
 
		$this->_table = $table;
		parent::__construct($id);
	}
}

Edit - et c'est ce que j'ai fait, c'est encore ce que je trouve de plus propre
On ferme.