Code improvements.

Signed-off-by: Franz Wilhelmstötter <franz.wilhelmstoetter@gmail.com>

Author: jenetics

facilejdbc/src/main/java/io/jenetics/facilejdbc/Dctor.java

@@ -37,7 +37,6 @@
 /**
  * This interface is responsible for <em>deconstructing</em> a given record, of
  * type {@code T}, to a DB-<em>row</em> ({@link ParamValues}).
- *
  * {@snippet lang="java":
  * final Dctor<Book> dctor = Dctor.of(
  *     Dctor.field("title", Book::title),

facilejdbc/src/main/java/io/jenetics/facilejdbc/Records.java

@@ -107,7 +107,7 @@ private Records() {
 	 * @throws IllegalArgumentException if there are duplicate fields defined
 	 */
 	public static <T extends Record> Dctor<T> dctor(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Function<? super RecordComponent, String> toColumnName,
 		final List<? extends Dctor.Field<? super T>> fields
 	) {
@@ -187,7 +187,7 @@ private static <T extends Record> Dctor.Field<? super T> toFiled(
 	 */
 	@SafeVarargs
 	public static <T extends Record> Dctor<T> dctor(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Function<? super RecordComponent, String> toColumnName,
 		final Dctor.Field<? super T>... fields
 	) {
@@ -233,7 +233,7 @@ public static <T extends Record> Dctor<T> dctor(
 	 */
 	@SafeVarargs
 	public static <T extends Record> Dctor<T> dctor(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Dctor.Field<? super T>... fields
 	) {
 		return dctor(type, Records::toSnakeCase, List.of(fields));
@@ -343,7 +343,7 @@ public static String toSnakeCase(final String name) {
 	 * @throws NullPointerException if one of the arguments is {@code null}
 	 */
 	public static <T extends Record> RowParser<T> parser(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Function<? super RecordComponent, String> toColumnName,
 		final Function<? super RecordComponent, ? extends RowParser<?>> fields
 	) {
@@ -357,7 +357,7 @@ public static <T extends Record> RowParser<T> parser(
 			.map(toColumnName)
 			.toArray(String[]::new);
 
-		final Constructor<T> ctor = ctor(type);
+		final var ctor = ctor(type);
 
 		return (row, conn) -> {
 			final Object[] values = new Object[components.length];
@@ -402,7 +402,7 @@ public static <T extends Record> RowParser<T> parser(
 	 * @throws NullPointerException if one of the arguments is {@code null}
 	 */
 	public static <T extends Record> RowParser<T> parser(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Map<? super String, String> toColumnName,
 		final Map<? super String, ? extends RowParser<?>> fields
 	) {
@@ -442,7 +442,7 @@ public static <T extends Record> RowParser<T> parser(
 	 * @throws NullPointerException if one of the arguments is {@code null}
 	 */
 	public static <T extends Record> RowParser<T> parserWithColumnNames(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Function<? super RecordComponent, String> toColumnName
 	) {
 		return parser(type, toColumnName, component -> null);
@@ -470,7 +470,7 @@ public static <T extends Record> RowParser<T> parserWithColumnNames(
 	 * @throws NullPointerException if one of the arguments is {@code null}
 	 */
 	public static <T extends Record> RowParser<T> parserWithColumnNames(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Map<? super String, String> toColumnName
 	) {
 		return parser(type, toColumnName, Map.of());
@@ -502,7 +502,7 @@ public static <T extends Record> RowParser<T> parserWithColumnNames(
 	 * @throws NullPointerException if one of the arguments is {@code null}
 	 */
 	public static <T extends Record> RowParser<T> parserWithFields(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Function<? super RecordComponent, ? extends RowParser<?>> fields
 	) {
 		return parser(type, Records::toSnakeCase, fields);
@@ -529,7 +529,7 @@ public static <T extends Record> RowParser<T> parserWithFields(
 	 * @throws NullPointerException if one of the arguments is {@code null}
 	 */
 	public static <T extends Record> RowParser<T> parserWithFields(
-		final Class<T> type,
+		final Class<? extends T> type,
 		final Map<? super String, ? extends RowParser<?>> fields
 	) {
 		return parser(type, Records::toSnakeCase, cmp -> fields.get(cmp.getName()));
@@ -550,7 +550,7 @@ public static <T extends Record> RowParser<T> parserWithFields(
 	 * @return a new row-parser for the given record {@code type}
 	 * @throws NullPointerException if one of the arguments is {@code null}
 	 */
-	public static <T extends Record> RowParser<T> parser(final Class<T> type) {
+	public static <T extends Record> RowParser<T> parser(final Class<? extends T> type) {
 		return parser(type, Records::toSnakeCase, component -> null);
 	}
 

facilejdbc/src/main/java/io/jenetics/facilejdbc/Reflections.java

@@ -36,7 +36,7 @@ final class Reflections {
 	private Reflections() {
 	}
 
-	static <T extends Record> Constructor<T> ctor(final Class<T> type) {
+	static <T extends Record> Constructor<? extends T> ctor(final Class<? extends T> type) {
 		final Class<?>[] columnTypes = Stream.of(type.getRecordComponents())
 			.map(RecordComponent::getType)
 			.toArray(Class<?>[]::new);

facilejdbc/src/main/java/io/jenetics/facilejdbc/RowParser.java

@@ -247,7 +247,7 @@ default ResultSetParser<Optional<T>> singleOpt() {
 	 * @throws NullPointerException if the given {@code factory} is {@code null}
 	 */
 	default  <C extends Collection<T>>
-	ResultSetParser<C> collection(final Supplier<C> factory) {
+	ResultSetParser<C> collection(final Supplier<? extends C> factory) {
 		return collection(factory, Function.identity());
 	}
 
@@ -269,7 +269,7 @@ default ResultSetParser<List<T>> list() {
 	 *
 	 * @return a new parser witch parses a whole selection result
 	 */
-	default ResultSetParser<T[]> array(final Class<T> type) {
+	default ResultSetParser<T[]> array(final Class<? extends T> type) {
 		return (rs, conn) -> list().parse(rs, conn).toArray(length -> {
 				@SuppressWarnings("unchecked")
 				final var array = (T[])Array.newInstance(type, length);
@@ -279,7 +279,7 @@ default ResultSetParser<T[]> array(final Class<T> type) {
 
 	private <C1 extends Collection<T>, C2 extends Collection<T>>
 	ResultSetParser<C2> collection(
-		final Supplier<C1> factory,
+		final Supplier<? extends C1> factory,
 		final Function<? super C1, ? extends C2> mapper
 	) {
 		requireNonNull(factory);
@@ -461,7 +461,7 @@ static <A, B, T> RowParser<T> compose(
 	 * Returns a parser for a scalar not-null value.
 	 * {@snippet lang="java":
 	 * final String name = Query.of("SELECT name FROM person WHERE id = :id")
-	 *     .on(value("id", 23))
+	 *     .on(Param.value("id", 23))
 	 *     .as(scalar(String.class).single(), conn);
 	 * }
 	 *
@@ -473,15 +473,15 @@ static <A, B, T> RowParser<T> compose(
 	 * @return a parser for a scalar not-null value
 	 * @throws NullPointerException if the give {@code type} is {@code null}
 	 */
-	static <T> RowParser<T> scalar(final Class<T> type) {
+	static <T> RowParser<T> scalar(final Class<? extends T> type) {
 		return scalar(1, type);
 	}
 
 	/**
 	 * Returns a parser for a scalar not-null value.
 	 * {@snippet lang="java":
 	 * final String name = Query.of("SELECT id, name FROM person WHERE id = :id")
-	 *     .on(value("id", 23))
+	 *     .on(Param.value("id", 23))
 	 *     .as(scalar(2, String.class).single(), conn);
 	 * }
 	 *
@@ -496,15 +496,15 @@ static <T> RowParser<T> scalar(final Class<T> type) {
 	 * @return a parser for a scalar not-null value
 	 * @throws NullPointerException if the give {@code type} is {@code null}
 	 */
-	static <T> RowParser<T> scalar(final int index, final Class<T> type) {
+	static <T> RowParser<T> scalar(final int index, final Class<? extends T> type) {
 		return (row, conn) -> row.getObject(index, type);
 	}
 
 	/**
 	 * Returns a parser for a scalar not-null value.
 	 * {@snippet lang="java":
 	 * final String name = Query.of("SELECT id, name FROM person WHERE id = :id")
-	 *     .on(value("id", 23))
+	 *     .on(Param.value("id", 23))
 	 *     .as(scalar("name", String.class).single(), conn);
 	 * }
 	 *
@@ -519,7 +519,7 @@ static <T> RowParser<T> scalar(final int index, final Class<T> type) {
 	 * @return a parser for a scalar not-null value
 	 * @throws NullPointerException if the give {@code type} is {@code null}
 	 */
-	static <T> RowParser<T> scalar(final String name, final Class<T> type) {
+	static <T> RowParser<T> scalar(final String name, final Class<? extends T> type) {
 		return (row, conn) -> row.getObject(name, type);
 	}
 
@@ -887,7 +887,7 @@ static RowParser<String> csvLine() {
 	 * @param <T> the record type
 	 * @throws NullPointerException if the give {@code type} is {@code null}
 	 */
-	static <T extends Record> RowParser<T> record(final Class<T> type) {
+	static <T extends Record> RowParser<T> record(final Class<? extends T> type) {
 		return Records.parser(type);
 	}
 

facilejdbc/src/main/java/io/jenetics/facilejdbc/Stored.java

@@ -22,8 +22,6 @@
 /**
  * This class combines a record, stored in the DB, with its primary key.
  *
- * @since 2.1
- *
  * @param <K> the key type
  * @param <T> the record type, stored in the DB
  * @param id the primary key