Fix SQL config defaults and validation

[jnsrnhld]

Denkst du, es ergibt Sinn, in den Configs Konstruktoren einzuführen, die auf die default-Werte Rücksicht nehmen, z.B. so:

@Data
@Builder
public class DatabaseConfig {

	private static final String DEFAULT_PRIMARY_COLUMN = "pid";

	private Dialect dialect;

	private String databaseUsername;

	private String databasePassword;

	private String jdbcConnectionUrl;

	@Builder.Default
	private String primaryColumn = DEFAULT_PRIMARY_COLUMN;

	@JsonCreator
	public DatabaseConfig(
			@JsonProperty("dialect") Dialect dialect,
			@JsonProperty("databaseUsername") String databaseUsername,
			@JsonProperty("databasePassword") String databasePassword,
			@JsonProperty("jdbcConnectionUrl") String jdbcConnectionUrl,
			@JsonProperty("primaryColumn") String primaryColumn
	) {
		this.dialect = dialect;
		this.databaseUsername = databaseUsername;
		this.databasePassword = databasePassword;
		this.jdbcConnectionUrl = jdbcConnectionUrl;
		this.primaryColumn = (primaryColumn != null) ? primaryColumn : DEFAULT_PRIMARY_COLUMN;
	}

}

Oder gibt es dafür eine elegantere Lösung? Mir war das vorher nicht so klar, dass Jackson die defaults (lombok Builder.Default, aber auch statische Werte) nicht einfach übernimmt.

[thoniTUB]

Hey @jnsrnhld ich habe das gerade zum ersten Mal gesehen: https://projectlombok.org/features/experimental/Jacksonized

Ein Test mit:

@Data
@Jacksonized
@Builder
public class DatabaseConfig {

	private static final String DEFAULT_PRIMARY_COLUMN = "pid";

	private Dialect dialect;

	private String databaseUsername;

	private String databasePassword;

	private String jdbcConnectionUrl;

	@Builder.Default
	private String primaryColumn = DEFAULT_PRIMARY_COLUMN;
}

und

class Scratch {
	public static void main(String[] args) throws IOException {
		final DatabaseConfig databaseConfig = Jackson.MAPPER.reader().readValue("{}", DatabaseConfig.class);

		System.out.println(databaseConfig);
	}
}

printet:

DatabaseConfig(dialect=null, databaseUsername=null, databasePassword=null, jdbcConnectionUrl=null, primaryColumn=pid)

[thoniTUB]

👌

[thoniTUB]

Sind hier logs dazugekommen? Ich sehe keinen logger im Diff

[jnsrnhld]

Nein, hatte ich nur zum Debugging drin und vergessen wieder raus zu nehmen.

[jnsrnhld]

Sehr cooler Fund mit dem Jacksonized!