MVC: CRUD подобное API для моделей – best practices?

interface iUser
{
    public function __construct($email);
    public function add($email, $password);
    public function edit($id, array $fields);
    public function delete($id);
}

edit($id, array $fields)

$fields = ['role' => 'moderator']

if (is_array($id)) {
    // генерация запроса для нескольких пользователей
} else {
    // работаем только с одним
}

<?php
namespace Core\Models;
 
use PDO;
 
class User
{
 
// ============================== CRUD =========================================
 
    public function __construct($email)
    {
        $sth = Db::$dbh->prepare('SELECT * FROM users WHERE email = :email LIMIT 1');
        $sth->execute(['email' => $email]);
        $data = $sth->fetch(PDO::FETCH_ASSOC);
 
        // recursive queries for some fields – they're MUST be objects!
        $propObjects = ['role', 'payment', 'delivery'];
 
        if (!empty($data)) {
            foreach ($data as $propName => $propValue) {
                if (in_array($propName, $propObjects)) {
                    $this->$propName = $this->getAttr($propName, $propValue);
                } else {
                    $this->$propName = $propValue;
                }
            }
        }
    }
 
    public function add(array $info)
    {
        $fields = array_keys($info);
 
        foreach ($fields as $f) {
            $v[] = ':' . $f;
        }
 
        $sql = 'INSERT INTO users (' . implode(', ', $fields) . ') VALUES (' . implode(', ', $v) . ')';
        $sth = Db::$dbh->prepare($sql);
        return $sth->execute($info);
    }
 
    public function edit(array $info)
    {
        $id = (!empty($info['id'])) ? $info['id'] : $this->id;
        unset($info['id'], $info['action']);
 
        $arr = [];
        foreach ($info as $f => $v) {
            $arr[] = $f . ' = :' . $f;
        }
        $sql = 'UPDATE users SET ' . implode(', ', $arr) . ' WHERE id = :id';
        $sth = Db::$dbh->prepare($sql);
 
        if (is_array($id)) {
            foreach ($id as $i) {
                $info['id'] = $i;
                $status[] = $sth->execute($info);
            }
        } else {
            $info['id'] = $id;
        }
        $status[] = $sth->execute($info);
 
        return !in_array(false, $status);
    }
 
    public function delete(array $ids)
    {
        $sql = 'DELETE FROM users WHERE (';
        foreach ($ids as $id) {
            $sql .= 'id = ? OR ';
        }
        $sql .= '1=0)';
 
        $sth = Db::$dbh->prepare($sql);
        return $sth->execute($ids);
    }
 
// ============================== OTHER ========================================
 
    public function getAll($role=null)
    {
        // filter by role
        if ($role==null) {
            $sth = Db::$dbh->prepare('SELECT * FROM users');
            $sth->execute();
        } else {
            $sth = Db::$dbh->prepare('SELECT * FROM users WHERE role = :role');
            $sth->execute(['role' => $role]);
        }
        $data = $sth->fetchAll(PDO::FETCH_OBJ);
 
        // recursive queries for some fields
        $propObjects = ['role', 'payment', 'delivery'];
 
        $users = [];
        foreach ($data as $user) {
            if (!empty($user)) {
                foreach ($user as $propName => $propValue) {
                    if (in_array($propName, $propObjects)) {
                        $user->$propName = $this->getAttr($propName, $propValue);
                    } else {
                        $user->$propName = $propValue;
                    }
                }
                $avatar = 'img/avatars/' . $user->id . '.jpg';
                $user->avatar = (is_readable($avatar)) ? $avatar : 'img/avatars/0.jpg';
            }
            $users[] = $user;
        }
        return $users;
    }
 
// ============================== PRIVATE ======================================
 
    private function getAttr($name, $id)
    {
        // add postfix "s" for multiply form of object == name of table
        $name .= 's';
        $sth = Db::$dbh->prepare('SELECT * FROM ' . $name . ' WHERE id = :id LIMIT 1');
        $sth->execute(['id' => $id]);
        return $sth->fetchObject();
    }
 
}

    public function delete(array $ids)
    {
        $sql = 'DELETE FROM '.$this->table.' WHERE (';
        foreach ($ids as $id) {
            $sql .= 'id = ? OR ';
        }
        $sql .= '1=0)';
 
        $sth = Db::$dbh->prepare($sql);
        return $sth->execute($ids);
    }

$user = new User($data); // $data - это массив с данными
echo $user->name; // вася
$user->save(); // есть свойство $this->id - UPDATE, нет - INSERT
$user->delete(); // удаление по $this->id
 
 // найти по емейл
$user = User::findByEmail($email);
// или по свзяке емейл-пароль
$user = User::findByEmailAndPass($email, $pass);

$user = User::findByEmail($email);

// Создать пользователя из рандомных данных
$user = new User((array)$data);
// Найти пользователя в БД и создать его объект
$user = new User((string)$email);

// Найти в БД по полям и создать с данными оттуда
$user = (new User)->createFromDb($data);
// Создать из полученных данных (по сути, обычный геттер)
$user = (new User)->createWithData($data);

$user->save(); // есть свойство $this->id - UPDATE, нет - INSERT
$user->delete(); // удаление по $this->id

$user = new User($_POST);
$user->save();

<?php
namespace Core\Controllers;
 
use Core\Models\User;
use Core\Views\View;
 
class Profile
{
    public function edit()
    {
        $user = new User($_POST);
        $user->save();
        $view = new View(['user' => $user]);
    }
}

<?php
namespace Core;
 
use Core\Models\User;
use Core\Rbac;
 
class Router
{
    // some filelds && methods
 
    public function run(User $user)
    {
        $routes = explode('/', $_SERVER['REQUEST_URI']);
 
        $object = (isset($routes['1']) && mb_strlen($routes['1'])>1) ? $routes['1'] : 'main';
        $action = (isset($routes['2']) && mb_strlen($routes['2'])>1) ? $routes['2'] : 'index';
 
        if ((new Rbac)->isAllowed($user, $object, $action)) {
            $object = 'Core\\Controllers\\' . $object;
            if (class_exists($object) && method_exists($object, $action)) {
                call_user_func([(new $object), $action]);
            } else {
                // if not allowed – send to 404 error page
                $this->error404();
            }
        } else {
            // if controller || method doesn't exist – send to 404 error page
            $this->error404();
        }
    }
 
}

<?php
namespace Core;
 
use Core\Models\User;
use Core\Models\Db;
 
class Rbac
{
 
    public function isAllowed(User $user, $object, $action)
    {
        $resource = $object . '/' . $action;
        $sql = 'SELECT role FROM rbac WHERE resource = :resource';
        $options = [
            'resource' => $resource,
        ];
        $sth = Db::$dbh->prepare($sql);
        $sth->execute($options);
        return ($user->role->id <= $sth->fetchColumn()) ? true : false;
    }
 
}

header('Location: http://' . $_SERVER['HTTP_HOST'] . '/cart/confirm/?order=' . $idOrder);

$comment = new Comment($_POST);
$comment->save();

$user = new User($_POST['user_email']);
$data = $_POST;
$data['user'] = (!empty($user->email)) ? $user->email : null;
$comment = new Comment($data);

public static function redirect(
    $object='main',
    $action='index',
    array $options=null,
    $type='GET',
    $sendHeader=false
)
{
    // Код для переадресации
}