Senior

public class Accumulator { private Map<String, Object> data = new HashMap<>(); // esamas datu kopums private List<Map<String, Object>> dataList = new ArrayList<>(); // datu partija private Map<Instant, String> responses = new HashMap<>(); // atbildes no servera, kas saņem datus private int dataId; /** * Pievieno lauku field ar vērtību value kopumam */ public void add(String field, String value) { data.put(field, value); } /** * Atgriež vērtību pēc lauka nosaukuma field */ public <T> T get(String field) { return (T) data.get(field); } /** * Atgriež atbildes no servera, kas saņem datus */ public String getResponses() { String out = ""; for (String resp : responses.values()) { out += "; " + resp; } return out; } /** * Pabeidz pašreizējo datu kopuma vākšanu un nosūta to */ public synchronized void push() { dataList.add(data); if (dataList.size() == 100) { send(); dataList.clear(); } data = new HashMap<>(); data.put("id", dataId++); } /** * Nosūta partiju uz serveri, kas saņem datus */ private void send() { RestTemplate restTemplate = new RestTemplate(); HttpEntity<List<Map<String, Object>>> request = new HttpEntity<>(dataList); String url = "localhost:20808/msg/incoming"; ResponseEntity<String> response = restTemplate.exchange(url, HttpMethod.GET, request, String.class); responses.put(Instant.now(), response.getBody()); } }

sobes.tech AI

Atsakymas iš AI

В данном классе есть несколько проблем и потенциальных улучшений:

Метод send() использует HTTP GET для отправки данных в теле запроса, что не соответствует стандартам HTTP. Для отправки данных следует использовать POST.
RestTemplate создается заново при каждом вызове send(), что неэффективно. Лучше создать один экземпляр и переиспользовать.
В методе getResponses() конкатенация строк через += в цикле неэффективна, лучше использовать StringBuilder.
В методе push() поле "id" добавляется в новый data после очистки, но dataId не инициализирован явно, что может привести к проблемам.
Отсутствует обработка ошибок при отправке данных.

Пример исправленного метода send():

private final RestTemplate restTemplate = new RestTemplate();

private void send() {
    HttpEntity<List<Map<String, Object>>> request = new HttpEntity<>(dataList);
    String url = "http://localhost:20808/msg/incoming";
    ResponseEntity<String> response = restTemplate.exchange(url, HttpMethod.POST, request, String.class);
    responses.put(Instant.now(), response.getBody());
}

Также стоит добавить обработку исключений и инициализацию dataId в конструкторе.

Таким образом, класс аккумулирует данные в пачки по 100 элементов и отправляет их на сервер, сохраняя ответы с временными метками.

Ankstesnis Kitas