Time Limit Exceeded

[andrir13]

I am finding and counting the last word from text file, it seems to be working on codeblocks but when running on mooshak it says time limit exceeded. Also when the text file has numbers like seven zeros it only counts six of them.

  #include<fstream>
#include<iostream>
#include<string>
#include<sstream>
#include<algorithm>
using namespace std;

int main()
{
    ifstream inFile;
    stringstream buffer;
    string fileName;
    string test;
    string word;
    string words;
    int counter = 0;

    cin >> fileName;

    inFile.open(fileName.c_str());
    if(inFile.fail())
    {
        return 0;
    }

    while(!inFile.eof())
    {
        inFile >> word;
    }
    inFile.close();

    word.erase(std::remove(word.begin(), word.end(), '.'), word.end());
    word.erase(std::remove(word.begin(), word.end(), ','), word.end());
    word.erase(std::remove(word.begin(), word.end(), '!'), word.end());
    word.erase(std::remove(word.begin(), word.end(), '?'), word.end());
    word.find_last_of(" ");
                      
    inFile.open(fileName.c_str());
    while(!inFile.eof())
    {
        buffer << inFile.rdbuf();
        test = buffer.str();

        getline(inFile, words);
    }
    inFile.close();

    int unsigned pos = 0;
    while(true)
    {

        pos =  test.find(word, ++pos);
        if (pos != std::string::npos)
        {
            counter++;
        }
        else break;
    }

    cout << word << ' ' << counter;
    return 0;
}

[jlb]

What are the assignment requirements?

[andrir13]

Opening a textfile from userinput, find the last word from the file and count the occurance of that word.

[andrir13]

And also if last word is a number then count the occurance of that number in the file

[jlb]

Please post a small sample of the input file.

But this seems to point me to a possible problem area:

main.cpp||In function ‘int main()’:|
main.cpp|53|warning: comparison is always true due to limited range of data type [-Wtype-limits]|
||=== Build finished: 0 errors, 1 warnings (0 minutes, 1 seconds) ===|

std::string::pos is a size_t, not an unsigned int. A size_t is a implementation defined unsigned type, but not necessarily an unsigned int. On my system, for example, it is a larger type, probably an unsigned long. You should use a size_t for functions that return this type to avoid this type of problem.

[andrir13]

For example this text file: This is what Mississippi is.
The output is: is 5

[andrir13]

Thank you! It works now with words.
But when counting numbers it seems to miss one number. For example the text file with seven zeros: 0000000
The output is 0 6

[andrir13]

And works now with numbers, thank you!