Pet-проект на Rust нуждается в конструктивной критике

Ну вот сразу не хватает плашек, ведущих на crates.io и docs.rs. Без этого не удобно изучать проект, и нужно его отдельно искать где он опубликован.

Кстати, а чем sea orm не устроил?

Это не статья, а просто ссылка на Github. Еще и хабы выбраны про программирование, а в статье ни одной строчки кода.

В статье нужно описать, чем ваше решение лучше аналогов (Diesel, SeaORM, Sqlx), какие походы использовали, с какими трудностями столкнулись, в чем подобие именно Hibernate и т.п.

На основании примера:

1. (Не критицизм кода) ORM - никогда их не любил, но как способ попрактиковаться - OK

2. Почему метод connect почему-то блокирующий, в то время как всё остальное асинхронное?

3. Prepared statement нет? В примере format и protect - есть гарантии что он корректно всё заэскейпит?

4. conn.query(query.as_str()) - можно использовать AsRef чтобы можно было передовать строки в разной форме

5. Очень много клонирования. Можно принимать и ссылки. Ну а лучше разрешить и то и то, опять же с помощью AsRef

ну из нечитаемого

        match value {
            Some(v_opt) => {
                match v_opt {
                    None => {
                        None
                    }
                    Some(v) => {
                        let r = Z::from_str(v.as_str());
                        match r {
                            Ok(res) => {
                                Some(res)
                            }
                            Err(_) => {
                                None
                            }
                        }
                    }
                }

            }
            None => {
                None
            }
        }

Должно использовать идиому if let Some(x) / if let Ok(x) .

Аналогично но с циклом while let Some(x) .

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

Преобразование ошибок выглядит не очень. Возможно стоит подсмотреть как используют какой-нибудь thiserror/anyhow либо сделать impl Into/From для ваших ошибок.

Санация ввода определённо требует много тестов и вероятно выделение в собственную сущность или как минимум отдельный трейт с имплементацией для char, str и String , чтобы можно быо делать что-то вроде "praise the sun \0/".orm_escape() .

Сейчас тестов вроде примерно нисколько. Да и в общем по проекту тестов как-то мало. Отсюда же отсутствие понимания у людей как это будет использоваться в дальнейшем.

Про сериализацию наверное стоит подсмотреть какой-нибудь miniserde ибо текущий вариант кажется имплементит только какой-то сабсет json спеки. Ввиду отсутвия тестов - непонятно.

вот тут https://github.com/igumnoff/parvati/blob/3c3e6ff32872326bbef818192cc3d634962c9381/lib/src/sqlite.rs#L89 у вас прямо SQL инъекция в чистом виде - я вам еще в прошлый раз об этом говорил, но видимо вы не поняли. НЕЛЬЗЯ данные неизвестно откуда использовать в запросах напрямую, пусть даже с санитайзером, ТОЛЬКО передавая их через параметры.

ORM не умеет генерировать create table … (просто недостаточно информации о точных типах полей), обратно тоже не умеет создавать описания растовских структур из существующих таблиц. В примере create_table_sqlite.sql нет ограничений not null, хотя по коду в расте они подразумеваются.

let user_opt: Option<User> = conn.find_one(user_from_db.id as u64).run().await?;

А если я захочу первичный ключ сделать с типом char(4 char)?

Ну и всякие мелочи по оптимизации:

• update user set name = \"Mike\",age = 30 where id = 1 - не отслеживается список ИЗМЕНЁННЫХ полей, правильно так: update user set name = \"Mike\" where id = 1

• select * from user - ORM же должен знать ВСЕ поля и давать возможность выбрать только НЕКОТОРЫЕ: select id, name, age from user и select id, name from user

• insert into user (name,age) values (\"John\",30) - опа, явный косяк. А где указанный мной id=0?!

• select * from user where rowid = 1 - зачем оно делает селект сразу же после запроса и откуда оно узнало rowid? Правильно так:

insert into user (
    name,
    age
) values (
    "John",
    30
) returning
    id;