Нужен совет пофессионалов своего дела

[boko]

Спасибо посмотрим

 

[fixxxer]

>класс для работы с БД должен быть типа одиночка.

о боже

помолчи пожалуйста во всех тредах касаемо ооп и архитектуры года этак три

 

[Духовность™]

а зачем вам класс для работы с БД типа не одиночка? какой в нем смысл, если 90% проектов пишутся под одну базу и работают с одной базой и с одним соединением? Вот у меня часто возникает потребность получить доступ к БД из разных классов. Одиночка для этого очень удобен. Если бы небыло одиночки, мне бы пришлось бы юзать какой-нибудь глобалс или регистри для получения экземпляра объекта.

 

[HraKK]

triumvirat
Тебе бы пришлось бы строить нормально архитектуру) Вот и все.
Вообще старайся избегать одиночек так часто как только можешь.

 

[Beavis]

Автор оригинала: triumvirat
PS: Вот мой класс

У тебя очень интересный подход к именованию переменных и методов))

 

[HraKK]

Хех, вот объясните мне зачем работать капитаном очевидностью?

 /**
    * Конструктор класса.
    * 
    * @access public
    * @param void
    * @return void
    */
    public function __construct()
    {
        $this->connect();
    }

/*
    * Получает количество рядов, 
    * задействованных в предыдущей MySQL-операции.
    * Возвращает количество рядов, 
    * задействованных в последнем запросе INSERT, UPDATE или DELETE.
    * Если последним запросом был DELETE без оператора WHERE, 
    * все записи таблицы будут удалены, но функция возвратит ноль.
    * 
    * @access public
    * @param void
    * @return int
    */
    public function getAffectedRows()
    {
        return mysql_affected_rows($this->lnk);
    }

Как по мне такие коменты только ухудшают восприятие кода.

оО

$this->query = @call_user_func_array('sprintf', $arg_list);

собачка?!!!

 /**
    * Аналог функци str_replace, но с единичной заменой подстроки.
    */
    protected static function str_replace_once($search, $replace, $subject)

Велосипед вместо preg_replace?

 

[Духовность™]

У тебя очень интересный подход к именованию переменных и методов

собачка?!!!

код старый, я его давно писал, я уже не помню зачем там собачки и кошечки..

Велосипед вместо preg_replace?

не знаю. надо посмотреть....

Хех, вот объясните мне зачем работать капитаном очевидностью?

ну разве это плохо?

 

[HraKK]

как по мне плохо -засоряет.

 

[Духовность™]

а у тебя код вообще без комментариев! :Р

 

[HraKK]

Как это без? tests папку видел? Вот это лучшие комментарии. Да и код - в 5 строчек не нуждается в комментировании. Но это конечно, имхо, к комментария у меня особых претензий нету - я лишь уточню, зачем такие.

 

[fixxxer]

>>Если бы небыло одиночки, мне бы пришлось бы юзать какой-нибудь глобалс или регистри

Все три перечисленных тобой способа идентично говняные. Что синглтон, что регистри - это те же глобалс, завернутые в красивую обертку.

Посмотри различия между symfony 1 и symfony 2 - товарищ идет по правильному пути. =)