Es gibt zwei Klassen Simple und ControlChart.
Die Klasse Simple liest aus einer Datei Stichproben ein.
Die Klasse ControlChart liest Stichprobenreihen ein bzw. bekommt Stichproben übergeben (Objekte von Sample).
Bsp einer Datei:
02.01.2014 18:40 5 40.1 40.12 39.98 40.02 39.89

Hier meine Klasse Simple:

#ifndef _SAMPLE_H
#define _SAMPLE_H
#include <string>
#include <iostream>
using namespace std;
class Sample {
public:
	float mean();
	float stdDev(float);
	void write(ostream& strm=cout);
	void read(istream& strm=cin);
private:
	string date;
	string time;
	int n;
	float m[5];
	float xm;
	float s;
};
ostream& operator<<(ostream& strm, Sample& s);
istream& operator>>(istream& in, Sample& s);

#endif

Meine Einlese Methode:

void Sample::read(istream& strm) {
	strm >> date >> time >> n;
	for (int i = 0; i < n; i++)
	{
	  strm >> m[i];
	}

}

ostream& operator<<(ostream& strm, Sample& s)
{
  s.write(strm);
  return (strm);
}

Aus den Eingelesenen Werten sollen der Durchschnittswert und die Standardabweichung berechnet werden. Und anschließend in eine Datei geschrieben werden.
Bis hier hin funktioniert alles perfekt.
Mein Problem ist die Klasse ControlChart, ich hab keine Ahnung wie das mit den vectoren klappen soll, dazu gleich mehr.
Macht es überhaupt Sinn Objekte vom Typ Sample in einen Vector zu packen?
Hier mal meine Klasse (sehr stark vereinfacht erst mal):

#ifndef _CONTROLCHART_H
#define _CONTROLCHART_H
#include "Sample.h"
#include <iostream>
#include <string>
#include <vector>
using namespace std;

class ControlChart {
public:
	int addSample(Sample& s);
	void writec(ostream& strm=cout);
	//void read(istream& strm=cin);
private:
	vector<int> Sample;

};
ostream& operator<<(ostream& strm, ControlChart& c);
istream& operator>>(istream& in, ControlChart& c);

#endif

Ich hab in meiner main-Datei versucht Stichproben einzulesen und für jede ein neues Objekt Sample zu erstellen. Diese Samples will ich in ControlChart in den Vector schreiben und als Test in eine Datei schreiben.

Hier mein Versuch:

int main()
{
  fstream strm ("daten", ios::in);
  Sample s1, s2, s3, s4;
  ControlChart c1;
  s1.read(strm);
  s2.read(strm);
  c1.addSample(s1);
  ofstream strmout ("datenaus", ios::out);

  c1.writec(strmout);
  strmout.close();
  return 0;
}

Hier noch meine ControlChart.cpp:

int ControlChart::addSample(Sample& s) {
	Sample.push_back(s);
}

void ControlChart::writec(ostream& strm) {
	for(int i = Sample.begin(); i != Sample.end(); i++)
	{
	  strm << *i << endl;
	}
}

ostream& operator<<(ostream& strm, ControlChart& c)
{
  c.write(strm);
  return (strm);
}

istream& operator>>(istream& in, ControlChart& c)
{
  c.read(in);
  return (in);
}

Wäre sehr dankbar wenn mir ein netter Herr auf die Sprünge helfen könnte.
Vielen Dank!

vector<Sample> samples;

oder?

Dann solltest du noch

int addSample(const Sample& s); // <- const hinzugefügt

verwenden und deine Write- und Read-Methoden so abändern, daß du passende Leerzeichen (bzw. irgendein anderes Trennzeichen) zwischen die einzelnen Felder schreibst.

Und die Main-Funktion solltest du auch noch so abändern, daß du nur noch ControlChart für das Einlesen und Schreiben benötigst.

PS: Sehe gerade noch, daß du in deinem Text immer von "Klasse Simple" anstatt "Sample" sprichst

Und noch ein PPS:
"using namespace std;" solltest du niemals in Header-Dateien verwenden (sondern dort immer explizit "std::" schreiben), da du sonst die "Namespaces" ad absurdum führst.

Du willst doch sicherlich (in der Klasse ControlChart)

vector<Sample> samples;

oder?

Ja stimmt! Hätten wir schon mal einen Übeltäter.

Th69 schrieb:

Und die Main-Funktion solltest du auch noch so abändern, daß du nur noch ControlChart für das Einlesen und Schreiben benötigst.

Ich will in ControlChart mit dem Durchschnittswert und Standardabweichung weiterarbeiten, die von Sample berechnet werden.
Aber ich könnte die berechneten Werte wieder in eine Datei schreiben und davon auslesen stimmt.

Th69 schrieb:

PS: Sehe gerade noch, daß du in deinem Text immer von "Klasse Simple" anstatt "Sample" sprichst

Uuppsi

Danke dir schon mal.
Bei weiteren Problemen schreibe ich wieder hier.

Hat jedes Sample 5 Elemente? Oder solltest Du nicht schon in Sample einen Vektor verwenden?

Gut aufgepasst. Ich finde es auch komisch, aber in den Anforderungen steht, dass jeder Sample 5 Elemente hat. Wozu die Anzahl dann trotzdem noch vor den Messwerten steht weiß ich nicht.

EDIT: Vermutlich muss Sample auch schon ein vector sein, damit es ein vector in einem vector wird und ich somit auf die einzelnen Elemente zugreifen kann oder? -.-

In einer Sample steht ja jetzt die komplette Stichprobe (ca. 10 Werte).
Wie stelle ich es denn am geschicktesten an wenn ich aus jedem Sample den Wert an Stelle 4 haben möchte?
Will nämlich den Durchschnittswert von x berechnen.

EDIT: Vermutlich muss Sample auch schon ein vector sein, damit es ein vector in einem vector wird und ich somit auf die einzelnen Elemente zugreifen kann oder? -.-

Ich kann Dir nicht folgen - was hat der Wert an Stelle 4 mit dem Durchschnitt zu tun?
Wer ist x?
Und wieso hat Sample jetzt 10 Werte? Ich dachte in der Aufgabe stünde fünf Werte pro Stichprobe?

Wie dem auch sei: Sample einen std::vector<float> für die Daten zu spendieren hielte ich für eine gute Idee - aber das ist mit Sicherheit kein Muss.

Ich kann Dir nicht folgen - was hat der Wert an Stelle 4 mit dem Durchschnitt zu tun?
Wer ist x?
Und wieso hat Sample jetzt 10 Werte? Ich dachte in der Aufgabe stünde fünf Werte pro Stichprobe?

Das Sample besteht aus Datum, Uhrzeit, Durchschnittswert, Standardabweichung, Messwert1, Messerwert2, Messwert3, Messwert4, Messwert5
Die Durschnittswerte (in Sample berechnet) aus jedem Sample soll ich wieder verrechnen zu einem Gesamtdurchschnitt (in Controlchart berechnet).

Furble Wurble schrieb:

Wie dem auch sei: Sample einen std::vector<float> für die Daten zu spendieren hielte ich für eine gute Idee - aber das ist mit Sicherheit kein Muss.

Wenn ich das Datum und Uhrzeit auch drin haben muss brauche ich einen vector<string> oder?

Das Sample besteht aus Datum, Uhrzeit, Durchschnittswert, Standardabweichung, Messwert1, Messerwert2, Messwert3, Messwert4, Messwert5
Die Durschnittswerte (in Sample berechnet) aus jedem Sample soll ich wieder verrechnen zu einem Gesamtdurchschnitt (in Controlchart berechnet).

Furble Wurble schrieb:

Wie dem auch sei: Sample einen std::vector<float> für die Daten zu spendieren hielte ich für eine gute Idee - aber das ist mit Sicherheit kein Muss.

Wenn ich das Datum und Uhrzeit auch drin haben muss brauche ich einen vector<string> oder?

Nein.
Dein erster Ansatz gefällt mir besser. Ich dachte an einen Vektor nur für die Messwerte.

Trotzdem stellt sich mir immer noch die Frage, warum der Wert an der vierten Stelle so wichtig ist? Ich sehe in Deinem Beispiel steht da der Durchschnitt. Ist das immer so?
Ich dachte das Format stünde einfach für

Datum Zeit N Messung1 Messung2 ... MessungN

Trotzdem stellt sich mir immer noch die Frage, warum der Wert an der vierten Stelle so wichtig ist? Ich sehe in Deinem Beispiel steht da der Durchschnitt. Ist das immer so?
Ich dachte das Format stünde einfach für

Datum Zeit N Messung1 Messung2 ... MessungN

So ist das Format beim einlesen von Sample. Sample schnappt sich die Messungswerte und berechnet den Durchschnitt und die Standardabweichung.
Das wird wieder in eine Datei geschrieben, aus der ControlChart lesen soll, denn dieser braucht den Durchschnitt und die Standardabweichung jedes Samples um weitere Berechnungen durchzuführen.
An die Vieten Stelle habe ich in meinem Beispiel den Durchschnittswert gesetzt.

Wie stelle ich das am besten an mit dem vector in Sample?
Soll ich direkt beim einlesen der Datei die Werte in den vector schreiben? (Bei der Methode brauche ich dann auch meine ganzen Attribute nicht mehr, stimmts?)

Wie stelle ich das am besten an mit dem vector in Sample?
Soll ich direkt beim einlesen der Datei die Werte in den vector schreiben? (Bei der Methode brauche ich dann auch meine ganzen Attribute nicht mehr, stimmts?)

Das mit dem vector ist kein Allheilmittel. Und hilft Dir auch gar nicht bei Deinem Problem, jedenfalls sehe ich nicht wie - letztlich scheint ja sample zu funktionieren und ControlChart macht Dir Probleme.

Ich würde das Pferd von hinten aufzäumen.
Was Du machen willst:
- Aus einer Datei eine Reihe von Samples auslesen.
- Diese speichern - in einer Klasse ControlChart.
- ControlChart ausgeben.

Richtig?

int main(){
  ControlChart cc;
  {
    std::ifstream in("daten");
    Sample s;
    while(in>>s)
      cc.push_back(s);
  }
  std::ofstream out("datenaus");
  out << cc;
}

Ich würde das Pferd von hinten aufzäumen.
Was Du machen willst:
- Aus einer Datei eine Reihe von Samples auslesen.
- Diese speichern - in einer Klasse ControlChart.
- ControlChart ausgeben.

Richtig?

- Aus einer Datei eine Reihe von Samples auslesen. Korrekt
- Durchschnitt der 5 Messungen zu den Messungen dazu schreiben.
- Diese speichern - in einer Klasse ControlChart. Korrekt
- Den Durchschnitt für jedes Sample auslesen und daraus den Gesamtdurchschnitt bestimmen.

Problem:
Wenn ein Sample aus der kompletten Zeichenkette aller Messwerte besteht (wie jetzt), dann kann ich damit ja nicht rechnen.

Gesamtdurchschnitt = 0;
for(int i = 0; i< y; i++)
{
   Gesamtdurchschnitt += samples[i][4]; //An [i][4] steht bei jedem Sample der Messwertedurchschnitt des Samples
}
Gesamtdurchschnitt /= i;

Der Gesamtdurchschnitt soll auch in den vector.
Anschließend sollen die Samplereihen (inkl. Gesamtdurchschnitt) in eine Datei geschrieben werden.

Problem:
Wenn ein Sample aus der kompletten Zeichenkette aller Messwerte besteht (wie jetzt), dann kann ich damit ja nicht rechnen.

im ControlChart haben. (y beschreibt Samples, x beschreibt Werte der Samples)

Damit ich z.B. Rechnen kann

Gesamtdurchschnitt = 0;
for(int i = 0; i< y; i++)
{
   Gesamtdurchschnitt += samples[i][4]; //An [i][4] steht bei jedem Sample der Messwertedurchschnitt des Samples
}
Gesamtdurchschnitt /= i;

Der Gesamtdurchschnitt soll auch in den vector.
Anschließend sollen die Samplereihen (inkl. Gesamtdurchschnitt) in eine Datei geschrieben werden.

Verstehe ich nicht.

Ich wuerde auch die Messwerte und die Durchschnitte getrennt halten.

Vielleicht hilft folgendes:

#include <vector>
#include <iostream>

struct sample{
  using container = std::vector<float>;
  container data;
};

struct controlchart{
  using container = std::vector<sample>;
  void push_back(const sample&s){
    samples.push_back(s);
  }
  // gibt den 4. Wert eines jeden Samples aus
  void print_4th() const {
    for(auto i=0u; i<samples.size(); ++i)
      if(3<samples[i].data.size())
	std::cout << samples[i].data[3] << '\n';
  }
private:
  container samples;
};

int main(){
  sample samples[] = {
    { {0.f, 1.f, 2.f, 33.f} },
    { {1.f, 2.f, 3.f, 44.f} },
    { {2.f, 3.f, 4.f, 55.f} }
  };
  controlchart cc;
  for(const auto& s : samples)
    cc.push_back(s);
  cc.print_4th();
}

Das Beispiel hat keinen tieferen Sinn - aber in Z. 18 greife ich über zwei Indizes in zwei verschiedenen Vektoren auf einen Wert zu.
Ungefähr das, womit Du Probleme zu haben scheinst.
Der Code benutzt hier und da C++11. Achte also darauf, dass Dein Compiler das unterstüzt (Z. 18 ist allerdings nicht speziell C++11).

Hoffentlich hilft es.
Wobei ich - wie schon gesagt - mit einigem, dass Du vorhast arge Bauchschmerzen habe...