1

Labirint - labirint.cpp
labirint.h - labirint.h
матрица - data.txt

Проблема сама в типах данных я так понимаю, но конкретно найти проблему не могу

При объявлении массива в заголовочном файле, результат такой: инициализация в заголовочном файле

При инициализации массива с файла, результат такой: введите сюда описание изображения

Ну и понятно же что лабиринт не проходит алгоритм, не знаю из-за чего

1
  • Какой алгоритм должен пройти лабиринт? Что вы ожидаете получить и фактически получаете? Почему нет возможности найти ошибки с помощью отладчика?
    – Cerbo
    30 сен 2015 в 16:24

2 ответа 2

3

Я думаю, что проблемы связаны с тем, что вы объявили внутри класса константы

const int sizeRow = 24;
const int sizeCol = 20;

но вместо этих констант в задании размерностей массива map используете магическое число 24

unsigned char map[24][24]
                  ^^^^^^^

хотя список инициализации для каждой строки содержит только sizeCol, то есть 20 элементов. Например

{ 1, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, },

Не следует использовать магические числа, а надо использовать именованные константы. Для этого они и объявлены.

Кстати сказать, лучше эти константы объявить как статические члены класса

static const int sizeRow = 24;
static const int sizeCol = 20;

Также у вас расхождение между представлениями считываемых значений из файла как символов, тогда как в определении массива у вас заданы не символы, а целые числа.

Вы могли бы считывать данные из файла предварительно в некоторую переменную типа unsigned int, а затем ее значение присваивать элементу массива. Например

unsigned int value;

ifs >> value;
this->map[i][j] = value;
8
  • я переделал по вашим замечаниям, но вот в этом участке for (int i = 0; i < this->sizeRow; i++) { for (int j = 0; j < 24; j++) { не хочет работать, если ставлю sizeCol (20)
    – jashka
    30 сен 2015 в 17:27
  • @jashka Не совсем понял, что в этом участке и почему опять используется число 24? 30 сен 2015 в 17:28
  • потому что без него вообще алгоритм не работает
    – jashka
    30 сен 2015 в 17:33
  • @jashka Почему не работает? У вас есть константа sizeCol. 30 сен 2015 в 17:35
  • в том то и дело что если вместо 24 написать this->sizeCol, то алгоритм вообще не проходит лабиринт
    – jashka
    30 сен 2015 в 17:36
1

Все очень просто: ваш текстовый файл содержит СИМВОЛЫ цифр, то есть состоит из байтов 0x31 или 0x30, к которым еще домешан пробел (0x20). Ваш заголовочный файл содержит массив, в котором записаны только байты 0x01 или 0x00. В итоге текстовик имеет ну совершенно иной формат.

2
  • как можно избежать этого ?
    – jashka
    30 сен 2015 в 16:29
  • @jashka Как можно избежать чего именно? Другого формата? Создавайте текстовик в нужном формате, делов-то. 30 сен 2015 в 16:29

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.