0

По домашней работе мне требуется добиться 100% покрытия тестами кода:


package ru.netology.statistic; public class StatisticsService {
    /**
     * Calculate index of max income
     *
     * @param incomes - array of incomes
     * @return - index of first max value
     */
    public long findMax(long[] incomes) {
        long current_max_index = 0;
        long current_max = incomes[0];
        for (long income : incomes)
            if (current_max < income)
                current_max = income;
        return current_max;
    } 
}

при прогоне через jacoco выдает 85 процентов покрытия и показывает , что строчка long current_max_index = 0; не используется в тестах. Как добиться 100% покрытия? Сам тест

void findMax() {
        StatisticsService service = new StatisticsService();
        long[] incomesInBillions = {12, 5, 8, 4, 5, 3, 8, 6, 11, 11, 12};
        long expected = 12;
        long actual = service.findMax(incomesInBillions);
        assertEquals(expected, actual);
    }
6
  • 2
    У вас в коде нет строчки long current_max_index = 0
    – ЮрийСПб
    18 янв 2021 в 11:51
  • Строчка long current_max_index = 0; идет перед long current_max = incomes[0];
    – Alina
    18 янв 2021 в 12:19
  • 1
    вы не могли бы отредактировать код в вопросе, чтобы он включал в себя проблемную строку, пожалуйста. Код должен быть идентичен проблемному
    – Andrew
    18 янв 2021 в 12:32
  • ну вообще по коду теста я не вижу использования данной строки, поэтому как вариант решения - убрать данную строку и вероятнее всего покрытие будет 100% :)
    – Andrew
    18 янв 2021 в 12:57
  • я отредактировала код) чтобы использовать эту строчку long current_max_index = 0; нужно написать тест так, чтобы он находил номер месяца с максимальной суммой, потом выводил значение этого месяца в моем понимании.
    – Alina
    18 янв 2021 в 12:57

1 ответ 1

2

Давайте поанализируем код вместе. Строки 1, 2, 3 исполняются при любом вызове метода findMax. Строка 4 вызовется только если массив incomes не пуст. В тесте ниже так и есть. Строка 5 вызывается когда... Никогда она не вызывается. В тесте максимальное значение идёт первым элементом. В строке 2 оно попадает в переменную current_max. Поэтому условие current_max < income всегда ложно. Строка 6 вызывается всегда.

Единственная строка которая не покрыта текущим тестом - 5. Чтобы исправить уменьшите первый элемент массива.

public long findMax(long[] incomes) {
/* 1 */    long current_max_index = 0;
/* 2 */    long current_max = incomes[0];
/* 3 */    for (long income : incomes)
/* 4 */        if (current_max < income)
/* 5 */            current_max = income;
/* 6 */    return current_max;
} 

...

long[] incomesInBillions = {12, 5, 8, 4, 5, 3, 8, 6, 11, 11, 12};
long actual = service.findMax(incomesInBillions);
1
  • да, так получилось) Спасибо большое!
    – Alina
    18 янв 2021 в 15:10

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.