Глюк QList или моя ошибка

[aliks-os]

Есть структура

Код:

struct ConData {
    QString name;
    QString driver;
    QString dbName;
    QString host;
    QString user;
    QString password;
    int port;
    QString charset;
    int charsetIdx;
};

Загнал ее в массив

Код:

QList<ConData> connectionList;

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

Код:

ConnectionDlg *dialog = new ConnectionDlg(this);
ConData co1 = connectionList.at(curCon);  //curCon - это номер строки в массиве
ConData co = dialog->showThis(co1);

Далее, пытаюсь полученное значение подменить в массиве. И вот тут проблемы...
если делаю вот так...

Код:

connectionList.replace(curCon,co);

то программа потом вылетает, трассировка из-за чего не помогает.
А если делаю вот так, то все ок

Код:

connectionList[curCon].host = co.host;
...
и так все поля.

Подскажите, где моя ошибка и как это побороть

[Fat-Zer]

вроде ничего криминального не видно...

Код:
connectionList.replace(curCon,co);

то программа потом вылетает, трассировка из-за чего не помогает.

странно что не помогает, хотя бы побольше информации о том как валится... сигфолт? тогда коллстек посмотрите, где именно.

Код:

connectionList[curCon].host = co.host;
...
и так все поля.

Подскажите, где моя ошибка и как это побороть

почему не просто

Код:

connectionList[curCon]=co;

[aliks-os]

Если напишу просто

Код:

connectionList[curCon]=co;

то у меня тоже возникает ошибка при повторном обращении....При дебуге, ошибка возникает в
файле qatomic_i386.h
а конкретно в функции

Код:

inline bool QBasicAtomicInt::ref()
{
    unsigned char ret;
    asm volatile("lock\n"
                 "incl %0\n"
                 "setne %1"
                 : "=m" (_q_value), "=qm" (ret)
                 : "m" (_q_value)
                 : "memory");
    return ret != 0;
}

Ошибка возникает как не странное при обращении к QTreeWidget, хотя они у меня ни как не связаны

[Fat-Zer]

хотя бы побольше информации о том как валится

в том коде, который вы привели явных ошибок нет.

[Igors]

Наиболее вероятно - битая куча. Кто-то пишет по удаленному указателю или др. невалидному адресу. В результате калечится объект(ы) который не имеет к пишущему никакого отношения. Нужно запрягать отладчик на предмет проверки кучи, хотя это не всегда помогат. Простой способ - воткнуть строку (connectionList[curCon]=co;) "до и после" и так пытаться локализоваться

[aliks-os]

ну то что куча битая, я это понял. уже победил, вот таким способом.

Код:

ConData *co1 = &(connectionList[curCon]);
ConData co = *dialog->showThis(co1);
connectionList[curCon] = co;

Стал передавать в функцию указатель и она его же возвращает

[Igors]

ну то что куча битая, я это понял. уже победил, вот таким способом.

Код:

ConData *co1 = &(connectionList[curCon]);
ConData co = *dialog->showThis(co1);
connectionList[curCon] = co;

Стал передавать в функцию указатель и она его же возвращает

Ну это Вы заткнули "до следующего милиционера". Нужно найти кто бьет и устранить. И на всякий случай поставить проверку curCon < connectionList.size()

[aliks-os]

Ну это Вы заткнули "до следующего милиционера". Нужно найти кто бьет и устранить. И на всякий случай поставить проверку curCon < connectionList.size()

Согласен, буду искать еще, проверку curCon < connectionList.size() сделаю обязательно, но в данном случае ошибка была не из-за него

[Авварон]

я бы конструктор копирования написал, класс-то не POD

[aliks-os]

а подробней можно?

[Пантер]

Копирующий конструктор типа:
ConData (const ConData& cd) : name (cd.name), и т.д. {
 
}

[Fat-Zer]

я бы конструктор копирования написал, класс-то не POD

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

[Авварон]

ну вообще-то будет, по умолчанию происходит что-то типа memcpy насколько я знаю, а конструкторы кустрингов вроде не вызовуться

[Fat-Zer]

ну вообще-то будет, по умолчанию происходит что-то типа memcpy насколько я знаю, а конструкторы кустрингов вроде не вызовуться

я до недавнего времени тоже так считал, но меня образумили))
1) конструктор копирования по умолчанию вызывает оные для всех членов.
2) в данном случае будет вызываться не конструктор копирования, а оператор присваивания.

пример:

Код

C++ (Qt)
#include <iostream>
 
using namespace std;
 
struct SubStruct
{
public:
	SubStruct()
	{
		cout<<"in substruct constructor\n";
	}
	SubStruct(const SubStruct&)
	{
		cout<<"in substruct copy constructor\n";
	}
	const SubStruct& operator=(const SubStruct&)
	{
		cout<<"in substruct operator=\n";
		return *this;
	}
};
 
 
struct MyStruct
{
public:
	SubStruct ss;
};
 
 
int main(int , char *[])
{
	cout<<"Creation:\n";
	MyStruct s1, s2;
	cout<<"Assigning:\n";
	s2=s1;
	cout<<"Copyconstructing=)\n";
	MyStruct s3(s1);
 
	return 0;
}
 

[Пантер]

Ну, вообще советуют:
1. Если переопределяешь конструктор копирования, то и присваивание тоже переопределяй.
2. Конструктор копирования реализуется через присваивание.