ну, у тебя тут всё в кучу свалено: и обработка строк и выполнение запросов, и их обработка, и чего здесь только нет.
Зачем наследуешь класс Db?

Я здесь вижу, как уже говорил, как минимум три, или даже четыре отдельно стоящих и организованных класса (StringProcessor_Composite, Db_Singleton, DataHandler, Error_Factory), собранных разными паттернами для облегчения дальнейшей разработки.
Дело в том, что класс Access_Facade - это, как бы, действие, которое выполняется средствами, некоторые из которых (например обработка строк) вообще висят в воздухе.

db Наследую чтобы можно было делать запрос вида $db->sql("запрос");
А как будет правильно выглядеть этот код можете показать пожалуйста, чтобы я знал как правильно писать в ооп) А то я всё в кучу собираю)

для примера могу привести класс регистрации для одной гостевухи. У тебя должно получиться примерно то же самое. Этот класс, кстати можно было уменьшить еще больше.

class Registration_Facade
{
    public static $data = array();
    private $compare    = null;
    public function __construct($post) {
        
        $this->post = (array)$post;
        $this->convert();
        
        $DE = new DefExp($this-> login);    // логин
        $DE -> check('[а-яa-z_-]+', 'iu')
            -> isEmpty('Пустой логин!'); 
            
        $DE = new DefExp($this-> password); // пароль
        $DE -> check('^.+$', '')
            -> isEmpty('Пустой пароль!');
            
        $DE = new DefExp($this-> name);     // имя
        $DE -> check('^[a-zа-я_-]{2,}$', 'ui')
            -> isEmpty('Пустое имя!');        
            
        $this->comparePass();
        DataBase::addUser();        
        showErrors();
    }
    private function convert() {
        
        foreach($this->post as $k=>$v) {
            $this->$k = array($k, $v);
        }
    }
    private function comparePass() {
        if($this->password[1] == $this->repass[1]){
            $this->compare = true;
        }
    }
}

Жесть)

ну, а чего ты хотел? Класс, он на то и класс, чтоб скопипастил его и он сразу же начал работать на/в (нужное подчеркнуть) другой системе без всяких доделок. Когда применяешь ООП, надо иметь четкое представление о том, нужно ли это. Насколько гибок и масштабируем должен быть код. Если такой задачи не стоит, то ООП нахрен не нужно

Мда, то же говнокод,
здесь мы и логику проводим,
здесь же и валидатор,
здесь мы и модель создаём,
здесь мы и выводим html код, хотя бы даже если он и выводит сообщение об ошибке.
Я представляю если мне захочется добавить более подробное освещение ошибки или стиль оформления это по всем классам лазить и менять код.

прикольно, дубляж а зачем

Error::getErrors();
Error::$errors

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

Дак вот именно что модель смешана с логикой регистрации и логикой валидации, если посмотреть здесь фигурирую три класса абстрактных данных,
класс пользовательской регистрации самой
класс связующийся с БД сама модель
и класс валидации
ну конечно класс ошибок

Вы имеете ввиду, что нужно разнести все по классам, а пока здесь это все в голом виде, правильно я Вас понял?

Вот только не приложу, как можно обыграть это

        $DE = new DefExp($this-> login);    // логин        
        $DE -> check('[а-яa-z_-]+', 'iu')            
            -> isEmpty('Пустой логин!'); 
. . .

Кажется, то, что есть сейчас, вполне удобно и можно оставить без изменений.