Thread: Homework question

g++ main.cpp user.cpp ad.cpp
user.h:29: error: ISO C++ forbids declaration of 'vector' with no type
user.h:29: error: expected ';' before '<' token

Here is my user.h file:

Code:

#ifndef _user_h_
#define _user_h_

#include <string>
#include <vector>
#include "ad.h"
using std::string;

class user {

public:

	// CONSTRUCTORS
	user(const string& _ID) : ID(_ID) {}

	// ACCESSORS
	const string& id() const { return ID; }

	// MUTATORS
	void add_keyword(const string& _keyword, const string& _URL, const float& _PC, const float& _CC, bool& _is_new);
	bool ad_for_keyword(const string& _keyword, const string& _URL, const float& _PC) const;
	bool click(const string& _keyword, const string& _URL) const;
	bool remove(const string& _keyword) const;
	void statistics(std::ostream & output_ofs);

private:
	// REPRESENTATION
	string ID;
	vector<ad> ads;
};

#endif

I'd appreciate it if someone could help me with this. Commenting out the bolded line of code makes my programs compile perfectly.

EDIT: Also, I'd appreciate if you didn't post any specific code to help me (stuff like the 2nd post is ok), not that I think this question needs it. My instructors allow me to seek help on comprehending error messages, but not get code from outside sources.

A few points:
* In general start class names with capitals.
* put the word explicit before 1 argument constructors, to prevent automatic conversion.
* don't put using std::string and similar statements in header files.

And for the fix to your problem:

Code:

std::vector<ad> ads;

Originally Posted by Jozrael

Code:

2: explicit user (const string& _ID) : ID(_ID) {}

^ like that?

Yep, that look right.

3: I'm not sure why I shouldn't put 'using blah' in my header files: don't I need it to define for the compiler what it needs to have?

Each header file serves as a module for linking functions and objects that it declares. As a modularization tool, it should provide the minimal impact on code that includes it -- it should only do what is necessary to use the attributed functions and objects. using statements go against this practice. There is no need for someone using your header to put string into the global namespace (which is what using statements do). It could lead to a name collision if the header file is included in a file that has a different global version of string.

It's all in the spirit of modular, reusable code.

I hope that's clear.

And thank you for the fix...but I'm still not sure why it was necessary, since I defined #include <vector>. It works, so thank you, but I'm still not sure why.

Thanks for your help mate

Vector is defined in the std namespace. To use it you must either preface every use with std:: or before you use it have using std::vector or using namespace std. The same applies to std:: string. Since this is a header file putting std:: is the best solution.

Read up on namespaces.