CODEWITHSUNDEEP
rusttraits
Slater Th
Slater Th

Reputation: 33

Cannot call rusqlite's query because it expects the type &[&rusqlite::types::ToSql]

I want to use a prepared statement with rusqlite. Rusqlite implements the trait ToSql for String, &str and a bunch of other types:

extern crate rusqlite;

use rusqlite::Connection;

fn main() {
    let mut connection = Connection::open("C:\\test_db.db").unwrap();

    let mut cached_statement = connection
        .prepare_cached("SELECT ?, ?, ? FROM test")
        .unwrap();

    let vec_values = vec![
        &"test1".to_string(),
        &"test2".to_string(),
        &"test3".to_string(),
    ];

    let rows = cached_statement.query(vec_values.as_slice()).unwrap();
}

This does not compile with the error:

error[E0308]: mismatched types
  --> src/main.rs:18:39
   |
18 |     let rows = cached_statement.query(vec_values.as_slice()).unwrap();
   |                                       ^^^^^^^^^^^^^^^^^^^^^ expected trait rusqlite::types::ToSql, found struct `std::string::String`
   |
   = note: expected type `&[&rusqlite::types::ToSql]`
              found type `&[&std::string::String]`

Upvotes: 3

Views: 2024

Answers (1)

Shepmaster
Shepmaster

Reputation: 431031

The compiler message isn't lying to you. You have a &[&String] not a &[&ToSql]. A trait object is a different type and often a different size from the underlying type; both are important considerations when packing values into a vector.

Another problem is that you cannot create a String, take a reference to it, then store that in a variable. The String would be deallocated immediately, leaving a dangling reference, so the compiler prevents that.

The easiest thing you can do is to create a new Vec that contains the trait object references:

let vec_values = vec![
    "test1".to_string(),
    "test2".to_string(),
    "test3".to_string(),
];

let query_values: Vec<_> = vec_values.iter().map(|x| x as &dyn ToSql).collect();

let _rows = cached_statement.query(&query_values).unwrap();

(complete example)

Or if you wanted an overly-generic function to perform the conversion:

fn do_the_thing<'a, I, T: 'a>(things: I) -> Vec<&'a dyn ToSql>
where
    I: IntoIterator<Item = &'a T>,
    T: ToSql,
{
    things.into_iter().map(|x| x as &dyn ToSql).collect()
}

let _rows = cached_statement.query(&do_the_thing(&vec_values)).unwrap();

(complete example)

In many cases, you can use the params! or named_params! macro:

let a = "test1".to_string();
let b = "test2".to_string();
let c = "test3".to_string();

let _rows = cached_statement.query(params![a, b, c]).unwrap();

(complete example)

Upvotes: 6

Related Questions