Есть следующий код и вопрос: "Как делать правильно?" Приведены два класса и main(). Выполняемая работа и проблема описана ниже.
class Getter
{
private:
Buffer buffer_; // тип Buffer - или vector<char>, или std::string
public:
//...
};
void Getter::getPacket(const char* message, size_t size)
{
//...
// Как-то обрабатываем пакет, по мере работы с пакетом сохраняем переработанный пакет в buffer_
// Варианты отправки пакета:
// 1)
sendPacket(buffer_.c_str(), buffer_.size()) // или то же самое sendPacket(buffer_.data(), buffer_.size())
// Минусы: не знаем как обрабатывается пакет в Sender::sendPacket,
// не знаем копируются ли там данные, можем перезаписать данные в buffer_ к моменту, когда в Sender::sendPacket начнется работа с предыдущими
// Но и не нужно чистить память.
// 2) Выделяем память в куче при помощи new []
char* data = new char[newSize];
memcpy(data, buffer_.data(), newSize);
sendPacket(data, newSize);
// Вопрос: кто должен чистить память?
//...
}
class Sender
{
private:
vector<string> packets_;
public:
//...
}
void Sender::sendPacket(const char* message, size_t size)
{
// Возможная работа с данными, которая нас устроила бы
// 1)
packets_.emplace_back(message, size);
// 2)
packets_.emplace_back(message, size);
delete [] message;
// Возможная работа с данными, которая нас не устраивает
// 1)
vector<pair<const char*, size_t> packets;
packets.emplace(message, size);
// 2) Не вызов delete []
}
int main()
{
Sender sender;
Getter getter(&sender);
while (true)
{
const char* data = GetDataFromDevice();
size_t size = GetSizeFromDevice();
// какое-нибудь условие для выхода
getter.getPacket(data, size);
}
}