probleme c+ code

Druckbare Version

01-02-2007, 18:52
Tuxist

probleme c+ code

Moin,
Versuche gerade eine cms mit c++ zu schreiben versuche jetzt funktionen in shared libary's zu schreibe, dies ist für mich neuland.

Mein c++ code

index.cpp

Code:

#include <iostream> #include <string> #include <fstream> #include "index.h" #include <dlfcn.h> #include "lib/libcss/include/css.h" using namespace std; int main() { dlopen("lib/libcss/libcss.so", RTLD_LAZY); cout << "Content-type: text/html" << endl << "<html>" << endl << "<head>" << endl << "<title>Wikommen auf Tuxist Mainbase</title>" << endl << "</head>" << endl; cout << cssheader; cout << "<body>" << endl << "<h1><em>" << endl << "Im Aufbau" << endl << "</em></h1>" << endl << "</body>" << endl << "</html>"; }

css.cpp

Code:

[/CODE#include <iostream> #include <fstream> #include <string> using namespace std; class libcssread { public: char cssheader; ifstream cssread; string csspath; }; int main() { char cssheader; ifstream cssread; string csspath; cssread.open(csspath.c_str()); while (cssread.eof() !=true) { cssheader = cssread.get(); cssread.close(); } } ]

css.h (css header)

Code:

class libcssread { public: char cssheader; ifstream cssread; string csspath; };

g++ output

Code:

make /usr/bin/gcc -Wall -fsigned-char -Ilib/libcss/include -Llib/libcss index.cpp lib/libcss/include/css.h:5: error: ‘ifstream’ does not name a type lib/libcss/include/css.h:6: error: ‘string’ does not name a type index.cpp: In function ‘int main()’: index.cpp:22: error: ‘cssheader’ was not declared in this scope make: *** [all] Fehler 1
01-02-2007, 20:05
TheHawk

Die Typen ifstream und string, welche in der css.h verwendet werden, gibt es nicht. Und die Variable cssheader in der index.cpp wurde in der Funktion main() nicht deklariert.
01-02-2007, 20:34
Tuxist

so die type fstream und string wurden mit #include <fstream> und # <string> bennant.
Und libcssread wurde alls klasse in css.h bennant und css.cpp verarbeitet.
Daher frage ich mich ?
css.h muss eigentlich kein #include stehen oder ?
warum kann von der klasse einzelne funtionen in der int main() nicht verwenden ?
Denn css.h dien leglich als api header für die libcss.
02-02-2007, 14:44
panzi

Zitat:

Zitat von Tuxist

so die type fstream und string wurden mit #include <fstream> und # <string> bennant.
Und libcssread wurde alls klasse in css.h bennant und css.cpp verarbeitet.
Daher frage ich mich ?
css.h muss eigentlich kein #include stehen oder ?
warum kann von der klasse einzelne funtionen in der int main() nicht verwenden ?
Denn css.h dien leglich als api header für die libcss.

Das hier ist C++, nix hihglevel artiges wie Java. Jeder der mit einen Objekt was macht, muss es kennen. Kennen tut er es nur wenn er Deklarationen dazu hat. Jeder, der's verwendet, muss die .h file includieren, sonst kennt ers net.

Und was machst mit dem dlopen? Da öffnest eine shared lib. und vergisst den handel drauf gleich wieder? Und dlopen ist für C geschrieben, nicht C++. D.h. du musst um C++ Funktionen extrahieren zu können entweder den gemangelten Namen wissen oder die Funktion als extern "C" deklarieren. dlopen ist wohl nur für plugins gut, ansonsten gibt's IMHO keinen Grund das in C++ zu verwenden.

Zur Verwendung von dlopen siehe:
http://www.die.net/doc/linux/man/man3/dlopen.3.html
02-02-2007, 20:07
anda_skoa

Zitat:

Zitat von Tuxist

so die type fstream und string wurden mit #include <fstream> und # <string> bennant.

In css.cpp und dort ist die Deklaration der Variablen auch erst nach dem

using namespace std

sonst gäbe es dort auch ein Problem.

Du kannst das in index.cpp auch machen, wenn du vor dem include von css.h den Namespace öffnest.

Normalerweise wird das include für direkte Instanzen, also wenn es keine Pointer oder Referenzen sind, direkt in der Header Datei gemacht, die die Deklaration enthält, in deinem Falle also css.h

Zitat:

Und libcssread wurde alls klasse in css.h bennant und css.cpp verarbeitet.

Der Compiler hat dort auch keinen Fehler gemeldet, sondern in index.cpp

css.cpp kann vermutlich einwandfrei kompiliert werden.

Zitat:

Daher frage ich mich ?
css.h mus
s eigentlich kein #include stehen oder ?

Korrekt, müssen nicht. Meistens macht man das, damit man diese includes nicht in jeder Datei wiederholen muß, die den Header (in deinem Fall css.h) inkludiert.

Aber wie oben gesagt kannst du alternativ vor dem inkludieren von css.h alle dort verwendeten Typen auch direkt in index.cpp inkludieren und entweder die Namespaces mit angeben oder die Namespaces öffnen.

Raten würde ich aber eher dazu, diese Typen gleich in css.h zu erledigen, d.h. in etwas so

Code:

#include <fstream> #include <string> class libcssread { public: char cssheader; std::ifstream cssread; std::string csspath; };

(Natürlich die Include Guards nicht vergessen)

Zitat:

warum kann von der klasse einzelne funtionen in der int main() nicht verwenden ?

Deine Klasse hat erstens keine Funktionen, sondern nur Daten, und du hast in der main Funktion in index.cpp keine Instanz dieser Klasse.

Etwas nicht vorhandenes kann nicht benutzt werden, ist in der Relalität meistens auch so ;)

Zitat:

Denn css.h dien leglich als api header für die libcss.

Ja, das ist auch ok.

Was neben dem unverständlichen dlopen() Aufruf, den panzi schon behandelt hat, noch aufgefallen ist, ist daß du zwei mal int main() definiert hast, was auf keine Fall linken wird.

Außerdem sieht der Compiler Aufruf nicht korrekt aus, dort ist gcc statt g++ zu sehen, was zwar den C++ Code noch einwandfrei kompilieren wird, aber im Linker Schritt nicht die C++ Standardbibliothek mit nimmt und daher mit einem Fehler abbrechen wird.

Ciao,
_
03-02-2007, 12:42
Tuxist

danke deine Antwort hat mir sehr geholfen :)

habe jetzt einiges geändert:
css.cpp

Code:

#include <iostream> #include <fstream> #include <string> using namespace std; void libcssread(void) { char cssheader; ifstream cssread; string csspath; cssread.open(csspath.c_str()); while (cssread.eof() !=true) { cssheader = cssread.get(); cssread.close(); } }

css.h

Code:

#include <fstream> #include <string> using namespace std; void libcssread(void) { char cssheader; std::ifstream cssread; std::string csspath; };

index.cpp

Code:

#include <iostream> #include "index.h" #include <dlfcn.h> using namespace std; #include "lib/libcss/include/css.h" int main() { cout << "Content-type: text/html" << endl << "<html>" << endl << "<head>" << endl << "<title>Wikommen auf Tuxist Mainbase</title>" << endl << "</head>" << endl; libcssread(); cout << "<body>" << endl << "<h1><em>" << endl << "Im Aufbau" << endl << "</em></h1>" << endl << "</body>" << endl << "</html>"; }

g++ output:

Code:

/usr/bin/g++ -Wall -fsigned-char -Ilib/libcss/include -Lbin/lib/libcss -llibcss index.cpp lib/libcss/include/css.h: In function ‘void libcssread()’: lib/libcss/include/css.h:8: warning: unused variable ‘cssheader’ /usr/bin/ld: cannot find -llibcss collect2: ld returned 1 exit status make: *** [all] Fehler 1

makefile index.cpp

Code:

CXX= /usr/bin/g++ CXXOPTFLAGS= CXXFLAGS=$(CXXOPTFLAGS) -Wall -fsigned-char -Ilib/libcss/include -Lbin/lib/libcss -llibcss lib_SRC= \ index.cpp all: $(CXX) $(CXXFLAGS) $(lib_SRC)
03-02-2007, 14:59
anda_skoa

Sehr fein.

Ein paar weitere Anmerkungen:

libcssread() ist jetzt eine Funktion. Eine Funktion brauchst du im Header lediglich zu deklarieren, sie sollte keinen Body (Implementierung) haben, wenn du diese Implementierung auch noch in einer anderen Datei hast.

Das führt nämlich dazu, daß die Funktion zweimal vorhanden ist und dann der Linker auch wieder ein Problem hat.

In css.h reicht jetzt das

Code:

void libcssread();

Am besten du benutzt css.h auch in css.cpp, dann hast du schon beim Erstellen der Bibliothek diese Doppeldeutigkeit als Fehler und siehst sie nicht erst bei einem Programm, das die Bibliothek benutzt.

Allerdings gehe ich davon aus, daß die mit den Daten auch noch was machen willst, im Moment, sind sie nur während der Abarbeitung von libcssread() gültig und die hat keinen Rückgabewert oder Ausgabeparameter.

Dein aktueller Linkerfehler sagt, daß die Bibliothek liblibcss.a bzw. liblibcss.so nicht gefunden wurde.
Vermutlich heißt deine Bibliothek libcss, daher ist bei der Linkerangabe nur das hier nötig

Code:

-lcss

Das liegt daran, daß der Basisname einer Bibliothek über Plattformen hinweg gleich ist, aber der tatsächliche Dateiname anders aussehen kann.

zB wird eine statische Bibliothek mit dem Basisnamen "css" unter Unix "libcss.a" heißen, unter Windows würde sie "css.lib" heißen.

Ciao,
_

P.S.: noch etwas zu Namespaces

Code:

#include <string> using namespace std; std::string foo;

Ist doppelt gemoppelt :)

In einem Header schreibt man eher so

Code:

#include <string> std::string foo;

Damit man dem Benutzer des Headers nicht dazu zwingt, den gesamten Namespace "std" geöffnet zu haben.

In einer Quelldatei ist es meist Geschmackssache, als Mittelweg macht man dort meistens das hier

Code:

#include <string> using std::string; string foo;
03-02-2007, 15:56
Tuxist

danke schön