check my code please

K

kittykat

Hi,
could someone please check my code? its asking the user to enter 3
letters, and check to see if these letters exist in the text file. i know
ive done something wrong. can someone fix my code please??

#include <fstream>
#include <string>
#include<iomanip>
#include <iostream>
#include<istream>
#include <cstdlib>
using namespace std;

int main()
{
char a, b, c;
ifstream myFile("data.txt");

cout << "Enter the pattern you are searching for: ";
cin.getline >> a & b & c;

cout << "Program will now search for: " << a << b << c << endl;

return 0;

}
 
H

Howard

kittykat said:
Hi,
could someone please check my code? its asking the user to enter 3
letters, and check to see if these letters exist in the text file. i know
ive done something wrong. can someone fix my code please??

#include <fstream>
#include <string>
#include<iomanip>
#include <iostream>
#include<istream>
#include <cstdlib>
using namespace std;

int main()
{
char a, b, c;
ifstream myFile("data.txt");

cout << "Enter the pattern you are searching for: ";
cin.getline >> a & b & c;

What in the world does that do? Isn't the format for getline:
getline(cin,str); ?
cout << "Program will now search for: " << a << b << c << endl;

Aren't you supposed to DO the search here???
return 0;

}

Is this some kind of joke? Trying to trick someone into doing your work for
you? Surely you don't expect us to think you don't see there isn't even one
line of code that searches in the text file for a match! At least TRY to do
your own work first. This is not an effort, it's just silly.

-Howard
 
Y

Yoon Soo

kittykat said:
Hi,
could someone please check my code? its asking the user to enter 3
letters, and check to see if these letters exist in the text file. i know
ive done something wrong. can someone fix my code please??

#include <fstream>
#include <string>
#include<iomanip>
#include <iostream>
#include<istream>
#include <cstdlib>
using namespace std;

int main()
{
char a, b, c;
ifstream myFile("data.txt");

cout << "Enter the pattern you are searching for: ";
cin.getline >> a & b & c;

cout << "Program will now search for: " << a << b << c << endl;

return 0;

}

have a look at this for some algorithms in pattern matching:

http://www.i.kyushu-u.ac.jp/~takeda/PM_DEMO/e.html

the naive algorithm should be easy enough to understand

greets

yoon
 
K

kittykat

Hi Howard,
this is my first week of trying anything in C++.

"Is this some kind of joke? Trying to trick someone into doing your work
for you?"

not a joke at all. i just wanted some help. my experience is very limited
and i am trying my best here.

"Isn't the format for getline: getline(cin,str);"

so in my case, would i write
getline(cin, a,b,c);
would that work?
 
E

E. Robert Tisdale

kittykat said:
Could someone please check my code?
It's asking the user to enter 3 letters
and check to see if these letters exist in the text file.
I know [that] I've done something wrong.
Can someone fix my code please?
expand kat.cc
#include <fstream>
#include <string>
#include <iomanip>
#include <iostream>
#include <istream>
#include <cstdlib>
using namespace std;

int main(int argc, char* argv[]) {
char a, b, c;
ifstream myFile("data.txt");

std::cout << "Enter the pattern you are searching for: ";
std::cin >> a >> b >> c;

std::cout << "Program will now search for: "
<< a << b << c << std::endl;

return EXIT_SUCCESS;
}
g++ -Wall -ansi -pedantic -o kat kat.cc
./kat
Enter the pattern you are searching for: a b c
Program will now search for: abc

Now write some code to search "data.txt" and post it here.
 
K

Karl Heinz Buchegger

kittykat said:
Hi Howard,
this is my first week of trying anything in C++.

If this is true (and I don't doubt it)
then the assignment is much to hard for you (as for
anybody else with just one week of experience).
"Is this some kind of joke? Trying to trick someone into doing your work
for you?"

not a joke at all. i just wanted some help. my experience is very limited
and i am trying my best here.

"Isn't the format for getline: getline(cin,str);"

so in my case, would i write
getline(cin, a,b,c);
would that work?

No.
Which books are you learning from?

Read 3 characters: When we are allowed to ignore error checking
for the moment, it is written:

char a, b, c;

cin >> a >> b >> c;


Your version cannot work. Mostly because as you say for yourself:
"Isn't the format for getline: getline(cin,str);"

getline takes 2 arguments. You try to feed it
getline(cin, a,b,c);

4 arguments. 2 - 4. That won't work. Also: getline expects
a *string* variable. You don't have one. You have 3 char
variables.
char - stores a single character
string - stores a sequence of characters.
 
H

Howard

kittykat said:
Hi Howard,
this is my first week of trying anything in C++.

"Is this some kind of joke? Trying to trick someone into doing your work
for you?"

not a joke at all. i just wanted some help. my experience is very limited
and i am trying my best here.

Ok, sorry if I was harsh. But we see a lot of students here simply asking
us to do their work for them. And your post only said "something is wrong",
when it looks pretty obvious to me that the main problem is that there's no
code there at all that does what you're saying the program should do (which
is to match up the pattern with the text in the file). See what I mean? If
the problem you were having was a compiler error on the call to getline,
then you need to tell us, so that we can concentrate on fixing that.

-Howard
 
K

kittykat

thanx so much. I am currently working on this, and will post it as soon as
i have finished. I have taken some advice from my previous "Pattern
Matching" post, and have decided to break the problem down into tiny bits.
once i have something completed, i will post it and see what you think.

Thanx so much once again for everyones help. i really appreciate it!
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

Forum statistics

Threads
474,181
Messages
2,570,971
Members
47,537
Latest member
BellCorone

Latest Threads

Top