Как лучше делать модели, на статических методах или на объектах?

Question

[Fagi]

Class user
{
public $user_id;

public function __construct($user_id = false)
{
if ($user_id) $this->user_id = $user_id;
else $this->user_id = USER_ID;
}

public function get_login()
{
// тут получение и возвращение логина пользователя
}
}

Или:

Class user
{
public static function get_login($user_id)
{
// тут получение и возвращение логина пользователя
}
}

Второй вариант для меня удобнее, так как меньше кода писать надо, как и при реализации классов, так и при их вызове. Но статические методы везде осуждают — говорят, что это плохая практика... Вот и задаю этот вопрос давно мучающий меня.

Answer 1

[Lander]

Вы вроде как используете классы, но всё равно пытаетесь писать в процедурном стиле. Не стоит использовать ООП, если вы его не понимаете, просто потому что это стильномодномолодёжно.

Answer 2

[Антон Шаманов]

Статика допустима в классах=хэлперах вроде коллекции методов для работы с массивами или файлами или фабриках\билдерах.

Answer 3

[Талян]

А как вы в статическом методе обратитесь к базе данных например? Прямо при вызове метода что ли передавать собрались? Не легче создать объект юзера, а потом получить в обычном методе this->id?

Comments

[Fagi]

Я так и делаю. Прямо при вызове передаю uid. Что в этом плохого?

[Талян]

Fagi,

я про id ничего не писал, я спросил про передачу соединения с БД

дак логин то как получаете? Передали вы ID, а дальше что в статической функции происходит - покажите. Возможно я дурак и что-то не понимаю, а у вас свой супер подход.

Покажите код вашей статичной функции.

Как из id получается login

[Fagi]

Анатолий, соединение с бд у меня глобально.

public static function get_login($uid)
{
   $sql = 'select login from users where id = ? limit 1';
   $sth = \db::$pdo->prepare($sql);
   $sth->execute([$uid]);
   return $sth->fetchColumn();
}

[Талян]

соединение с бд у меня глобально

Тогда в вашем случае - пофигу. Если конечно у класса нет неймспейсов, и для вызова функции не придется писать каждый раз

Vendor\Forum\Database\UsersActions\User::get_login($id);

Плюс никто нр мешает использовать self.

Кроме того как я понимаю, если у вас есть что-то типа CRM, которая работает с дочерними CRM в разных городах, и у каждой дочерней - своя БД, тоже будет не все так радужно.

В общем есть два путя:
1) делать так, как общепринято, и сделано это не просто так
2) либо делать как вам удобно, а для других проектов - всё в движке менять. Дело ваше - хоть на процедурном стиле пишите - ничего плохого в этом нет.

[Fagi]

Анатолий, буду таки делать на объектах. Хочу стремиться к более правильному коду. У меня еще такой вопрос есть, с которым я никак не могу разобраться, касамый моделей.

Рекомендуется на каждую таблицу делать свою модель. Правильно? То есть, в такой моделе не должно быть ничего другого, кроме как методов исключетельно для работы именно с этой таблицей? А что если мне понадобится получить данные одновременно с двух или более таблиц, то я путаюсь, непонятно в какой модели делать соответствующий метод? Или создавать новую модель, уже для конкретной задачи, а не для таблицы?

[Талян]

Fagi, нуу это рекомендуется со стороны ActiveRecord. Я этому правилу не следую. Мне западло создавать отдельно класс для юзера, отдельно для списка его полномочий, а потом что-то там с этим делать. У меня есть таблица юзеры, таблица user_privilegies, и всё это обрабатывается в одном классе users.

[Fagi]

Анатолий, Понятно. Спасибо вам за подсказки, Анатолий!