diff --git a/libraries-http-2/src/main/java/com/baeldung/okhttp/download/BinaryFileDownloader.java b/libraries-http-2/src/main/java/com/baeldung/okhttp/download/BinaryFileDownloader.java index a2e02c3be8..a7ef60f124 100644 --- a/libraries-http-2/src/main/java/com/baeldung/okhttp/download/BinaryFileDownloader.java +++ b/libraries-http-2/src/main/java/com/baeldung/okhttp/download/BinaryFileDownloader.java @@ -4,7 +4,6 @@ import okhttp3.OkHttpClient; import okhttp3.Request; import okhttp3.Response; import okhttp3.ResponseBody; -import org.jetbrains.annotations.NotNull; import java.io.IOException; import java.util.Objects; @@ -22,36 +21,13 @@ public class BinaryFileDownloader implements AutoCloseable { } public long download(String url) throws IOException { - Request request = createRequest(url); - Response response = executeRequest(request); - ResponseBody responseBody = getResponseBodyOrFail(response); - double length = getResponseLength(response); - return write(responseBody, length); - } - - @NotNull - private Request createRequest(String url) { - return new Request.Builder().url(url).build(); - } - - @NotNull - private Response executeRequest(Request request) throws IOException { - return client.newCall(request).execute(); - } - - private ResponseBody getResponseBodyOrFail(Response response) { + Request request = new Request.Builder().url(url).build(); + Response response = client.newCall(request).execute(); ResponseBody responseBody = response.body(); if (responseBody == null) { throw new IllegalStateException("Response doesn't contain a file"); } - return responseBody; - } - - private double getResponseLength(Response response) { - return Double.parseDouble(Objects.requireNonNull(response.header(CONTENT_LENGTH, "1"))); - } - - private long write(ResponseBody responseBody, double length) throws IOException { + double length = Double.parseDouble(Objects.requireNonNull(response.header(CONTENT_LENGTH, "1"))); return writer.write(responseBody.byteStream(), length); }