Polishing

Rearrange methods and properties by name.

[#296]

Author: mp911de

src/main/java/io/r2dbc/postgresql/PostgresqlConnectionConfiguration.java

@@ -85,6 +85,8 @@ public final class PostgresqlConnectionConfiguration {
 
     private final int port;
 
+    private final int preparedStatementCacheQueries;
+
     private final String socket;
 
     private final String username;
@@ -95,15 +97,13 @@ public final class PostgresqlConnectionConfiguration {
 
     private final boolean tcpNoDelay;
 
-    private final int preparedStatementCacheQueries;
-
     private PostgresqlConnectionConfiguration(String applicationName, boolean autodetectExtensions, @Nullable Duration connectTimeout, @Nullable String database, LogLevel errorResponseLogLevel,
                                               List<Extension> extensions,
                                               ToIntFunction<String> fetchSize, boolean forceBinary,
                                               LogLevel noticeLogLevel, @Nullable String host,
-                                              @Nullable Map<String, String> options, @Nullable CharSequence password, int port, @Nullable String schema,
-                                              @Nullable String socket, boolean tcpKeepAlive, boolean tcpNoDelay, String username, SSLConfig sslConfig,
-                                              int preparedStatementCacheQueries) {
+                                              @Nullable Map<String, String> options, @Nullable CharSequence password, int port, int preparedStatementCacheQueries, @Nullable String schema,
+                                              @Nullable String socket, boolean tcpKeepAlive, boolean tcpNoDelay,
+                                              String username, SSLConfig sslConfig) {
         this.applicationName = Assert.requireNonNull(applicationName, "applicationName must not be null");
         this.autodetectExtensions = autodetectExtensions;
         this.connectTimeout = connectTimeout;
@@ -122,12 +122,12 @@ private PostgresqlConnectionConfiguration(String applicationName, boolean autode
 
         this.password = password;
         this.port = port;
+        this.preparedStatementCacheQueries = preparedStatementCacheQueries;
         this.socket = socket;
         this.username = Assert.requireNonNull(username, "username must not be null");
         this.sslConfig = sslConfig;
         this.tcpKeepAlive = tcpKeepAlive;
         this.tcpNoDelay = tcpNoDelay;
-        this.preparedStatementCacheQueries = preparedStatementCacheQueries;
     }
 
     /**
@@ -216,6 +216,10 @@ int getPort() {
         return this.port;
     }
 
+    int getPreparedStatementCacheQueries() {
+        return this.preparedStatementCacheQueries;
+    }
+
     @Nullable
     String getSocket() {
         return this.socket;
@@ -255,7 +259,6 @@ boolean isTcpNoDelay() {
     boolean isUseSocket() {
         return getSocket() != null;
     }
-
     SSLConfig getSslConfig() {
         return this.sslConfig;
     }
@@ -271,10 +274,6 @@ ConnectionSettings getConnectionSettings() {
             .build();
     }
 
-    int getPreparedStatementCacheQueries() {
-        return this.preparedStatementCacheQueries;
-    }
-
     private static String obfuscate(int length) {
 
         StringBuilder builder = new StringBuilder();
@@ -323,6 +322,8 @@ public static final class Builder {
 
         private int port = DEFAULT_PORT;
 
+        private int preparedStatementCacheQueries = -1;
+
         @Nullable
         private String schema;
 
@@ -354,8 +355,6 @@ public static final class Builder {
         @Nullable
         private String username;
 
-        private int preparedStatementCacheQueries = -1;
-
         private Builder() {
         }
 
@@ -402,9 +401,10 @@ public PostgresqlConnectionConfiguration build() {
             }
 
             return new PostgresqlConnectionConfiguration(this.applicationName, this.autodetectExtensions, this.connectTimeout, this.database, this.errorResponseLogLevel, this.extensions,
-                this.fetchSize, this.forceBinary, this.noticeLogLevel, this.host, this.options, this.password, this.port, this.schema, this.socket, this.tcpKeepAlive, this.tcpNoDelay, this.username
-                , this.createSslConfig(),
-                this.preparedStatementCacheQueries);
+                this.fetchSize, this.forceBinary, this.noticeLogLevel, this.host, this.options, this.password, this.port, this.preparedStatementCacheQueries, this.schema, this.socket, this.tcpKeepAlive, this.tcpNoDelay, this.username
+                ,
+                this.createSslConfig()
+            );
         }
 
         /**
@@ -580,6 +580,19 @@ public Builder port(int port) {
             return this;
         }
 
+        /**
+         * Configure the preparedStatementCacheQueries. The default is {@code -1}, meaning there's no limit. The value of {@code 0} disables the cache. Any other value specifies the cache size.
+         *
+         * @param preparedStatementCacheQueries the preparedStatementCacheQueries
+         * @return this {@link Builder}
+         * @throws IllegalArgumentException if {@code username} is {@code null}
+         * @since 0.8.1
+         */
+        public Builder preparedStatementCacheQueries(int preparedStatementCacheQueries) {
+            this.preparedStatementCacheQueries = preparedStatementCacheQueries;
+            return this;
+        }
+
         /**
          * Configure the schema.
          *
@@ -722,19 +735,6 @@ public Builder username(String username) {
             return this;
         }
 
-        /**
-         * Configure the preparedStatementCacheQueries. The default is {@code -1}, meaning there's no limit. The value of {@code 0} disables the cache. Any other value specifies the cache size.
-         *
-         * @param preparedStatementCacheQueries the preparedStatementCacheQueries
-         * @return this {@link Builder}
-         * @throws IllegalArgumentException if {@code username} is {@code null}
-         * @since 0.8.1
-         */
-        public Builder preparedStatementCacheQueries(int preparedStatementCacheQueries) {
-            this.preparedStatementCacheQueries = preparedStatementCacheQueries;
-            return this;
-        }
-
         @Override
         public String toString() {
             return "Builder{" +
@@ -751,8 +751,8 @@ public String toString() {
                 ", parameters='" + this.options + '\'' +
                 ", password='" + obfuscate(this.password != null ? this.password.length() : 0) + '\'' +
                 ", port=" + this.port +
+                ", preparedStatementCacheQueries='" + this.preparedStatementCacheQueries + '\'' +
                 ", schema='" + this.schema + '\'' +
-                ", username='" + this.username + '\'' +
                 ", socket='" + this.socket + '\'' +
                 ", sslContextBuilderCustomizer='" + this.sslContextBuilderCustomizer + '\'' +
                 ", sslMode='" + this.sslMode + '\'' +
@@ -762,7 +762,7 @@ public String toString() {
                 ", sslHostnameVerifier='" + this.sslHostnameVerifier + '\'' +
                 ", tcpKeepAlive='" + this.tcpKeepAlive + '\'' +
                 ", tcpNoDelay='" + this.tcpNoDelay + '\'' +
-                ", preparedStatementCacheQueries='" + this.preparedStatementCacheQueries + '\'' +
+                ", username='" + this.username + '\'' +
                 '}';
         }