Refactor FCP dialog

[jFCPlib.git] / src / main / java / net / pterodactylus / fcp / quelaton / SubscribeUskCommandImpl.java

diff --git a/src/main/java/net/pterodactylus/fcp/quelaton/SubscribeUskCommandImpl.java b/src/main/java/net/pterodactylus/fcp/quelaton/SubscribeUskCommandImpl.java
index cf9d10b..9977163 100644 (file)
--- a/src/main/java/net/pterodactylus/fcp/quelaton/SubscribeUskCommandImpl.java
+++ b/src/main/java/net/pterodactylus/fcp/quelaton/SubscribeUskCommandImpl.java
@@ -4,8 +4,7 @@ import java.io.IOException;
 import java.util.Optional;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.ExecutorService;
-import java.util.concurrent.atomic.AtomicBoolean;
-import java.util.concurrent.atomic.AtomicReference;
+import java.util.function.Supplier;
 
 import net.pterodactylus.fcp.IdentifierCollision;
 import net.pterodactylus.fcp.SubscribeUSK;
@@ -22,14 +21,18 @@ import com.google.common.util.concurrent.MoreExecutors;
  */
 public class SubscribeUskCommandImpl implements SubscribeUskCommand {
 
-       private static final RandomIdentifierGenerator IDENTIFIER = new RandomIdentifierGenerator();
        private final ListeningExecutorService threadPool;
        private final ConnectionSupplier connectionSupplier;
-       private final SubscribeUSK subscribeUSK = new SubscribeUSK(IDENTIFIER.generate());
+       private final SubscribeUSK subscribeUSK;
+       private final ActiveSubscriptions activeSubscriptions;
 
-       public SubscribeUskCommandImpl(ExecutorService threadPool, ConnectionSupplier connectionSupplier) {
+       public SubscribeUskCommandImpl(
+               ExecutorService threadPool, ConnectionSupplier connectionSupplier, Supplier<String> identifierGenerator,
+               ActiveSubscriptions activeSubscriptions) {
+               this.activeSubscriptions = activeSubscriptions;
                this.threadPool = MoreExecutors.listeningDecorator(threadPool);
                this.connectionSupplier = connectionSupplier;
+               subscribeUSK = new SubscribeUSK(identifierGenerator.get());
        }
 
        @Override
@@ -44,43 +47,28 @@ public class SubscribeUskCommandImpl implements SubscribeUskCommand {
 
        private Optional<UskSubscription> executeDialog() throws IOException, ExecutionException, InterruptedException {
                try (SubscribeUskDialog subscribeUskDialog = new SubscribeUskDialog()) {
-                       return subscribeUskDialog.send(subscribeUSK).get();
+                       if (subscribeUskDialog.send(subscribeUSK).get()) {
+                               UskSubscription uskSubscription = activeSubscriptions.createUskSubscription(subscribeUSK);
+                               return Optional.of(uskSubscription);
+                       }
+                       return Optional.empty();
                }
        }
 
-       private class SubscribeUskDialog extends FcpDialog<Optional<UskSubscription>> {
-
-               private final AtomicBoolean finished = new AtomicBoolean();
-               private final AtomicReference<SubscribedUSK> subscribedUSK = new AtomicReference<>();
+       private class SubscribeUskDialog extends FcpDialog<Boolean> {
 
                public SubscribeUskDialog() throws IOException {
-                       super(threadPool, connectionSupplier.get());
-               }
-
-               @Override
-               protected boolean isFinished() {
-                       return finished.get();
-               }
-
-               @Override
-               protected Optional<UskSubscription> getResult() {
-                       return Optional.ofNullable(subscribedUSK.get()).map(subscribedUSK -> new UskSubscription() {
-                               @Override
-                               public String getUri() {
-                                       return subscribedUSK.getURI();
-                               }
-                       });
+                       super(threadPool, connectionSupplier.get(), false);
                }
 
                @Override
                protected void consumeSubscribedUSK(SubscribedUSK subscribedUSK) {
-                       this.subscribedUSK.set(subscribedUSK);
-                       finished.set(true);
+                       setResult(true);
                }
 
                @Override
                protected void consumeIdentifierCollision(IdentifierCollision identifierCollision) {
-                       finished.set(true);
+                       finish();
                }
 
        }