Thread: Comparing a struct array...

Did you overload the == operator?

operator==
for market?

If this method doesn't work for comparing markets.

Why not just add an integer or something, whenever a book has a market.

Or better yet, check if bArray[i].market == NULL... to see if it has been set.

The market could have a "invalid" value, such as 0, for books that have no market.
(Since I assume market is just an integer of sorts, or something?)
If it's a class, then you can overload operator == and keep track of which books have a market or not (ie a member variable, m_HasMarket or something).

>>some of the books do not have a market associated with it, or contain invalid text

So I assume the 'market' field contains text type data (char array or string).

If 'market' is a string type class (CString, String etc) your code will work (as these classes define an equals operator) otherwise try 'strcmp' or similar (to compare char arrays).

I have added a flag to find books without a market. What you do with them is up to you.....

Code:

for (c = 1; c < MARKETS; c++)
{
	bFound=false;				
	for (i = 1; i < BOOK_IN_LIST; i++)
	{
		if (strcmp (bArray[i].market, mArray[c].market) == 0)
		{
		        input_to_file << bArray[i].item_name << endl;
		        bFound=true;
		}					
	}
	//did we find a market?
	if(bFound==false)//no market
	{
		//handle no market found
	}
}

Is there a reason you are starting at one not zero? (C++ arrays start at 0 to num elements -1)