Code review

1

Bardzo prosze o rady dot. poprawy jakosci kodu w moim projekcie.
[...]

1

sciagnelam i przejzalam, musze przyznac ze jestem pod wrazeniem, biorac pod uwage ze w innym watku pytales sie o mozliwosci stazu :)
planujesz to rozwijac?

1

Całkiem, całkiem. Choć musiałbym przyjść dokładniej i się temu przyjrzeć.

1

https://github.com/javafanboi[...]trategy/SwapStrategyImpl.java
implementacje tego subscribe() wyciągnąłbym do metody, i dzieki method reference to może wyglądać tak:

  @Override
    public void execute() {
        swapStream
                .getCutOff()
                .observeOn(scheduler)
                .subscribe(costam::costam);
    }

btw co to w ifach się powtarza więc też można wyciągnąć do metody.

1

W ogóle w tym miejscu ta drabinka ifów wygląda źle. Szczególnie, że w obu elsach robisz dokładnie to samo.

1

Ogólnie to tam nie ma za wiele do oceny. Masz braki walidacji np

public void run(String[] commandLineParameters) {
        BigDecimal managedBTC = new BigDecimal(commandLineParameters[0]);
        BigDecimal offset = new BigDecimal(commandLineParameters[1]);
        String publicKey = commandLineParameters[2];
        String privateKey = commandLineParameters[3];

        SwapBroker swapBroker = swapBrokerFactory.create(publicKey, privateKey);
        SwapStrategy swapStrategy = swapStrategyFactory.create(swapBroker, managedBTC, offset);
        swapStrategy.execute();
    }

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0