Насколько правильно такое считывание бинарного файла

[Alex_C]

Т.к. перехожу с дельфи, хочу правильно писать программы на Qt.
Хочу чтоб сказали, на сколько правильный код. Нужно считать бинарный фаил, описывающий массив "точек", каждая точка представляет из себя такую структуру:

Код:

struct MapPoint
{
    quint16 code;
    qint16 lat;
    qint16 lon;
};

считываю так

Код:

    QFile f(fileName);
    if(!f.open(QIODevice::ReadOnly))
    {
        QMessageBox::critical(0, "Error!", "Can not open " + fileName);
        return false;
    }

    quint64 poinCount = f.size() / (sizeof(quint16) + sizeof(qint16) * 2);

    mapPoints = new QVector<MapPoint>;
    mapPoints->resize(poinCount);

    QDataStream in(&f);

    int i = 0;
    MapPoint m;
    while (!in.atEnd()) {
        in >> m.code >> m.lat >> m.lon;
        mapPoints->append(m);
        i++;
    }

[Igors]

Вы не должны делать никаких предположений о длине данных на диске - она может очень отличаться от sizeof. Ну и f.size() вообще не годится если в файле что-то еще

А главное - все проще. Определите операторы << и >> для Вашей структуры (элемента вектора) и спокойно используйте >> для вектора

Код:

QDataStream in(&f);
in >> mapPoints;

И все дела

[m_ax]

Лучше так:

Код

C++ (Qt)
    QFile file(fileName);
    if(!file.open(QIODevice::ReadOnly)) {
        QMessageBox::critical(0, "Error!", "Can not open " + fileName);
        return false;
    }
 
    QVector<MapPoint> mapPoints;
    QDataStream in(&file);
    in >> mapPoints;
 

Нужно только перегрузить операторы:

Код

C++ (Qt)
QDataStream &operator<<(QDataStream & out, const MapPoint &mp) {
    out << mp.code << mp.lat << mp.lon;
    return out;
}
 
QDataStream &operator>>(QDataStream & in, MapPoint &mp) {
    in >> mp.code >> mp.lat >> mp.lon;  
    return in;
}
 
 

[Blackwanderer]

И не забывайте про QDataStream::setVersion()

[sidsukana]

Можно так еще, старт позицию и длину сам укажешь. Длина обычно sizeof() - опять же если ты уверен в том что там именно эти данные.

Код

C++ (Qt)
QByteArray buffer = file.readAll();
MapPoint point = *reinterpret_cast<MapPoint*>(buffer.mid(startpos, length).data());

[Alex_C]

Всем большое спасибо за ответы!
Вопрос по последнему варианту

Код:

QByteArray buffer = file.readAll();
MapPoint point = *reinterpret_cast<MapPoint*>(buffer.mid(startpos, length).data());

Зачем после mid еще data стоит? mid ведь и так QByteArray возвращает?

[sidsukana]

А у тебя MapPoint в QByteArray разве?) Приведение типа происходит из сырых данных.

[Igors]

Можно так еще, старт позицию и длину сам укажешь. Длина обычно sizeof() - опять же если ты уверен в том что там именно эти данные.

Код

C++ (Qt)
QByteArray buffer = file.readAll();
MapPoint point = *reinterpret_cast<MapPoint*>(buffer.mid(startpos, length).data());

Это "как не надо делать" - блочное чтение в стиле C пригодное только для POD структур, да и то не всегда. Суть никак не меняется от того что использованы классы и reinterpret_cast вместо "просто приведения"

[sidsukana]

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

[m_ax]

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

А можно просто один раз по-человечески сделать и забыть)

[Igors]

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

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

А можно просто один раз по-человечески сделать и забыть)

За спиной мощного инструментария легко быть смелым 

[sidsukana]

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

Ну я сейчас тоже перевожу свои тулзы на стримеры))

[m_ax]

За спиной мощного инструментария легко быть смелым 

Скорее спокойным за то, что в один прекрасный день, в самый ответственный момент не произойдёт грохинга программы)