QMap и масив

[Old]

Я только читаю данные с бинарников ничего туда не пишу.
Или я Вас неправильно понял?

Пожалуйста, а как это соотносится с хранением данных?
Вы пытаетесь засунуть их в обычный массив, этот массив в map, а где будете хранить размерности массивов?
А я предлагаю это объединить в отдельную сущность с удобными методами их использования.

[shadow]

Я только читаю данные с бинарников ничего туда не пишу.
Или я Вас неправильно понял?

Пожалуйста, а как это соотносится с хранением данных?
Вы пытаетесь засунуть их в обычный массив, этот массив в map, а где будете хранить размерности массивов?
А я предлагаю это объединить в отдельную сущность с удобными методами их использования.

Ну размеры массивов я знаю перед добавлением в словарь.
А вообще получилось спасибо!

Код:

class FileData
{
public:    
    FileData();
    void LoadKvdopData(float **arr,int Row,int Col);   
    void RetKvdopData(float **arr,int Row,int Col);  
    ~FileData();
private:
    bool isKvdopExist;
    float **Kvdop;
};

filedata.cpp

Код:

#include <QDebug>
#include "filedata.h"
FileData::FileData()
{
isKvdopExist=false;
}
void FileData::LoadKvdopData(float **arr,int Row,int Col)
{
    Kvdop = new float*[Row];
    for (int i = 0; i < Row; i++)
        Kvdop[i]  = new float [Col];

    for (int i = 0; i < Row; i++)
        for (int j = 0; j < Col; j++)
            Kvdop[i][j]=arr[i][j];
    //memcpy(Kvdop,arr,Row*Col);
    isKvdopExist=true;
}
void FileData::RetKvdopData(float **arr,int Row,int Col)
{
    if (isKvdopExist){
    for (int i = 0; i < Row; i++)
        for (int j = 0; j < Col; j++)
            arr[i][j]=Kvdop[i][j];
    //memcpy(arr,Kvdop,Row*Col);
    }else
     qDebug()<<"Массив не создан!";
}

Хочу вместо цикла использовать memcpy но программа вываливается... в чем может быть проблема?
Или по производительности это будет то-же самое?

[Igors]

Хочу вместо цикла использовать memcpy но программа вываливается... в чем может быть проблема?

У Вас каждая строка распределена отдельно, поэтому memcpy не прокатит.

Вы сделали "косметический ремонт", старые си-шные массивы-то остались, Вы будете продолжать разбивать о них ноги (массив создан! Ах нет, не создан). Вот Вам заготовочка в аттаче. Если я где ошибся - подправите. Также избегайте всяких QMap::value() и др методов возвращающих ПО ЗНАЧЕНИЮ (не слушайте если будут рассказывать что, мол, в Qt так можно и.т.п.)

[shadow]

У Вас каждая строка распределена отдельно, поэтому memcpy не прокатит.

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

Вы сделали "косметический ремонт" , старые си-шные массивы-то остались

Но массивы теперь динамические?

Аттач посмотрел...
Ну мне чтобы это понять еще надо подтянуть матчасть....

[__Heaven__]

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

Указатель на что (тип) ?

[kambala]

Также избегайте всяких QMap::value() и др методов возвращающих ПО ЗНАЧЕНИЮ (не слушайте если будут рассказывать что, мол, в Qt так можно и.т.п.)

если в словарь складывать указатели, то можно и value()

[shadow]

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

Указатель на что (тип) ?

На float.

[__Heaven__]

На float.

А если повнимательнее на какой тип указывает Kvdop?

[shadow]

На float.

А если повнимательнее на какой тип указывает Kvdop?

На  указатель? Указатель на указатель.

[__Heaven__]

Да. Давайте для упрощения будем называть Kvdop массивом указателей (на float).
Теперь смотрите на memcpy. Эта функция копирует информацию из массива А в массив Б.
Как мы видим, в массиве Kvdop хранятся указатели. Значит что мы копируем? Верно, указатели вместо желаемых float.

Но это полбеды. Вы в memcpy передаёте количество элементов, которое хотите скопировать, а на самом деле она ожидает количество байт подлежащие копированию.


Функция memcpy, конечно же, мощная, но она потенциально опасная. Поэтому, если у вас нет острой необходимости в повышении скорости, то от неё лучше отказаться.
Очень советую обратить внимание на замечания Old.
Если бы передо мной стояла такая задача, то я бы написал класс матрицы, либо воспользовался готовым извне.

[shadow]

Да. Давайте для упрощения будем называть Kvdop массивом указателей (на float).
Теперь смотрите на memcpy. Эта функция копирует информацию из массива А в массив Б.
Как мы видим, в массиве Kvdop хранятся указатели. Значит что мы копируем? Верно, указатели вместо желаемых float.

Но это полбеды. Вы в memcpy передаёте количество элементов, которое хотите скопировать, а на самом деле она ожидает количество байт подлежащие копированию.


Функция memcpy, конечно же, мощная, но она потенциально опасная. Поэтому, если у вас нет острой необходимости в повышении скорости, то от неё лучше отказаться.
Очень советую обратить внимание на замечания Old.
Если бы передо мной стояла такая задача, то я бы написал класс матрицы, либо воспользовался готовым извне.

Спасибо за совет надо подумать...

[__Heaven__]

Я, кстати, так и не понял, почему вы QMap используете. В чём выгода?

[Igors]

Я, кстати, так и не понял, почему вы QMap используете. В чём выгода?

Нормально если каких-то массивов может не оказаться (напр не загрузились)

[shadow]

Я, кстати, так и не понял, почему вы QMap используете. В чём выгода?

Ну я новичок. Может поэтому...Выгода в простоте для меня. QMap первым попался, может QVector лучше я пока не знаю. А вообще я выше писал, что раньше я писал структуру на диск... это напряжно и я захотел от этого избавится...обратился за помощью к джедаям..

[shadow]

Я, кстати, так и не понял, почему вы QMap используете. В чём выгода?

Нормально если каких-то массивов может не оказаться (напр не загрузились)

Ненормально конечно.