Замечания по программе Labyrinth

[alexey-malov]

https://github.com/KernelMrex/oop_labs/blob/d90bd79fcf5888ea9eccf70b4b56df96d9cf4a7c/lab1_labyrinth/main.cpp#L8-L17

• [x] По возможности не используйте макросы. Вместо них используйте enum-ы или constexpr-выражения:

constexpr int LABYRINTH_CELL_NOT_DEFINED = -3;

[alexey-malov]

https://github.com/KernelMrex/oop_labs/blob/d90bd79fcf5888ea9eccf70b4b56df96d9cf4a7c/lab1_labyrinth/main.cpp#L24

• [x] Лучше использовать структуру с полями row и column

[alexey-malov]

https://github.com/KernelMrex/oop_labs/blob/d90bd79fcf5888ea9eccf70b4b56df96d9cf4a7c/lab1_labyrinth/main.cpp#L26-L29

• [x] Нет необходимости объединять значение и координаты клетки. Совместно они нигде не используются. Достаточно в алгоритме хранить координаты

[alexey-malov]

https://github.com/KernelMrex/oop_labs/blob/d90bd79fcf5888ea9eccf70b4b56df96d9cf4a7c/lab1_labyrinth/main.cpp#L240-L251

• [x] Лучше получить ссылку на элемент массива и тогда не придётся в двух местах ссылаться по длинным индексам на нее

[alexey-malov]

https://github.com/KernelMrex/oop_labs/blob/a56d2df4c1b1d6c4f7c849b02f2c3caca4d38f38/lab1_labyrinth/main.cpp#L28-L32

• [ ] Вот так можно в списке инициализации сделать:

  CellCoordinates(int row, int column)
      :row(row), column(column)
  {
  }

[alexey-malov]

• [ ] В SearchNextNeighbour встречается частично дублирующийся код.