Суммируйте часы и минуты Java

Question

Суммирование часов и минут приводит к неправильному ответу.

Вот 2 списка с часами.

[[0 часов: 0 минут: 0 секунд, 1 часов: 16 минут: 22 секунд, 0 часов: 1 минут: 53 секунд, 23 часов: 54 минут: 18 секунд], [0 часов: 8 минут: 22 секунд, 0 часы: 8 минут: 22 секунды, 0 часов: 8 минут: 22 секунды]] это ЧАСЫ

Затем я делю его на подсписки, чтобы рассчитать для каждого. Вот один из подсписок.

[0 часов: 0 минут: 0 секунд, 1 часов: 16 минут: 22 секунд, 0 часов: 1 минут: 53 секунд, 23 часов: 54 минут: 18 секунд]

Итак, ответ, который я получаю для этого подсписка: 1 час: 12 минут: 33 секунды, что неверно. Я предполагаю, что на это уйдет около 25 часов плюс несколько минут.

public List<String> getTheHoursWorked() {
    DateTimeFormatter parser = DateTimeFormatter.ofPattern("H 'hours :' m 'mins :' s 'sec'", Locale.US);
    final DateFormat dt = new SimpleDateFormat("HH:mm:ss");
    final Calendar c = Calendar.getInstance(TimeZone.getDefault(), Locale.getDefault());

    c.clear();
    long startingMS = c.getTimeInMillis();
    int counter = 0;

    for (int k = 0; k < hours.size(); k++){
        List<Object> shorter = new ArrayList<>();
        List<Object> temp;
        temp = (List<Object>) hours.get(k);
        
        long milliseconds = 0;
        for (int m = 0; m < shorter.size(); m++) {
            LocalTime odt = LocalTime.parse((CharSequence) shorter.get(m), parser);
            DateTimeFormatter formatter =
                    DateTimeFormatter.ofPattern("HH:mm:ss");
            String printDate = formatter.format(odt);
            try {
                milliseconds = milliseconds + (dt.parse(printDate).getTime() - startingMS);
                System.out.println(milliseconds + "MILISECONDS");
            } catch (ParseException e) {
                e.printStackTrace();
            }
        }
        hoursToString.add(String.valueOf(shorter));
        String s = milliseconds / 1000 % 60   + " seconds";
        String m = milliseconds /(60 * 1000) % 60 + " minutes";
        String h = milliseconds / (60 * 60 * 1000) % 24 + " hours";
        String together = h+":"+m+":"+s;
        togetherHours.add(together);
    }
    return togetherHours;
}

Answer 1

Шаги решения должны быть

1. Начните со времени 0:0
2. Выполните итерации списка и добавьте все часы, минуты и секунды по отдельности после анализа временных строк с помощью соответствующего средства форматирования.
3. Отрегулируйте часы, минуты и секунды, если минуты и / или секунды превышают 60.

Демо:

import java.time.LocalTime;
import java.time.format.DateTimeFormatter;
import java.util.List;

public class Main {
    public static void main(String[] args) {
        List<String> list = List.of("0 hours : 0 mins : 0 sec", "1 hours : 16 mins : 22 sec",
                "0 hours : 1 mins : 53 sec", "23 hours : 54 mins : 18 sec");

        // Start with a time of 0:0
        int sumHours = 0;
        int sumMinutes = 0;
        int sumSeconds = 0;

        // Iterate the list and add all hours, minutes and seconds separately after
        // parsing the time strings using the corresponding formatter
        DateTimeFormatter timeFormatter = DateTimeFormatter.ofPattern("H' hours : 'm' mins : 's' sec'");
        for (String strTime : list) {
            LocalTime time = LocalTime.parse(strTime, timeFormatter);
            sumHours += time.getHour();
            sumMinutes += time.getMinute();
            sumSeconds += time.getSecond();
        }

        // Adjust hour, minutes and seconds if minute and/or second exceed 60
        sumMinutes += sumSeconds / 60;
        sumSeconds %= 60;
        sumHours += sumMinutes / 60;
        sumMinutes %= 60;

        String strSum = String.format("%d hours : %d mins : %d sec", sumHours, sumMinutes, sumSeconds);
        System.out.println(strSum);
    }
}

Вывод:

25 hours : 12 mins : 33 sec

Answer 2

Один из способов приблизиться к этому - использовать Duration. Преобразуйте каждое значение времени в продолжительность, а затем просуммируйте их. Предоставленные значения времени показаны в виде массива из двух строк.

String[] hours = {
        "0 hours : 0 mins : 0 sec, 1 hours : 16 mins : 22 sec, 0 hours : 1 mins : 53"
                + "sec, 23 hours : 54 mins : 18 sec",
        "0 hours : 8 mins : 22 sec, 0 hours : 8"
                + " mins : 22 sec, 0 hours : 8 mins : 22 sec" };

Преобразование выполняется с помощью потоков. Процесс объяснен в комментариях.

List<Duration> times = Arrays.stream(hours)
        // remove all the spaces in each string
        .map(str -> str.replaceAll("\\s+", ""))

        // split each string on the commas.  At this
        // point, each string is processed independently
        // of the other in a separate stream
        .map(str -> Arrays.stream(str.split(","))
                
                // parse the time
                .map(tm -> LocalTime.parse(tm,
                        DateTimeFormatter.ofPattern(
                                "H'hours':m'mins':s'sec'")))

                // convert each time to a duration
                .map(lt -> Duration
                        .between(LocalTime.of(0, 0, 0), lt))

                // sum the durations
                .reduce(Duration.ZERO, (a, b) -> a.plus(b)))
        // and collect in a list
        .collect(Collectors.toList());

На этом этапе вы можете распечатать продолжительность в стандартном формате Duration.

times.forEach(System.out::println);

Печать

PT25H12M33S
PT25M6S

Чтобы распечатать их в более привычном формате, можно использовать следующую лямбду.

Function<Duration, String> durationToString = d -> {
    StringBuilder timeString = new StringBuilder();   
    long h = d.toDaysPart()*24+d.toHoursPart();
    timeString.append(h == 1 ? (h + " hour ") : h > 1 ?  (h + " hours ") : "");
    long m = d.toMinutesPart();
    timeString.append(m == 1 ? (m + " minute ") : m > 1 ? (m + " minutes ") : "");
    long s = d.toSecondsPart();
    timeString.append(s == 1 ? (s + " second ") : s > 1 ? (s + " seconds ") : "");
    return timeString.toString();
};

Печать

25 hours 12 minutes 33 seconds 
25 minutes 6 seconds 

Answer 3

1. Логическая ошибка в вашей программе -%24, поэтому вы удаляете полные дни.

2. Ваш код содержит много бесполезных частей, таких как temp, и не является чистым.

3. Создание парсера в цикле - очень плохой стиль.

4. Вы делаете очень много вещей, которые не требуются, а работа с миллисекундами и календарем бесполезна. Вот две версии, возвращающие ожидаемый результат. Во-вторых, для более новых версий Java.

    public static List<String> getTheHoursWorked() {
    final DateTimeFormatter parser = DateTimeFormatter.ofPattern("H 'hours :' m 'mins :' s 'sec'", Locale.US);
    final List<String> togetherHours = new LinkedList<>();
    for (int k = 0; k < hours.size(); k++){
        final List<String> shorter = hours.get(k);
        long seconds = 0;
        for (int m = 0; m < shorter.size(); m++) {
            final LocalTime odt = LocalTime.parse(shorter.get(m), parser);
            seconds += odt.getHour() * 3600 + odt.getMinute() * 60 + odt.getSecond();
        }
        final String s = (seconds       ) % 60 + " seconds";
        final String m = (seconds /   60) % 60 + " minutes";
        final String h = (seconds / 3600)      + " hours";
        final String together = h+":"+m+":"+s;
        togetherHours.add(together);
    }
    return togetherHours;
    }
   
    public static List<String> getTheHoursWorked_new() {
    final DateTimeFormatter parser = DateTimeFormatter.ofPattern("H 'hours :' m 'mins :' s 'sec'", Locale.US);
    final List<String> togetherHours = new LinkedList<>();
    for (final var shorter: hours){
        long seconds = 0;
        for (final var duration : shorter) {
            final LocalTime odt = LocalTime.parse(duration, parser);
            seconds += odt.getHour() * 3600 + odt.getMinute() * 60 + odt.getSecond();
        }
        final String s = (seconds       ) % 60 + " seconds";
        final String m = (seconds /   60) % 60 + " minutes";
        final String h = (seconds / 3600)      + " hours";
        final String together = h+":"+m+":"+s;
        togetherHours.add(together);
    }
    return togetherHours;
    }