Ммм... немного расплывчатый вопрос, что ты хочеш делать с данными пришедшими из формы? Работа с базами данных? mysqli, pdo. Проверки на пустоту и прочее тучи функций есть, что сложного принять данные из формы, проверить и присвоить их переменной? Не понимаю... И при чем тут ООП?

Какая же у меня мания оптимизации =)

bulgakov
Что я имею ввиду, вот есть класс обработки данных из формы (допустим 3 поля)
Как это выглядит у меня (напомню я только начинаю учить ООП)

   function __construct($name, $pass) {
        $this->name = !empty($name)?mysql_real_escape_string($name):NULL;
        $this->pass = !empty($pass)?mysql_real_escape_string($pass):NULL;
        $this->encode = !empty($pass)?md5($pass):NULL;          
   }  

И заносим это в БД

   function insert_user() {
    $sql = "INSERT INTO `".self::$table."` SET 
            `name` = '".$this->clean_text($this->name)."',
            `pass` = '".$this->clean_text($this->pass)."',
            `encode` = '".$this->encode."'
            ";
    return connect_Db::sql($sql);
   }

Вот! То есть ели я захочу добавить еще одно поле например - email, то мне придется вручную переписать кусок класса, то есть нужно будет что бы уже методвы были вот такие

   function __construct($name, $pass, $email) {
        $this->name = !empty($name)?mysql_real_escape_string($name):NULL;
        [b]$this->email = !empty($email)?mysql_real_escape_string($email):NULL;[/b]
        $this->pass = !empty($pass)?mysql_real_escape_string($pass):NULL;
        $this->encode = !empty($pass)?md5($pass):NULL;          
   }  

и другой метод соответственно -

   function insert_user() {
    $sql = "INSERT INTO `".self::$table."` SET 
            `name` = '".$this->clean_text($this->name)."',
            `pass` = '".$this->clean_text($this->pass)."',
            `encode` = '".$this->encode."',
            `email` = '".$this->email."'
            ";
    return connect_Db::sql($sql);
   }

Так же?
А мне интересно реально ли написать класс кот. бы сам все это делал! А мне осталось бы только вставить в форму новый input.

Для этого и нужны модели в MVC. В модели мы можем описать правила для var, а в шаблоне сделать просто
$model = new Model;
echo CHtml::textField($model->var);

(пример для YiiFramework)

Т.е. здесь присутствует модель, объект модели с предопределнными правилами проверки перед отправкой, генератор html полей формы

ADiel
Хм... MVC - говорите. Но мне кажется не в этом дело. Я то пока что работаю только с одним классом и одной формой... Хотя даже если и MVC то как класс должен выгдядеть. Я думаю что тут необходимы переворачивать массивы и разбираться с $_POST.

Ну здесь уже не один класс. Должен быть контроллер как минимум, библиотека правил, сама модель, обработчик, класс работы с базой данных... По сути изобретение фреймворка, велосипедFramework =)

Я бы не стал писать в бд все, что приходит постом. Нужно делать какой то паттерн входящих данных.

А поля лучше делать массивом, например
<input type = "text" name = "loginform[password]" />
<input type = "text" name = "loginform[login]" />

И получим на входе массив $_POST['loginform'], а если тправляем сразу несколько форм, или сохранем несколько моделей в одной форме, будет очень удобно.

ADiel
Ну и ладно! Часика полтора у меня есть, все равно отдыхаю, почему бы не побезобразничать. Значица так. Что касается обработки. Начало (только не ржать)

    public $key;
    
    public function __construct($post){
        foreach ($post as $key=>$value){
                $this->key[$key] = !empty($value)?$value:NULL;
        }
    }

Количество сайтов в сети растёт в геометрической прогрессии и некоторые программисты удивляются, почему потратив столько времени и гордясь тем что сайт работает по принципу MVC и код круче крутого уокера. Но однако нисмотря на силы, вложенные в раскрутку, нет посещаемости, ведь столько сил потрачено на написание удобной формы регистрации причём на ajax с подтверждением по почте. Вот тут и есть корень зла. А зачем мне, ленивому пользователю регистрироваться если на следующей ссылке в гугле я получу всё, что мне нужно без всякого напряга.
Однако новички, с бараньим упорством, продолжают лепить формы регистрации, а тот главный вопрос - для кого сделан сайт, для себя или для пользователя, даже не возникает.
Есть много способов решить проблемму без всяких напрягов пользователя, но то ли фантазии не хватает, то ли эго мешает.
Это ИМХО и сорри если оффтоп.

А если массив? Добавь is_array и рекурсию
Guest, Конечно оффтоп! Мы тут над хорошей вещью думаем, а ты влез не по теме.
m4a1fox, отключи возможность писать гостям в теме =)

Кстати, зачем нам нужны пустыен значения, может просто не добавлять их в массив класса? И я не думаю, что в $this->key будут ключи. Может data назвать его или еще как?

Гостю - а если это форма для добавления новости в админке???? Я же писал в начале, форма - всему голова
ADiel

Ладно! Собственно как буду мучаться... Вот метод

    public $key;
    public $error = '<font color=red>Some errors isset</font>';
    
    public function __construct($post){
        foreach ($post as $key=>$value){
            if(!empty($value)){
                $this->key[$key] = !empty($value)?$value:NULL;
            }else{
                return $this->error;
            }
            
        }
    }
function Show(){
        return $this->key;
    }
}
$ex = new Experement($_POST);

И теперь форма

echo $ex->__construct($_POST);
print_r($ex->Show());

?>
<form action="" method="post">
<input name="name" type="text" />
<input name="text" type="text" />
<input type="submit" name="send" />
</form>

Вот теперь сколько бы не добавлял input'ов в массиве они все равно будут!!!

Вот проверка и возвращение многомерного массива.
Если массив должен быть одномерным, можно убрать
$data[$key] = $this->verifyArr($value);

class Example
{

public $key = array();

function __construct($post){
    $this->key = $this->verifyArr($post);
}

private function verifyArr($arr) {
    $data = array();
    foreach ($arr as $key=>$value) {
        if (is_array($value))
            $data[$key] = $this->verifyArr($value);
        else 
            $data[$key] = !empty($value)?$value:NULL;
    }
    return $data;
}


}

$array = new Example($_POST);

var_dump($array->key);

ADiel
Хорошо! Это как бы понятно.... а дальше гораздо интереснее.... Как занести все это в БД????
Прикиним -

    public function Insert(){
        $sql = "INSERT INTO `".self::$table."` 
        SET `".(вот тут первые значения массива)."` = '".(вот тут второе значение)."'
         ";
        echo $sql;
    }

И по окончанию, кол-ва значений в массиве - это должно останавливаться.... ))))
Мдя.... написал аж сам растроился... По ходу это - нереально!

Рано сдаешься. Реально.
Небезопасно так сразу писать в бд. Нужно писать правила проверки для полей ИМХО

Тут к нам на помощь приходят паттерны таблицы.
т.е. мы пишем в бд только то, что предопределено в самой таблице. Так же в verifyArr лучше сделать экранирование кавычек для предотвращение sql инъекций

ADiel
Ща сделам! Вот так

    public function Insert(){
        $sql = "INSERT INTO `".self::$table."`";
        $tree = $this->key;
        foreach($tree as $key=>$value){
        $sql .= '"'.$key.'" = "'.$value.'",';    
        }
        
        echo $sql;
    }

Гляньте а?! А я курить! :)

ну если о безопасности не задумываться совсем, то да.
Только опять же... is_array.
Пример если массив, пишем в другую таблицу.. Уже реляционность получается =)

Только есть одна проблема - с запятой в самом конце. И да, пока что совсем не думаю о безопасности!

/**
* @packade Велосипед 0.0.0.1а
*/
$tmp = array();
foreach($tree as $key=>$value){
        $tmp[]= '"'.$key.'" = "'.$value.'"';    
        }
$sql = implode(",",$tmp);

Логичнее разделить на класс проверок и конструктор sql запросов


$tree = $this->key; не стоит делать. Мы копируем массив, следовательно умножаем количество памяти, занятой $this->key на 2

Конструктор запросов будет выглядеть идеально так:
$sql = new Constructor("tablename");
$sql->command = "INSERT";
$sql->values($array);
$sql->insert($limit);

ADiel
Давай те не забывать что еще есть и send(submit)

    public function Insert(){
        $sql = "INSERT INTO `".self::$table."` SET ";
        $tree = $this->key;
        foreach($tree as $key=>$value){
            if($key != 'send'){
        $sql .= "`".$key."` = '".$value."', ";
        } 
        }
        echo $sql;
    }

Вот как тут заменить???

P.S Прикололо

/**
* @packade Велосипед 0.0.0.1а
*/

Так же сделаю )

Я ошибку допустил сначала. Посмотри мой пост

Что-то с implode у меня не ладится!

Что не так?
Посмотри еще раз на пост. я там правил код. Сначала была ошибка

Вроде получилось с implode

    public function Insert(){
         $sql = "INSERT INTO `".self::$table."` SET ";
         $tmp = array();
        $tree = $this->key;
        foreach($tree as $key=>$value){
            if($key != 'send'){
        $tmp[] = "`".$key."` = '".$value."'";
        } 
        }
        $sql .= implode(", ",$tmp);
        return $sql;
    }
    

Вот пишу код, правлю, а потом школота в плохом смысле этого слова, прашивает, почему за них код никто не пишет =)

Ответ очевиден, я думаю, сорри за оффтоп

ADiel
Это где такое, а то я могу и на свой счет это записать!

Везде. Дают весь код в тегах CODE без разметки вперемешку с html и пишут, типа мне нужно сюда добавить rand(1, 10) функций

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

Дальше по теме что?

Вот так ошибка ушла и вроде все заработало

    public function __construct($post){
        foreach ($post as $key=>$value){
            if(!empty($value)){
                $this->key[$key] = !empty($value)?$value:NULL;
            }else{
                return $this->error;
            }
            
        }
        [b]return true;[/b]
    }

Вот так применяем

 if(isset($_POST['send'])){
if($ex->__construct($_POST) === true){
echo $ex->Insert();
}else{
    echo $ex->__construct($_POST);
}
}

зачем 2 раза !empty($value) ?

ADiel
А.... не знаю! Привычка с тернарного вылезла! А вы как считаете правильно? (ну понятно что считает что один раз) но где убрать?

Ага! Вот так?

    public function __construct($post){
        foreach ($post as $key=>$value){
            if(!empty($value)){
                $this->key[$key] = $value;
            }else{
                return $this->error;
            }
        }
        return true;
    }

Именно.

return $this->error; не думаю, что уместно.
return заканчивает выполнение функции

ADiel
Впереди планеты всей....) только изменил

    public function __construct($post){
        foreach ($post as $key=>$value){
            if(!empty($value)){
                $this->key[$key] = $value;
            }else{
                return false;
            }
        }
        return true;
    }

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

ADiel

Паттерны. Завтра придумаю, как лучше реализовать.

ADiel
Продолжаем????????
Что там про паттерны?

Да, вот только сейчас совсем нет времени писать код.
Короче смотри:

   - Класс проверки
   - Абстрактная модель
   - Модель
      - Массив полей таблицы, содержащий правила проверки вида:

// Паттерн
$rules = array(
   // Поле row1 обязательное для заполнения, только цифры
   'row1'=>array("requrie", "numeric"),
   // Поле 2 может быть пустым, должно содержать буквы
   'row2'=>array("string"),
);

Для записи вызываем Модель, которая наследует функции абстракной модели, в которых прописаны дефолтные правила

Далее при проверке входящих данных, просто перебираем $rules и вызываем функции проверки

К примеру вызов проверок будет выглядеть так:

foreach ($model->rules as $var=>$rules) {
    $this->errors[] = VerifyClass::post($_POST[$var], $rules);
}

if (empty($this->errors)) {
    $model->genSql;
} else {
    $this->getErrors();
}

ADiel
Хм... Надо разбираться!

Далее просто наращиваем VerifyClass проверками

Значит нужно создавать еще один файл!!!! То есть делаем так. Есть класс, кот. заносит информацию в БД. Но перед тем как заносить в БД, нужно провести проверку данных.... Пока что есть единственный вопрос.... Вот верите, до сих пор не могу понять необходимость абстрактных методов.... Зачем они нужны?

Для определения основного функционала. Например есть 2 класса, первый абстрактный

abstract class A {
    private function verify($val) {
        return $val;
    }

    private function delete($val) {
        unset($val);
    }

}

class B extends A {

    public function test($var) {
        if (!$this->verify($val))
            $this->verify($val);
       }

}

Это нужно, если еть много классов, имеющих схожие или одинаковые функции.

ADiel
то есть абстрактные метода - это по сути некий вид проверки?????

Нет. Это своего рода библиотека методов.

ADiel
А! Если это библиотека методов(функций) тогда в ней можно создать некий метод, кот. не работает с реальными объектами... и потом применять его выполняя определенные действия... то есть по сути - функция в процедурном программировании....

Именно. Будем вызывать статическую функцию

Нарвался тут случайно, но если тема еще актуальна то почитай PHP_praktika_sozdaniya_web_saitov, Кузнецов, в частности глава 6