Как бы Вы написали такой код?

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

Как бы Вы написали такой код?

Есть класс пользователь.

class user
{
    public function __construct($id_user)
    {
        $this->data = $db->query('SELECT * FROM users WHERE id_user = '.$id_user);
    }
}

Нужно вывести список пользователей. Имеем 2 варианта:

1. Структурный, простой, 1 запрос

$users = $db->query('SELECT * FROM users ...')->fetch_assoc();

foreach($users as $user)
{
    echo '<tr><td>'.$user['name'].'</td></tr>';
}

2. ОО-подход, 1 + N-юзеров запросов:

$res = $db->query('SELECT id_user FROM users ...');

while($id_user = $res->getOne())
{
    $users[] = new User($id_user);
}

foreach($users as $user)
{
    echo '<tr><td>'.$user->getByName('name').'</td></tr>';
}

что тут правильно как идеологически, так и практически?

 

[флоппик]

М. _ИМХО_, идеологически верно для работы со списком пользователей, сделать класс UserList ?

-~{}~ 27.08.08 15:02:

Который будет иметь метод getUser() возвращающий обьект типа User ?

 

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

Подумалось тут, что `беда` ООП в том, что на каждый элементарный запрос типа

$users = $db->query('SELECT * FROM users ...')->fetch_assoc();

надо писать

класс UserList

Скорее всего ты прав, да.

 

[флоппик]

Класс надо писать на одну сущность. Список пользователей вполне подходит под это описание )

 

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

флоппик
а контроллеры обычно тоже пишут под каждый action?

вот у меня есть 3 действия:
- список пользователей
- редактирование пользователей
- удаление пользователей

модель "пользователь" одна, а контроллер надо делать под каждое действие, так?

или твой класс в данном случае будет моделью для контроллера "список пользователей"?

 

[Gas]

triumvirat
в контроллере может использоваться несколько моделей, нет жёсткой привязки "один контроллер = одна модель".

модель "пользователь" одна, а контроллер надо делать под каждое действие, так?

не нужны отдельные контроллеры тут, сделай разные actions.

 

[флоппик]

или твой класс в данном случае будет моделью для контроллера "список пользователей"?

Да.

У тебя есть действия:
- список пользователей
- редактирование пользователя
- удаление пользователя

и, да:

в контроллере может использоваться несколько моделей

ЗЫ. Как то мы странно с ООП на MVC перескочили.

 

[Gas]

ЗЫ. Как то мы странно с ООП на MVC перескочили.

та я думаю triumvirat'у обе темы интересны.

 

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

не нужны отдельные контроллеры тут, сделай разные actions.

и как это может выглядеть? у меня сейчас один контроллер приходится для одного действия. а каждое действие - в отдельном подключаемом файле.

 

[Gas]

и как это может выглядеть

например, контроллер - класс, action - метод.

triumvirat
mvc патерн не описывает доскональной реализации и каждый делае как хочет, но посмотри лучше как сделано у других, хоть в zend framework.

 

[akxxiv]

У меня модель поддерживает и список объектов. в зависимости от запроса. Например:

Объект пользователя:

$obj = new Users($id); $obj->getName ......

или

$obj = new Users;  $obj->findById($id)->getName ......

или

$obj = new Users;  $obj->setWhere("AND id=?d", $id)->findRow()->getName.....

Список пользователей:

$users = new Users; 
$list = $users->setWhere(...)->find()->getObjectList();

foreach($list as $obj)
{
     $obj->getName ......
}

 

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

посмотри лучше как сделано у других, хоть в zend framework.

не не осилю... я ленивый на изучение чужого кода. мне его и так с лихвой хватило в моей жизни

 

[HraKK]

мне его и так с лихвой хватило в моей жизни

Йопть да у тебя код такой же гавнокод как и в битриксе и тд.
Лучше не занимайся херней, а юзай ZF

 

[флоппик]

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

 

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

Лучше не занимайся херней, а юзай ZF

да не хочу, хочу свое писать, пусть кривое, но свое.