C / C ++ освобождает указатель, вызывая ссылку или напрямую

Я думаю, что совет использовать конструкторы/деструкторы вместе с вектором - это тот, который вы должны предпринять, чтобы решить эту проблему. Тем не менее, я думаю, что в этом контексте вы должны смотреть на две важные вещи, потому что в какой-то момент вам может потребоваться ручное распределение.

RAII-Идиома (Инициализация ресурсов)

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

Умные указатели

В этом случае это также может быть вариантом для вас, если вы хотите внести минимально возможные изменения. В c++ вы всегда должны использовать интеллектуальный указатель, если вы можете это сделать (что почти всегда). В новом стандарте есть два типа интеллектуальных указателей: уникальный указатель и общий указатель. Оба заслуживают долгого объяснения, но я отложу это (уникальный указатель, общий указатель). Было бы разумно читать те, кто даже за пределами определений, которые я связывал, потому что эти два очень мощные. Уникальный указатель позволяет иметь указатель, который не может быть скопирован, и который будет обрабатывать освобождение объекта, на который он указывает, когда он выходит из области видимости. Общий указатель хранит счетчик ссылок и освобождает объект, на который он указывает, как только счет достигает 0. В сочетании с RAII эти указатели обычно позволят вам писать код без явного распределения вручную, что обычно требуется в c++. Если вы не можете использовать интеллектуальные указатели из c++ 11, вам следует подумать о повышении интеллектуальных указателей.

Blockquote Я всегда кодировал в C, и я действительно начинаю делать C++

Вам нужно написать C++ как C++, где, хотя вы также в какой-то мере несете ответственность за управление памятью (т. C++ Сбор мусора вообще отсутствует), вы обычно не просто удаляете свою память, а используете предоставленную библиотеку заботиться о памяти для вас.

Какими бы ни были cups, похоже, что ваш класс State содержит два вектора.

class State
{
     // note these members are private

   std::vector< int > maxLiters_;
   std::vector< int > nowLiters_;

 public:
   explicit State( size_t cups )
         : maxLiters_( cups )
         ,  nowLiters_( cups )
   {
   }

   std::vector<int> const& maxLiters() const
  {
    return maxLiters_;
   }

   std::vector< int > const& nowLiters() const
  {
      return nowLiters ;
  }
};

int main()
{
    State state( 6 );

   // whatever
}

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

C++ - это другой язык, и вам нужно научиться его использовать и не думать о том, как вы его реализуете на C (или Java, С#, Python, Perl или любой другой язык).

Кстати, с вашей реализацией (даже если C если вы удаляете ссылки и т.д.) У вас возникла проблема, если первый malloc преуспеет, а второй - не удался. Вы знаете, что на самом деле не удалось? Знаете ли вы, free первый или нет?

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