Как найти ошибку в C/C++ коде? Видимо где-то забыл ; или " или }

[xintrea]

Здравствуйте!


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

Проблема в том, что компилятор gcc, в случае забывчивости программиста поставить ; или " или }, обычно не показывает то место, где эта ошибка произошла, и догадываться приходится по коственным признакам.

Вот сейчас у меня такая же ситуация. Я столько кода наменял, что вспомнить, где надо перепроверить, нет возможности. А скомпилить не могу, из-за неизвестного места, в котором есть проблема.

Что делать? Слышал, вроде есть какие-то дополнительные анализаторы кода, но те что я нашел - это для аудита безопасности кода.

Вот вывод ошибок

Код:

src/treeitem.h:75: ошибка: ISO C++ запрещает декларации 'recordtabledata' без типа
src/treeitem.h:75: ошибка: expected ';' before '*' token
src/treeitem.h:89: ошибка: 'recordtabledata' does not name a type
src/recordtablemodel.h:38: ошибка: 'recordtabledata' не был декларирован
src/recordtablemodel.h:43: ошибка: ISO C++ запрещает декларации 'recordtabledata' без типа
src/recordtablemodel.h:43: ошибка: expected ';' before '*' token
src/recordtablescreen.h:35: ошибка: 'recordtabledata' не был декларирован
src/recordtablescreen.h:125: ошибка: ISO C++ запрещает декларации 'recordtabledata' без типа
src/recordtablescreen.h:125: ошибка: expected ';' before '*' token

src/recordtablescreen.cpp:173: ошибка: prototype for 'void recordtablescreen::set_tabledata(recordtabledata*)' does not match any in class 'recordtablescreen'
src/recordtablescreen.h:35: ошибка: претендент: void recordtablescreen::set_tabledata(int*)

src/recordtablescreen.cpp:173: предупреждение: unused parameter 'rtdata'

src/recordtablescreen.cpp: In member function 'void recordtablescreen::select(const QModelIndex&)':
src/recordtablescreen.cpp:217: ошибка: нет декларации 'table' в этой области видимости
src/recordtablescreen.cpp: In member function 'void recordtablescreen::add_new(int, QString, QString, QString, QString, QString)':
src/recordtablescreen.cpp:312: ошибка: нет декларации 'table' в этой области видимости
src/recordtablescreen.cpp: In member function 'void recordtablescreen::edit_field_context()':
src/recordtablescreen.cpp:341: ошибка: нет декларации 'table' в этой области видимости
src/recordtablescreen.cpp: In member function 'void recordtablescreen::edit_field(int, QString, QString, QString, QString)':
src/recordtablescreen.cpp:373: ошибка: нет декларации 'table' в этой области видимости
src/recordtablescreen.cpp: In member function 'void recordtablescreen::delete_records()':
src/recordtablescreen.cpp:426: ошибка: нет декларации 'table' в этой области видимости
src/recordtablescreen.cpp: In member function 'void recordtablescreen::copy()':
src/recordtablescreen.cpp:459: ошибка: нет декларации 'table' в этой области видимости
src/recordtablescreen.cpp: In member function 'void recordtablescreen::moveup()':
src/recordtablescreen.cpp:642: ошибка: нет декларации 'table' в этой области видимости
src/recordtablescreen.cpp: In member function 'void recordtablescreen::movedn()':
src/recordtablescreen.cpp:662: ошибка: нет декларации 'table' в этой области видимости

По какой-то причине невозможна работа с классом recordtabledata. Обычно такие ошибки возникают, если забыть подключить заголовок recordtabledata.h через include, или в прототипе класса забыть за закрывающей скобкой поставить ; . Везде где нужно (и ненужно) заголовок подключен. В pro файле в HEADERS и SOURCES добавлены recordtabledata.h и recordtabledata.cpp соответсвенно.


Обращаю внимание на странную ошибку

Код:

prototype for 'void recordtablescreen::set_tabledata(recordtabledata*)' does not match any in class 'recordtablescreen'
src/recordtablescreen.h:35: ошибка: претендент: void recordtablescreen::set_tabledata(int*)

Претендента set_tabledata(int*) у меня нигде на самом деле в коде нет, а есть
 

Код:

 // Прототип в классе recordtablescreen (файл recordtablescreen.h)
 void set_tabledata(recordtabledata *);

 // Реализация (файл recordtablescreen.cpp)
 void recordtablescreen::set_tabledata(recordtabledata *rtdata)
 {
  ...
 }

Поток ошибок с table связано с тем, что член класса table имеет тип recordtabledata.


В общем, все указывает на проблему в классе recordtabledata. Я его излазил вдоль и поперек, ошибки не вижу. Но почему-то с ним происходят вот такие странные вещи. И почему-то в нем компилятор ошибок не обнаруживает.


Вопрос. Что делать, как найти ошибку?

[pastor]

1. Начать комментирорвать код проекта, пока не исчезнет ошибка. Это давольно кропотлиный процесс
2. Выложеть код, и пусть глянет общественность

[Rcus]

А еще есть крайнее средство, когда больше ничего не помогает:
 Посмотреть на компиляции какого файла происходит ошибка, запустить его компиляцию с флагом -E (Только препроцессирование - без компиляции, ассемблирования, компоновки) и читать

[BRE]

А выложи файлик где описывается recordtabledata.

[xintrea]

1. Начать комментирорвать код проекта, пока не исчезнет ошибка. Это давольно кропотлиный процесс

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

2. Выложеть код, и пусть глянет общественность

Чтоб сильно не напрягать общественность, прошу попробовать скомпилить код под виндой (т.е. не в gcc под Linux), может другой компилер скажет более конкретно. Прога на чистом Qt, никаких дополнительных библиотек не используется.

Архив приложен.

[panAlexey]

Начать пользоваться системой хранения версий.
простой диф тебе покажет где ты чего менял, а следовательно и проанализировать правки с момента нормальной компиляции.
ПС. Я один раз скобку на наймспейсе забыл закрыть, ох и понаслушался....

[xintrea]

Начать пользоваться системой хранения версий.
простой диф тебе покажет где ты чего менял, а следовательно и проанализировать правки с момента нормальной компиляции.
ПС. Я один раз скобку на наймспейсе забыл закрыть, ох и понаслушался....

Наскока я понимаю, в систему хранения версий имеет смысл коммитить как минимум только рабочий (компилируемый) код. Это, вроде как, основное правило пользования системы контроля версий. Так что, когда глобально перелопачиваешь код, в vcs лучше не коммитить промежутки. Не знаю, так ли это или нет, ибо вместо vcs использую rdiff-backup, но и им я не делаю копий пока код не заработает хоть в каком виде.

[panAlexey]

linetmp=table.at(pos)); // Раньше было index.row()
Спасибо креатору Поиск меньше минуты занял...

[xintrea]

linetmp=table.at(pos)); // Раньше было index.row()
5 баксов. Спасибо креатору ))))
http://unnstudio.googlecode.com/files/spasCrea.JPG

Только что хотел спросить, как ты это нашел!

Щас поставлю креатор для вот такой валидации... Там он еще не показывает ошибку? А то компиляция аналогично пока завершается.

[panAlexey]

все, больше не могу искать. работать надо.
больше ничего не нашел.

[xintrea]

все, больше не могу искать. работать надо.
больше ничего не нашел.

Спасибо большое, дальше сам.

[panAlexey]

В принцыпе можно написать скрипт, который подсчитывает парные скобки.
Не шибко сложно, должно помочь...

[Eugene Efremov]

Убрать зависимости хедеров друг от друга, т.е. заменить строки вида «#include "recordtabledata.h"» на «class recordtabledata;». А сами инклуды перенести из h в cpp. Это позволит, по крайней мере, локализовать хедер, в котором ошибка... возможно. Мда, это надо было сделать до того как она появилось, а сейчас есть риск новые насажать.

И на будущее — не менять код сразу такими большим кусками... Вообще, не исключено, что проще будет откатить на последнюю работающую версию и переписать заново, с учетом вышесказанного.

P.S. Просмотрел код под колорером, явных синтаксических ошибок (т.е. таких, которые бы колорер обнаруживал) помимо той скобки там нет.

[lit-uriy]

Наскока я понимаю, в систему хранения версий имеет смысл коммитить как минимум только рабочий (компилируемый) код. Это, вроде как, основное правило пользования системы контроля версий.

Нет такого правила. Правила ты придумываешь сам. Есть сложившиеся тактики, одна из них заключается в копировании рабочего каталога (trunk) или его части в ветки (branch). А там изгаляйся как тебе удобно (в том числе любое кол-во промежуточных правок). После окончания эксперементов делаешь слияние ветки с рабочим каталогом.

[Mikhail]

Убрать зависимости хедеров друг от друга, т.е. заменить строки вида «#include "recordtabledata.h"» на «class recordtabledata;». А сами инклуды перенести из h в cpp. Это позволит, по крайней мере, локализовать хедер, в котором ошибка... возможно. Мда, это надо было сделать до того как она появилось, а сейчас есть риск новые насажать.

И на будущее — не менять код сразу такими большим кусками... Вообще, не исключено, что проще будет откатить на последнюю работающую версию и переписать заново, с учетом вышесказанного.

P.S. Просмотрел код под колорером, явных синтаксических ошибок (т.е. таких, которые бы колорер обнаруживал) помимо той скобки там нет.

Скорее всего отсутствие логики в проекте.
У тебя в каждом включаемом файле включаются в основном все остальные включаемые файлы.
Вернись к рабочей версии и по шагам иди вперед.
Избегай включать ненужные в хедеры все другие.
Если уж надо, то используй конструкцию
class ttt;
а инклуд уже в cpp файл.