VPF::Укоротить код по получению массива засчет класса

Win MK 32

Дата 10.6.2014, 14:47 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 492
Регистрация: 15.7.2002

Репутация: нет
Всего: нет

Есть примерно такой код

Код


class sgm_users{
//...
    public function get_by_id($id){
        if(is_int($id)){
            $this->query = $this->dbconnection->prepare("SELECT * FROM sgm_users WHERE id=".$id.";");
            $this->query->execute();
            $this->query->setFetchMode(PDO::FETCH_ASSOC);
            if($row = $this->query->fetch()) return $row;
        }
    }
//...
}
//...
$users_obj = new sgm_users();
array $user101 = $users_obj->get_by_id(101);

Как я это понимаю? $users_obj - это некий объект класса sgm_users. Он был нужен, а потом вдруг не нужОн стал smile

А вот $user101 - некий объект или массив с данными пользователя (поля профиля, пароль и т.д.), не связанный с классом алиментами и прочими штучками-свистелками.

Я хочу чтобы подобная кондиция достигалась более коротко:

Код

$user101 = new sgm_users(101);

Как это сделать?
P.S.
Очень прошу, НЕ думайте, что задача сводится к конвертации объекта в массив

Это сообщение отредактировал(а) Win MK 32 - 10.6.2014, 14:47

Aliance

Дата 10.6.2014, 15:25 (ссылка)

(нет голосов)

Загрузка ...

I ♥ <script>

Профиль
Группа: Модератор
Сообщений: 6418
Регистрация: 2.8.2004
Где: spb

Репутация: 14
Всего: 137

Ну так напишите логику в конструкторе.

Код


class sgm_users {
//...
    public function __construct($id) {
        return $this->_get_by_id($id);
    }

    private function _get_by_id($id) {
        if(is_int($id)){
            $this->query = $this->dbconnection->prepare("SELECT * FROM sgm_users WHERE id=".$id.";");
            $this->query->execute();
            $this->query->setFetchMode(PDO::FETCH_ASSOC);
            if($row = $this->query->fetch()) return $row;
        }
    }
//...
}
//...
$user101 = new sgm_users(101);

И что этот вопрос делает в треде "для профи" не понятно. Переношу в общие вопросы.

Добавлено через 3 минуты и 10 секунд
Далее по качеству кода...

1. Метод get_by_id возвращает значение, только если принимает числовой аргумент. В противном случе ничего не возвращается, но и не кидается исключение. Это логически не верно.
2. Какой смысл PDO пользоваться подготовленными выражениями, если вы делаете уязвимую конкатенацию в запросе? Читать пример здесь.

--------------------

• Все о событиях
• Объекты и массивы
• ОКНА И ФРЕЙМЫ

JAVA ≠ JavaScript

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| PHP: Общие вопросы \| Следующая тема »