void f()
{
	std::ifstream istr;

	vector <Point> points;
	Point x,y;
	while (istr>>x>>y) points.push_back(x,y);
}

int main()
{
    f();

	keep_window_open();
    return 0;
}

Laut der Übung muss man einen Datentyp Point definieren das zwei Koordinatenmembern x und y hat, was auch gemacht wurde.

Nein. Da sollte oben sowas stehen wie

Class Point
{
   double x;
   double y;
public:
   Point(

Lg freeG

EDIT: Da war Nexus schneller...allerdings braucht deine Klasse auch ein Konstruktor der die beiden Werte entgegennimmt und die Member damit initialisiert.

vector <Point> points;
Point p;
while (istr>>p) points.push_back(p);

Da habe ich dann die Aufgabe falsch verstanden.

z.B.: so

class Point
{
   public:
     Point( int _x, int _y ) : x(_x), y(_y) {}
   private:
     int x;
     int y;
}

cout << "Geben Sie bitte den Namen der Eingabedatei ein:";
string name;
cin >> name;
ifstream istr(name.c_str());
if(!istr)error ("Fehler beim Oeffnen der Eingabedatei ",name);

vector <Point> points;
Point p;
while(istr>>p)points.push_back(p);

cout << "Geben Sie bitte den Namen der ausgabedatei ein:";
string oname;
cin >> oname;
ofstream ostr(oname.c_str());
if(!ostr) error("Fehler beim Oeffnen der Ausgabedatei",oname);

Definieren Sie dazu zuerst den Datentyp Point, mit den beiden Koordinatenmembern x und y.

Ohne das Buch zu kennen, gehe ich stark davon aus das vorher beschrieben wird, wie man eigene Datentypen erstellt (Vermutlich mittels struct oder class).

Da laut google-Books das Kapitel "Klassen und mehr" (8) vorher kommt, solltest du dir dies nochmals anschauen.

Lg freeG

Das heißt das man es so machen soll. Laut meinem Buch (Einführung in die Programmierung mit C++ von Bjarne Stroustrup) ist das geschrieben: Legen Sie ein neues Projekt an, das mit Punkten arbeitet (siehe §10.4).

Ja, und an der Stelle, an der Du jetzt bist steht:

winux schrieb:

Definieren Sie dazu zuerst den Datentyp Pont, mit den beiden Koordinatenmembern x und y.

Wenn Du nicht weißt, dass Du hierfür:

vector <Point> points;
Point p;
while (istr>>p) points.push_back(p);

erst mal einen entsprechenden Datentyp benötigst, dann fang das Buch noch mal von vorne an (ich nehme an, dass das schon erwähnt worden ist; aber ich kenne das Buch nicht).

void f()
{
	ifstream istr;

	class Point 
    { 
    public: 
       Point( int xx, int yy ) : x(xx), y(yy) {}
	   vector <Point> original_points;
	   int x2(){return x;}
	   int y2(){return y;}
    private: 
       int x,y;  
    };
	int x3,y3;
	cout << "Geben Sie sieben (x,y)-Paare ein." << endl;
	while (istr>>x3>>y3){
		Point::original_points.push_back(Point(x3,y3));};
}

int main()
{
    f();

	keep_window_open();
    return 0;
}

Und da sagt der Compiler push_back muss sich eine Klasse/Struktur/Union befinden. Wenn ich aber Point:: weglasse dann sagt der Compiler es findet original_points nicht.

ifstream istr;

	class Point 
    { 
    public: 
       Point( int xx, int yy ) : x(xx), y(yy) {}
	   int x2(){return x;}
	   int y2(){return y;}
    private: 
       int x,y;  
    };
	vector <Point> original_points;
	int x2,y2;
	cout << "Geben Sie sieben (x,y)-Paare ein." << endl;
	while (istr>>x2>>y2){
                if (x2 <= 0 || x2 >= 10000) error("ungueltige eingabe"); 
		original_points.push_back(Point(x2,y2));};

Doch wieso kann man keine werte eingeben?

Hier mal ein Beispiel-Pseudo-Code:

Point.h

//hier definierst du deine Klasse

main.cpp

#include "Point.h"
#include <iostream>
#include <vector>

int main()
{
   vector<Point> original_points;
   for( int i = 0; i < 7; ++i )
   {
     cout << "Bitte geben sie einen x- und einen y-Wert ein: ";
     int x, y;
     cin >> x;
     cin >> y;
     original_points.push_back( Point(x,y) );
   }
}

Lg freeG

Lg freeG

winux schrieb:

Und das habe ich bis jetzt gemacht

Es deutet vieles darauf hin, das du das Buch nochmal von Anfang an intensiv lesen solltest.

1. Deklariere den Datentyp nicht lokal (Es sei den du weist wirklich was du da tust).
2. Der Vector gehört nicht in die Punktklasse hinein, was ganz davon abgesehen, so auch nicht so funktionieren kann.
3. Für die Verwendung des Vectors sollte die Klasse einen Standardkonstruktor (Sprich einen ohne Argumente, oder mit Defaultargumenten) haben.
4. Selbst wenn 2 funktionieren würde, könntest du nicht auf Nicht-Statische Membervariablen ohne eine Instanz zugreifen (Ist dir der Unterschied zwischen Klasse und Objekt [im OO-Sinne] bzw. Instanz klar?).

So jetzt geht alles, danke für deine Hilfe. Denke auch zurückblättern hilft immer weil man auch sehr viel Sachen vergisst. Doch wie der Autor gesagt hat Programmieren lernt man in dem man Code schreibt und nicht wenn man ein Buch liest.

Es stimmt zwar, das man nicht alleine vom Bücher lesen lernt, aber in deinen Fall scheint mir eher zu sein das du das Buch im allerbesten Fall überflogen hast (Und wenn das Kapitel 10 auf die anderen Kapitel übertragbar ist, werden in jedem davon Übungen gemacht).

void f()
{
   vector<Point> original_points; 
   for( int i = 0; i < 7; ++i ) 
   { 
     cout << "Bitte geben sie einen x- und einen y-Wert ein: "; 
     int x, y; 
     cin >> x >> y;
	 if (x  >= 10000 || x <= 0 || y <= 0 || y >= 10000) {cout << "ungueltiger wert" << endl; break;}
     original_points.push_back( Point(x,y) );
	 }
   for ( int i = 0; i < original_points.size(); ++i)
	     cout << original_points[i] << "\n";
}

int main()
{
    f();

	keep_window_open();
    return 0;
}

...Doch wie ich es weiß...

Du rätst dich hier doch nur durch. Ob man dir aktuell helfen sollte wage ich eigentlich zu bezweifeln (da dies nur mehr fragen nach sich zieht, die beim "echten" Lesens des Buches mit Sicherheit nicht auftreten würden.

Nur ein Tip: Der operator<< unterstützt direkt weder den Vector- noch deine Point-Klasse. Du kannst aber durchaus die Einzelwerte jedes Elementes einzeln ausgeben...

Jetzt verstehe was ihr meint. Da der Vektor ein benutzerdefinierte Typ ist hat cout Probleme die Werte auszugeben, die Lösung wäre ein benutzerdefinierter Ausgabeoperator.

Deine Lösung is richtig, dein Verständnis leider nicht;-)
Nicht cout hat irgendein Problem damit, sondern der Compiler, da der <<-Operator so nicht auf deine Klasse angwendet werden kann, weil das ganze weder deklariert noch definiert ist. Musst also wie meine Vorredner schon gesagt haben und du richtig erkannst hast, den <<-Operator für deine Klasse überladen.

Lg freeG

void f()
{
   ostream& operator<<(ostream os, const vector <Point>&  original_points)
   {
	   for ( int i = 0; i < original_points.size(); ++i)
	   return os << original_points[i] << "\n";
   }

   vector<Point> original_points; 
   for( int i = 0; i < 7; ++i ) 
   { 
     cout << "Bitte geben sie einen x- und einen y-Wert ein: "; 
     int x, y; 
     cin >> x >> y;
	 if (x  >= 10000 || x <= 0 || y <= 0 || y >= 10000) {cout << "ungueltiger wert" << endl; break;}
     original_points.push_back( Point(x,y) );
	 }
   cout << original_points;   
}