void calc(vector <double>& price,vector <double>& weight){
	double val;
	val = price[0] * weight[0];
    cout << val << endl;
	val = price[1] * weight[1];
	cout << val << endl;
	val = price[2] * weight[2];
	cout << val << endl;
	val = price [3] * weight[3];
	cout << val << endl;

	for (int i = 0;weight.size() <= price.size(); ++i) val = i;
	cout << val << endl; 
}

int main()
{
    vector <double> price;
	vector <double> weight;

    price.push_back(45.56);
	price.push_back(4.5);
	price.push_back(66.89);
    price.push_back(0.4555);

	weight.push_back(32.89);
	weight.push_back(2.2);
	weight.push_back(99.45);
	weight.push_back(56.6);

	calc(price,weight);

	keep_window_open();
    return 0;
}

Nein, sondern wenn ich es richtig verstehen es soll einen Wert errechnen aus price und weigth nur das weight.size()<=price.size() gelten muss.

Und was genau soll das bedeuten?

if (weight.size() <= price.size()){ 
		val = price[] * weight[] ;
		cout << val << endl;}

for (int i = 0;weight.size() <= price.size(); ++i) val = i;

Das ist jedenfalls Blödsinn. Ich würde stark empfehlen dass du dir nochmal anschaust was genau eine for Schleife tut.

Mir ist klar was du damit beabsichtigst aber du kannst nicht einfach zwei Arrays miteinander multiplizieren. Du kannst nur zwei Elemente der beiden Arrays explizit ansprechen und diese dann multiplizieren, also z.B. sowas machen:

double product = price[1] * weight[1];

das multipliziert das zweite Element aus price mit dem zweiten Element aus weight.

if (weight.size() <= price.size()){ 
		val = price.size() * weight.size();
		cout << val << endl;}

void calc(const vector <double>& price,const vector <double>& weight){ 
	double val;
	val = price[0] * weight[0];
    cout << val << endl;
	val = price[1] * weight[1];
	cout << val << endl;
	val = price[2] * weight[2];
	cout << val << endl;
	val = price [3] * weight[3];
	cout << val << endl;
	cout << " " << endl;
	if (weight.size() <= price.size()){   // For the calculation
		val = price.size() * weight.size();
		cout << val << endl;}
	cout << " " << endl;
    if (weight[0] <= price[0]){
		val = price[0] * weight[0];
		cout << val << endl;}
	if (weight[1] <= price[1]){
		val = price[1] * weight[1];
		cout << val << endl;}
	if (weight[2] <= price[2]){
		val = price[2] * weight[2];
		cout << val << endl;}
	if (weight[3] <= price[3]){
		val = price[3] * weight[3];
		cout << val << endl;}
}

int main()
try{
    vector <double> price;
	vector <double> weight;

    price.push_back(45.56);
	price.push_back(4.5);
	price.push_back(66.89);
    price.push_back(0.4555);

	weight.push_back(32.89);
	weight.push_back(2.2);
	weight.push_back(99.45);
	weight.push_back(56.6);

	calc(price,weight);

	keep_window_open();
    return 0;
}
catch (runtime_error e) {	
	keep_window_open("~");	
}
catch (...) {	
	cout << "exiting\n";
	keep_window_open("~");	
}

Wenn du das nächste mal Programmcode postest, solltest du deine Stichpunktliste ebenso mit abtippen, damit wir dir das korrigieren können.

//edit deine zuletzt genannte Lösung ist Schrott. In deiner Aufgabe steht explizit "die Summe". Dein Programm enthält aber nicht ein einziges "+".

Auch beachtest du nicht die von dir genannte Restriktion. Stell dir vor, price hat nur 3 Elemente. Was passiert, wenn du dann price[3] aufrufst? Und was machst du, wnen beide Vektoren 10 Elemente haben?

#include <numeric>
#include <vector>

double sumOfProducts (std::vector<double>& weight, std::vector<double>& price) {
	if (weight.size()==price.size())
		return std::inner_product(weight.begin(),weight.end(),price.begin(),0.0);
	throw std::runtime_error("vector sizes mismatch");
}
// ... snipsnap

Eine andere Möglichkeit wäre:

double sumOfProducts2 (std::vector<double>& weight, std::vector<double>& price) {
	if (weight.size()<=price.size()) {
		double sum = 0.0;
		for (int i = 0; i!= weight.size(); ++i) 
			sum += weight[i]*price[i];
		return sum;
	}
	throw std::runtime_error("weight-vector is greater than price-vector");
}

for (int i = 0; i!= weight.size(); ++i)

Wenn das nicht gleich ist dann wird nichts gemacht?

@inter2k3 und was hat er jetzt daraus gelernt? Könnte er jetzt die nächste Aufgabe aus dem Buch besser lösen?

@winux was ist denn deine Muttersprache?

//edit deine For-Schleife lässt sich wie folgt zerlegen (ich gehe einfach davon aus, dass du bereits while kennst):

int i=0;
while (i!= weight.size()){
   //tue das was in der For-Schleife zwischen den {...} steht
   //...
   ++i;
}

Hilft dir das weiter?

*Seufz*

@inter2k3 und was hat er jetzt daraus gelernt? Könnte er jetzt die nächste Aufgabe aus dem Buch besser lösen?

Hoffentlich, dass es für jedes Problem mehrere Lösungen gibt (auch wenn ich nur zwei präsentiert habe).
Ausserdem war offensichtlich, dass er aufgrund der beschränkten Deutschkenntnisse die Aufgabe nicht richtig verstanden hat. Meine Hoffnung war, dass dies aus dem code (vorallem dem zweiten Beispiel) hervorgeht, was gefordert war.
Dass die absoluten Grundlagenkenntnisse (z.b. for-schleife) noch nicht fundiert sind konnte ich ja nicht ahnen.
Du erklärst die Funktionsweise der For-Schleife ja auch anhand von code

for (int i = 0; i!= weight.size(); ++i)

Aber so etwas kenne ich schon:

for (int i = 0; i<= weight.size(); ++i)

Alle 3 Schleifen tun das gleiche. Es ist nur eine unterschiedliche Formulierung.

Ich würde aber zur ersten Variante raten.

for (int i = 0; i < vec.size(); ++i)
	{
		std::cout << vec[i] << std::endl;
	}

Die zweite Variante ist unnötig kompliziert:

for (int i = 0; i <= vec.size() - 1; ++i)
	{
		std::cout << vec[i] << std::endl;
	}

Und die dritte Variante empfinde ich als unschön. Ich vergleiche nur bei Iteratoren auf ungleich.

In C++ werden Intervalle (ranges) immer als Halboffen [begin, end) angegeben.
Das bedeutet, das begin noch dazugehört und end nicht.

Deswegen sind

for (int i = 0; i<= weight.size(); ++i)

und

for (int i = 0; i != weight.size(); ++i)

nicht das selbe! Die Merkregel ist: bei 5 Durchläufen bei 0 anfangen und
auf i != 5 oder auf i < 5 testen. Und am besten in eigenem Code
das immer ganz genauso machen. Dann muss man nämlich nicht jedes mal überlegen.