Thread: Newbie needs help removing elements from an array

Use a std::vector, which has an erase() member function.

You would then use a std::vector<tv_in>

Maybe you can create a vector of tv_ins and have a loop that passes

Erasing element by element can be rather slow.

Considering that the serial number should probably be a string rather than an int, I would suggest:

Code:

struct tv_in{
	std::string serialNum;
	std::string tvBrand;
	int screenSize;
};

class serialNumIsEqual
{
public:
	serialNumIsEqual(const std::string& serialNum) : serialNum_(serialNum) {}
	bool operator()(const tv_in& x) const
	{
		return x.serialNum == serialNum_;
	}
private:
	std::string serialNum_;
};

Then the actual erasing would be:

Code:

new_tv.erase(
	std::remove_if(new_tv.begin(), new_tv.end(), serialNumIsEqual("5432167890")),
	new_tv.end());

What is new_tv? In my example it is a std::vector<tv_in>

Then, the erase member function either takes a single iterator, or a pair of iterators to denote a range.

This is the error I receive when I use the tv_in as my pointer.


error C2143: syntax error : missing ';' before '.'
error C2143: syntax error : missing ';' before '.'

The new_tv is the pointer I was using for that array when it was an array.

My data structure looks like this:

Code:

:

struct tv_in{
 int serialNum;
 string tvBrand;
 int screenSize;
}new_tv [IN_TV];

This is the code declaring that struct a vector

Code:

std::vector<tv_in>;

This is the function I have defined to erase the elements using the tv_in

Code:

void deleteTV()
{
	string deleteSerialNum;
	int d = 0;

	cout << "Please select the serial number you would like to search:" << endl;
	cin >> deleteSerialNum;
	
	for (int d= 0; d < IN_TV; d++)
	{
		if (deleteSerialNum == new_TV[d].inSerialNum)
		{
			     tv_in.erase(new_tv[d].inSerialNum,new_tv[d].inTvBrand,new_tv[d].inScreenSize);
		}
	}
}

My data structure looks like this:

It is more than a definition of a struct, but also creates an array named new_tv with IN_TV number of tv_in objects.

This is the code declaring that struct a vector

One cannot declare a struct to be a vector. One can create a vector of structs. In this case you have a declaration that does not really declare anything, but if you did provide a name, then it would be a vector of vehicles_in.

This is the function I have defined to erase the elements using the tv_in

1. new_tv is an array. Arrays do not have an erase member function.
2. new_tv[d] is the dth element of that array, and is a tv_in. A tv_in is not an iterator.
3. the erase member function of std::vector takes either one or two arguments, not three.

You may find the reference material on cppreference.com and SGI's Standard Template Library Programmer's Guide useful.