Russian Qt Forum

Здравствуйте.
Столкнулся с одной проблемой, которую не в силах объяснить, поэтому и название темы не информативное.

Задача куска кода: пробежаться по всем позициям спутника за заданный интервал и посчитать в каких местах радиосигнала накапливается больше.

satList - это QList из QList'ов.
Каждый элемент satList'a - это QList, состоящий из:

activeSats - это QList, в котором хранятся индексы элементов satList, выбранных для отображения.
( на форме просто чекбокс, в котором выбирается что отображать, а что - нет )
activeSatsNum - это activeSats.count();

ephIndex - это индекс структуры из элемента satList'a.
intervalBegin и intervalEnd - это с какого по какой индексы нужно отобразить структуры.

patterns - это QList, состоящий из структур вида:

В этом листе лежат шаблоны.
Шаблон есть матрица 180 на 360, в которой true установлена для ячеек, соответствующих зоне, в которой сигнал от спутника есть.
Т.е. есть набор шаблонов, соответствующих широте от 0 до 90, и, в зависимости от текущей широты, для которой происходят вычисления, выбирается тот или иной шаблон.
Поскольку форма зоны радиовидимости для широт от 0 до 90 и от 0 до -90 - зеркальная, то при отрицательной широте флаг inverted устанавливается в true;

cm - это тот номер столбца, для которого был рассчитан шаблон,  т.е. отталкиваясь от него и зная текущий номер столбца ( текущую долготу подспутниковой точки ) + текущий номер столбца матрицы шаблона, можно было бы корректно преобразовать координаты позиции в результирующей матрице.

Соответственно, patternIndex - это индекс шаблона для конкретной широты.



rm->addToMatrix(int row, int col) добавляет в ячейку(row)(col) результирующей матрицы еденицу, отмечая этим что в данном кванте поверхности радиосигнал присутствовал.
Если представить циллиндрическое изображения Земли, то на него как бы наложена матрица, каждая ячейка которой представляет собой квант поверхности 1° на 1°.
( не совсем так, конечно, т.к. в приполярный зонах 1° - это совсем не один квадрат, а вся линия. )
Также в rm есть проверка на границы переданных значений и, если столбец превышает 359 или меньше 0, то "перебрасывает" его через нулевой меридиан.

Так вот, в чём вся штука, если в месте, которое я отметил, написать:
, то всё работает, иначе - вываливается с ошибкой.
Причем не в конкретно этом месте, а на следующей интерации цикла.

Переменные объявлены вне циклов т.к. это я уже плясками с бубном занимался.
Не уверен, стоило ли использовать QList для таких целей, но как-то он мне понравился.
Раньше всё работало, а стоило мне поменять размерность матриц с [360][720] на [180][360] - началось такое.
Сами шаблоны составлены корректно, проверял как они отображаются по одному.

Не знаю даже в чём проблема.
Если нужно - приведу скрин ошибки.
Возможно, объяснил не очень, если возникнут вопросы - задавайте.

Вы сами понимаете что вникнуть во все подробности задачи постороннему человеку затруднительно. Можно посоветовать

- убрать все "числа", использовать как положено константы (на худой конец #define) т.е. напр  m[180][360] заменить на m[LAT_DIV][LONG_DIV]. Конечно это всем известно, но частенько думается "та ладно, и так работает"

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

C++ (Qt)
assert(i >= 0 && i < LAT_DIV);
assert(j >= 0 && j < LONG_DIV);
int val = m[i][j];
 

Ну в общем, писать культурно  :)

Да, прекрасно понимаю, да и с точки зрения просто логики всё впоряде здесь.
Мне и казалось изначально, что дело в каких-то тонкостях, которых я просто не знаю по неопытности.

Сделал как вы сказали, размерность матрицы для шаблона через константы задал, циклы тоже до констант бегать пустил.
Два раза всё отработало нормально, потом всё снова начало падать, только с разными ошибками.

Первый раз слышу, что предпочтительнее использовать контанты в таких случаях. Чем числа плохи?
Про assert не совсем понял, какое именно место вы имеете ввиду?

И можно ли где-нибудь почитать о подобных вещах? Хочу научиться писать культурно. Собственно, первая моя серьёзная программа.

Лично меня смущает, что приходится использовать столько QList'ов, столько много получается обращений к разным их элементам.
И почему-то кажется, что лучше использовать не структуру, а класс, и все внутренние переменные сделать private, а для доступа к ним написать соответствующие методы.

Как посоветуете поступить?
Спасибо.

Не цитирую чтобы не раздувать пост, здесь и так понятно на что ответы

Числа плохи тем что по мере разрастания кода их потом хрен найдешь, возможно Вы и стали жертвой. По поводу private - удобный случай изучить принципы ООП на деле. Только не лепить тупо get/set, а конкретно. Напр у Вас есть такое m[ i ][ j ].  Это никак не контролируется, если произошла запись за границы - память испорчена и падения могут быть где угодно. Вот и сделайте m private, а в методах get/set воткните assert

Используй std::vector и метод at. При выходе за грани массива, at кидает исключение.
Вычисления точно должны происходить в геодезических координатах, а не в госк'е?

Igors, перед тем, как смотреть true или false значение в матрице, у меня стоит assert() на и i и j, так что дело там вряд ли в выходе за границы, но всё равно попробую таки сделать, "для четкости".

DmitryM, так QList - почти тот же вектор, не?
По поводу координат - да.

Так.
Попробовал использовать QVector вместо QList для хранения элементов.
Прочитал в документации, что если вес элемента больше указателя или стандартных типов, то лучше использовать QVector.
Не помогло.

Решил каждый элемент списка создавать через new и в списка хранить указатели на элементы.
Тоже не помогло.

Заметил, что дело не только в обращении к файлу шаблона по индексу, но и то же самое происходит при попытке "дёрнуть" подспутниковую точку по индексу.

Дело не в выходе за границы списков и.т.п., проверял они в таком случае выкидывают ошибку, похожую на assert'овскую.

Скрины ошибок во вложении.

Заметил, что если загружать не больше 20 подспутниковых точек, то всё работает.
И ещё, кусок кода сначала нормально отрабатывает, показывается результат, а потом выкидывает ошибку номер 2. ( ели успел заметить, что отработала графика )
1-ая же ошибка получается когда объём данных увеличивается.

Перепробовал, как мне кажется, всё, куда копать - ума не приложу.

Ах, и да, кусок кода где не идёт поиндексное обращение в цикле к подспутниковым точкам, работает нормально.

надо смотреть свои методы в стеке вызовов (если они там конечно есть), а не библиотечные. думаю происходит обращение к удалённому элементу.

Но в ходе выполнения программы ничего не удаляется.
Поэтому-то я и в замешательстве.
Вроде ведь даже для каждого элемента отвёл своё место в куче.
Думал сначала, что, может, какие-то проблемы с распределением памяти.

Сори, возможно лучше заменить обычные массивы в  eph, pattern.
Стандартные контейнеры бросают исключения, а Qt'шные нет.
По поводу координат, наверное спутники по углу возвышения в другом месте отбрасываются.

Заменить обычные массивы? Но на что?
В плане выхода за границы Qt'шные тоже бросают.

Попробуйте "вцепиться" в ошибку, такая возможность есть. Ну вот напр 2.png - явно виджет испорчен, возможно повторное удаление. Добавляете отладочную печать в деструкторе

C++ (Qt)
MyWidget::~MyWidget( void )
{
 qDebug() << "destructor" << this;
}
 

А так менять здесь и там "а вдруг поможет" можно долго но безуспешно

Проверил, деструктор не вызывается. ( Хотя какой виджет проверять? )
Проверял тот, в который значения добавляются, ну и главный.

Есть ещё класс, код которого абсолютно такой же как и тот, который я пытаюсь использовать при ошибке.
Если подставить элемент класса-копии, то всё работает.
Т.е. написать в цикле, например, не
, а
, то всё работает.

Т.е. если проблема в классе, то непонятно в чём именно, т.к. код в точности идентичный.

Создаются элементы этих двух классов одинаково, обрабатывается всё в разных слотах одинаково.

Класс-копию я создал скопировав код "проблемного" класса и поменяв имена. ( Он для других целей используется, но похжих. )

Выкладывайте минимальный сбойный проект, а то прогресса нет

Попробуй запустить cppcheck (http://cppcheck.sourceforge.net/), статические анализаторы могут дать что-нибудь, для размышления.

Хорошо, спасибо, сейчас попробую покопаться.

Вопрос немного не по теме, но: есть задача создать некоторое подобие "слоёв" ( как в ФШ ), т.е. слой с одной графикой, слой с другой и.т.п.
Делаю я сейчас это так: виджет с переопределённым paintEvent, у него внутри QImage, графику рисую на QImage, а в paintEvent() виджета рисую эту QImage на сам виджет.
Можно ли как-нибудь сделать это более "культурно" или и так норм?

Выкладываю минимальный сбойный проект.
Танцов с бубном там было ну очень много, многое сделал иначе, но всё равно не работает.
Вот ссылка: http://rghost.net/39728031
В архиве сам проект, 8 файлов с данными, представляющими спутники.
Файл с данными - это смоделированные положения подспутниковых точек с течением времени, в моём случае рассчитанный для не реальных данных.
Считываю я из него широту+долготу ПТ, время, и ещё 37 точек, представляющих зону радиовидимости спутника. ( 1 точка - это одна точка на фигуре проекции ЗРВ на поверхность Земли, рассчитанны они с шагом в 10 градусов и последняя == первая. )

Шаблоны подрузятся сами, нужно будет нажать "Загрузить данные" и выбрать входной файл. Если смотреть ресурс для одного/несколькоих спутников, но за малое время, то можно будет увидеть как они двигаются.

Целый день сегодня с этим просидел, голова уже не работает, вот и решил "сдаться".

Любые комментарии по поводу того как можно было бы сделать "культурнее" приветствуются :)

Пробовал считать ресурс не в класс, а во внутреннюю матрицу основного класса, - всё равно вылетает.

Сейчас попробую ещё шаблонов наделать для каждого кванта, хоть и памяти много сожрёт, но каким образом ещё заставить это работать я уже не знаю.

Дайте пожалуйста pro файл (я не на Вындоуз)

Разобрался.
Это я дурак, опять.
Даже стыдно как-то стало.
С вычислением строк в случае инвертирования шаблона ( когда широта отрицательная ) напутал.
Если всё ещё интересно - могу скинуть файл.

Ну чего же смотреть когда проблемы нет? Подождем пока возникнет. А самобичевание ни к чему  :)

Да я так, почти шутя :)
Находишь глупую ошибку, из-за которой долгое время мучался, и даже не знаешь, что чувствовать.