Skip to content

Commit 352c9ba

Browse files
committed
better refactor
1 parent 28cfdc5 commit 352c9ba

File tree

1 file changed

+41
-14
lines changed
  • AndroidSDKCore/src/main/java/com/leanplum/internal

1 file changed

+41
-14
lines changed

AndroidSDKCore/src/main/java/com/leanplum/internal/Request.java

Lines changed: 41 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -487,32 +487,59 @@ private static class RequestsWithEncoding {
487487
String jsonEncodedString;
488488
}
489489

490-
private RequestsWithEncoding getRequestsWithEncodedString() {
490+
private RequestsWithEncoding getRequestsWithEncodedStringForErrors() {
491+
491492
List<Map<String, Object>> unsentRequests = new ArrayList<>();
492493
List<Map<String, Object>> requestsToSend;
493-
// Check if we have localErrors, if yes then we will send only errors to the server.
494-
if (localErrors.size() != 0) {
495-
String uuid = UUID.randomUUID().toString();
496-
for (Map<String, Object> error : localErrors) {
497-
error.put(UUID_KEY, uuid);
498-
unsentRequests.add(error);
499-
}
500-
requestsToSend = unsentRequests;
501-
} else {
502-
unsentRequests = getUnsentRequests();
503-
requestsToSend = removeIrrelevantBackgroundStartRequests(unsentRequests);
494+
String jsonEncodedRequestsToSend;
495+
496+
497+
String uuid = UUID.randomUUID().toString();
498+
for (Map<String, Object> error : localErrors) {
499+
error.put(UUID_KEY, uuid);
500+
unsentRequests.add(error);
504501
}
502+
requestsToSend = unsentRequests;
503+
jsonEncodedRequestsToSend = jsonEncodeUnsentRequests(unsentRequests);
504+
505+
RequestsWithEncoding requestsWithEncoding = new RequestsWithEncoding();
506+
requestsWithEncoding.unsentRequests = unsentRequests;
507+
requestsWithEncoding.requestsToSend = requestsToSend;
508+
requestsWithEncoding.jsonEncodedString = jsonEncodedRequestsToSend;
509+
510+
return requestsWithEncoding;
511+
}
505512

506-
String jsonEncodedRequestsToSend = jsonEncodeUnsentRequests(unsentRequests);
513+
private RequestsWithEncoding getRequestsWithEncodedStringStoredRequests() {
514+
List<Map<String, Object>> unsentRequests;
515+
List<Map<String, Object>> requestsToSend;
516+
String jsonEncodedRequestsToSend;
507517

508518
RequestsWithEncoding requestsWithEncoding = new RequestsWithEncoding();
519+
unsentRequests = getUnsentRequests();
520+
requestsToSend = removeIrrelevantBackgroundStartRequests(unsentRequests);
521+
jsonEncodedRequestsToSend = jsonEncodeUnsentRequests(unsentRequests);
522+
523+
509524
requestsWithEncoding.unsentRequests = unsentRequests;
510-
requestsWithEncoding.requestsToSend= requestsToSend;
525+
requestsWithEncoding.requestsToSend = requestsToSend;
511526
requestsWithEncoding.jsonEncodedString = jsonEncodedRequestsToSend;
512527

513528
return requestsWithEncoding;
514529
}
515530

531+
private RequestsWithEncoding getRequestsWithEncodedString() {
532+
RequestsWithEncoding requestsWithEncoding;
533+
// Check if we have localErrors, if yes then we will send only errors to the server.
534+
if (localErrors.size() != 0) {
535+
requestsWithEncoding = getRequestsWithEncodedStringForErrors();
536+
} else {
537+
requestsWithEncoding = getRequestsWithEncodedStringStoredRequests()
538+
}
539+
540+
return requestsWithEncoding;
541+
}
542+
516543
private void sendRequests() {
517544
RequestsWithEncoding requestsWithEncoding = getRequestsWithEncodedString();
518545

0 commit comments

Comments
 (0)