Bardzo prosze o rady dot. poprawy jakosci kodu w moim projekcie.
[...]
1
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/bitalgo/blob/master/src/main/java/bitalgo/strategy/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();
}