How to search sentence in cpp

[siabapet]

how to make program like this:


in the cpp file there is a sentence:
Dave buy 2 mangoes
Felix buys 1 basketball
Sara sells shoes for 200 dollars
~
when I run it correctly: Enter your sentence "Dave buy 2 mangoes"
in the output: The sentence buy 2 mangoes Founded

but when I input the wrong sentence like this: Enter your sentence "my fruit is 2"
in the output: The sentence my fruit is 2 Not Found

#include <iostream>
using namespace std;

char A[][2000] = {"Dave buy 2 mangoes", "Felix buys 1 basketball", "Sara sells shoes for 200 dollars"};
string x;
int i;
main()
{
cout << "Enter your sentence : "; cin >> x;
for (i=0; i<1; i++)
{
if (x == A[i])
{ cout << "The sentence" << x << "Founded"; }
else
{ cout << "The sentence" << x << "Not Found"; }
}
    return 0;
}

The output doesn't work whatsoever

[seeplus]

Post your current code so that we have have a look so that we can advise.

[siabapet]

edited~

[seeplus]

A problem is that cin >> x only reads the chars from the keyboard until the first space or new line - which is not what you want here as you want the whole line. There's also a problem with the if statement in that when iterating through the array A, you only know it's not found when the whole of the array has been searched.

Consider for C++17:

#include <iostream>
#include <string>
#include <iterator>
using namespace std;

const char* A[] {"Dave buy 2 mangoes", "Felix buys 1 basketball", "Sara sells shoes for 200 dollars"};

int main()
{
	string x;

	cout << "Enter your sentence : ";
	getline(cin, x);

	bool got {};

	for (int i = 0; i < size(A); ++i)
		if (x == A[i]) {
			cout << "The sentence '" << x << "' Founded";
			got = true;
			break;
		}

	if (!got)
		cout << "The sentence '" << x << "' Not Found";
}

where size() gets the number of elements in the array. If this causes a compiler error, make sure you are compiling as C++17.

[Handy Andy]

Hello siabapet,

You should have compiled you code before posting it. Then you could aks about any errors that you do not understand.

If your objective is to write your code for the compiler this shloud shave a few milliseconds off the compile time.

#include <iostream>
using namespace std; char A[][2000]{ "Dave buy 2 mangoes", "Felix buys 1 basketball", "Sara sells shoes for 200 dollars" }; string x; int i; main() { cout << "Enter your sentence : "; cin >> x; for (i = 0; i < 1; i++) { if (x == A[i]) { cout << "The sentence" << x << "Founded"; } else { cout << "The sentence" << x << "Not Found"; } }return 0; }

Oh BTW there are several errors in line 2.

If you would do something more like this:

#include <iostream>

using namespace std;

main()
{
    char A[][2000]{ "Dave buy 2 mangoes", "Felix buys 1 basketball", "Sara sells shoes for 200 dollars" };
    string x;  // <--- Should have a better name.
    //int i;  // <--- Should be defined in the for loop.

    cout << "Enter your sentence : ";
    std::getline(std::cin, x);

    for (int i = 0; i < 1; i++)
    {
        if (x == A[i])
        {
            cout << "\n The sentence " << x << " Found";  // <--- Changed "founded".
        }
        else
        {
            cout << "\n The sentence \"" << x << "\" Not Found";
        }
    }

	// A fair C++ replacement for "system("pause")". Or a way to pause the program.
	// The next line may not be needed. If you have to press enter to see the prompt it is not needed.
	//std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');  // <--- Requires header file <limits>.
	std::cout << "\n\n Press Enter to continue: ";
	std::cin.get();

	return 0;  // <--- Not required, but makes a good break point.

}

It makes the problems with the code easier to spot.

First you should avoid using global variables unless they start with "const" or "constexpr". Later on when your programs get bigger and you are using functions. Any line of code that follows the global variable definition can change the value. This could take hours or even days to track down and fix. The second part of this is that global variables are not needed.

Next I noticed that you wrote: string x; // <--- Should have a better name. . By its-self this may not be a problem, but in the "cin" and "cout" statements it is a problem because they do not know how to handle a "std::string" with out the "<string>" header file, which is missing.

In modern C++ it is written as: int main(). The "int" is required.

Just like "x" "A" should have a better name. Also the capital "A" gives the impression that it is defined as a constant when it is not. The 2D "char" array is not needed. A 1D array of "std::string"s will work better. It also allows the use of the functions defined in the string class.

I am surprised that line 16 works since you are comparing a "std::string" to a "char" array, but it does seem to work. Comparing 2 "std::strings" would be better.

Your for loop works, but produces this output:

Enter your sentence : Felix buys 1 basketball

 The sentence "Felix buys 1 basketball" Not Found
 The sentence Felix buys 1 basketball Found
 The sentence "Felix buys 1 basketball" Not Found

 Press Enter to continue:

You also have a problem with the for loop as it is set up to run 1 time. That is 1 of the reasons that you are not getting the correct output.

seeplus's idea of the for loop is something to consider.

Now that you have a better idea of what is wrong it should help to fix what is needed.

Andy

[siabapet]

sorry for late answer.
thank you everyone I appreciate it.