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

[ ] Тесты падают (в наборе отсутствует файл map1.txt)

map1_not_found

[ ] Для хранения ячеек лабиринта лучше использовать std::vector<std::vector<int>>. Так же, при использовании вектора, исчезнет необходимость в методе

void FillTheMapWithZeros(Map &map)
{
    for (int row = 0; row != SIZE_MAP - 1; ++row)
    {
        for (int colum = 0; colum != SIZE_MAP - 1; ++colum)
        {
            map[row][colum] = 0;
        }
    }
}

ибо его можно будет заменить.

Map(SIZE_MAP, std::vector<int>(SIZE_MAP, 0)

Возможно, стоит явно объявить конструктор по умолчанию у структуры Labirinth, где, в списках инициализации, задать размер вектору, при этом заполнив его нулями.

[ ] Операции над лабиринтом лучше вынести в отдельный файл.
[ ] int colum = 0 Наверное, всё-таки column
[ ] Для хранения магических -1, -2, -3... лучше использовать константы

            if (map[y][x] == -1)
            {
                outputFile << '#';
            }
            else if (map[y][x] == 1)
            {
                outputFile << 'A';
            }
            else if (map[y][x] == -2)
            {
                outputFile << 'B';
            }
            else if(map[y][x] == -3)
            {
                outputFile << '.';
            }
            else
            {
                outputFile << ' ';
            }

Например, WALL Так же, стоит подумать об использовании константы вида std::map для хранения пар, где ключом будет являться число типа int, а значением char. Например:

std::map<int, char> exampleMap = {
    {WALL, '#'},
    {START, 'A'},
    {END, 'B'},
    ...
}

Тогда, описанное выше ветвление можно будет заменить чем-то вроде этого:

    auto exampleResult = exampleMap.find(3);
    if (exampleResult == exampleMap.end())
    {
        outputFile << ' ';
    }
    else    
    {   
        outputFile << exampleResult->second;    
    }

[ ] Для проверки инициализации переменной можно использовать boost::optional. Это позволит избавиться от велосипеда

            case 'A':
                labirinth.map[row][colum] = 1;
                labirinth.startPoint = { row, colum };
                if (!haveStart)
                {
                    haveStart = true;
                }
                else
                {
                    outputFile << "We found more than one starting points!\n";
                    wasError = true;
                }
                break;

И превратит кусок кода, расположенный чуть выше, во что-то вроде:

            case 'A':
                if (labirinth.startPoint.is_initialized())
                {
                    outputFile << "We found more than one starting points!\n";
                    wasError = true;
                }
                labirinth.map[row][colum] = 1;
                labirinth.startPoint = { row, colum };
                break;

где labirinth.startPoint имеет тип boost::optional<Point>

[ ] No path between points A and B! в выходном файле в том случае, если путь не найден. При отсутствии пути между точками A и B необходимо вывести лабиринт без указания пути.

Пока k = 0.9

gagik12 / OOP

Замечания по Labyrinth #3