Ich möchte folgende Aufgabe gerne Lösen, es wäre nett wenn ihr mir dabei etwas behilflich sein könntet.
Ich möchte allerdings keine Komplettlösung haben denn sonst lerne ich dabei ja nichts.
Es wäre nett wenn ihr mir hin und wieder einfach etwas auf die Sprünge helfen könntet.

Aufgabe : Lesen einer Datei und Bestimmung des Minimalwertes

Eine Datei MESSWERTE.TXT enthält eine unbekannte Anzahl n von float-Messwerten.
a) Bestimmen Sie die Anzahl n der Messwerte in der Datei. Schließen Sie dann die Datei wieder.
b) Definieren Sie (mit dem new-Operator) eine Reihung messwerte für die n Messwerte.
c) Öffnen Sie die Datei erneut und lesen Sie die Messwerte der Datei in messwerte ein.
d) Geben Sie die Messwerte zur Kontrolle aus. Sollten es mehr als 20 Messwerte sein, geben Sie nur die ersten 10 und die letzten 3 Messwerte aus.
e) Bestimmen Sie das minimum der Messwertreihe.
f) Geben Sie die Anzahl n und den Werte von minimum aus.

Lösungsansatz:

a) // ich hab das jetzt erstmal mit Ganzzahl messwerte gemacht, ich weis leider nicht wie man das mit gleitkommazahlen machen könnte ( wegen dem einlesen von Datei)

//Aufgabe 5 a)
#include <iostream>
#include <fstream>
#include <string>

using namespace std;

int main()
{
	string dateiname = "MESSWERTE.TXT"; 
	ifstream input;

	input.open(dateiname.c_str(), ios_base::in);

	if (!input)
	{
	cout << "Datei kann nicht geöffnet werden";
	return -1;
	}

int i;
char zeichen;

for (i=0; !input.eof(); i++)
	{
	input.get(zeichen);
	}

cout << "Die Datei enthaelt " << i/2 << " Zeichen" << endl;

input.close();

system("pause");

return 0;

}

MESSWERTE.TXT sieht so aus:

1
2
3
4
5
6
7
8
9
10

b)
Hier hab ich schon meine Probleme das mit dem new Operator hatten wir noch nicht so genau.
Ich weis zwar das man mit diesem Operator dynamischen Speicher im Heap reservieren kann aber wie ich das genau mache weis ich nicht.

Warum kann man das denn nicht nur mit einem normalen float Array machen?

An dieser Stelle komme ich momentan nicht mehr weiter, ich weis halt nicht was ich nun machen soll.

Könnt ihr mir da etwas auf die Sprünge helfen?

dankeschön im voraus

mfg fraggelfragger

float f;
int nAnzahl = 0;
while(true)
{
    if(!(input << f))
      break;

    nAnzahl++;
}

Ich habe extra nicht auf input.eof() abgefragt, sondern nur, ob der Stream noch in Ordnung ist. Bei Buchstaben in der Datei gäbe es sonst eine Endlosschleife.

zu b:
Da du ja nicht weißt, wieviele Werte in der Datei stehen, kannst du ja nicht ein statisches Array benutzen.

Du mußt also dann 'nAnzahl' float-Werte allokieren:

float *pValues = new float[nAnzahl];

// Zugriff auf die einzelnen Werte analog zu einem statischen Array
for(int i=0; i<nAnzahl; i++)
  pValues[i] = 0;

// bzw. aus der Datei lesen
input << pValues[i];

// und am Ende nach allen Berechnungen wieder freigeben
delete [] pValues;

(25) : error C2784: "std::basic_ostream<_Elem,_Traits> &std::operator <<(std::basic_ostream<_Elem,_Traits> &,const std::basic_string<_Elem,_Traits,_Alloc> &)": template-Argument für "std::basic_ostream<_Elem,_Traits> &" konnte nicht von "std::ifstream" hergeleitet werden

//edit muss dazu evtl. noch ne andere Bibiothek includieren? wegen input << f

#include <iostream>
#include <fstream>
#include <string>

using namespace std;

int main()
{
	string dateiname = "MESSWERTE.TXT"; 
	ifstream input;

	input.open(dateiname.c_str(), ios_base::in);

	if (!input)
	{
	cout << "Datei kann nicht geöffnet werden";
	return -1;
	}

float f;
int nAnzahl = 0;

while(true)
{
    if(!(input << f))
      break;

    nAnzahl++;
}
cout << "Anzahl der Werte lautet: " << nAnzahl << endl;

input.close();

system("pause");

return 0;

}

was ist da nun falsch?

Richtig ist es so:

input >> f;

so ich habs nun soweit:

//b) Definieren von einer Reihung Messwerte
float *pmesswerte = new float[nAnzahl];

//c) lesen der Datei in messwerte
ifstream input2;
input2.open(dateiname.c_str(), ios_base::in)
if (!input2)
		{
		cout << "Datei kann nicht geöffnet werden";
		return -1;
		}

for(int i=0; i < nAnzahl; i++)
input2 >> pmesswerte[i];
input2.close(); // edit hatte ich vergessen

somit wäre ja dann c) abgehackt oder?
Eine Frage habe ich allerdings dazu noch, hab ich das nun richtig verstanden das er mit dem befehl input immer eine komplette Zeile einliest? Wie müsste ich es machen wenn die Messwerte.txt so aussehen würde:

1 2 3 4 5 6 7 8 ...

so in der Art --> lese alles ein bis zum nächsten Leerzeichen?!

mfg fraggelfragger

//edit deklaration von input in input2 umgeändert wegen Namensdoppelbelegung
//edit2 habe das Program c) mal in Einzelschritt ausgeführt leider schreibt der nun in den Pointer messwerte nur den ersten Wert von der Text Datei rein. Alle anderen Werte gehen unter ( es scheint so das die Schleife immer wieder bei der ersten Zeile anfängt.

Laß dir doch danach einfach mal die Meßwerte ausgeben:

for(int i=0; i < nAnzahl; i++)
{
  input2 >> pmesswerte[i];
  cout << pmesswerte[i] << endl;
}

Bei der Datei ist es egal, ob zwischen den Zahlenwerten Leerzeichen, Zeilenenden oder sonstige sog. Whitespaces stehen, diese werden alle überlesen.

Hast du evtl. beim Debuggen immer nur den Wert von 'pmesswerte' dir angeschaut, dieser entspricht ja einfach dem ersten Wert des Arrays.
Du mußt schon explizit 'pmesswerte[i]' dir anschauen (watch).

Ich werd mich nun mal an den nächsten Aufgabenteil ranmachen.

An dieser Stelle möchte ich mich aber erstma noch bedanken für deine Hilfe.

mfg fraggelfragger

//b) Definieren von einer Reihung Messwerte
float *pmesswerte = new float[nAnzahl];

//c) messwerte einlesen
ifstream input2;
input2.open(dateiname.c_str(), ios_base::in);
if (!input2)
		{
		cout << "Datei kann nicht geöffnet werden";
		return -1;
		}

for(int i=0; i < nAnzahl; i++)
	{
	input2 >> pmesswerte[i];

		if (nAnzahl <= 20)
			{
			cout << pmesswerte[i]<< endl;
			}
		else;
	}

input2.close();

//d) Bei mehr als 20 Werte --> nur ersten 10 und letzte 3 ausgeben
if (nAnzahl > 20) 
{
	for(int j=0; j < 10 ; j++)
		{
		cout << pmesswerte[j]<< endl;
		}

	for(int k=nAnzahl-3; k < nAnzahl ; k++)
		{
		cout << pmesswerte[k]<< endl;
		}
}

bei der e) hab ich noch etwas Probleme, kann man das mit der getExtremes() Funktion machen?
oder doch eher mit Schleife und einzelne Werte vergleichen und vertauschen?

mfg
fraggelfragger

Du solltest die zwei Ausgabe-Schleifen am besten einfach als if ... else ... realisieren (die Ausgabe aus der Einleseschleife wieder entfernen):

if(anzahl <= 20)
{
  for(int i=0; i < nAnzahl; i++)
    cout << pmesswerte[i]<< endl;
}
else
{
  // Schleife mit ersten 10
  // Schleife mit letzten 3
}

Tipp: Du solltest dir für jede Teilaufgabe eine eigene Funktion implementieren
(Idealerweise sogar eine Klasse dafür definieren, falls du schon weißt, was das ist?)

zu e:
im Header <algorithm> gibt es die Funktionen std::min_element() und std::max_element.
Die Funktion 'getExtremes()' kenne ich nicht.

und f sollte dann ja auch einfach für dich sein...

P.S. Anhand welches Tutorials oder welchen Buchs lernst du denn?

Tipp: Du solltest dir für jede Teilaufgabe eine eigene Funktion implementieren
(Idealerweise sogar eine Klasse dafür definieren, falls du schon weißt, was das ist?)

okey ich werde das mal ändern, Klassen habe ich auch schon gehört aber praktisch in der Programierung angewend habe ich diese leider noch nicht.
Ist dir evtl. ein Buch bekannt in dem Übungen mit Lösungen vorhanden sind? Denke so könnte ich am besten lernen.

Th schrieb:

P.S. Anhand welches Tutorials oder welchen Buchs lernst du denn?

Ich lerne nach einem Skript, dabei musste ich mich allerdings einem Skript von einem anderen Professor bedienen, da mein eigener Prof kein Skript hat ( der gute Mann ist schon etwas älter und war schon im Ruhestand --> er wurde zurückbeordert)

Deshalb muss ich mich halt da etwas durchbeisen, hast du irgendwelche Tips für mich ? Buchtips?

Aber gerade Funktionen sollten schon am Anfang erklärt werden, bevor man z.B. komplexe Ein- und Ausgaben macht.

Je größer die Programme werden, desto mehr Wert muß man auf eine gute Struktur legen (und dies wird durch gut benannte Funktionen bzw. Klassen unterstützt).

Und dann das Programm mit Unterfunktionen realisieren.

mfg
fraggelfragger

was könnte ich da noch besser machen? ist das alles so korrekt?

Mit Klassen würd ichs glaube ich auch gerne versuchen. Weil es ja noch Professioneller dann ist.

#include <iostream>
#include <fstream>
#include <string>
#include <algorithm>

using namespace std;

//Prototypen
long Werte_Anzahl(string name);
int einlesen(string name,int b,float *pmesswerte);
void ausgabe(int b, float *pmesswerte); 
float minimum(int b, float *pmesswerte);

int main()
{
string dateiname = "MESSWERTE.TXT"; 

//a) Anzahl der Messwerte bestimmen
int nAnzahl = Werte_Anzahl(dateiname);

//b) Definieren von einer Reihung Messwerte
float *pmesswerte = new float[nAnzahl];

//c) Messwerte einlesen
einlesen(dateiname,nAnzahl,pmesswerte);

//d) 20 Werte --> ersten 10 und letzten 3 ausgeben ansonsten alle
ausgabe(nAnzahl,pmesswerte);

//e)Minimum suchen
float min = minimum(nAnzahl,pmesswerte);

//f) nAnzahl ausgeben und minmum ausgeben
cout << "Anzahl der Werte: " << Werte_Anzahl(dateiname) << endl;
cout << "Minimum: " << min << endl;

delete [nAnzahl] pmesswerte;

return 0;
}

//Unterfunktionen

long Werte_Anzahl(string name)
{
	//Datei öffnen
	ifstream input;
	input.open(name.c_str(), ios_base::in);

	//Wenn Datei nicht vorhanden
	if (!input)
		{
			cout << "Datei kann nicht geöffnet werden";
			return -1;
		}

float f;
int nAnzahl = 0;

//Werte-Anzahl bestimmen
while(true)
	{
    		if(!(input >> f))
    			break;

    		nAnzahl++;
	}

//Datei schließen
input.close();

//Rückgabe von Anzahl der Werte in Datei
return nAnzahl;
}

int einlesen(string name,int b,float *pmesswerte)
{
ifstream input2;
input2.open(name.c_str(), ios_base::in);
if (!input2)
        {
        	cout << "Datei kann nicht geöffnet werden";
        	return -1;
        }

for(int i=0; i < b; i++)
    {
    	input2 >> pmesswerte[i];
    }

input2.close();
}

void ausgabe(int b, float *pmesswerte)
{
if (b > 20) 
	{
		for(int j=0; j < 10 ; j++)
			{
				cout << pmesswerte[j]<< endl;
			}

		for(int k=b-3; k < b ; k++)
			{
				cout << pmesswerte[k]<< endl;
			}
	}
else
	{
		for(int l=0; l < b ; l++)
			{
				cout << pmesswerte[l]<< endl;
			}
	}
}

float minimum(int b, float *pmesswerte)
{
float minimum = *min_element(pmesswerte, pmesswerte+b);

return minimum;
}

∞

int main()
{
    while(true)
    {
         if(flag)
         {
            f();
         }
         else
         {
            g();
         }
    }
}

Aber inhaltlich ist dein Code schon sehr gut aufgeteilt auf die Funktionen (du könntest höchstens noch einige Variablen noch besser benennen, z.B. 'b').

Und wenn du dich in Klassen eingelesen hast, dann wirst du merken, daß du dann nicht mehr bei jeder Funktion die Parameter 'pmesswerte' angeben mußt.

Aber du bist auf jeden Fall auf einen guten Weg zum erfolgreichen C++ Programmierer -)

Deine Code-Formatierung ist gewöhnungsbedürftig. Normalerweise rückt man alle Teile innerhalb der Blockklammern um einen Tabulator (oder wahlweise Leerzeichen) ein

okey, ich werde das nächste mal versuchen, es komplett so zu machen, und das mir so anzugewöhnen.

Th schrieb:

du könntest höchstens noch einige Variablen noch besser benennen, z.B. 'b').

Da hast du recht, habe das etwas einfallslos benannt.
Ich wollte hauptsächlich das bei den Funktionsdefinitionen, nicht unbedingt die Variablen stehen die auch übergeben werden.

Th schrieb:

Aber du bist auf jeden Fall auf einen guten Weg zum erfolgreichen C++ Programmierer -)

freut mich zu hören

Vielen dank das du nochmal über den Code drüber geschaut hast und mir etwas Feedback zukommen gelassen hast.
Es hilft wirklich viel wenn man von einem erfahrenen Programmierer Tips bekommt