А зачем тебе класс для insert'a? Мне кажется там одного метода хватит?

PHPprogramer
Как я понимаю, метод - это не функция, значит если Вы написали слово метод, то я на автомате принимаю это, как то, что есть class.
Вот собственно класс, кот. у меня есть

class Member extends connect_Db {
    function return_user() {
        $sql = "SELECT * FROM `login`";
        $res = $this->sql($sql);
        return $res;
    }
}

class showLogin extends Member {
    function show_user() {
        $res = $this->return_user();
        if(mysql_num_rows($res) > 0) {
            while($row = mysql_fetch_assoc($res)) {
                $name[] = $row;
            }
        } else {
            echo 'String in db is empty';
        }
        return $name;
    }
}
$user = new showLogin;

Как я только что понял, это не верно, необходимо все писать в одном классе... так же получается? А потом просто вытаскивать методы и все...

А я вот не понимаю, что вы хотите сделать.
Чем вас не устраивает mysql_query("INSERT INTO ...") ? А если устраивает - то как-то подробнее надо описывать, зачем вам класс и что вы от него хотите.

вот тоже интересно можно ли как-то допустим по колл.элементов массива, определить имена колонок в таблице, чтоб ну вообще под всё подходило?

PHPprogramer, не соглашусь.

Я вообще против наследования классов, состоящий только из настроек подключений и классов работы с базой данных.
Это же разные сущности. Один класс работает с базой данных, а второй класс работает с пользователями. И вот второй класс просто пользуется услугами первого. Поэтому надо либо делать методы класс работы с бд статическими (лучше так), либо передавать объект этого класса в конструктор другим классам.

PHPprogramer
Поддерживаю... вот так оформил

class showLogin extends connect_Db {
    
        function return_user() {
        $sql = "SELECT * FROM `login`";
        $res = $this->sql($sql);
        return $res;
    }
    
                    
    function show_user() {
        $res = $this->return_user();
        if(mysql_num_rows($res) > 0) {
            while($row = mysql_fetch_assoc($res)) {
                $name[] = $row;
            }
        } else {
            echo 'String in db is empty';
        }
        return $name;
    }
}
$user = new showLogin;

Для вас neadekvat, меня полностью устраивает

neadekvat

neadekvat

PHPprogramer
ну так тоже можно, но я наследованием решил сделать?!

Ну говорят же что наследование здесь не нужно.

PHPprogramer
Знаю, что говорят. Тогда как без наследования?!
Вот класс для соединения с БД

    private $connection;
    
    function __construct() {
        $this->open_connection();
        //echo "Connection is set";
    }
    
    private function open_connection() {
        $this->connection = mysql_connect($this->DB_HOST, $this->DB_USER, $this->DB_PASS);
        if(!$this->connection) {
            die("Database connection faild: ".mysql_error());
        }
        else {
            $db_select = mysql_select_db($this->DB_NAME);
            if(!$db_select) {
                die ("Database selected faild: ".mysql_error());
            }
        }
        mysql_query("set names utf8") or die ("set names utf8 faild");
    }
    
    public function sql ($query) {
        $result = mysql_query($query, $this->connection);
        if(!result) {
            die("Database query failed: ".mysql_error());
        }
        return $result;
    }
}

$db = new connect_Db;

класс настроек default.php

class defaultVar extends Debug {
    var $BASE_URL   = "localhost/oop";
    var $DB_HOST    = "localhost";
    var $DB_USER    = "user";
    var $DB_PASS    = "";
    var $DB_NAME    = "oop";
}

И соббственно класс select'a

class showLogin extends connect_Db {
    
        private function return_user() {
        $sql = "SELECT * FROM `login`";
        $res = $this->sql($sql);
        return $res;
    }
    
                    
    function show_user() {
        $res = $this->return_user();
        if(mysql_num_rows($res) > 0) {
            while($row = mysql_fetch_assoc($res)) {
                $name[] = $row;
            }
        } else {
            echo 'String in db is empty';
        }
        return $name;
    }
}
$user = new showLogin;

Объясните, как туту быть без наследования?!

Мне кажется, что метод sql в классе соединение вообще не к месту.

PHPprogramer
Согласен! Попробую переделать!?

Здесь обращаешься к статической переменной.

PHPprogramer, потому что я лишь показал, как использовать без наследования, но весь код не перечитывал.
Верно будет обращаться как к статическим данным.

neadekvat
Да и как к статической, все равно ошибка?!

---

Спустя 1 минута, 59 секунд m4a1fox написал(а):

---

А ну ка... для меня тупого, чем плохо наследование?!

В общем где то бред но он работает :)

    function insert_user($post) {
        $sql = "INSERT INTO `login` SET `name` = '".$post['name']."',
                `pass` = '".$post['pass']."',
                `encode` = '".$post['email']."'
                ";
        //echo $sql;
        $this->sql($sql);
        return true;
    }

*всплеснул руками*

neadekvat
А как тогда?????????? Я уже в догадках теряюсь?!

neadekvat
Спасибо вам огромное за подсказки и терпение, но мне кажется, так как вы предложили для меня еще очень сложно.... извините.

neadekvat
хорошо! Наверно так и сделаю!

neadekvat по твоему коду, немного комментов:

1. Подключение к БД лучше вывести с класса в константы, либо переменные.
2. Для вывода ошибок лучше сделать отдельный метод.
3. Сделать возможность управлять выводом ошибок.

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

А как тогда ты выучишь ООП, если не будешь пытаться учить и тестить? Дерзай!