Хорошо ли сортировать список в тесте тестового примера, чтобы проверить граничные данные?

Ответ 1

Я вижу, как ваша главная проблема состоит в том, чтобы написать unit test с самой простой логикой. Это приведет к повышению уровня доверия, когда unit test сообщает об успехе или неудаче, что действительно означает, что запрос возвращает хорошие или плохие результаты, а не то, что вы закодировали ошибку в логике вашего unit test. Для вас даже не может быть безупречной производительности.

Если это ваш случай, я предлагаю использовать ваш очень прямой вариант # 1, где вы просто проверяете каждую дату/время последовательно и не выполняете unit test, как только вы сталкиваетесь с датой/временем, которое не находится в пределах минимальная/максимальная дата/время. Но я бы скорректировал метод сравнения двух наборов даты/времени следующим образом, чтобы логика сравнения была очень простой:

Объединить и форматировать каждую дату/время в следующий формат строки: YYYY-MM-DD hhmm (например: 2015-01-01 0700). Другими словами, ваша строка отформатирована с нулевым дополнением, так что она всегда будет иметь длину 15. После этого формата точно есть очень удобное свойство, которое, если вы сравните две такие строки с использованием встроенного метода String.compareTo(), будет точно сравнивать ваши даты.

Это позволяет вам сохранить вашу логику очень простой и удобочитаемой. Вот пример того, как это могло бы выглядеть (вы не указали, поэтому я предполагаю, что ваша дата - это строка, а время - число, но вы можете приспособиться к вашим фактическим типам):

// starting boundary values
String dateFrom = "2015-01-01";
int hourFrom = 700;
String dateTo = "2015-01-03";
int hourTo = 600;

String formatString = "%s %04d"; // adjust as necessary.

String fromDateTime = String.format(formatString, dateFrom, hourFrom);
String toDateTime = String.format(formatString, dateTo, hourTo);

// execute query here

while (rs.next()) {
    String dateTime = String.format(formatString, rs.getString("date"), rs.getInt("hour"));

    if (fromDateTime.compareTo(dateTime) > 0 || toDateTime.compareTo(dateTime) < 0) {
        throw new Exception("fail unit test");
    }
}

Ответ 2

Оба варианта, которые вы предложили, верны. Опция, в которой вы сортируете данные, будет проще реализовать.

Ответ 3

Нет ничего плохого в сортировке результатов после их извлечения. Это подход, который я бы взял, но проверка каждой строки также будет работать.

Ответ 4

Поскольку это Date, лучше использовать java-объекты Date,

import java.io.IOException;
import java.text.ParseException;
import java.util.Calendar;
import java.util.Date;
import java.util.GregorianCalendar;

public class ATest {

    public static void main(String[] args) throws IOException, ParseException {
        String dateFrom = "2015-01-01";
        String dateTo = "2015-01-03";

        // those are actually string as i see.
        String hourFrom = "700";
        String hourTo = "600";


        Date from = str2Date(dateFrom, hourFrom);
        Date to = str2Date(dateTo, hourTo);

        Date any = new GregorianCalendar().getTime();

        // now testing any date using 'before' 'after'
        assert(any.after(from) && any.before(to));

        // or using compareTo
        assert(any.compareTo(from) > 0 && any.compareTo(to) < 0);

        // or simpy comparing epocs, i would use this
        long low = from.getTime();
        long high = from.getTime();

        assert( any.getTime() > low && any.getTime() < high);
    }

    private static Date str2Date(String date_input, String time_input) {

        // probably this is an overkill.
        while (time_input.length() < 4){
            time_input = "0" + time_input;
        }

        String parts[] = date_input.split("-");
        int year = Integer.parseInt(parts[0]); 
        int month = Integer.parseInt(parts[1]); 
        int date = Integer.parseInt(parts[2]);

        int hourOfDay = Integer.parseInt(time_input.subSequence(0, 2).toString()); 
        int minute = Integer.parseInt(time_input.subSequence(2, 4).toString());  
        int second = 0;

        Calendar cal = GregorianCalendar.getInstance();
        cal.set(year, month - 1, date, hourOfDay, minute, second);

        return cal.getTime();
    }
}

Сортировка или нет для меня не важна (если количество элементов в списке не очень велико).

Ответ 5

Go с опцией 1. По мере прохождения данных запишите максимальные и минимальные значения. Один раз можно переходить только по списку. Если ваша цель состоит в том, чтобы найти существование несовместимых данных, тогда возможно, что вы не можете завершить обход до конца, если вы уже нашли его. Это более эффективно, чем сортировка списка. В худшем случае O (n).