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

[alexey-malov]

• [x] Запуск fillTest.exe по окончании сборки должен быть прописан в проекте fillTest, а не fill

[alexey-malov]

• [x] В тестах используйте raw string literal, вместо строк с escape-последовательностями. Так будет лучше читаться:

  string fieldString = R"(
  # G#
  # ##
  ###)";

[alexey-malov]

    std::vector<std::vector<char>> m_fields;

• [x] Дать имя m_field или m_rows

[alexey-malov]

    if (CanFill({ current.x - 1, current.y }))
    {
        coordsStack.push({ current.x - 1, current.y });
    }
    if (CanFill({ current.x + 1, current.y }))
    {
        coordsStack.push({ current.x + 1, current.y });
    }
    if (CanFill({ current.x, current.y - 1 }))
    {
        coordsStack.push({ current.x, current.y - 1 });
    }
    if (CanFill({ current.x, current.y + 1 }))
    {
        coordsStack.push({ current.x, current.y + 1 });
    }

• [x] Устранить дублирование кода

[alexey-malov]

• [x] Вместо

    while (m_fields.size() <= paintCoords.y)
    {
        m_fields.push_back({});
    }

лучше сделать if () m_field.resize(paintCoords.y);

    while (m_fields[paintCoords.y].size() <= paintCoords.x)
    {
        m_fields[paintCoords.y].push_back(EMPTY_CHAR);
    }

• [x] У вектора есть метод insert http://www.cplusplus.com/reference/vector/vector/insert/