Странная ошибка с const

0

1

Не хочет компилироваться следующий метод:

// Преобразование таблицы конечных записей в Dom документ
QDomElement RecordTableData::exportDataToDom(QDomDocument *doc) const
{
  // Если у ветки нет таблицы конечных записей, возвращается пустой документ
  if(tableData.size()==0)
    return QDomElement();

  QDomElement recordTableDomData=doc->createElement("recordtable");

  // Пробегаются все записи в таблице
  for(int i=0; i<tableData.size(); i++)
    recordTableDomData.appendChild( tableData.at(i).exportDataToDom( doc ) ); // К элементу recordtabledata прикрепляются конечные записи

  // qDebug() << "In export_modeldata_to_dom() recordtabledata " << doc.toString();

  return recordTableDomData;
}

Ошибка выглядит так:

src\models\recordTable\RecordTableData.cpp: In member function 'QDomElement RecordTableData::exportDataToDom(QDomDocument*) const':
src\models\recordTable\RecordTableData.cpp:308:74: error: passing 'const Record' as 'this' argument of 'QDomElement Record::exportDataToDom(QDomDocument*)' discards qualifiers [-fpermissive]
     recordTableDomData.appendChild( tableData.at(i).exportDataToDom( doc ) ); // Рљ СЌР»РµРјРµРЅС‚Сѓ recordtabledata РїСЂРёРєСЂРµРїР»СЏСЋС‚СЃСЏ РєРѕРЅРµС‡РЅС‹Рµ Р·Р°РїРёСЃРё

Не пойму чего хочет компилер. Сначала думал, что проблема в том, что метод константный, и вызывает неконстантный метод

tableData.at(i).exportDataToDom( doc )

Но потом понял, что вызываемый неконстантный метод - это метод другого класса, так что он влиять не может.

И так же убирание const из определения вышеприведенного метода проблему не решает.

Вопрос: как заставить компилироваться код?

Исходники: https://github.com/xintrea/mytetra_dev/commits/experimental
Коммит: 3c674e61423616ae3ca8e4191c09cfaed945a8ef

Ссылка

←	libc fwrite fflush как узнать размер буфера?

Макросы в расте

→

внезапно: tableData.at(i) возвращает const Record &

dt1 ★★
(15.02.16 17:07:49 MSK)

Ссылка

Во-первых метод at возвращает константную ссылку. Вот он на это и ругается в первую очередь.

Во-вторых, тут recordTableDomData.appendChild ты вроде пытаешься поменять поле класса?

grondek ★
(15.02.16 17:10:46 MSK)

Ссылка

Вот эта хрень - exportDataToDom( doc ) не константная
А вот эта -tableData.at(i) наоборот, константная, потому как at(i) возвращает константную ссылку.

AF ★★★
(15.02.16 17:14:40 MSK)

Ответ на: комментарий от AF 15.02.16 17:14:40 MSK

Ну я для интереса заменил at(i) на [ i ]:

recordTableDomData.appendChild( tableData[i].exportDataToDom( doc ) );

но ошибка осталась:

recordTableDomData.appendChild( tableData[i].exportDataToDom( doc ) );

../mytetra/src/models/recordTable/RecordTableData.cpp: In member function 'QDomElement RecordTableData::exportDataToDom(QDomDocument*) const':
../mytetra/src/models/recordTable/RecordTableData.cpp:308:71: error: passing 'const Record' as 'this' argument of 'QDomElement Record::exportDataToDom(QDomDocument*)' discards qualifiers [-fpermissive]
     recordTableDomData.appendChild( tableData[i].exportDataToDom( doc ) ); // К элементу recordtabledata прикрепляются конечные записи

https://github.com/xintrea/mytetra_dev/commit/bdb2f926dcf18bbf2ba33720bf1b613...

Xintrea ★★★★★
(15.02.16 20:20:28 MSK) автор топика

Ответ на: комментарий от Xintrea 15.02.16 20:20:28 MSK

друже, тебе глаголят, что метод exportDataToDom не константный, однако, сучилось так, что tableData есть константа, поскольку находится в константном методе класса. Для константы возбранятеся вызывать не константный метод. Либо метод на неконстентный, сиречь убери «const», либо соизволь сделать tableData mutable. В общем в консерватории что-то не так.

anonymous
(15.02.16 20:24:08 MSK)

Нельзя коммитить некомпилирующийся код!!!

vzzo ★★★
(15.02.16 20:24:22 MSK)

Ответ на: комментарий от vzzo 15.02.16 20:24:22 MSK

можно, только не скомпилируется.

anonymous
(15.02.16 20:25:50 MSK)

Ответ на: комментарий от anonymous 15.02.16 20:24:08 MSK

сиречь убери «const»

Блин я столько с ними мучался чтоб расставить.

либо соизволь сделать tableData mutable

А это что имеешь в виду?

Xintrea ★★★★★
(15.02.16 20:30:06 MSK) автор топика

Ответ на: комментарий от Xintrea 15.02.16 20:30:06 MSK

    // Таблица записей (в нормальном виде содержит только "легкие" объекты записей)
    QList< Record > tableData;

-->

    // Таблица записей (в нормальном виде содержит только "легкие" объекты записей)
    mutable QList< Record > tableData;

но это коряво. Убери лучше конст, если ты меняешь объект.

anonymous
(15.02.16 20:32:43 MSK)

Ссылка

Ответ на: комментарий от Xintrea 15.02.16 20:20:28 MSK

Ну я для интереса заменил at(i) на [ i ]:

Ты забыл, шо:
1) T & QList::operator[](int i)
2) const T & QList::operator[](int i) const

В твоем случае компилятор выбрал второй вариант, потому, как QDomElement RecordTableData::exportDataToDom(QDomDocument *doc) const

AF ★★★
(15.02.16 20:33:34 MSK)

Ссылка

Ответ на: комментарий от Xintrea 15.02.16 20:30:06 MSK

самое забавное, что у тебя явно константные методы НЕКОНСТАНТНЫ! пример:

// Получение значения текста указанной записи
// Метод возвращает расшифрованные данные
// Если возникнет проблема, что файла с текстом записи нет, будет создан пустой файл
QString RecordTableData::getText(int pos)
{
 // Если индекс недопустимый, возвращается пустая строка
 if(pos<0 || pos>=size()) 
   return QString();

 return tableData[pos].getTextDirect();
}

anonymous
(15.02.16 20:34:11 MSK)

Ответ на: комментарий от Xintrea 15.02.16 20:30:06 MSK

сиречь убери «const»

Как только я его убираю, у меня по цепочке приходится убирать консты со всех геттеров. А это нехорошо.

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

Не знаю как это обойти.

Xintrea ★★★★★
(15.02.16 20:35:40 MSK) автор топика

Ответ на: комментарий от Xintrea 15.02.16 20:35:40 MSK

так на кого дьявола у тебя геттер меняет объект? Геттер - получение. Не должен он ничего менять.

anonymous
(15.02.16 20:37:42 MSK)

Ответ на: комментарий от Xintrea 15.02.16 20:35:40 MSK

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

Почитай про ключевое слово mutable.

anonymous
(15.02.16 20:38:27 MSK)

Ссылка

Ответ на: комментарий от anonymous 15.02.16 20:37:42 MSK

так на кого дьявола у тебя геттер меняет объект? Геттер - получение. Не должен он ничего менять.

Вполне может, как раз в случае кэширования/ленивого чтения. И такие поля просто имеют спецификатор mutable.

anonymous
(15.02.16 20:39:27 MSK)

Ответ на: комментарий от anonymous 15.02.16 20:34:11 MSK

самое забавное, что у тебя явно константные методы НЕКОНСТАНТНЫ!

Я знаю об этом. В данном месте должен быть конст, тут надо подправить. Но у меня данные хранятся как в памяти, так и на диске (т. н. «легкие» данные - в памяти, а «тяжелые» - на диске). И я пока не могу сообразить, как с ними обращаться с точки зрения конста. С данными в памяти все понятно. А вот с данными на диске - нет, потому что они не хранятся в свойствах класса.

Xintrea ★★★★★
(15.02.16 20:40:18 MSK) автор топика