Сортировка

[Diana528]

почему программа не хочет удалять мне лекарства по имени?

void PharmV::del_name(const wchar_t *f)
{
	int q=0;
	int i = 0;
	pmedicines = vmedicines.begin();
	while (pmedicines != vmedicines.end())
	{
		if (wcscmp(vmedicines[i].get_name(), f) == 0)
		{
			vmedicines.erase(pmedicines);
			q++;
		}

		pmedicines++;
		i++;
	}
	if (q == 0) { cout << "не найдены данные" << endl; }
};

[nikmoon]

Было бы неплохо увидеть хотя бы объявление метода get_name()

[Diana528]

class Medicine//Класс Лекарство
{
  private: 
      static int count; 
      wchar_t name[lname];  
      int price; 
      friend class Pharmacy;
      
  public:  
         Medicine(); 
         ~Medicine();
         
void set_name(const wchar_t*); 
wchar_t* get_name();

void set_price(int); 
int get_price(); 

void show(); 
void read();
};

и определение

void Medicine::set_name(const wchar_t *r) 
{wcscpy(name, r);};
wchar_t *Medicine::get_name() 
{return &name[0];};

[nikmoon]

Не знаю, код нормальный.

[Diana528]

пишет такую ошибку - vector iterator not incrementable

[nikmoon]

Хрень какая-то, или эта ошибка относится к коду, который не показан.

[Son Of Pain]

vmedicines у тебя все еще вектор, так же? )

Итератор для вектора фактически является указателем на элемент. Когда ты удаляешь что-то из средины вектора - следующие элементы сдвигаются на единицу ближе к началу. И итератор показывает уже на следующий элемент. А ты после этого увеличиваешь его еще раз, пропуская таким образом одно из значений.
Это продолжает работать (хоть как-то, с пропуском некоторых элементов) до тех пор, пока ты не удалишь последний элемент. После этого итератор показывает уже на vector.end(). И тогда он становится действительно not incrementable, потому что куда ж еще увеличивать-то. )
При этом стандарт языка вообще говорит о том, что любой итератор, показывающий на место за удаляемым элементом, перестает быть валидным. И такой код может падать вообще на первой итерации.
Чтобы как-то разрулить это, они решили возвращать из erase итератор на следующий за удаляемым элемент (или на vector.end(), если такого нет). Потому самый простой способ починить этот цикл -

	while (pmedicines != vmedicines.end())
	{
		if (wcscmp(pmedicines->get_name(), f) == 0)
		{
			pmedicines = vmedicines.erase(pmedicines);
			q++;
		} else //увеличиваем только если не было удаления
		pmedicines++;
	}

P.S. Вот из-за такой фигни в C++ не рекомендуется писать подобные вещи руками, для этого есть std::remove_if.

[Diana528]

теперь ошибку пишет - Ошибка 8 error C2039: get_name: не является членом "std::vector<_Ty>" d:\мои документы\visual studio 2010\projects\laba.9\laba.7\pharma. cpp 541 1 Laba.9

Ошибка 10 error C2232: ->std::vector<_Ty>::erase: левый операнд имеет тип "class", используйте "." d:\мои документы\visual studio 2010\projects\laba.9\laba.7\pharma. cpp 543 1 Laba.9

Ошибка 7 error C2819: тип "std::vector<_Ty>" не имеет перегруженный "operator ->" d:\мои документы\visual studio 2010\projects\laba.9\laba.7\pharma. cpp 541 1 Laba.9

[Son Of Pain]

Покажи весь код.

[Diana528]

спасибо всем огромное) я разобралась..теперь все работает