Cannot find logical operator

[Dasttann777]

Here is my code:

#include <iostream>
#include <string>
using namespace std;
string repeat;
string repeatnum;
string prompt = "|-[]->";
int main()
{
    string entry;
    bool Running = true;
    while(Running == true)
    {
    cout << "\n";
    cout << prompt;
    cin >> entry;
    if(entry == "Exit") return 0;
    if(entry == "Help") cout << "HELP:\nThsi is a simple program, try an input";
    if(entry == "ChangePrompt")
    {
        cout << "What do you want to change the prompt to?: ";
        cin >> prompt;
    }
    if(entry == "Repeat" || "repeat")
    {
        cout << "What string do you want to repeat?: ";
        cin >> repeat;
        cout << "How many times do you want to repeat" << repeat << "(1-9)?: ";
        cin >> repeatnum;
        if(repeatnum > 0){}
    }
    }
    char f;
    cin >> f;
    return 0;
}

Here are the error messages:

||=== test, Debug ===|
C:\Users\Packard Bell\Desktop\test\main.cpp||In function 'int main()':|
C:\Users\Packard Bell\Desktop\test\main.cpp|29|error: no match for 'operator>' in 'repeatnum > 0'|
||=== Build finished: 1 errors, 0 warnings ===|

[MrHutch]

You're trying to use the > operator on a string.

[TheIdeasMan]

Hi,

This bit isn't right either:

if(entry == "Repeat" || "repeat")

should be this:

if(entry == "Repeat" || entry == "repeat")

Instead of a series of if's you need, an if, some else if's, then an else.

This bit needs to be a for loop:

if(repeatnum > 0){}

HTH

[ThangDo]

You know

//you can declare repeatnum as an int instead
int repeatnum;

cin >> repeatnum;

if(repeatnum > 0)
{
     //Whatever
}

I dont see the need to declare repeatnum as string.

[Dasttann777]

Thanks guys,
I feel stupid now. :0