Структура с указателем внутри

Указатель на структуру разобрали. Теперь задачка посложнее =)

Код (C++):

struct Payload_t                    // Структура с данными
{
    uint8_t    *Data;                // Указатель на данные
    uint8_t    Length;               // Размер данных в байтах
    uint8_t   Channel;                // Данные с этого канала
};

Я хочу по указателю в поле Data записать Length данных.
Пишем:

Код (C++):

    struct Payload_t Payload;
    Payload.Length = SEND_LEN;
    Payload.Data = malloc(sizeof(Payload.Length));

Записываю данные с помощью функции:

Код (C++):

void SPI_ReadArray(uint8_t num, uint8_t *data)

вот так:

Код (C++):

SPI_ReadArray(Payload.Length, Payload.Data);

а читаю так

Код (C++):

uint8_t Data3       = *(Payload.Data);

Я же всё правильно сделал?

 

Делай лучше так

Указатель на структуру разобрали. Теперь задачка посложнее =)

Код (C++):

struct Payload_t                    // Структура с данными
{
   uint8_t   *Data;               // Указатель на данные
   uint8_t   Length;             // Размер данных в байтах
   uint8_t  Channel;               // Данные с этого канала

  Payload_t(){memset(this,0,sizeof(Payload_t));};
  ~Payload_t(){ delete [] Data;};
};

Я хочу по указателю в поле Data записать Length данных.
Пишем:

Код (C++):

   struct Payload_t Payload;
    Payload.Length = SEND_LEN;
    Payload.Data =  new uint8_t[SEND_LEN];

   //  ошибка, у тебя здесь выделится 1 байт ->    malloc(sizeof(Payload.Length));
  //   а не SEND_LEN, убери sizeof из скобок
 

Записываю данные с помощью функции:

Код (C++):

void SPI_ReadArray(uint8_t num, uint8_t *data)

вот так:

Код (C++):

SPI_ReadArray(Payload.Length, Payload.Data);

а читай так

Код (C++):

uint8_t Data3      = Payload.Data[N];

 

Дед, тогда ещё конструктор нужен, который туда NULL загонит. А то если структура не статическая, то NULL туда никто за нас не засунет. А если там точно NULL, то if в деструкторе не нужен. delete на NULL вполне адекватно работает.

 

Не буду спорить с Вами, в таких тонкостях не разбираюсь, век живи, век учись.
Я думал, конструктор по умолчанию обнулит все поля свои сам, но, видимо, я ошибаюсь с классом хранения этого типа.
Поправил.

 

Тогда уж и конструктор копий и оператор присваивания непременно

Код (C++):

struct ByteArray_t {
    ByteArray_t() {
        this->size = 0;
        this->data = 0;
    }
    ByteArray_t(const ByteArray_t& org) {
        this->size = 0;
        this->data = 0;
        operator = (org);
    }
    ByteArray_t& operator = (const ByteArray_t &rhs) {
        if (this != &rhs) {
            clear();
            this->size = rhs.size;
            if (rhs.size) {
                this->data = new uint8_t[rhs.size];
                memcpy(this->data, rhs.data, rhs.size);
            }
        }
        return *this;
    }
    ~ByteArray_t() {
        clear();
    }
    void clear() {
        delete[] this->data;
        this->data = 0;
        this->size = 0;
    }
    size_t size;
    uint8_t* data;
};
 

 

Это ты щас типа блеснул?

 

Это же си плас плас?

Да, точняк. Надо прекращать кодить в 2 часа ночи) А больше некогда(

Я там умолчал - в конкретно данном случае мне нужен всего один (он же первый) байт. А компилятор не скажет

я же массив явно не объявлял?

 

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

 

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

 

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

 

Я указал на потенциально опасный код, конечно же лучше потрольть, даже веселее становится.

 

я прошу прощения.

 

Оправданием мне может служить только то, что я не то что вапще не программист, но еще и спахмелью. Пардоньте-с.

 

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

 

Кто или что мешает сделать это в структуре?

 

Согласен, меня с толку сбило что C++ обрабатывает обратную совместимость с программами на C, и там всё было паблик.

 

А через указатель нельзя?
А чем работа через имя структуры сакрально отличается от работы через структурную переменную? Я в первом случае не смогу вот так инициализировать:

Код (C++):

typedef struct Payload_t {...};

.......
struct Payload_t Payload = (Payload_t)
{
    .*Data = malloc ()...,            
    .Length = ...,            
    .Channel = ...,            
};
 

?

 

Шта?

 

Объявляю приватным и где то в коде вызываю оператор присваивания

Код (C++):

struct ByteArray_t {
  ByteArray_t() {
    this->size = 0;
    this->data = 0;
  }
  ~ByteArray_t() {
    if (this->data) {
      delete[] this->data;
    }
    this->data = 0;
    this->size = 0;
  }
  size_t size;
  uint8_t* data;
private:
  ByteArray_t(const ByteArray_t& org) {
  }
  ByteArray_t& operator = (const ByteArray_t &rhs) {
    return *this;
  }
};
void setup() {
  struct ByteArray_t a, b;
  a = b;
}
void loop() {
}

компилятор это не пропустит

Код (C++):

/tmp/arduino_modified_sketch_193172/sketch_jan22a.ino: In function 'void setup()':
sketch_jan22a:18:16: error: 'ByteArray_t& ByteArray_t::operator=(const ByteArray_t&)' is private
   ByteArray_t& operator = (const ByteArray_t &rhs) {
                ^
sketch_jan22a:24:5: error: within this context
   a = b;
     ^
exit status 1
'ByteArray_t& ByteArray_t::operator=(const ByteArray_t&)' is private
 

 

достаточно, памойму, сделать их protected.