В Idea всё проходит, ответ соответствует, но валик не принимает :(

Вопрос по задаче

Java Multithreading, 5 уровень, 2 лекция

Решен

Реализовал методы так, но валик не принимает. Может быть я что-то не понимаю:

public static List<Column> getVisibleColumns() {
        List<Column> result = new LinkedList<>();
        for (int i = 0; i < realOrder.length; i++) {
            if (realOrder[i] != -1) {
                if (realOrder[i] > result.size()) {
                    result.add(Column.values()[i]);
                } else if (realOrder[i] == result.size()) {
                    result.add(realOrder[i] - 1, Column.values()[i]);
                }
                else {
                    result.add(realOrder[i], Column.values()[i]);
                }
            }
        }
        return result;
    }
...
public void hide() {
        if (this.isShown()) {
            int order = realOrder[this.ordinal()];
            for (int i = 0; i < realOrder.length; i++) {
                if (i == this.ordinal()) {
                    realOrder[i] = -1;
                } else if (realOrder[i] > order && realOrder[i] > 0) {
                    realOrder[i]--;
                }
            }
        }
    }

Реализуй интерфейс Columnable у Column, описание методов смотрите в джавадоках.
Реализуй логику метода Column.getVisibleColumns.
Метод Column.configureColumns уже реализован, его не меняй.
Создавать дополнительные поля нельзя.
Метод main не участвует в тестировании.

Требования:

Интерфейс Columnable менять нельзя.
Класс Column должен реализовывать интерфейс Columnable.
Создавать дополнительные поля в классе Column нельзя.
Метод Column.configureColumns реализован. Менять его не нужно.
Метод Column.getVisibleColumns должен возвращать список отображаемых колонок в скофигурированом порядке.
Метод Column.getColumnName должен возвращать полное имя колонки.
Метод Column.isShown должен возвращать true, если колонка видимая, иначе false.
Метод Column.hide должен скрывать колонку и сдвигать индексы остальных отображаемых колонок.

package com.javarush.task.task25.task2503; import java.util.*; public enum Column implements Columnable { Customer("Customer"), BankName("Bank Name"), AccountNumber("Account Number"), Amount("Available Amount"); private String columnName; private static int[] realOrder; private Column(String columnName) { this.columnName = columnName; } /** * Задает новый порядок отображения колонок, который хранится в массиве realOrder. * realOrder[индекс в энуме] = порядок отображения; -1, если колонка не отображается. * * @param newOrder новая последовательность колонок, в которой они будут отображаться в таблице * @throws IllegalArgumentException при дубликате колонки */ public static void configureColumns(Column... newOrder) { realOrder = new int[values().length]; for (Column column : values()) { realOrder[column.ordinal()] = -1; boolean isFound = false; for (int i = 0; i < newOrder.length; i++) { if (column == newOrder[i]) { if (isFound) { throw new IllegalArgumentException("Column '" + column.columnName + "' is already configured."); } realOrder[column.ordinal()] = i; isFound = true; } } } } /** * Вычисляет и возвращает список отображаемых колонок в сконфигурированом порядке (см. метод configureColumns) * Используется поле realOrder. * * @return список колонок */ public static List<Column> getVisibleColumns() { List<Column> result = new LinkedList<>(); for (int i = 0; i < realOrder.length; i++) { if (realOrder[i] != -1) { if (realOrder[i] > result.size()) { result.add(Column.values()[i]); } else if (realOrder[i] == result.size()) { result.add(realOrder[i] - 1, Column.values()[i]); } else { result.add(realOrder[i], Column.values()[i]); } } } return result; } @Override public String getColumnName() { return this.columnName; } @Override public boolean isShown() { return realOrder[this.ordinal()] != -1; } @Override public void hide() { if (this.isShown()) { int order = realOrder[this.ordinal()]; for (int i = 0; i < realOrder.length; i++) { if (i == this.ordinal()) { realOrder[i] = -1; } else if (realOrder[i] > order && realOrder[i] > 0) { realOrder[i]--; } } } } }

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

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

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

Павел Безумный учёный

Expert

21 декабря 2022, 13:47

решение

В логике метода getVisibleColumns() есть изъян, который поможет выявить следующий тест. Задайте начальную конфигурацию колонок следующим образом:

Column.configureColumns(Column.Customer, Column.BankName, Column.AccountNumber);

после чего вызовите метод getVisibleColumns(). На мой взгляд, в реализации метода для привязки объектов колонок к их текущим позициям удобно было бы использовать TreeMap. В этом случае логика метода может быть такой: перебираем массив realOrder, и для каждого неотрицательного значения (то есть видимой колонки) создаём в отображении новую пару типа <Integer, Column>, где в качестве ключа передаём порядковый номер колонки в текущей конфигурации. А поскольку в отображении TreeMap пары автоматически сортируются по ключу, то после отработки цикла остаётся извлечь из отображения коллекцию значений (колонок) и добавить её в выходной список result. Для реализации метода hide() достаточно изменить значение в "своей" ячейке в массиве realOrder на -1 :

realOrder[this.ordinal()] = -1;