unit test. code improvement.

Привет. Нужна помощь, хочу написать юнит тест для метода(ниже прикреплен), но я не знаю как это сделать, помогите написать юнит тест. Так же хочу немного улучшить код который я написал в методе, можно ли его переписать полностью используя stream api, сам я пробовал, но не получилось, в итоге написал вот так:

// принимает на вход массив целых чисел,
// возвращает только те числа, которые положительны (>=0),
// отсортировать их по убыванию.
    public static int[] getPositiveSortedNumbers(int[] arrayOfNumbers){

        int[] positiveSortedReversedArray = Arrays.stream(arrayOfNumbers).filter(i -> i >= 0).toArray();

        for (int i = 0; i < positiveSortedReversedArray.length; i++) {
            for (int j = i; j < positiveSortedReversedArray.length; j++) {
                if (positiveSortedReversedArray[j] > positiveSortedReversedArray[i]) {
                    int temporaryValue = positiveSortedReversedArray[j];
                    positiveSortedReversedArray[j] = positiveSortedReversedArray[i];
                    positiveSortedReversedArray[i] = temporaryValue;
                }
            }
        }

        return positiveSortedReversedArray;
    }

Комментарии (8)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Alex

Уровень 27

4 ноября 2022, 09:04

Привет, Vladimir и Денис, я обязательно погляжу ваши ответы!! Просто спешу еще магистерскую работу написать(80 листиков), и еще куча дел(предметов) сдать. + прохожу стажировку в группе, в компании какой то)) так что конкретно эту задачу еще пока не пробовал переделать, но думаю сегодня займусь.

Alex

Уровень 27

11 ноября 2022, 08:18

p.s. задачу переделал, сделал тест для нее, и выполнил все задачи что давали на стажировке, и написал к ним junit тесты).

Vladimir

Уровень 40

Master

3 ноября 2022, 20:05

Молчишь, видимо так и не выполнил задачу. Если выполнил, покажи свой вариант. В данном примере сортировка по возрастанию. По убыванию сделай сам для тренировки. :)

import java.util.Arrays;
import java.util.function.IntPredicate;

public class PositiveNumbers {
    public static int[] getPositiveSortedNumbers(int[] arrayOfNumbers) {
        IntPredicate predicate = n -> n >= 0;
        Arrays.stream(arrayOfNumbers)
                .filter(predicate)
                .sorted()
                .forEach(System.out::println);
        return arrayOfNumbers;
    }
}

И тестируем:

import org.junit.jupiter.api.Test;

import java.util.Arrays;

import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.*;

class PositiveNumbersTest {

    @Test
    void getPositiveSortedNumbers() {
        var expected = PositiveNumbers
             .getPositiveSortedNumbers(new int[]{-2, 1, 2, 12, 0, -2, 4});

        var actual = Arrays.stream(expected)
             .toArray();

        assertThat(expected, is(actual));
    }
}

Денис Java Developer

3 ноября 2022, 21:00

Ты уверен что твоя реализация метода соответствует его названию? Подозреваю здесь скорее нужно было бы что-то такое (вывод оставил чисто по аналогии) ну если говорить о сортировке по возрастанию конечно.

public static int[] getPositiveSortedNumbers(int[] arrayOfNumbers) {
    return Arrays.stream(arrayOfNumbers)
                 .filter(n -> n >= 0)
                 .sorted()
                 .peek(System.out::println)
                 .toArray();
}

Ну или для реверса

public static int[] getPositiveSortedNumbers(int[] arrayOfNumbers) {
    return Arrays.stream(arrayOfNumbers)
                 .filter(n -> n >= 0)
                 .boxed()
                 .sorted(Collections.reverseOrder())
                 .mapToInt(Integer::intValue)
                 .toArray();
}

Тест кстати тоже поправить бы стоит. А то сейчас он будет зеленым независимо от того что вернет тестируемый метод (разве что hamcrest делает какую-то магию, но не уверен).

Alex

Уровень 27

4 ноября 2022, 09:05

выше ответил...

Alex

Уровень 27

4 ноября 2022, 09:05

выше ответил...

Vladimir

Уровень 40

Master

5 ноября 2022, 04:59

@Test
void getPositiveSortedNumbers() {
    var expected = PositiveNumbers
    .getPositiveSortedNumbers(new int[]{-2, 1, 2, 12, 0, -2, 4});

    var actual = new int[] {0, 1, 2, 4, 12};

    assertThat(expected, is(actual));
}

😉🤝

Vladimir

Уровень 40

Master

2 ноября 2022, 14:18

Как вариант, можно при помощи функционального интерфейса Predicate сделать элегантно и красиво. Попробуй и покажи что получится.