2

Подскажите, пожалуйста, каким образом возможно инициализировать матрицу в методе init() класса Matrix, чтобы строки и столбцы имели значения из метода main? Получается, что сразу вызывается конструктор родительского класса RunnableTask, потом идет переход в метод init() класса Matrix и матрица имеет нулевой размер. А нужно чтобы в методе init() класса Matrix, матрица имела размер 5*5.

public static void main(String[] args) {
    RunnableTask matrix = new Matrix(5, 5);
}

public abstract class RunnableTask {
    public abstract void runTask();
    public abstract void init();

    public void run() {
        runTask();
    }

    public RunnableTask() {
        init();
    }
}

public class Matrix extends RunnableTask {
    int rows;
    int cols;

    int matrix[][];

    public Matrix(int rows, int cols) {
        this.rows = rows;
        this.cols = cols;
    }

    @Override
    public void init() {
        matrix = new int[rows][cols];

        for (int i = 0; i < rows; i++) {
            for (int j = 0; j < cols; j++) {
                matrix[i][j] = i * j;
            }
        }
    }

    @Override
    public void runTask() {
        for (int i = 0; i < rows; i++) {
            System.out.println();

            for (int j = 0; j < cols; j++) {
                System.out.print(" " + matrix[i][j] + " ");
            }
        }
    }
}
5
  • Может убрать метод init из RunnableTask? По сути он дублирует функционал конструктора.
    – diralik
    Commented 6 сент. 2017 в 20:08
  • Нельзя к сожалению убирать метод init(). Вот пытаюсь понять, возможно ли это ? Или просто пошутили, дав такое тестовое.
    – user266161
    Commented 6 сент. 2017 в 20:18
  • инициализация в runTask() также не допустима
    – user266161
    Commented 6 сент. 2017 в 20:32
  • Вообще это очень плохая практика - вызывать из конструктора родителя переопределяемые методы т.к. сначала будет вызван конструктор родительского класса, а потом только конструктор наследника и может быть непредсказуемое поведение (как в данном примере). Не знаю насколько сильно можно менять код в Вашем задании, но я бы предложил через конструкторы передавать rows, cols
    – damintsew
    Commented 6 сент. 2017 в 20:44
  • Инициализация матрицы и заполнение должны происходить в методе init() класса Matrix. Менять к сожалению нельзя. Было бы можно, уже бы решил.
    – user266161
    Commented 6 сент. 2017 в 20:52

1 ответ 1

2

Конструктор родительского класса вызывается перед выполнением операторов зависимого класса. В данном случае, конструктор класса RunnableTask вызовется перед тем, как будут заданы размеры матрицы. Поэтому возможно только 3 варианта решения:

1 Вызывать init() в конструкторе Matrix:

public Matrix(int rows, int cols) {
    this.rows = rows;
    this.cols = cols;
    init();
}

2 Поскольку init объявлен public, то можно производить инициализацию RunnableTask после создания объекта

 public static void main(String[] args) {
        RunnableTask matrix = new Matrix(5, 5);
        matrix.init();
        matrix.run();
    }

3 Заточить класс RunnableTask под матрицу, например с помощью наследования RunnableMatrixTask

3
  • DaysLikeThis большое спасибо. Подходит по условию задачи и работает так как нужно именно первый вариант. Спасибо.
    – user266161
    Commented 7 сент. 2017 в 7:04
  • хотел проголосовать, но пока не могу, рейтинга(репутации) не хватает
    – user266161
    Commented 7 сент. 2017 в 7:16
  • @java_jun Вы можете отметить ответ как решение Commented 7 сент. 2017 в 7:23

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.