Beg Class challenge for a returning student

[mchoya]

I'm so frustrated. I'm beginning school next week and I have been working
on a simple program for several days now without being able to complete it.
(Though I have brushed up on a lot of C++ while trying.)

I realize this code is long but, I'm not sure of any other way to display
the issue.

The header file and ?implementation file work. (This assignment was
building on another in which I created the header and implementation with a
simple executable, i.e. display the read data.)

This executable is supposed to 1. create an array of Patient objects, then
2. present a user menu to a. read Patient data from a user specified file,
b. display data. c. quit program.

Facts: max patients = 1000.

Requirements: cannot change header or implement file; use an array; handle
user errors

So here it is, any assistance would be extremely appreciated.

HEADER
#ifndef PATIENT_H
#define PATIENT_H

#include <iostream>
#include <string>

using namespace std;

class Patient
{
public:
Patient();
Patient(string fName, string lName, string idString, int bYear);

string getName() const;
int getAge() const;
string getID() const;

void setFirstName (string fname);
void setLastName (string lname);
void setID(string idString);
void setBirthYear(in t bYear);
private:
string firstName;
string lastName;
string ID;
int birthYear;
};
#endif

HEADER DEFINITIONS
#include <iostream>
#include <string>
#include "patient.h"

using namespace std;

Patient::Patien t() {
setFirstName("F irst");
setLastName("La st");
setID("Z9999");
setBirthYear(99 99);
}
Patient::Patien t(string fName, string lName, string idString, int bYear) {
setFirstName(fN ame);
setLastName(lNa me);
setID(idString) ;
setBirthYear(bY ear);
}
string Patient::getNam e() const {
return lastName + ", " + firstName;
}
int Patient::getAge () const {
return birthYear;
}
string Patient::getID( ) const {
return ID;
}
void Patient::setFir stName(string fName) {
firstName = fName;
}
void Patient::setLas tName(string lName) {
lastName = lName;
}
void Patient::setID( string idString) {
int i = idString.size() ;
if (i == 5) {
ID = idString;
}
else {
cerr <<"\nIllegal Patient ID: " <<idString<<" using Z9999"<<endl;
ID = "Z9999";
}
}
void Patient::setBir thYear(int Year) {
if ((Year >1873) && (Year < 2003)) {
birthYear = 2003 - Year;
}
else {
cerr<<"\nIllega l Year: "<<Year<<" using 9999"<<endl;
birthYear = 9999;
}
}

**** Problem Driver ***********
#include <iostream>
#include <string>
#include <fstream>
#include <iomanip>
#include "patient.h"

using namespace std;

char resp;
void GetList();
bool getPatient(ifst ream &fin, Patient &P);
const int MaxPatientSize = 1000;

int main()
{

Patient P[MaxPatientSize];
GetList();

do{
cout <<"\nWould you like to read another Patient File? (y or n):";
cin >> resp;
}while ((resp != 'y') && (resp != 'Y') && (resp != 'n') && (resp != 'N'));
if ((resp == 'y') || (resp == 'Y'))
GetList();
else
cout <<"Good Bye!"<< endl;

return 0;
}
void GetList()
{
cout<<"Enter name of Patient List to be read: ";
cin.getline(
ifstream fin(file_name.c _str());
if (!fin)
{
cerr<<"\nCannot open Patient List. Good Bye\n";
exit(1);
}
do
{
cout<<"Would you like to display your Patient List? (y or n): ";
cin >> resp;
}while ((resp != 'y') && (resp != 'Y') && (resp != 'n') && (resp !=
'N'));
if ((resp == 'y') || (resp == 'Y'))
{
for (P = Patient(),
while(getPatien t(fin, P))
{
cout<<"\n\n\n"< <setw(25) <<setiosflags(i os::left)<<"Pat ient Name"
<<setw(20)
<<setiosflags(i os::left)<<"Pat ient ID" <<setw(15)
<<setiosflags(i os::left)<<"Pat ient Age"<<endl;
cout <<setw(25)<<set iosflags(ios::l eft)<<P.getName () <<setw(20)
<<setiosflags(i os::left)<<P.ge tID() <<setw(15)
<<setiosflags(i os::left)<<P.ge tAge()<<"\n\n";

fin.close();
P = Patient();
}
else
return;
}
bool getPatient(ifst ream &fin, Patient &P)
{
bool isOK = true;
string name = "";

if (getline(fin, name, '\n') && isOK)
fam.setFirstNam e(name);
else
isOK = false;

if (getline(fin, name, '\n') && isOK)
fam.setLastName (name);
else
isOK = false;

if (getline(fin, name, '\n') && isOK)
fam.setID(name) ;
else
isOK = false;

/*f (getline(fin, name, '\n') && isOK)
fam.setBirthYea r(name);
else
isOK = false; */

return isOK;
}

[Mike Wahler]

"mchoya" <mi******@sbcgl obal.net> wrote in message
news:ah******** ****@newssvr16. news.prodigy.co m...

I'm so frustrated. I'm beginning school next week and I have been working
on a simple program for several days now without being able to complete it. (Though I have brushed up on a lot of C++ while trying.)

I realize this code is long but, I'm not sure of any other way to display
the issue.
You haven't told us what issues you're having.

The header file and ?implementation file work. (This assignment was
building on another in which I created the header and implementation with a simple executable, i.e. display the read data.)

This executable is supposed to 1. create an array of Patient objects, then
2. present a user menu to a. read Patient data from a user specified file,
b. display data. c. quit program.

Facts: max patients = 1000.

Requirements: cannot change header or implement file; use an array; handle
user errors

So here it is, any assistance would be extremely appreciated.

Tell us what problems you're having. Be specific.

-Mike

[rossum]

On Sun, 29 Aug 2004 17:22:14 GMT, "mchoya" <mi******@sbcgl obal.net>
wrote:

I'm so frustrated. I'm beginning school next week and I have been working
on a simple program for several days now without being able to complete it.
(Though I have brushed up on a lot of C++ while trying.)

I realize this code is long but, I'm not sure of any other way to display
the issue.

The header file and ?implementation file work. Yes, the .cpp file is called the implementation file.
(This assignment was
building on another in which I created the header and implementation with a
simple executable, i.e. display the read data.)

This executable is supposed to 1. create an array of Patient objects, then
2. present a user menu to a. read Patient data from a user specified file,
b. display data. c. quit program.

Facts: max patients = 1000.

Requirements : cannot change header or implement file; use an array; handle
user errors

So here it is, any assistance would be extremely appreciated. Help us to help you by clearly explaining the problem you are having.
I will give some comments but since I do not know what problem you
want us to look at, I do not know if my comments are relevant.

HEADER
#ifndef PATIENT_H
#define PATIENT_H

#include <iostream> Why are you including iosteam? You do not use it in this file.
#include <string>

using namespace std; AAARRGGGHHHH! NEVER put this in a header file. You are likely to
give yourself big problems with clashing names if you do this with any
sort of sizeable program. By doing this you undo all the good work
that namespaces can do for you by keeping names separate.

class Patient
{
public:
Patient();
Patient(string fName, string lName, string idString, int bYear);

string getName() const;
int getAge() const;
string getID() const;

void setFirstName (string fname);
void setLastName (string lname);
void setID(string idString);
void setBirthYear(in t bYear);
private:
string firstName;
string lastName;
string ID;
int birthYear;
};
#endif

HEADER DEFINITIONS
#include <iostream>
#include <string>
#include "patient.h"

using namespace std;

Patient::Patie nt() {
setFirstName("F irst");
setLastName("La st");
setID("Z9999");
setBirthYear(99 99);
} A point of style, better to use some vertical space to separate
functions, it makes the code easieer to read. Also on a point of
style, it is unusual to use as little as one space for indentation,
three or four is more usual.
Patient::Patie nt(string fName, string lName, string idString, int bYear) {
setFirstName(fN ame); You don't have to use the functions, you can assign directly to the
data if you want. setLastName(lNa me);
setID(idString) ;
setBirthYear(bY ear);
}
string Patient::getNam e() const {
return lastName + ", " + firstName;
} All these one-liners are ideal for inlining.int Patient::getAge () const {
return birthYear;
}
string Patient::getID( ) const {
return ID;
}
void Patient::setFir stName(string fName) {
firstName = fName;
}
void Patient::setLas tName(string lName) {
lastName = lName;
}
void Patient::setID( string idString) {
int i = idString.size() ;
if (i == 5) {
ID = idString;
}
else {
cerr <<"\nIllegal Patient ID: " <<idString<<" using Z9999"<<endl;
ID = "Z9999";
}
}
void Patient::setBir thYear(int Year) {
if ((Year >1873) && (Year < 2003)) {
birthYear = 2003 - Year;
}
else {
cerr<<"\nIllega l Year: "<<Year<<" using 9999"<<endl;
birthYear = 9999;
}
}

**** Problem Driver ***********
#include <iostream>
#include <string>
#include <fstream>
#include <iomanip>
#include "patient.h"

using namespace std;

char resp;
void GetList();
bool getPatient(ifst ream &fin, Patient &P);
const int MaxPatientSize = 1000;

int main()
{

Patient P[MaxPatientSize];
GetList();

do{
cout <<"\nWould you like to read another Patient File? (y or n):";
cin >> resp;
}while ((resp != 'y') && (resp != 'Y') && (resp != 'n') && (resp != 'N'));
if ((resp == 'y') || (resp == 'Y'))
GetList();
else
cout <<"Good Bye!"<< endl;

return 0;
}
void GetList()
{
cout<<"Enter name of Patient List to be read: ";
cin.getline( You have missed off the parameters, closing parenthesis and semicolon.
You compiler should start complaining about this point. Please
either post compilable code or copy the compiler's messages along with
the code.
ifstream fin(file_name.c _str());
if (!fin)
{
cerr<<"\nCannot open Patient List. Good Bye\n";
exit(1);
}
do
{
cout<<"Would you like to display your Patient List? (y or n): ";
cin >> resp;
}while ((resp != 'y') && (resp != 'Y') && (resp != 'n') && (resp !=
'N'));
if ((resp == 'y') || (resp == 'Y'))
{
for (P = Patient(), You have left off much of this for statement. Again your compiler
should be complaining.
while(getPatien t(fin, P))
{
cout<<"\n\n\n"< <setw(25) <<setiosflags(i os::left)<<"Pat ient Name"
<<setw(20)
<<setiosflags(i os::left)<<"Pat ient ID" <<setw(15)
<<setiosflags(i os::left)<<"Pat ient Age"<<endl;
cout <<setw(25)<<set iosflags(ios::l eft)<<P.getName () <<setw(20)
<<setiosflags(i os::left)<<P.ge tID() <<setw(15)
<<setiosflags(i os::left)<<P.ge tAge()<<"\n\n";

fin.close();
P = Patient();
}
else
return;
}
bool getPatient(ifst ream &fin, Patient &P)
{
bool isOK = true;
string name = "";

if (getline(fin, name, '\n') && isOK) getline defaults to '\n' as delimiter so it is not necessary to
include it explicitly. fam.setFirstNam e(name); What is "fam"? Where is it defined? Your compiler should be
complaining about fam also.
else
isOK = false;

if (getline(fin, name, '\n') && isOK)
fam.setLastName (name);
else
isOK = false;

if (getline(fin, name, '\n') && isOK)
fam.setID(name) ;
else
isOK = false;

/*f (getline(fin, name, '\n') && isOK)
fam.setBirthYea r(name);
else
isOK = false; */

return isOK;
}

rossum
--

The ultimate truth is that there is no Ultimate Truth

[mchoya]

Of course your right, sorry.

Well, I'm receiveing "unresolved external links" to the header file getAge,
getID, getName.

But , I guess I'm looking for direction in the following, regarding the
executable file:
can I declare an array while initializing an object; ex: Patient
P[MaxSize], where MaxSize has been declared.(my goal being to limit the
number of Patient files (think hospital beds) to MaxSize
have I chosen appropriate parameters for the function getList in the
executable file

Thanks

"mchoya" <mi******@sbcgl obal.net> wrote in message
news:ah******** ****@newssvr16. news.prodigy.co m...

I'm so frustrated. I'm beginning school next week and I have been working
on a simple program for several days now without being able to complete it. (Though I have brushed up on a lot of C++ while trying.)

I realize this code is long but, I'm not sure of any other way to display
the issue.

The header file and ?implementation file work. (This assignment was
building on another in which I created the header and implementation with a simple executable, i.e. display the read data.)

This executable is supposed to 1. create an array of Patient objects, then
2. present a user menu to a. read Patient data from a user specified file,
b. display data. c. quit program.

Facts: max patients = 1000.

Requirements: cannot change header or implement file; use an array; handle
user errors

So here it is, any assistance would be extremely appreciated.

HEADER
#ifndef PATIENT_H
#define PATIENT_H

#include <iostream>
#include <string>

using namespace std;

class Patient
{
public:
Patient();
Patient(string fName, string lName, string idString, int bYear);

string getName() const;
int getAge() const;
string getID() const;

void setFirstName (string fname);
void setLastName (string lname);
void setID(string idString);
void setBirthYear(in t bYear);
private:
string firstName;
string lastName;
string ID;
int birthYear;
};
#endif

HEADER DEFINITIONS
#include <iostream>
#include <string>
#include "patient.h"

using namespace std;

Patient::Patien t() {
setFirstName("F irst");
setLastName("La st");
setID("Z9999");
setBirthYear(99 99);
}
Patient::Patien t(string fName, string lName, string idString, int bYear) {
setFirstName(fN ame);
setLastName(lNa me);
setID(idString) ;
setBirthYear(bY ear);
}
string Patient::getNam e() const {
return lastName + ", " + firstName;
}
int Patient::getAge () const {
return birthYear;
}
string Patient::getID( ) const {
return ID;
}
void Patient::setFir stName(string fName) {
firstName = fName;
}
void Patient::setLas tName(string lName) {
lastName = lName;
}
void Patient::setID( string idString) {
int i = idString.size() ;
if (i == 5) {
ID = idString;
}
else {
cerr <<"\nIllegal Patient ID: " <<idString<<" using Z9999"<<endl;
ID = "Z9999";
}
}
void Patient::setBir thYear(int Year) {
if ((Year >1873) && (Year < 2003)) {
birthYear = 2003 - Year;
}
else {
cerr<<"\nIllega l Year: "<<Year<<" using 9999"<<endl;
birthYear = 9999;
}
}

**** Problem Driver ***********
#include <iostream>
#include <string>
#include <fstream>
#include <iomanip>
#include "patient.h"

using namespace std;

char resp;
void GetList();
bool getPatient(ifst ream &fin, Patient &P);
const int MaxPatientSize = 1000;

int main()
{

Patient P[MaxPatientSize];
GetList();

do{
cout <<"\nWould you like to read another Patient File? (y or n):";
cin >> resp;
}while ((resp != 'y') && (resp != 'Y') && (resp != 'n') && (resp != 'N')); if ((resp == 'y') || (resp == 'Y'))
GetList();
else
cout <<"Good Bye!"<< endl;

return 0;
}
void GetList()
{
cout<<"Enter name of Patient List to be read: ";
cin.getline(
ifstream fin(file_name.c _str());
if (!fin)
{
cerr<<"\nCannot open Patient List. Good Bye\n";
exit(1);
}
do
{
cout<<"Would you like to display your Patient List? (y or n): ";
cin >> resp;
}while ((resp != 'y') && (resp != 'Y') && (resp != 'n') && (resp !=
'N'));
if ((resp == 'y') || (resp == 'Y'))
{
for (P = Patient(),
while(getPatien t(fin, P))
{
cout<<"\n\n\n"< <setw(25) <<setiosflags(i os::left)<<"Pat ient Name"
<<setw(20)
<<setiosflags(i os::left)<<"Pat ient ID" <<setw(15)
<<setiosflags(i os::left)<<"Pat ient Age"<<endl;
cout <<setw(25)<<set iosflags(ios::l eft)<<P.getName () <<setw(20)
<<setiosflags(i os::left)<<P.ge tID() <<setw(15)
<<setiosflags(i os::left)<<P.ge tAge()<<"\n\n";

fin.close();
P = Patient();
}
else
return;
}
bool getPatient(ifst ream &fin, Patient &P)
{
bool isOK = true;
string name = "";

if (getline(fin, name, '\n') && isOK)
fam.setFirstNam e(name);
else
isOK = false;

if (getline(fin, name, '\n') && isOK)
fam.setLastName (name);
else
isOK = false;

if (getline(fin, name, '\n') && isOK)
fam.setID(name) ;
else
isOK = false;

/*f (getline(fin, name, '\n') && isOK)
fam.setBirthYea r(name);
else
isOK = false; */

return isOK;
}

[Thomas Matthews]

Thanks.

top-post.
don't
Please

mchoya wrote:
[Re-arranged]

"mchoya" <mi******@sbcgl obal.net> wrote in message
news:ah******** ****@newssvr16. news.prodigy.co m...
[snip]

This executable is supposed to 1. create an array of Patient objects, then
2. present a user menu to a. read Patient data from a user specified file,
b. display data. c. quit program.

Facts: max patients = 1000.

Requirement s: cannot change header or implement file; use an array; handle
user errors

So here it is, any assistance would be extremely appreciated.

HEADER
#ifndef PATIENT_H
#define PATIENT_H

#include <iostream>
#include <string>

using namespace std;

class Patient
{
public:
Patient();
Patient(string fName, string lName, string idString, int bYear);

string getName() const;
int getAge() const;
string getID() const; int bYear() const;

void setFirstName (string fname); Use const reference to avoid the overhead of passing a whole
string structure:
void setFirstName(co nst string& first_name);
void setLastName (string lname); See above.
void setID(string idString); See above.
void setBirthYear(in t bYear);
private: My preference is to include some whitespace between
the public, private and protected sections.
string firstName;
string lastName;
string ID;
int birthYear; You _may_ want this as an unsigned signed int since
people can't be born in a negative year.
};
#endif

HEADER DEFINITIONS
#include <iostream>
#include <string>
#include "patient.h"

using namespace std;

Patient::Pati ent() {
setFirstName("F irst");
setLastName("La st");
setID("Z9999");
setBirthYear(99 99);
} You may want to leave the fields empty for the default constructor.

Patient::Pati ent(string fName, string lName, string idString, int bYear) {
setFirstName(fN ame);
setLastName(lNa me);
setID(idString) ;
setBirthYear(bY ear);
} Prefer to use initialization lists:
Patient::Patien t(const string& fName,
const string& lName,
const string& idString,
unsigned int bYear)
/* here is the initialization list */
: firstName(fname ),
lastName(lname) ,
ID(idString),
birthYear(bYear )
{
}

If initialization is more complicated, then use an
initialization method and have each constructor call
it.

string Patient::getNam e() const {
return lastName + ", " + firstName;
} You may want to change the title of this method.
Some users may want to have both last name first
and first then last layouts.

int Patient::getAge () const {
return birthYear;
}
string Patient::getID( ) const {
return ID;
}
void Patient::setFir stName(string fName) {
firstName = fName;
}
void Patient::setLas tName(string lName) {
lastName = lName;
}
void Patient::setID( string idString) {
int i = idString.size() ;
if (i == 5) { Where did this magic number come from?
ID = idString;
}
else {
cerr <<"\nIllegal Patient ID: " <<idString<<" using Z9999"<<endl; Explain to the user why the ID is illegal:
cerr << "Patient ID string too long.".
Although you may want to through an exception because
this method assumes that cerr will be available and
the user will be looking at it. With an exception,
the function using Patient::setID will get to choose
what to do with this error.

ID = "Z9999";
}
}
void Patient::setBir thYear(int Year) {
if ((Year >1873) && (Year < 2003)) {
birthYear = 2003 - Year;
}
else {
cerr<<"\nIllega l Year: "<<Year<<" using 9999"<<endl;
birthYear = 9999;
}
} Why bother altering the year. There is nothing to
gain, but much more to lose. Just check for boundary
conditions and throw an exception. An integer, signed
or unsigned, has enough capacity for a persons birth
year.


**** Problem Driver *********** Hmmm, does the driver have a problem? :-)

#include <iostream>
#include <string>
#include <fstream>
#include <iomanip>
#include "patient.h"

using namespace std;

char resp;
void GetList();
bool getPatient(ifst ream &fin, Patient &P);
const int MaxPatientSize = 1000;

int main()
{

Patient P[MaxPatientSize];
GetList();

do{
cout <<"\nWould you like to read another Patient File? (y or n):";
cin >> resp;
}while ((resp != 'y') && (resp != 'Y') && (resp != 'n') && (resp !=

'N')); Do yourself a favor and research the std::toupper and std::tolower
methods. This will simplify to:
cin >> resp;
resp = std::toupper(re sp);
} while (resp != 'Y' && resp != 'N');


if ((resp == 'y') || (resp == 'Y')) if (resp == 'Y') /* see std::toupper */

GetList();
else
cout <<"Good Bye!"<< endl;

return 0; See EXIT_SUCCESS in <cstdlib>
return EXIT_SUCCESS;

}
void GetList()
{
cout<<"Enter name of Patient List to be read: ";
cin.getline( This line is an error. The compiler should have said something.

ifstream fin(file_name.c _str());
if (!fin)
{
cerr<<"\nCannot open Patient List. Good Bye\n";
exit(1); See EXIT_FAILURE in <cstdlib>
exit(EXIT_FAILU RE);
This is bad karma to have a function terminate the program.
The function should throw an exception or return a status.
Let the main() function be the only function that terminates
the program.

}
do
{
cout<<"Would you like to display your Patient List? (y or n): ";
cin >> resp;
}while ((resp != 'y') && (resp != 'Y') && (resp != 'n') && (resp !=
'N'));
if ((resp == 'y') || (resp == 'Y')) std::toupper or std::tolower. 'Nuff said.

{
for (P = Patient(),
while(getPatien t(fin, P)) What language is this?
1. Don't mix while and for loops.
2. A "for" loop implies one or more iterations.
3. A "while" loop implies zero or more iterations.
More bad karma.

{
cout<<"\n\n\n"< <setw(25) <<setiosflags(i os::left)<<"Pat ient Name"
<<setw(20)
<<setiosflags(i os::left)<<"Pat ient ID" <<setw(15)
<<setiosflags(i os::left)<<"Pat ient Age"<<endl;
cout <<setw(25)<<set iosflags(ios::l eft)<<P.getName () <<setw(20)
<<setiosflags(i os::left)<<P.ge tID() <<setw(15)
<<setiosflags(i os::left)<<P.ge tAge()<<"\n\n"; Looks like your Patient class would really benefit by overloading
operator <<. This would simplify to:
cout << P;

fin.close(); So, why is the file closed after reading one patient?

P = Patient(); This isn't necessary. No reason to clear an object when
it will be overwritten; just a waste of time and energy.
Poor karma.

}
else
return;
} bool getPatient(ifst ream &fin, Patient &P)
{ Note: This function could be replaced by overloading
the operator>> in the Patient class.
bool isOK = true;
string name = "";

if (getline(fin, name, '\n') && isOK) The first line does not need to check the isOK
variable.
Beware of using cut and paste.
fam.setFirstNam e(name);
else
isOK = false;

if (getline(fin, name, '\n') && isOK)
fam.setLastName (name);
else
isOK = false;

if (getline(fin, name, '\n') && isOK)
fam.setID(name) ;
else
isOK = false;

/*f (getline(fin, name, '\n') && isOK)
fam.setBirthYea r(name);
else
isOK = false; */

return isOK;
}

My suggestion is to implement two methods in your
Patient class that load and store to a string.

For example, your above input function will fail
if the data fields are all on one line separated
by tabs or commas. You will need another method
when retrieving the data from a database or
other input source.

My experience is to:
1. Input the data to a string.
2. Have the object initialize its members from
the given string.

In your program, the above simplifies to:
std::string text_line;
std::string patient_record;
std::vector<Pat ient> patient_list;
while (fin)
{
for (unsigned int i = 0;
(i < LINES_PER_RECOR D) && (getline(fin, text_line);
++i;
{
if (i > 0)
{
patient_record += '\t';
}
patient_record += text_line;
}
if (i < LINES_PER_RECOR D)
{
// Handle case of not enough data.
}
else
{
Patient new_patient;
new_patient.loa d_from(text_lin e);
patient_list.pu sh_back(new_pat ient);
}
}


Of course your right, sorry.

Well, I'm receiveing "unresolved external links" to the header
getAge, getID, getName.

But , I guess I'm looking for direction in the following, regarding the
executable file:
can I declare an array while initializing an object; ex: Patient
P[MaxSize], where MaxSize has been declared.(my goal being to limit the
number of Patient files (think hospital beds) to MaxSize
have I chosen appropriate parameters for the function getList in the
executable file

Thanks

You also have to make sure that your linker is fed all object
modules or your project includes all your source files.

--
Thomas Matthews

C++ newsgroup welcome message:
http://www.slack.net/~shiva/welcome.txt
C++ Faq: http://www.parashift.com/c++-faq-lite
C Faq: http://www.eskimo.com/~scs/c-faq/top.html
alt.comp.lang.l earn.c-c++ faq:
http://www.comeaucomputing.com/learn/faq/
Other sites:
http://www.josuttis.com -- C++ STL Library book