0
require_once('adodb/adodb.inc.php');

class adoDbConnector
{

    private static $instance = null;

    public static function getInstance()
    {
    if (self::$instance == NULL)
    {
        self::$instance = new adoDbConnector();
    }

    return self::$instance;
    }

    private function __construct() { }

    public function __clone()
    {
        trigger_error('No __clone()!', E_USER_ERROR);
    }

    public function __wakeup()
    {
        trigger_error('No __wakeup()!', E_USER_ERROR);
    }

    public function connect($connectionType, $connectionData)
    {
        switch($connectionType)
        {
            case 0: 
            return self::generalConnect($connectionData);
            break;

                    case 1: 
            return self::dsnConnect($connectionData);
            break;

                    case 2: 
            return self::xmlConnect($connectionData);
            break;

                    default:
            throw new Exception('Wrong type of connection!');
        }
    }

    private static function generalConnect($connectionData)
    {   
        if(is_array($connectionData))
        {
            $conn = &ADONewConnection($connectionData[0]);

            $conn->PConnect($connectionData[1], $connectionData[2], 
                                $connectionData[3], $connectionData[4]);

            return $conn;
        }
        else throw new Exception('Wrong type of connection data, must be an array!');
    }

    private static function dsnConnect($connectionData)
    {       
        if(is_string($connectionData))
        {
            $conn = ADONewConnection($connectionData);

            return $conn;
        }
        else throw new Exception('Wrong type of connection data, must be a string!');
    }

    private static function xmlConnect($connectionData)
    {
        if(is_string($connectionData) && file_exists($connectionData))
        {
            $xml = simplexml_load_file($connectionData);

            foreach($xml as $x)
            {
                $connArr[] = trim($x);
            }

            $conn = &ADONewConnection($connArr[0]);

            $conn->PConnect($connArr[1], $connArr[2], $connArr[3], $connArr[4]);

            return $conn;
        }
        else throw new Exception('Wrong file name or connection type!');
    }

}
9
  • Лично меня смущают ссылки... Зачем?
    – user3545
    29 мар 2012 в 18:07
  • Лично меня смущает, что Вы пытаетесь создать Singleton таким ужасным образом. Почитайте [здесь][1] [1]:hashcode.ru/questions/92217/…
    – ReinRaus
    29 мар 2012 в 18:09
  • меня пустой конструктор, почему допустим не выполнять connect при создании экземпляра класса, к тому же можно сделать его опциональным
    – FLK
    29 мар 2012 в 18:10
  • FLK, поясните пожалуйста про опциональность, каким образом? Что именно? Да, действительно, возможно будет лучше выполнять его при создании экземпляра, хотя тут еще нужно подумать, но спасибо большое за идею. :) На счет ссылок - пример из мануала, в принципе они не нужны. Спасибо за ссылку на синглтон.
    – Archont12
    29 мар 2012 в 18:17
  • 1
    Комментариев не нашел. В общем-то поэтому дальше оценивать сложно - о смысле некоторых объектов (переменных, функций) можно либо только догадываться, либо долго втыкать в код. А хороший стиль нужен именно потому, чтобы не заниматься ни тем, ни другим.
    – northerner
    29 мар 2012 в 18:26

2 ответа 2

1

Так же отсутствие комментариев к коду.

Возможно стоит обратить внимание на стандарт CakePHP или выбрать любой другой.

0
1
  1. Хромает форматирование
  2. Да и вот такой стиль (перевод { на новую строку) честно говоря убивает
7
  • 1
    Стиль мне тоже не нравиться, но тем не менее, как я замечал - большинство java, C#, PHP(пишущих на Zend'e) разработчиков пишут именно так, так что стиль это всего-лишь ИМХО.
    – user3545
    29 мар 2012 в 18:17
  • @AlexWindHope, это ужасно, что пишут так.
    – Oleg
    29 мар 2012 в 18:19
  • Действительно пишу на Zend'e и С#. )
    – Archont12
    29 мар 2012 в 18:22
  • Мне приходилось заниматься аудитом кода в компании, использующей стиль RSDN (там как раз Allman-style). Так вот, когда 12 часов подряд вычитываешь чужой код - этот стиль офигенно удобен, а значит и в плане сопровождения тоже. Хотя по жизни я использую K&R (1TBS). Так что это просто вопрос внутренний соглашений в команде разработчиков.
    – northerner
    29 мар 2012 в 18:22
  • 3
    @northerner, не знаю, не знаю.. Вот такой стиль: if ( condition ) { // some code } Гораздо лучше. Опять же - ИМХО.
    – Oleg
    29 мар 2012 в 18:24

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.