search in vector .. anyidea.??

hi everyone ... first of all, i let you guyz know what i am doing here... and will list the problem tat i got....

well i want to performs a search for the seqence of words finding files containing all of the search words...

in my implememntation (.cpp)

Code:

#include "inverted_index.h"
#include "osdir.h"

// InvertedIndex::tally
//   Pre: a word and file pair as strings
//   Post: adds the file to the index under word
void InvertedIndex::tally(string word, string file) 
{
     inputType input(word, file);
     list.push_back( input );
}

// InvertedIndex::search 
// Pre: a vector of strings as search terms
// Post: a vector of strings being the files that all of the
//      input words occur in in the index.
StringSet InvertedIndex::search(vector <string> words) 
{
     for(int i = 0; i < list.size(); i++)
     {
           if(list.word[i] == words)
              return list.file[i];
     }        
}

and the main is

Code:

#include "inverted_index.h"

int main() 
{
  // test the InvertedIndex 
 InvertedIndex idx;
 idx.tally("cat", "cats.txt");
 idx.tally("cat", "animals.txt");
 idx.tally("cat", "everything.txt");
 idx.tally("dog", "dogs.txt");

   // searching idx for cat should give us three files
  vector <string> query;
  query.push_back("cat");
  StringSet si = idx.search(query);
  if (si.contains("cats.txt") && si.contains("animals.txt") && si.contains("everything.txt") )
    cout << "ok - search('cat')\n";
  else 
    cout << "fail - search('cat')\n";

  if (si.contains("dogs.txt"))
    cout << "fail - search('cat') -- dogs.txt found\n";
  else 
    cout << "ok - search('cat') no dogs\n";


  system("PAUSE");
  return 0;
}

can someone check my code for the red font.... please//

Code:

system("pause")

Please don't use this, use

Code:

cin.get();

Only use a system command if you really have to, but there is always another way around using one.

Originally Posted by swgh

Code:

system("pause")

Please don't use this, use

Code:

cin.get();

Only use a system command if you really have to, but there is always another way around using one.

why is that ? please explain... thank you...

system pause is known to be a bad choice for feezing the console window. Read the FAQ on this site for a detailed explaination. I will not go into an argument about why it should / should not be used but in general program practice, system pause is frowned upon. Cin.get() is a cleaner and more efficient way to pause the console window.

As I stated above, read the FAQ for a good reason to not use system pause. I belive it has somthing to do with program security issues.

can someone give me an idea of how to design this function please?

What's an inputType? What is the type of the list member variable? What is a StringSet? An Inverted Index?

Really, post the smallest and simplest compilable program that demonstrates the error/problem.

you have PM laserlight.....

Just post here, it makes me, I mean me, feel left out.

Code:

           if(list.word[i] == words)
              return list.file[i];

Unless list.words[i] has an operator vector<string>() and an operator StringSet, that section of code will not work. In fact, that whole thing is not logical. Edited.