ich bin noch C++ Anfänger und muss mich jetzt damit auseinandersetzen.

Ich habe 3 x-Werte, welche normiert werden müssen. Anschließend muss mit dem neuen xi-Werten weitergerechnet werden.

Hier ist der aktuelle Code:

[code]

#include <iostream>
#include <cmath>

double norm(double summe, double xi);
double  xi[4], summe;
using namespace std;

double norm(double *xi)
 {
   int i;
   for(i = 1; i <= 3; i++)
     summe += xi[i];
	 xi[i] = xi[i]/summe;
   return  (xi, summe);
 }

int main(){

	xi[1] = 0.26;
	xi[2] = 0.26;
	xi[3] = 0.26;

	norm( xi);
	cout<<summe ;
	cout <<"\n	xi1 = "<< xi[1];

	cout << "\n\n Mit ENTER beenden!";

	cin.sync();
	cin.clear();
	cin.get();
return 0;
}

Die Frage ist, warum der ursprüngliche Wert von x1 ausgegeben wird und nicht der neu berechnete. Ein andere Frage ist, warum die Funktion nicht nach main() geschrieben werden kann.

Vielen Dank für die Hilfe!

mfg Rico

p.s. Derzeit nutze ich VS 2010 express.

//Deklaration
typ in_vect(typ vect[], int size); // Eindimensional
typ in_matrix(typ matrix[][dim2], int size); // Mehrdimensional
//Aufruf
in_vect(typ *vect,sizeof(vect)/sizeof(vect[0]); //Eindimensional
in_matrix(typ *matrix[dim2],dim1); // Mehrdimensioinal

- Komisch: Du deklarierst eine Funktion norm(double,double); welche nie wieder auftaucht
- Komisch: Du definierst globale Variablen xi und summe
- Falsch: Du setzt summe vorher nicht auf 0
- Falsch: Du brauchst zwei Schleifen. Einmal für die Summe und ein zweites Mal für das Dividieren
- Komisch: Du legst ein Array mit 4 Elementen an, benutzt aber nur drei

return  (xi, summe); // geht nich!

Achte wie die Schleifen verlaufen müssen.

#include <iostream>
#include <cmath>

void norm(double *xi,int size);
double  xi[3], summe;
using namespace std;

void norm(double *xi,int size)
{
	for(int i = 0; i < size; i++){ summe += xi[i]; }
	for(int i = 0; i < size; i++) { xi[i] /= summe; }
}

int main(){

	xi[0] = 0.26; // Die erste Position im Array ist "0"
	xi[1] = 0.26;
	xi[2] = 0.26;

	norm(xi,sizeof(xi)/sizeof(xi[0]));
	// summe kannst du hier ausgeben weil sie oben global definiert ist.
	cout << summe << '\n' << "xi[1] = " << xi[1] << '\n' << "Mit ENTER beenden!";

	cin.sync();
	cin.clear();
	cin.get();
	return 0;
}

Sieht dann so aus:

#include <iostream>
#include <vector>

void norm(std::vector<double> &xi);

using namespace std;

vector<double> xi;
double summe;

void norm(vector<double> &xi)
{
	vector<double>::iterator i;
	for(i=xi.begin();i!=xi.end();i++){ summe += *i; } // ist es am besten

	for(int j=0;j<xi.size();j++) {
		xi[j] /= summe; // so geht das auch, wenn bereits werte vorhanden sind
	}
}

int main(){

	xi.push_back(1.23); // schiebe einen Wert in die letzte Position
	xi.push_back(2.34);
	xi.push_back(3.45);

	norm(xi); //  keine größenangabe nötig

	cout << "summe = " << summe << '\n';
	cout << "xi[0] = " << xi.at(0) << '\n';
	cout << "xi[2] = " << xi.at(2) << '\n';
	cout << "Mit ENTER beenden!";

	return 0;
}

Am besten du gewöhnst dich gleich an std::vector, er ersetzt in den meisten Fällen das den *array.

Und übergibst diesen bitte nicht als Kopie, sondern je nach Anforderung als Referenz oder konstante Referenz. Und verwendest auch keine globalen Variablen.

Nach noch ein paar Änderungen könnte dies wie folgt aussehen:

#include <iostream>
#include <vector>

void norm(
    std::vector<double> & xi, // Soll geändert werden, daher Referenz
    double & summe);          // Soll geändert werden, daher Referenz

int main()
{
    std::vector<double> xi;
    xi.push_back(1.23);
    xi.push_back(2.34);
    xi.push_back(3.45);

    double summe = 0.0;
    norm(xi, summe);

    std::cout << summe << std::endl;
    std::cout << "xi[0] = " << xi[0];
    // ...
} 

void norm(
    std::vector<double> & xi,
    double & summe)
{
    // Was auch immer der Sinn von dieser Funktion sein sollte, ich
    // behaupte, die Berechnung ist nicht so, wie der OP sich das
    // vorstellt (ZUDEM: Arrays etc. beginnen beim Index 0).
    for(int pos=0, size=xi.size(); pos<size; ++i) {
        summe += xi[pos];
        xi[pos] /= summe;
    }
}

Und übergibst diesen bitte nicht als Kopie, sondern je nach Anforderung als Referenz oder konstante Referenz. Und verwendest auch keine globalen Variablen.

So grausam wollte ich erstmal nicht sein

asc schrieb:

Und übergibst diesen bitte nicht als Kopie, sondern je nach Anforderung als Referenz oder konstante Referenz. Und verwendest auch keine globalen Variablen.

So grausam wollte ich erstmal nicht sein

Nur ist dein Programm schlicht und ergreifend falsch. Da nur die summe (da global) dauerhaft geändert wird. Der Vektor wurde als Kopie übergeben, und nur diese Kopie (die am Ende der Funktion zerstört wird) wurde angepasst.

Nur ist dein Programm schlicht und ergreifend falsch. Da nur die summe (da global) dauerhaft geändert wird. Der Vektor wurde als Kopie übergeben, und nur diese Kopie (die am Ende der Funktion zerstört wird) wurde angepasst.

Stimmt, da fehlt ein & in der Funktion

void norm(std::vector<double> &xi);

und beim Aufruf

void norm(vector<double> &xi){
...
}

Normieren tut aber dennoch richtig.

Zunächst einmal:
Wenn du Arrays übergibst, dann musst du immer die Größe mit angeben.

Nicht wenn wie hier die Größe vorher schon festgelegt ist. Er weiß ja dass genau 3 Elemente drin sein sollen.

Was hier ein problem ist:
- du deklarierst eine Funktion norm(double, double) und definierst danach eine Funktion norm(double*), die eine andere Signatur hat! Wozu?
- du definierst ein Array mit 4 Elementen obwohl du sagst du hast nur 3 - wozu? (das ist kein char-Array wo man ein Element mehr braucht für das abschließende '\0')
- deine for-Schleife ist zwar eingerückt als solle sie über zwei Zeilen laufen, de facto läuft sie aber nur über die erste. -> Klammern setzen!
- die Rückgabe der Funktion tut nicht was du erwartest. Wobei nicht klar ist, was du überhaupt erwartest. (xi, summe) hat den Wert von summe, weril beim Komma-Operator der wert des letzten Ausdrucks den Wert des gesamtausdrucks ergibt.
- summe als globale variable ist ungünstig bzw. schlechtes Design. Noch dazu initialisierst du die Variable nirgends, es kann also alles mögliche bei rauskommen.

Das alles zusammen sieht nicht so aus als ob du genau wüsstest was du tust. Hast du dir die zugehörigen Kapitel im Buch schon genau durchgelesen und verstanden?

- summe als globale variable ist ungünstig bzw. schlechtes Design. Noch dazu initialisierst du die Variable nirgends, es kann also alles mögliche bei rauskommen.

Nicht deswegen "kann alles mögliche rauskommen". summe wird, nachdem das Programm in den Arbeitsspeicher geladen wurde, 0 sein. Es gibt aber woanders undefiniertes Verhalten...

for(i = 1; i <= 3; i++)
  summe += xi[i];
  xi[i] = xi[i]/summe; // <-- gehört nicht zur Schleife, i ist 4.

Denn xi[4] gibt es nicht.

Alles in allem sind da ziemlich viele Würmer drin -- soviele, dass man nicht genau weiß, was das Programm eigentlich machen sollte.

Danke.