Iterator problem..

Har ikke super meget styr på c++ fejl, men sådan her plejer jeg at bruge iteratorer. Det kan være det virker bedre.

for (vector<string>::const_iterator t(vector.begin()), t_end(vector.end()); t != t_end; ++t)
{
  cout << *t << endl;
}

Den kode her er i øvrigt også hurtigere end den du postede

Får stadig samme fejl s:

Dette virker:

void Products::browseProducts(vector<string> Vector)
{
	system("CLS");
	cout << "Browsing Products.." << endl;
	Sleep(3000);

    vector<string>::iterator iter;
	for(iter = Vector.begin(); iter != Vector.end(); iter++)
	{
		cout << *iter << endl;
	}
}

Det første problem var at parameteren hed "vector" det giver navne sammenfald.
Det andet problem var at du have skrevet , i stedet for ; i dit for statement.

>Jonas, man skal være meget varsom med at påstå at det ene eller andet er "hurtigere" end noget andet, i dit eksempel er der (sandsynligvis) ingen forskel, til gengæld er det ikke så let at læse (det er naturligvis kun min mening).

Dette virker:

Kode 

void Products::browseProducts(vector<string> Vector)
{
	system("CLS");
	cout << "Browsing Products.." << endl;
	Sleep(3000);

    vector<string>::iterator iter;
	for(iter = Vector.begin(); iter != Vector.end(); iter++)
	{
		cout << *iter << endl;
	}
}

Det første problem var at parameteren hed "vector" det giver navne sammenfald.
Det andet problem var at du have skrevet , i stedet for ; i dit for statement.

>Jonas, man skal være meget varsom med at påstå at det ene eller andet er "hurtigere" end noget andet, i dit eksempel er der (sandsynligvis) ingen forskel, til gengæld er det ikke så let at læse (det er naturligvis kun min mening).

Tusind tak.
Hvad mener du med at det giver navne sammenfald? Altså mener du med den anden parameter - newProduct parametern?
Fordi kan det ik være ligemeget når parametern alligevel bliver udskiftet med den vecter man selv bestemmer at bruge til det?

Jeg mener at parameteren til funktionen hed "vector" med lille v, så var der to ting der hed "vector", parameteren og typen. Ved at kalde parameteren for Vector (med stort V) er der ikke længere to ting der hedder det samme.

Der er ikke nogen "newProduct" parameter, der er kun en parameter (vector/Vector). Jeg forstår ikke "Fordi kan det ik være ligemeget når parametern alligevel bliver udskiftet med den vecter man selv bestemmer at bruge til det?"

Jeg forstår ikke "Fordi kan det ik være ligemeget når parametern alligevel bliver udskiftet med den vecter man selv bestemmer at bruge til det?"

Altså du bestemmer selv hvilken vector du vil bruge senere hen til funktionen

vector<string> whatever;

browseProduct(whatever);

Nåh, men nu har jeg kun 1 error tilbage.
Dette er når jeg prøver at kalde newProduct funktionen:

main:

//Vector
	vector<string> newprod;
//Laver ny structure
	Products products;
	if(Name == "admin" || Name == "Admin")
	{
		cout << "New or browse?" << endl;
		string answer;
		cin >> answer;
		if(answer == "1")
		{
			products.newProduct(newprod);
		}
			
		// ADMINISTRATOR!
	}

productsfunction.cpp

void Products::newProduct(vector<string> &Vector)
{
	cout << "Enter the name of the new product: ";
	string name;
	getline(cin, name);

	Vector.push_back(name);
}

products.h

#include "includer.h"

class Products {
public:
	Products();

	// Nyt produkt som skal skrives ind i vector
	void newProduct(vector<string> &Vector);

	// Mulighed for at søge vectoren igennem
	int searchProduct(vector<string> &vector);

	void browseProducts(vector<string> Vector);
};

Log:

1>------ Build started: Project: PayProgram, Configuration: Debug Win32 ------
1>Compiling...
1>main.cpp
1>Linking...
1>main.obj : error LNK2019: unresolved external symbol "public: __thiscall Products::Products(void)" (??0Products@@QAE@XZ) referenced in function _main
1>F:\c++ programmer\c++ Projekter\PayThing\PayProgram\Debug\PayProgram.exe : fatal error LNK1120: 1 unresolved externals
1>Build log was saved at "file://f:\c++ programmer\c++ Projekter\PayThing\PayProgram\Debug\BuildLog.htm"
1>PayProgram - 2 error(s), 0 warning(s)
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

I Products class'en har du en prototype for en default constructor, men du har ingen implementation for den.

Så du skal enten lave linien (i product.h):
Products();

Om til :
Products() {}
Eler bare slette linien, for den bliver ikke rigtigt brugt.

Du skal muligvis lige have lidt styr på termerne.

Hvis du har:
void Func(int parm)
{
Så er "parm" en parameter

Når du så kalder den:
Func(val)
Så er val et argument.

Du skal muligvis lige have lidt styr på termerne.

Hvis du har:
void Func(int parm)
{
Så er "parm" en parameter

Når du så kalder den:
Func(val)
Så er val et argument.

Har fået det hele til at virke nu. Tusind tak.