Issue #1043: Reduced too high amount of constructor arguments of `Imm…

…utableTopicPath`. * `ImmutableTopicPathBuilder` is now a private nested class of `ImmutableTopicPath` to be able to refer to to its instance field in constructor. * An instance of `ImmutableTopicPath` can now only be obtained via builder. Thus removed static factory methods. * Moved parsing of TopicPath string representation from `ProtocolFactory` to `ImmutableTopicPath` to make use of the nested builder. * Reworked parsing of `TopicPath` from string to provide meaningful context in case of errors. * Adjusted some Javadoc comments to reflect the code. Signed-off-by: Juergen Fickel <juergen.fickel@bosch.io>
eclipse-ditto · Apr 30, 2021 · 434f598 · 434f598
1 parent a5ef80e
commit 434f598
Show file tree

Hide file tree

Showing 11 changed files with 1,086 additions and 804 deletions.
diff --git a/protocol-adapter/src/main/java/org/eclipse/ditto/protocoladapter/ImmutableTopicPath.java b/protocol-adapter/src/main/java/org/eclipse/ditto/protocoladapter/ImmutableTopicPath.java
diff --git a/...ol-adapter/src/main/java/org/eclipse/ditto/protocoladapter/ImmutableTopicPathBuilder.java b/...ol-adapter/src/main/java/org/eclipse/ditto/protocoladapter/ImmutableTopicPathBuilder.java