Skip to content
Snippets Groups Projects
Commit 2c32c38c authored by Martin Vogel's avatar Martin Vogel
Browse files

chore: remove static field initializers from enums, simplify fromValue loop

parent a35d68c1
No related branches found
No related tags found
1 merge request!30Resolve "Enum mit Static Field / Static Initializer"
Showing
with 58 additions and 116 deletions
...@@ -3,6 +3,8 @@ package de.fitko.fitconnect.api.domain.model.jwk; ...@@ -3,6 +3,8 @@ package de.fitko.fitconnect.api.domain.model.jwk;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.Arrays;
public enum AlgEnum { public enum AlgEnum {
PS512("PS512"), PS512("PS512"),
...@@ -27,11 +29,9 @@ public enum AlgEnum { ...@@ -27,11 +29,9 @@ public enum AlgEnum {
@JsonCreator @JsonCreator
public static AlgEnum fromValue(final String value) { public static AlgEnum fromValue(final String value) {
for (final AlgEnum algEnum : AlgEnum.values()) { return Arrays.stream(AlgEnum.values())
if (algEnum.value.equals(value)) { .filter(enumValue -> enumValue.value.equals(value))
return algEnum; .findFirst()
} .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
}
throw new IllegalArgumentException("Unexpected value '" + value + "'");
} }
} }
...@@ -3,6 +3,8 @@ package de.fitko.fitconnect.api.domain.model.jwk; ...@@ -3,6 +3,8 @@ package de.fitko.fitconnect.api.domain.model.jwk;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.Arrays;
public enum EEnum { public enum EEnum {
AQAB("AQAB"); AQAB("AQAB");
...@@ -25,11 +27,9 @@ public enum EEnum { ...@@ -25,11 +27,9 @@ public enum EEnum {
@JsonCreator @JsonCreator
public static EEnum fromValue(final String value) { public static EEnum fromValue(final String value) {
for (final EEnum eEnum : EEnum.values()) { return Arrays.stream(EEnum.values())
if (eEnum.value.equals(value)) { .filter(enumValue -> enumValue.value.equals(value))
return eEnum; .findFirst()
} .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
}
throw new IllegalArgumentException("Unexpected value '" + value + "'");
} }
} }
...@@ -3,6 +3,8 @@ package de.fitko.fitconnect.api.domain.model.jwk; ...@@ -3,6 +3,8 @@ package de.fitko.fitconnect.api.domain.model.jwk;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.Arrays;
public enum KeyOpsEnum { public enum KeyOpsEnum {
VERIFY("verify"), VERIFY("verify"),
...@@ -27,11 +29,9 @@ public enum KeyOpsEnum { ...@@ -27,11 +29,9 @@ public enum KeyOpsEnum {
@JsonCreator @JsonCreator
public static KeyOpsEnum fromValue(final String value) { public static KeyOpsEnum fromValue(final String value) {
for (final KeyOpsEnum keyOpsEnum : KeyOpsEnum.values()) { return Arrays.stream(KeyOpsEnum.values())
if (keyOpsEnum.value.equals(value)) { .filter(enumValue -> enumValue.value.equals(value))
return keyOpsEnum; .findFirst()
} .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
}
throw new IllegalArgumentException("Unexpected value '" + value + "'");
} }
} }
...@@ -3,6 +3,8 @@ package de.fitko.fitconnect.api.domain.model.jwk; ...@@ -3,6 +3,8 @@ package de.fitko.fitconnect.api.domain.model.jwk;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.Arrays;
public enum KtyEnum { public enum KtyEnum {
RSA("RSA"); RSA("RSA");
...@@ -25,11 +27,9 @@ public enum KtyEnum { ...@@ -25,11 +27,9 @@ public enum KtyEnum {
@JsonCreator @JsonCreator
public static KtyEnum fromValue(final String value) { public static KtyEnum fromValue(final String value) {
for (final KtyEnum ktyEnum : KtyEnum.values()) { return Arrays.stream(KtyEnum.values())
if (ktyEnum.value.equals(value)) { .filter(enumValue -> enumValue.value.equals(value))
return ktyEnum; .findFirst()
} .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
}
throw new IllegalArgumentException("Unexpected value '" + value + "'");
} }
} }
...@@ -3,8 +3,7 @@ package de.fitko.fitconnect.api.domain.model.metadata; ...@@ -3,8 +3,7 @@ package de.fitko.fitconnect.api.domain.model.metadata;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.HashMap; import java.util.Arrays;
import java.util.Map;
public enum EidasAdesProfile { public enum EidasAdesProfile {
...@@ -14,13 +13,6 @@ public enum EidasAdesProfile { ...@@ -14,13 +13,6 @@ public enum EidasAdesProfile {
HTTP_URI_ETSI_ORG_ADES_191_X_2_LEVEL_BASELINE_B_LTA("http://uri.etsi.org/ades/191x2/level/baseline/B-LTA#"); HTTP_URI_ETSI_ORG_ADES_191_X_2_LEVEL_BASELINE_B_LTA("http://uri.etsi.org/ades/191x2/level/baseline/B-LTA#");
private final String value; private final String value;
private final static Map<String, EidasAdesProfile> CONSTANTS = new HashMap<>();
static {
for (final EidasAdesProfile c : values()) {
CONSTANTS.put(c.value, c);
}
}
EidasAdesProfile(final String value) { EidasAdesProfile(final String value) {
this.value = value; this.value = value;
...@@ -38,12 +30,10 @@ public enum EidasAdesProfile { ...@@ -38,12 +30,10 @@ public enum EidasAdesProfile {
@JsonCreator @JsonCreator
public static EidasAdesProfile fromValue(final String value) { public static EidasAdesProfile fromValue(final String value) {
final EidasAdesProfile constant = CONSTANTS.get(value); return Arrays.stream(EidasAdesProfile.values())
if (constant == null) { .filter(enumValue -> enumValue.value.equals(value))
throw new IllegalArgumentException(value); .findFirst()
} else { .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
return constant;
}
} }
} }
...@@ -3,8 +3,7 @@ package de.fitko.fitconnect.api.domain.model.metadata; ...@@ -3,8 +3,7 @@ package de.fitko.fitconnect.api.domain.model.metadata;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.HashMap; import java.util.Arrays;
import java.util.Map;
public enum SignatureFormat { public enum SignatureFormat {
...@@ -13,14 +12,8 @@ public enum SignatureFormat { ...@@ -13,14 +12,8 @@ public enum SignatureFormat {
PDF("pdf"), PDF("pdf"),
ASIC("asic"), ASIC("asic"),
JSON("json"); JSON("json");
private final String value;
private final static Map<String, SignatureFormat> CONSTANTS = new HashMap<>();
static { private final String value;
for (final SignatureFormat c : values()) {
CONSTANTS.put(c.value, c);
}
}
SignatureFormat(final String value) { SignatureFormat(final String value) {
this.value = value; this.value = value;
...@@ -38,12 +31,10 @@ public enum SignatureFormat { ...@@ -38,12 +31,10 @@ public enum SignatureFormat {
@JsonCreator @JsonCreator
public static SignatureFormat fromValue(final String value) { public static SignatureFormat fromValue(final String value) {
final SignatureFormat constant = CONSTANTS.get(value); return Arrays.stream(SignatureFormat.values())
if (constant == null) { .filter(enumValue -> enumValue.value.equals(value))
throw new IllegalArgumentException(value); .findFirst()
} else { .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
return constant;
}
} }
} }
\ No newline at end of file
...@@ -3,8 +3,7 @@ package de.fitko.fitconnect.api.domain.model.metadata.attachment; ...@@ -3,8 +3,7 @@ package de.fitko.fitconnect.api.domain.model.metadata.attachment;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.HashMap; import java.util.Arrays;
import java.util.Map;
public enum Purpose { public enum Purpose {
...@@ -12,15 +11,8 @@ public enum Purpose { ...@@ -12,15 +11,8 @@ public enum Purpose {
ATTACHMENT("attachment"), ATTACHMENT("attachment"),
REPORT("report"); REPORT("report");
private final String value; private final String value;
private final static Map<String, Purpose> CONSTANTS = new HashMap<>();
static { Purpose(final String value) {
for (Purpose c : values()) {
CONSTANTS.put(c.value, c);
}
}
Purpose(String value) {
this.value = value; this.value = value;
} }
...@@ -35,12 +27,10 @@ public enum Purpose { ...@@ -35,12 +27,10 @@ public enum Purpose {
} }
@JsonCreator @JsonCreator
public static Purpose fromValue(String value) { public static Purpose fromValue(final String value) {
Purpose constant = CONSTANTS.get(value); return Arrays.stream(Purpose.values())
if (constant == null) { .filter(enumValue -> enumValue.value.equals(value))
throw new IllegalArgumentException(value); .findFirst()
} else { .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
return constant;
}
} }
} }
...@@ -3,20 +3,12 @@ package de.fitko.fitconnect.api.domain.model.metadata.attachment.signature; ...@@ -3,20 +3,12 @@ package de.fitko.fitconnect.api.domain.model.metadata.attachment.signature;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.HashMap; import java.util.Arrays;
import java.util.Map;
public enum AttachmentSignatureType { public enum AttachmentSignatureType {
SHA_512("sha512"); SHA_512("sha512");
private final String value; private final String value;
private final static Map<String, AttachmentSignatureType> CONSTANTS = new HashMap<>();
static {
for (final AttachmentSignatureType c : values()) {
CONSTANTS.put(c.value, c);
}
}
AttachmentSignatureType(final String value) { AttachmentSignatureType(final String value) {
this.value = value; this.value = value;
...@@ -34,12 +26,10 @@ public enum AttachmentSignatureType { ...@@ -34,12 +26,10 @@ public enum AttachmentSignatureType {
@JsonCreator @JsonCreator
public static AttachmentSignatureType fromValue(final String value) { public static AttachmentSignatureType fromValue(final String value) {
final AttachmentSignatureType constant = CONSTANTS.get(value); return Arrays.stream(AttachmentSignatureType.values())
if (constant == null) { .filter(enumValue -> enumValue.value.equals(value))
throw new IllegalArgumentException(value); .findFirst()
} else { .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
return constant;
}
} }
} }
...@@ -3,20 +3,12 @@ package de.fitko.fitconnect.api.domain.model.metadata.data; ...@@ -3,20 +3,12 @@ package de.fitko.fitconnect.api.domain.model.metadata.data;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.HashMap; import java.util.Arrays;
import java.util.Map;
public enum DataSignatureType { public enum DataSignatureType {
SHA_512("sha512"); SHA_512("sha512");
private final String value; private final String value;
private final static Map<String, DataSignatureType> CONSTANTS = new HashMap<>();
static {
for (final DataSignatureType c : values()) {
CONSTANTS.put(c.value, c);
}
}
DataSignatureType(final String value) { DataSignatureType(final String value) {
this.value = value; this.value = value;
...@@ -34,12 +26,10 @@ public enum DataSignatureType { ...@@ -34,12 +26,10 @@ public enum DataSignatureType {
@JsonCreator @JsonCreator
public static DataSignatureType fromValue(final String value) { public static DataSignatureType fromValue(final String value) {
final DataSignatureType constant = CONSTANTS.get(value); return Arrays.stream(DataSignatureType.values())
if (constant == null) { .filter(enumValue -> enumValue.value.equals(value))
throw new IllegalArgumentException(value); .findFirst()
} else { .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
return constant;
}
} }
} }
...@@ -3,21 +3,14 @@ package de.fitko.fitconnect.api.domain.model.metadata.data; ...@@ -3,21 +3,14 @@ package de.fitko.fitconnect.api.domain.model.metadata.data;
import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue; import com.fasterxml.jackson.annotation.JsonValue;
import java.util.HashMap; import java.util.Arrays;
import java.util.Map;
public enum MimeType { public enum MimeType {
APPLICATION_JSON("application/json"), APPLICATION_JSON("application/json"),
APPLICATION_XML("application/xml"); APPLICATION_XML("application/xml");
private final String value;
private final static Map<String, MimeType> CONSTANTS = new HashMap<>();
static { private final String value;
for (final MimeType c : values()) {
CONSTANTS.put(c.value, c);
}
}
MimeType(final String value) { MimeType(final String value) {
this.value = value; this.value = value;
...@@ -35,11 +28,9 @@ public enum MimeType { ...@@ -35,11 +28,9 @@ public enum MimeType {
@JsonCreator @JsonCreator
public static MimeType fromValue(final String value) { public static MimeType fromValue(final String value) {
final MimeType constant = CONSTANTS.get(value); return Arrays.stream(MimeType.values())
if (constant == null) { .filter(enumValue -> enumValue.value.equals(value))
throw new IllegalArgumentException(value); .findFirst()
} else { .orElseThrow(() -> new IllegalArgumentException("Unexpected value '" + value + "'"));
return constant;
}
} }
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment