A maneira mais elegante para percorrer uma lista para comparar 2 elementos próximos um do outro

Joe D:

Eu tenho um método que funciona da seguinte maneira:

1. Tome-se como um argumento 3 params - uma lista com datas (classificados em ordem crescente), unidade de intervalo e valor do intervalo
2. Verifique se o próximo elemento não ultrapasse a data anterior (intervalo). Em outras palavras, tendo em conta o intervalo de 30 minutos, prev - 10:00, em seguida 10:29 - iteração adicional. se o próximo é 10:31 - quebrá-lo e retornar o contador de datas em uma fileira.

O código para ele é abaixo:

public static void main(String[] args)
{
    Date d1 = new Date();
    Date d2 = addOrSubtractTimeUnitFromDate(d1, Calendar.MINUTE, 10, true);
    Date d3 = addOrSubtractTimeUnitFromDate(d2, Calendar.MINUTE, 10, true);
    Date d4 = addOrSubtractTimeUnitFromDate(d3, Calendar.MINUTE, 10, true);
    Date d5 = addOrSubtractTimeUnitFromDate(d4, Calendar.MINUTE, 10, true);
    Date d6 = addOrSubtractTimeUnitFromDate(d5, Calendar.MINUTE, 10, true);

    List<Date> threeDates = new ArrayList<>();
    threeDates.add(d1);
    threeDates.add(d2);
    threeDates.add(d3);
    threeDates.add(d4);
    threeDates.add(d5);
    threeDates.add(d6);

    System.out.println(returnDatesInARowCounter(threeDates, Calendar.MINUTE, 30));
}

private static int returnDatesInARowCounter(List<Date> allDates, int intervalBetween2DatesTimeUnit, int intervalValue)
{
    int datesInARowCounter = allDates.size() > 0 ? 1 : 0; // esp. this line (in case allDates is empty)
    Date lastDate = null;
    Date nextDate;

    Iterator<Date> iter = allDates.iterator();

    while (iter.hasNext())
    {
        nextDate = iter.next();

        if (lastDate != null) // both lastDate и nextDate are initialized now
        {
            if(isNextIncidentInIntervalWithLastOneOrNot(lastDate, nextDate, intervalBetween2DatesTimeUnit, intervalValue, true))
            {
                datesInARowCounter += 1;
            }
            else break;
        }

        lastDate = nextDate;
    }

    return datesInARowCounter;
}

public static Date addOrSubtractTimeUnitFromDate(Date dateToAddToOrSubtractFrom, int calendarTimeUnit, int value, boolean isAdd)
{
    if(!isAdd)
    {
        value = -value;
    }
    Calendar cal = Calendar.getInstance();
    cal.setTime(dateToAddToOrSubtractFrom);
    cal.add(calendarTimeUnit, value);
    return cal.getTime();
}

private static boolean isNextIncidentInIntervalWithLastOneOrNot(Date lastIncidentRegDate, Date nextIncidentRegDate, int intervalTimeUnit, int intervalValue, boolean isBetween)
{
    Date currentIncidentPlusInterval = addOrSubtractTimeUnitFromDate(lastIncidentRegDate, intervalTimeUnit, intervalValue, true);
    boolean betweenBool = isDateBetween(nextIncidentRegDate, lastIncidentRegDate, currentIncidentPlusInterval);

    return isBetween == betweenBool;
}

private static boolean isDateBetween(Date targetDate, Date startDate, Date endDate)
{
    return targetDate.compareTo(startDate) >= 0 && targetDate.compareTo(endDate) <= 0;
}

No entanto, os olhares de código peculiar para mim. É a alguma maneira de torná-la mais legível?

Quaffel:

Se você estiver usando Java 8 ou mais recente, você pode usar o java.time-API em seu lugar. Ele é construído com suporte para "períodos de tempo" torna a implementação real muito mais simples.

static int daysInARow(List<Instant> allInstants, Duration maxDifference) {
        int counter = allInstants.size() > 0 ? 1 : 0;
        Instant previous = allInstants.get(0);

        for (int i = 1; i < allInstants.size(); i++) {
            Instant current = allInstants.get(i);
            if (Duration.between(previous, current).compareTo(maxDifference) > 0)
                break;
            counter++;
            previous = current;
        }

        return counter;
    }

Se você estiver usando java.util.Dateem outras partes do seu projeto, você pode facilmente converter entre Instants usando

Date#from(Instant)

e

Date#toInstant()