Add QueryParams which deprecates HttpParameters

Motivation: - #1567 Modifications: - Fork our HTTP headers implementation into: - `QueryParams` - `QueryParamGetters` - `QueryParamsBase` - `QueryParamsBuilder` - .. and their implementations. - Note that it is intentional that I didn't introduce any common classes between `HttpHeaders` and `QueryParams`. They are just similar to each other accidentally and thus share nothing semantically close in common in my opinion. Please let me know if you do not like this approach - we should discuss. - Add support for `QueryParams` to annotated services. - Deprecate `HttpParameters`. Result: - Closes #1567 To-dos: - Add encoding/decoding methods like we did for `Cookie`. - Update `annotated-services.rst`
line · trustin · Dec 20, 2019 · Dec 10, 2019 · Dec 11, 2019 · Dec 11, 2019
commit 959f4023f5d7781360d5535934720e32e076d33c
diff --git a/core/src/main/java/com/linecorp/armeria/common/DefaultQueryParams.java b/core/src/main/java/com/linecorp/armeria/common/DefaultQueryParams.java
@@ -0,0 +1,51 @@
+/*
+ * Copyright 2019 LINE Corporation
+ *
+ * LINE Corporation licenses this file to you under the Apache License,
+ * version 2.0 (the "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at:
+ *
+ *   https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations
+ * under the License.
+ */
+package com.linecorp.armeria.common;
+
+import javax.annotation.Nullable;
+
+@SuppressWarnings("checkstyle:EqualsHashCode")
+final class DefaultQueryParams extends QueryParamsBase implements QueryParams {
+
+    static final DefaultQueryParams EMPTY = new DefaultQueryParams();
+
+    /**
+     * Creates an empty parameters.
+     */
+    private DefaultQueryParams() {
+        // Note that we do not specify a small size hint here, because a user may create a new builder
+        // derived from an empty parameters and add many parameters. If we specified a small hint,
+        // such a parameters would suffer from hash collisions.
+        super(DEFAULT_SIZE_HINT);
+    }
+
+    /**
+     * Creates a shallow copy of the specified {@link QueryParamsBase}.
+     */
+    DefaultQueryParams(QueryParamsBase params) {
+        super(params, true);
+    }
+
+    @Override
+    public QueryParamsBuilder toBuilder() {
+        return new DefaultQueryParamsBuilder(this);
+    }
+
+    @Override
+    public boolean equals(@Nullable Object o) {
+        return o instanceof QueryParams && super.equals(o);
+    }
+}