Как правильно работать с QList?

Дело не в том, как правильно работать с QList, а в том как правильно работать с указателями.

graph = new QList<QwtPlotCurve>;
// ...
(*graph)[i].attach(ui->graphWidget);

И зачем вообще делать указатель на QList? Он все равно хранит непосредственные данные в куче, а не на стеке. А, это ты видимо пытаешься создать десять элементов внутри QList - ну-ну. Нужно создать QList и далее методом append() или push_back() вставлять элементы.

Далее. Метод at() ведь описан как константный и при этом возвращающий ссылку на константу https://doc.qt.io/qt-5/qlist.html#at. Чего ж тут удивляться?

Поэтому я всегда советую: прежде всего нужно почитать что-нибудь по основам C++, а потом уже браться за Qt.

Видимо дело вот в этой строке?

graph = new QList<QwtPlotCurve>[10];

Которую читать нужно как

Создать указатель на массив из 10 QList<QwtPlotCurve>

А по хорошему нужно

QList<QwtPlotCurve*> graph;

Хотя пусть C++-ники поправят, хорошая ли идея хранить экземпляр компонента в контейнере

может конечно не к месту…но в современном cpp, использование указателя, значит недоделанность/неправильность конструктива.

объявление «QList< QwtPlotCurve > *graph» хреновый знак….

плюсы работают пока не трогают указатели

Да, в целом вы правы.

Нужно создать QList и далее методом append() или push_back() вставлять элементы.

Можно ту по подробнее объяснить?

И вопрос такой:

А не будет ли проще просто создать массив указателей на QwtPlotCurve, что нибудь типа QwtPlotCurve *cruve[10];, или я совсем туплю?

private:
…
QList *graph;

Зачем указатель? Положи на стек.

graph[i].

Ты сейчас итерируешься по массиву QList’ов, а не по его содержимому. У тебя здесь выход за границу памяти. Тогда уж так (*graph)[i]. Но лучше избавится от указателя, как я в начале написал.

Не знаешь семантику цепепе.

Не очень понятно к чему вы пишете. Похоже у вас тут два акка, и вы запутались.

Про

Ты сейчас итерируешься по массиву QList’ов, а не по его содержимому. У тебя здесь выход за границу памяти. Тогда уж так (*graph)[i].

я уже понял.

Вопрос встал немного шире:

А не будет ли проще просто создать массив указателей на QwtPlotCurve, что нибудь типа QwtPlotCurve *cruve[10];, или я совсем туплю?

И вообще избавится от QList.

Ты зачем то создал не список, а массив списков.

для Qlist нет конструктора, в котором бы сразу происходил resize, за исключением случая создания из другого объекта и потому выходит в две команды:

graph = new QList<QwtPlotCurve>;
graph.reserve(10);

При этом ты не устанавливаешь размер листа. Это резервирование памяти.

А не будет ли проще просто создать массив указателей на QwtPlotCurve, что нибудь типа QwtPlotCurve *cruve[10];, или я совсем туплю?

Можно и так, но по мне проще через QList, т.к. не нужно вручную заниматься выделением/освобождением памяти (что способствует возникновению утечек памяти и т.п.).

Вообще QList и подобные высокоуровневые контейнеры как раз и нужны для того, чтобы избавится от головной боли и удобно работать.

сам qlist всегда аллоцируется в heap, делай проще вроде как:

private:
    ...
    QList<QwtPlotCurve *> graph;

cpp:
for(int i=0; i < 10; i++)
{
    QwtPlotCurve *curve = new QwtPlotCurve();
    graph.insert(i, curve);
    graph.at(i)->attach(m_ui->graphWidget);
}

А не будет ли проще просто создать массив указателей на QwtPlotCurve, что нибудь типа QwtPlotCurve *cruve[10];, или я совсем туплю?

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

напишешь в деструкторе

qDeleteAll(grapth);
grapth.clear();

плюсы работают пока не трогают указатели

Просто руки должны быть достаточно прямыми, иначе да, лучше писать на похапэ, и С++ обходить за километр.

Есть подходящие инструменты, такие как умные указатели или контейнеры. Какого хера ими не пользоваться? Чтобы сказать, что ТЫЖПI$ДАTЫЙПРОГРАММИСТ С ПРЯМЫМИ РУКАМИ?

а) Где я возражал против использования контейнеров?

б) Говорить, что в Qt, где свой подсчет ссылок работает через parent уже 20 лет, и не особо совместим с умными указателями,

использование указателя, значит недоделанность/неправильность конструктива

это просто расписаться в непонимании того, как всё устроено.

использование указателя, значит недоделанность/неправильность конструктива

Ну как я понял, он имел ввиду не конкретно Qt, а в принципе использование указателей без нужды.

graph.attach(ui->graphWidget);

graph->attach(ui->graphWidget);

graph->at(i).attach(m_ui->graphWidget);

graph.at(i)->attach(m_ui->graphWidget);

1. Прочитать документацию https://doc.qt.io/qt-5/containers.html

2. Не использовать указатели и new без надобности

3. Почитать про контейнеры STL тоже https://en.cppreference.com/w/cpp/container

объявление «QList< QwtPlotCurve > *graph» хреновый знак….

Как тут можно не иметь в виду Qt?

А разве подсчет ссылок/указателей в Qt (тот что через parent у класса QObject) каким-то боком касается контейнера QList? Так он мог таким же образом написать:

std::vector<int>* cont;

Поэтому я бы сказал, что это касается не только Qt-шных контейнеров.