C++ Klasse in einer Klasse, Deklarationsfehler

10rotator01

10rotator01

Grünschnabel
Hallo Leute,

ich habe folgendes Problem. Ich habe ein Klasse geschrieben, die ich innerhalb anderer Klassen verwenden möchte.
Soweit so gut, es erkennt auch die Klasse, aber beim Kompilieren meint der Compiler, das einiges nicht deklariert sei:

Also:
Header 1:
Code: 
#pragma once
class point
{
private:
    int pos_x;
    int pos_y;

public:
    point();
    point(int x, int y);
	void setCoordinates(int x, int y);
};

CPP 1:
Code: 
#include "point.hpp"


point::point(void)
{
	pos_x = 0;
    pos_y = 0;
}

point::point(int x, int y)
{
	pos_x = x;
    pos_y = y;
}
void point::setCoordinates(int x, int y)
{
	pos_x = x;
	pos_y = y;
}

Header 2:
Code: 
#pragma once
class rob
{
private:
	point *pos;

public:
	rob(int x, int y);
	void moveTo(int x, int y);
};

CPP 2:
Code: 
#include "rob.hpp"
#include "point.hpp"


rob::rob(int x, int y)
{
	pos = new point(x, y);
}

void rob::moveTo(int x, int y)
{
	pos->setCoordinates(x, y);
}

Der Compiler spuckt mir folgendes aus:
Fehler 3 error C2065: 'pos': nichtdeklarierter Bezeichner f:\programmierung\c++\labrob1\labrob1\rob.cpp 7 1 labrob1
Und verweist mir auf die beiden pos im CPP2.

Ich habe jetzt schon eine Weile probiert, aber ich komme nicht mehr weiter.
Mein Problem wäre auch gelöst, wenn mir jemand sagen könnte, ob es in C++ die vordefinierte Klasse Point gibt wie in C#.
hab danach gegoogelt, aber nichts gefunden.

lg,
10rotator01

Nachtrag:
ok, habe meinen Fehler jetzt doch gefunden...
Ich war so schlau und habe die Headerfiles in der falschen Reihenfolge inkludiert...
 
Zuletzt bearbeitet:
Hi

warum machst du kein include "point.hpp" in der rob.hpp?

So auf die Reihenfolge achten zu müssen macht schwer änderbaren Code.
 
Hi,

ich weißt nicht wo, aber ich hab mal gelesen, bei Klassen sollte man die includes in der cpp halten, weiß aber auch nicht mehr warum...

Inwiefern macht das einen Unterschied?

lg
 
Hö? Man inkludiert in den .cpp-Dateien die zugehörigen .h(pp)-Dateien, in den .h(pp)-Dateien werden die Header für die .cpp-Datei inkludiert. Zumindest mache ich das so.

Beispiel:
C: 
//a.hpp
#ifndef A_H
#define A_H
#include <stdio.h>
//usw.

class a
{
    //blabla
};
#endif


//a.cpp

#include "a.hpp"

a::a()
{

}
//blabla

Gruss
cwriter

//EDIT: Ja, lese ich auch immer wieder. Ich verstehe aber nicht, warum. Meist muss man ja für die Typdefinitionen sowieso ein paar Header in der Headerdatei inkludieren, da ist es m.M.n. einfacher, einfach alles in eine Datei zu schreiben.
Unterschied macht es - bis auf einige wenige Fälle - keinen.
 
Zuletzt bearbeitet:
Technisch gesehen machts überhaupt keinen Unterschied.
Aber in einem Projekt mit, Hausnummer, 100 Codedateien wird man froh sein,
nicht jedesmal die Reihenfolge überlegen zu müssen.

Verarbeitet muss die point.hpp sowieo werden, egal ob es wegen der rob.hpp oder rob.cpp ist.
Man spart dem Compiler dadurch auch nichts.
Und falls die Datei zweimal inkludiert ist: Dafür ist dann das pragma-once
Verhindert mehrmalige Abarbeitung; bzw. der Compiler merkt sich, welche Dateien er schon hat.

edit:
Gibt natürlich verschiedene Meinungen dazu, wie zu so vielen Sachen im Programmierbereich.
Persönliche Vorgehensweise: Eine Headerdatei überall dort inkludieren,
wo etwas aus dieser Headerdatei verwendet wird.
Überall = Egal ob Cpp, hpp...
 
Hallo 10rotator01,

da du in deinem Header 2 die point-Klasse als Zeiger verwendet hast, würde auch eine sogenannte Vorwärtsdeklaration der point-Klasse reichen und du kannst das Include in der cpp belassen.
C++: 
#pragma once

class point; // Vorwaertsdeklaration

class rob
{
private:
    point *pos;
 
public:
    rob(int x, int y);
    void moveTo(int x, int y);
};
Gruß
MCoder
 
Hallo,

ich denke die Sache mit "... bei Klassen sollte man die includes in der cpp halten ... " macht am meisten Sinn, bei nicht-open source Projekten wo nur so wenig an Klassen-Layout preisgegeben werden soll wie möglich. Dann muss man Header-Dateien nicht mit raus geben, wenn sie nur von internen Klassen includiert werden die zum Beispiel in einer ausgelieferten Library liegen. Dann kann man auch die Übersicht für Kunden erleichtern und bläht auch die API Dokumentation (falls es eine gibt) nicht mit Funktionen/Methoden etc. auf, auf die der Kunde eigentlich eh keinen Zugriff haben sollte.

Gruß,
Der Wolf
 
Um antworten zu können musst du eingeloggt sein.

Neue Beiträge

Zurück