не знаю как исправить ошибуку QList<T>::at: "index out of range", при попытке вчитать QPoint с текстового файла

Qt, QPoint

QFile file("C:/Users/kamil/Desktop/data.txt");
QList<QPointF> pointsList;



if(file.open(QIODevice::ReadOnly | QFile::Text)){

    QTextStream  in(&file);

    while (!in.atEnd())

    {
        QString line = in.readLine();
        QStringList points =  line.split(';');

     if (points.length() > 0)

     {
         QPointF point(points.at(0).toDouble(), points.at(1).toDouble());
                         pointsList.append(point);  // и добавляем её в список
     }

        }

    file.close();

}

Рекомендуем хостинг TIMEWEB

Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Никогда не используйте метод at. это не безопасно, в место него используйте метод value. скорее всего в вашем случае у вас нет елемента под номером 1 в вашем списке.

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

В коде ошибка в неправельном использовании сепаратора:

        QStringList points =  line.split(';');

если значения точки разделины ",".

        QStringList points =  line.split(',');

Добрый день.

Я правильно понимаю, что по факту в том файле ваши три точки выглядят так?

186,77;
662,308;
645,195;

То есть в столбик, а не в строку?

Если так, то возможно стоит или сразу считать весь файл, то есть так QString line = in.readAll();

Или сначала считать в вектор все строчки, а потом сформировать точки. Но сначала уточните по содержимому файла.

Просто получается, что по факту этот кусок QStringList points = line.split(';'); не формирует правильный список точек и там внутри всего одна строчка, поскольку по факту точки написаны в столбик.

P/S/ Используйте пожалуйста диалог вставки программного кода, это одна из кнопок в тулбаре редактора сообщений. Чтобы разметка правильно добавлялась для программного кода.

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

Поэтому я бы не был так категоричен, чтобы не использовать at вообще.

К тому же вопрос производительности. Создание нового объекта также может быть затратным, гораздо затратнее, чем взять константную ссылку на объект.

у меня идет отрисовка фигур и на пример к каждому треугольнику три точки в линию следующий треугольник опять линия с тремя точкая и тд.
Выходит как вы говорите столбик но каждая линия в нем отдельная фигура.

195,76;437,282;428,124; // первый треугольник
195,76;437,282;428,124;// второй треугольник 

Это два отдельных сепаратора
класс RectToDraw, закоментировала пример точки и ее сепараторов.

readFromString = "144,81:626,232"
const QString &rectInString = ':'  
char pointsSplitter = ','


void RectToDraw::readFromString(const QString &rectInString, char pointsSplitter, char xySplitter)
{
    QStringList point = rectInString.split(pointsSplitter);
    //point ("144,81", "626,232")
    if(point.count()!=2) return;
    QStringList xy1 = point.at(0).split(xySplitter);//xy1("144","81")
    QStringList xy2 = point.at(1).split(xySplitter);

    setTopLeft(QPointF(xy1.at(0).toFloat(), xy1.at(1).toFloat()));
    setBottomRight(QPointF(xy2.at(0).toFloat(), xy2.at(1).toFloat()));

}