Класс гостевой

[(Sandr)]

Подскажите, как в этом классе избавиться от switch'ей?

Может с помощью if'ов?

PHP код:

<?php session_start();
require_once "connect.php";
class Messages
{
        public $name,$mess,$date,$sel,$ins,$upd,$del,$id;
        public function Show()
        {
                $obj->sel = "SELECT * FROM `gb`";
                $sel = mysql_query($obj->sel);
                while($arr = mysql_fetch_array($sel))
                {
                        echo $arr['name'].'<br>'.$arr['mess'].'<br><hr>';
                }
        }
 
        public function Add($name,$mess)
        {
                $obj->name = $name;
                $obj->mess = $mess;
                $obj->ins = "INSERT INTO `gb` (name, mess)VALUES('{$obj->name}','{$obj->mess}')";
                if($obj->name == NULL or $obj->mess == NULL)
                {
                        echo 'Ошибка! Одна из переменных пуста!';
                }
                else
                {
                        $ins = mysql_query($obj->ins);
                        if($ins == true)
                        {
                                echo 'Запись успешно произведена';
                        }
                        else
                        {
                                echo 'Ошибка!';
                        }
                }
        }
 
        public function Del($id)
        {
                $obj->id = $id;
                $obj->del = "DELETE FROM `gb` WHERE id='{$obj->id}'";
                if(empty($obj->id))
                {
                        echo 'Ошибка! Ид сообщения пуст!';
                }
                else
                {
                        $del = mysql_query($obj->del);
                        if($del == true)
                        {
                                echo 'Сообщение удалено.';
                        }
                        else
                        {
                                echo 'Ошибка! Сообщение не удалено!';
                        }
                }
        }
        
        public function Edit($id)
        {
                $obj->id = $id;
                $obj->sel = "SELECT * FROM `gb` WHERE `id`='{$obj->id}'";
                $sel = mysql_fetch_array(mysql_query($obj->sel));
                echo '<form action="" method="POST">
                        <input type="text" name="name" value="'.$sel['name'].'"><br>
                        <input type="text" name="mess" value="'.$sel['mess'].'"><br>
                        <input type="submit">
                        </form>';
        }
}
 
switch($_GET['a'])
{
        case 'form':
        echo '<form action="?a=write" method="POST">
        <input type="text" name="name"><br>
        <input type="text" name="mess"><br>
        <input type="submit">
        </form>';
        exit;
        break;
        
        case 'write':
        $obj = new Messages;
        $obj->Add($_POST['name'],$_POST['mess']);
        echo '<br><a href="/">Главная</a>';
        exit;
        break;
}
 
$obj = new Messages;
echo $obj->Show();
echo '<a href="?a=form">Добавить</a>';
?>

[ssdm]

Может с помощью if'ов?

именно.
А чем switch не угодил ?

[(Sandr)]

Сообщение от ssdm

именно.
А чем switch не угодил ?

Мне один из прогаммистов сказал это: "ООП изобрели для того, чтобы избавиться от огромного количества switch-ей, по сути каждый case-блоков должен быть отдельным классом, которые наследуют от единого класса, а вы просто вызываете метод (полиморфизм - методы у них одинаковые), не заботясь что там за класс. Понятно, что тут скорее всего вам будет сложно избавиться от switch, так как вы вступаете в противоречие с моделью Интернет, которая не является объектно-ориентированной. Но стремиться нужно к тому, чтобы switch-ей в ООП коде не было."

[ssdm]

Предложите этому программисту в вашем примере убрать switch и переписать все используя полиморфизм. Интересно будет на это посмотреть.

ООП изобрели для того, чтобы избавиться от огромного количества switch-ей,

- нет, имхо это одна из причин и далеко не главная..

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

- бред..

В общем если хотите, оставmте switch, хотя я бы предпочел if else .

[(Sandr)]

ssdm, спасибо за ответ. Не мог бы ты ещё подсказать несколько вещей?
- Есть ли в самом классе (не в switch'е) г-код? Если есть, то можешь подсказать, как его исправить?
- Мне сказали, что желательно сделать так, чтоб одна форма могла использоваться в разных случаях? Например, как можно использовать форму, которая находится в " case 'form' " для создания и редактирования сообщения?
- как можно отделить код от вёрстки?

[ssdm]

как можно отделить код от вёрстки?

используй любой mvc фреймворк

- Мне сказали, что желательно сделать так, чтоб одна форма могла использоваться в разных случаях? Например, как можно использовать форму, которая находится в " case 'form' " для создания и редактирования сообщения?

Хранить её в темплейте ?

В общем воспользуйтесь каким нибудь фреймворком.

- Есть ли в самом классе (не в switch'е) г-код? Если есть, то можешь подсказать, как его исправить?

Почитайте про sql инъекции

[(Sandr)]

Сообщение от ssdm

используй любой mvc фреймворк

Хранить её в темплейте ?

В общем воспользуйтесь каким нибудь фреймворком.


Почитайте про sql инъекции

Что такое темплейт?
Насчёт скуль я знаю, просто решил не делать защиту, этот скрипт врятли где будет использоваться. А вот насчёт фреймворка(или шаблонизатора), то я как раз и спрашиваю про то как сделать свой простенький.. не хочу чужим чем-то ползоваться. Я просто пока что учусь, и решил осваивать ООП на примере гостевой.

[ssdm]

Что такое темплейт?

- template (шаблон).
Если у тебя цель использовать шаблонизатор или фреймворк в будущем , то не надо изобретать велосипед ,возьми что нить готовое. Тут я тебе посоветовать ничего не смогу , так как сам пишу под Perl используя Catalyst.
Если все таки упорно хочешь писать шаблонизатор сам, то как вариант хранить html код с местами для вставки динамического контента. Что то типа

<html>
<..some content..>
[%_content_%]
<..some content..>
</html>

А потом пропускть это через регулярку заменяя [%_content_%] на нужный тебе код.
Хотя возможно это не самая лучшая идея.

[8oOoRPM]

шблонизатор для шаблонизатора... забавно...

[8oOoRPM]

фрэймворк для гостевой... удачи...