Class
- Tip&Tech
전체댓글수 14
-
송효진 09-08-18 20:40
소스보기
-
인자 수가 가변이라는 건가요?
그러면 인자를 객체로 넘기면 어떨까요?
$args = (object)null;
$args->userid = 'shimbird';
$args->email = 'asdf@cddd.com';;
$args->display = true;
아예 클래스화 시켜서 default 값을 주거나 validating 까지 할 수도 있겠고요.
-
MC심새 09-08-18 22:22
소스보기
-
음. 역시 전달이 잘 안 되는군요. 글빨이 워낙 부족하다 보니.. ^^;;
classA{
function __construct()
}
classB{
function __construct($id)
}
classC{
function __construct($id, $key)
}
classD{
function __construct($id, $key, $params)
}
라는 클래스가 여러개 있고, 아래와 같이 싱글톤을 사용하려는 경우
Singleton::getInstance('classB', $id);
Singleton::getInstance($className){
$instance = new $className();
}
형식으로 구현하는 경우 classA만 이상 없고 나머지는 오류 메시지가 뜨면서 인스턴스가 생성되지 않죠.
classB, C, D에서는 argument가 필수로 있어야 하니까요.
생성자에 값을 전달해서 정상적으로 인스턴스를 만들기 위함입니다.
설명이 이상해서 이것도 전달이 잘 안 될듯. ㅠ,.ㅠ
javascript의 apply와 비슷한 용도입니다.
-
그네 09-08-18 22:59
소스보기
-
ReflectionClass 가 정답 맞습니다.
$args 는 배열이어야 하고요 인자 순서대로 들어가야 할겁니다.
또한
생성자가 없을 경우는
$class = $reflector->newInstance ();
하심 됩니다.
생성자가 있느냐 없느냐를 채크한후 두가지를 상황에 맞게 쓰심 됩니다.
아마
메서드 확인하는게
$reflector->hasMethod('methodname'); 형태 일겁니다.
-
지나가는자 09-08-18 23:37
소스보기
-
논점을 좀 벗어난 이야기라 죄송합니다.
singleton이 흔하게 발생하는 경우는 아닌듯 하고
꼭 필요하다면
classA{
public function __construct() {}
public static function getInstance() {}
}
형태로 해당 클래스에서
classA::getInstance();
로 접근하는것이 좋지 않나 생각합니다.
꾸뻑 -
송효진 09-08-18 23:45
소스보기
-
누가 만든 클래스건 상관없이 싱글턴으로 쓰기 위한 클래스인듯 합니다.
그러고보니 인자가 들어가면 그 인자까지 고려해서 싱글턴 변수를 관리해야 겠네요.
효율이 많이 떨어지는 것 같은데...
-
지나가는자 09-08-19 01:46
소스보기
-
강제하는것은 아닐테고 각자의 해석이 다를수 있겠습니다만
GoF의 디자인 패턴 책중 싱글톤 패턴 관련 섹션?에
자신의 유일한 인스턴스에 접근하는 방법을
클래스 자체적으로 관리하게 한다는 부분이 있습니다.
그 이점의 범위를 넓히고자 하는 의도는 작성자께서 밝히셨으나
Zend Framework 내부에서도 유일성이 보장되어야할 클래스에서는
자체 singleton 접근법(Zend_XXX::getInstance())을 제시하고 있습니다.
Zend_Registry, Zend_Controller, Zend_Auth 등등...
관련 메소드가 존재하지 않을경우 부득이하게
class My_ClassA extends Zend_ClassA
{
public static function getInstance($id,$key) {
if (null === self::$_instance) {
self::$_instance = new self($id, $key);
}
return self::$_instance;
}
}
$a= My_ClassA::getInstance($id, $key);
정도로 각각 확장해보는것도 괜찮을듯 하고...
단한번 인스턴스 되어야 한다면
역시 명시적으로 관리되어야 하지 않을까 해서 드린말씀입니다.
꼭 필요하다면
class Zend_ClassA {
function __construct($id) {
echo '<pre>';
print_r(func_get_args());
echo '</pre>';
}
}
class Zend_ClassB {
function __construct($id, $key) {
echo '<pre>';
print_r(func_get_args());
echo '</pre>';
}
}
class singleton {
private static $instances = array();
public static function getInstance() {
$params = func_get_args();
$params_md5 = md5(serialize($params));
$class_name = array_shift($params);
/* 알려주신 ReflectionClass 덕분에 삭제
$paramstr = '';
foreach ($params as $i => $param){
$paramstr.= ($i > 0 ? ', ' : '').'$params['.$i.']';
}
*/
if(array_key_exists($params_md5, self::$instances)) {
echo 'same';
} else {
$reflector = new ReflectionClass($class_name);
self::$instances[$params_md5] = $reflector->newInstanceArgs($params);
//self::$instances[$params_md5] = eval("new ".$class_name."(".$paramstr.");");// 역시삭제
}
return self::$instances[$params_md5];
}
}
singleton::getInstance('Zend_ClassA', 'a');
singleton::getInstance('Zend_ClassB', 'a','b');
singleton::getInstance('Zend_ClassB', 'a',array(0,1,2));
singleton::getInstance('Zend_ClassB', 'a','b');
이와 유사한 코드가 나올수밖에 없을테구요...
알려주신 ReflectionClass로 하니 훨 간결하네요. 감사합니다. ^^
그러고 보니 Zend_Registry 로도 만들수 있겠네요
하지만
class My_ClassA extends Zend_ClassA
{
// .....
public static function getInstance($id,$key) {
if (null === self::$_instance) {
self::$_instance = new self($id, $key);
}
return self::$_instance;
}
}
My_ClassA::getInstance($id, $key);
이방법 보단 못하단 생각을 지울수가 없습니다.
수정을 몇번하다보니 송효진님 댓글에 있다가
옮겨오게 되어 죄송한댓글이 되었습니다.
더불어 오타및 문법오류는 애교 ^^
꾸벅 -
MC심새 09-08-19 10:28
소스보기
-
고견 감사 드립니다. ^^
위 코드가 나왔던 이유는, ClassLoader 역할이 컸습니다.
$db = ClassLoader('db.mysql'); 형식으로 사용 했었죠..
구현을 하고 보니, 같은 클래스를 여러번 생성시키길래
$db = ClassLoader::getInstance('db.mysql'); 형식의 싱글톤으로 변경하였습니다.
메소드 내에서 실행하기에, 훨씬 더 명시적이고 여러번 호출되더라도 걱정 없더군요..
나중에 ClassLoader::clear(); 등으로 생성된 instance를 전부 제거할 수도 있구요..
이게 2004년쯤 코드로 생각됩니다.
그 후 이 부분에 대해서는 의심한적 없었고, zend나 php.net의 예제 및 코드를 보면서도 의심을 안 했네요..
Zend_Loader를 보면서도, 왜 instance를 발생시키지 않을까?. 라고만 생각했으니까요..
2004년에 비해 프로그램 규모가 엄청 커지다 보니, 그에 맞춰 확장하게 되면서 올린건데요..
지나가는자님 말씀을 듣고 보니,
왜 위 코드들을 보면서 의심을 안 했는지, 부끄럽네요..
지금 보니, loader 역할과 singleton 역할 두개를 한번에 하고 있었는데... 흐음.
다시 좀 찬찬히 분석해 봐야 하겠습니다.
님 말씀과 같이 My_ClassA::getInstance($id, $key); 형식이 훨씬 더 명시적인건 분명합니다.
다시 예전 코드를 사용할 수도 있겠지만, 충분한 고려 없이 사용한 것과의 차이는 엄청나다고 생각합니다.
잘못된 생각 집어주셔서 정말 감사합니다.
ps) 참고로 예전부터 제가 구현한 방식은 singleton이라기 보다 Fly Weight 패턴이 아닌가 의심은 하고 있었습니다. 말씀 듣고 보니 확실히 그쪽에 더 가까워 보이네요.. -
지나가는자 09-08-19 10:57
소스보기
-
덕분에 저도 많은 공부가 되었습니다.
감사합니다 ^^ -
낭망백수 09-08-19 14:58
소스보기
-
이 이슈는 본문의 요점과는 무관한 싱클톤 관련 이슈죠? 명시화하는게 좋다는?
헷갈려서 물어봅니다.
꾸벅~! -
지나가는자 09-08-19 15:43
소스보기
-
네 뭐 용도가 좀 다르다 쯤이 저의 의견입니다.
==꼭 필요하다면== 이후의 코드는
원글에 언급된 내용중 args의 갯수에 제한 받지 않고
ReflectionClass를 대체할수 있는 고전적 방법도 주석처리된 구문에서
제시하고 있습니다.
function &singleton() {
static $instants;
// .....
}
형태로 바꾸면 php4버전에서도 사용할수 있구요..
감사합니다.
-
게시물 279건
12345678910
