How to debug my word search code in C++? - c++

Question

I need some help with my C++ word search code. I got it to work but not as what I intended.
When I run the code it scans my text file I inputted and only output one of the words in the text file that match the array in my code. When I add other words to the text file that is in the array it gives me a error.
And can someone please help me change it so it more in c++ code?
This is what I want to look like:
And this is what I get:
#include <fstream>
#include <iostream>
#include <cmath>
#include <iomanip>
#define SIZE 30
using namespace std;
const char *Phish[SIZE] ={"Amazon","official","bank","security",
"urgent","Alert","important","inform ation", "ebay", "password", "credit", "verify",
"confirm", "account","bill", "immediately", "address", "telephone","SSN", "charity",
"check", "secure", "personal", "confidential",
"ATM", "warning","fraud","Citibank","IRS", "paypal"};
int point[SIZE] = {2,2,1,1,1,1,2,2,3,3,3,1,1,1,1,1,2,2,3,2,1,1,1,1,2,2,2,2,2,1};
int totalPoints[SIZE];
void outputResults();
int main(void)
{
FILE *cPtr;
char filename[100];
char message[5000];
char *temp[100];
int i;
int counter=0;
int words=0;
char *tokenPtr;
cout << "Enter the name of the file to be read: \n";
cin >> filename;
if ( (cPtr = fopen(filename,"rb")) == NULL)
{
cout <<"File cannot be opened.\n";
}
else
{
fgets(message, 5000, cPtr);
tokenPtr = strtok(message, " ");
temp[0] = tokenPtr;
while (tokenPtr!=NULL)
{
for(i=0; i< SIZE; i++)
{
if(strncmp(temp[0], Phish[i], strlen(Phish[i]))==0)
{
totalPoints[i]++;
break;
}
tokenPtr =strtok(NULL, " ");
temp[0] = tokenPtr;
words++;
}
outputResults();
cout << "\n";
return 0;
}
}
}
void outputResults()
{
int i;
int count =0;
int a;
cout<<left<<setw(5) << "WORD "
<< setw(7)<<"# OF OCCURRENCE "
<< setw(15)<<"POINT TOTAL";
for(i=0; i<SIZE; i++)
{
if(totalPoints[i] !=0)
{
cout<<"\n"<<left << setw(10)<< Phish[i]
<< setw(11)<< totalPoints[i]
<< setw(13)<< point[i]*totalPoints[i];
count += point[i] * totalPoints[i];
}
}
cout<< "\nPoint total for entire file: \n"<< count;
}

Answer 1

First, your for loop is not written correctly. What you want to do is take a word from the file, and search your phishing array for that word. Right now, you are going through your phishing array without starting at the beginning of this array when you get a word.
Instead you should be taking a word, and then loop through the phishing array to see if the word exists. If it's not found, then get the next word, check if it's in the array starting from the beginning, etc.
fgets(message, 5000, cPtr);
tokenPtr = strtok(message, " ");
while (tokenPtr != NULL)
{
// start the search in the array
for (i = 0; i < SIZE; i++)
{
if (strncmp(tokenPtr, Phish[i], strlen(Phish[i])) == 0)
{
totalPoints[i]++;
break;
}
}
// get the next word in the file
tokenPtr = strtok(NULL, " ");
words++;
}
outputResults();
You also don't need temp[0] at all in the program. Just use tokenPtr.