Collection of DTOs as input parameters

[dmitrydrugi]

Hi Team,

The library supports var args to pass multiple objects:

var account1 = new Account(1, "John");
var account2 = new Account(2, "Mary");
client.forSql("INSERT INTO accounts(accountId, name) VALUES(:accountId, :name)")
         .withParamSets(account1, account2)
         .execute();   

If we need to pass Collection we have to call toArray():

List<Account> accounts = List.of(new Account(1, "John"), new Account(2, "Mary")....);
client.forSql("INSERT INTO accounts(accountId, name) VALUES(:accountId, :name)")
         .withParamSets(accounts.toArray())
         .execute();   

Batch saving quite often is used with a list of objects and it would help to avoid extra call toArray().

Could you please add overloaded withParamSets method or a new method to pass java.util.Collection, for instance:

List<Account> accounts = List.of(new Account(1, "John"), new Account(2, "Mary")....);
client.forSql("INSERT INTO accounts(accountId, name) VALUES(:accountId, :name)")
         .withParamCollection(accounts)
         .execute();   

Thank you.

[sp0l]

Hi, there is already an overloaded method:

public Executor withParamSets(List<Object> params)

[dmitrydrugi]

Hi,

Okay, thank you.

I have found a bug. If we pass a list with one item only it doesn't work:

List<Account> accounts = List.of(new Account(1, "John"));
client.forSql("INSERT INTO accounts(accountId, name) VALUES(:accountId, :name)")
         .withParamSets(accounts)
         .execute();   

However if we use the second method it works as expected:

List<Account> accounts = List.of(new Account(1, "John"));
client.forSql("INSERT INTO accounts(accountId, name) VALUES(:accountId, :name)")
         .withParamSets(accounts.toArray())
         .execute();   

Thank you.