Наследование классов - правильный вызов методов

Я пришел из долгого периода программирования в Java. Я решаю проблему кодирования и пытаюсь написать абстрактный класс решения, который будет расширен тремя классами:

var isValidSudoku = function(board) {
  return new CheckRows(board).isValid()
  // && checkCols(board)
  // && checkBoxes(board);
};

class AbstractSolver {
  constructor(board) {
    this._board = board;
    this._numSet = new Set();
    this._state = {
      x: 0,
      y: 0,
    }
  }

  getCell() {
    const numString = this._board[this._state.y][this._state.x];
    return isNumBetween0And9(numString) ?
      {
        isNum: true,
        num: parseInt(numString, 10),
      } :
      {
        isNum: false,
      };
  }

  nextCell() {}

  nextBlock() {}

  isBlockFinish() {}

  isBoardFinish() {}

  isValid() {
    while (this.isBoardFinish() == false) {
      while (this.isBlockFinish() == false) {
        const {
          isNum,
          num,
        } = this.getCell();
        if (isNum == false) {
          // do nothing
        } else if (this._numSet.has(num)) {
          return false;
        } else {
          this._numSet.add(num);
        }
        this.nextCell();
      }
      this.numSet.clear();
      this.nextBlock();
    }
    return true;
  }
}

function check(a) {
  return f => f(a);
}

function isNumBetween0And9(i) {
  const checkNum = check(i);
  return checkNum(Number.isInteger) && checkNum(x => x >= 0) && checkNum(x => x <= 9);
}

class CheckRows extends AbstractSolver {
  constructor(board) {
    super(board);
    this._boardLen = 9;
  }

  nextCell() {
    this._state = {
      x: this._state.x + 1,
      y: this._state.y,
    };
  }

  nextBlock() {
    this._state = {
      x: 0,
      y: this._state.y + 1,
    };
  }

  isBlockFinish() {
    this._state.x >= this._boardLen;
  }

  isBoardFinish() {
    this._state.x >= this._boardLen && this.state.y >= this._boardLen;
  }
}

const testParam = [
  ["5", "3", ".", ".", "7", ".", ".", ".", "."],
  ["6", ".", ".", "1", "9", "5", ".", ".", "."],
  [".", "9", "8", ".", ".", ".", ".", "6", "."],
  ["8", ".", ".", ".", "6", ".", ".", ".", "3"],
  ["4", ".", ".", "8", ".", "3", ".", ".", "1"],
  ["7", ".", ".", ".", "2", ".", ".", ".", "6"],
  [".", "6", ".", ".", ".", ".", "2", "8", "."],
  [".", ".", ".", "4", "1", "9", ".", ".", "5"],
  [".", ".", ".", ".", "8", ".", ".", "7", "9"]
];
const testParam2 = [
  ["5", "3", "3", ".", "7", ".", ".", ".", "."],
  ["6", ".", ".", "1", "9", "5", ".", ".", "."],
  [".", "9", "8", ".", ".", ".", ".", "6", "."],
  ["8", ".", ".", ".", "6", ".", ".", ".", "3"],
  ["4", ".", ".", "8", ".", "3", ".", ".", "1"],
  ["7", ".", ".", ".", "2", ".", ".", ".", "6"],
  [".", "6", ".", ".", ".", ".", "2", "8", "."],
  [".", ".", ".", "4", "1", "9", ".", ".", "5"],
  [".", ".", ".", ".", "8", ".", ".", "7", "9"]
];
console.info(isValidSudoku(testParam2));

Проблема в том, что когда запускается метод isValid из classCheckRows, он вызывает метод isValid из AbstractSolver, который запускает его метод isValid и вызывает все нереализованные «абстрактные» методы суперкласса вместо вызова переопределенных методов подкласса. Это сработало бы в Java. Есть ли способ исправить это в JS? И что еще более важно: есть ли лучшая передовая практика?

Лучше всего для поиска проблем, подобных описанным выше, использовать отладчик, встроенный в вашу IDE и/или браузер. Это показало бы вам, что вызываются правильные методы, и сделало бы вас на шаг ближе к пониманию проблемы.

T.J. Crowder 20.07.2019 13:05
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
0
1
36
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Проблема не в том, что вызываются неправильные методы (правильные вызывают), проблема в том, что методы не возвращают значение. Как и в Java, вам нужно ключевое слово return, чтобы вернуть значение из метода:

isBlockFinish() {
  return this._state.x >= this._boardLen;
//^^^^^^
}

isBoardFinish() {
  return this._state.x >= this._boardLen && this.state.y >= this._boardLen;
//^^^^^^
}

Стрелочные функции имеют неявный возврат, если вы используете краткое тело функции (например, () => value), а методы — нет.


После этого есть и другие проблемы (numSet вместо _numSet, индекс выходит за пределы диапазона), но это проблема, которая (предположительно) заставила вас думать, что вызываются абстрактные методы, а не переопределенные.

Другие вопросы по теме