Телеграмм чат группы javarush страница 5027

Cyclone

да

но, так работает. спасибо. попытаюсь разобраться теперь в этом)

20:22пожаловаться #1

IF

```

    public Offer[] removeById(int id) {
        int length = offers.length - 1;
        Offer[] temp = new Offer[length];
        int index = 0;
        for (Offer offer : offers) {
            if (offer.getId() != id) {
                temp[index] = offer;
                index++;
            }
        }
        offers = temp;
        return offers;
    }

```

если я пытаюсь произвести удаление по несуществующему ид, то получаю ошибку длины массива. так как на деле массив остается не изменным, а исходя из метода мы получаем длинна - 1. как быть?

счет массива идет с 0, допустим если в массиве 3 элемента то они лежат под индексами 0, 1, 2 , length вернет 3- пушо там 3 элемента, поэтому length -1

ЯФ

счет массива идет с 0, допустим если в массиве 3 элемента то они лежат под индексами 0, 1, 2 , length вернет 3- пушо там 3 элемента, поэтому length -1

это я понимаю))) тут идет вопрос в том, что у меня есть метод по удалению элемента в массиве. значит мы заведомо знаем что у нас массив будет короче на 1 единицу. и это типа работает. но если я вдруг пытаюсь удалить не существующий элемент массива, то массив тупо купируется и от сюда происходит ошибка

java.lang.ArrayIndexOutOfBoundsException: Index 4 out of bounds for length 4

20:24пожаловаться #3

IF

это я понимаю))) тут идет вопрос в том, что у меня есть метод по удалению элемента в массиве. значит мы заведомо знаем что у нас массив будет короче на 1 единицу. и это типа работает. но если я вдруг пытаюсь удалить не существующий элемент массива, то массив тупо купируется и от сюда происходит ошибка

java.lang.ArrayIndexOutOfBoundsException: Index 4 out of bounds for length 4

ну да, но как это умные люди решили я думаю можно посмотреть где то в исходниках аррай листа

20:25пожаловаться #4

N

Northwood in Java & Co

это я понимаю))) тут идет вопрос в том, что у меня есть метод по удалению элемента в массиве. значит мы заведомо знаем что у нас массив будет короче на 1 единицу. и это типа работает. но если я вдруг пытаюсь удалить не существующий элемент массива, то массив тупо купируется и от сюда происходит ошибка

java.lang.ArrayIndexOutOfBoundsException: Index 4 out of bounds for length 4

Логично. Если ты обращаешься к несуществующему элементу, то кидает эксепшн.

20:26пожаловаться #5

ЯФ

Northwood

Логично. Если ты обращаешься к несуществующему элементу, то кидает эксепшн.

да. а как это ошибку можно в таком случае отловить? мне насамом деле нужно тест сделать на удаление несуществующего. и значит я должен получить ошибку.... какой-то assert?

20:27пожаловаться #6

IF

да. а как это ошибку можно в таком случае отловить? мне насамом деле нужно тест сделать на удаление несуществующего. и значит я должен получить ошибку.... какой-то assert?

в метод добавь throws ArrayIndexOutOfBoundsException

20:28пожаловаться #7

ЯФ

а куда именно? у меня есть такой тест:

    @Test
    void shouldRemoveByNoneExistId() {
        Offer[] expected = new Offer[]{tick1, tick2, tick3, tick4, tick5};
        assertArrayEquals(expected, repository.removeById(6));
    }

но быть может что-то можно типа assertNull. только с out of bounds что есть?

20:30пожаловаться #8

N

Northwood in Java & Co

а куда именно? у меня есть такой тест:

    @Test
    void shouldRemoveByNoneExistId() {
        Offer[] expected = new Offer[]{tick1, tick2, tick3, tick4, tick5};
        assertArrayEquals(expected, repository.removeById(6));
    }

но быть может что-то можно типа assertNull. только с out of bounds что есть?

Junit какой?

ЯФ

5

20:31пожаловаться #10

IF

https://howtodoinjava.com/junit5/expected-exception-example/

а куда именно? у меня есть такой тест:

    @Test
    void shouldRemoveByNoneExistId() {
        Offer[] expected = new Offer[]{tick1, tick2, tick3, tick4, tick5};
        assertArrayEquals(expected, repository.removeById(6));
    }

но быть может что-то можно типа assertNull. только с out of bounds что есть?

20:32пожаловаться #11

ЯФ

ооо. спс. ща попробую

20:33пожаловаться #12

N

Northwood in Java & Co

Блин junit exception test в гугле запомни. я постоянно забываю эту лямбда конструкцию. а ментор требует максимального покрытия тестами.

20:35пожаловаться #13

ЯФ

https://howtodoinjava.com/junit5/expected-exception-example/

Ilya Frontend

еее... спасибо большое))))

    @Test
    void shouldRemoveByNoneExistId() {
        assertThrows(ArrayIndexOutOfBoundsException.class, () -> repository.removeById(10));
    }

20:35пожаловаться #14

ЯФ

Northwood

Блин junit exception test в гугле запомни. я постоянно забываю эту лямбда конструкцию. а ментор требует максимального покрытия тестами.

да. я тоже забыл про эту лямбду. предыдущая ДЗ была с этим же. пока Илья не ткнул носом, не вспомнил((( надеюсь практикой вошьется под корку🤣

20:36пожаловаться #15

N

Northwood in Java & Co

кстати тесты помогают писать код более чисто. например прописывать обнуления статических полей класса или избавляться от кондишенов в wile(true) лупах, которые junitом не тестируются.

20:39пожаловаться #16

ЯФ

ну вот именно по этому я и решил что в цикле что-то не так. а вот с советом от @ILyaCyclone работает. но нужно понять. что-то пока у меня с коллекциями не складывается... сложно пока

20:42пожаловаться #17

ЯФ

в любом случае спасибо. не первый раз помогаете :)

20:43пожаловаться #18

AB

Aleksey Budaev in Java & Co

После нажатия на submit мне в doPost как аттрибут искать ? который я в doGet передавал ?

21:09пожаловаться #19

AB

Aleksey Budaev in Java & Co

А если я допустим checkbox выделил и сабмит нажал, что в аттрибутах реквеста искать ? value чекбокса?