ich versuch mich grad an nem Funktionszeiger, aber das will nicht so wie ich das will. Ich hab eine Klasse CUtils, die einige Methoden zur Verfügung stellt, die andere Klassen benutzen können. Eine dieser Methoden soll als Parameter einen Funktionszeiger bekommen, der dann in der Methode aufgerufen wird. Das Ganze sieht so aus:

typedef void (*FileHandler)(const char*);

class CUtils{	
public:
    static void searchJPGFiles(const char* path, bool searchSubDir, FileHandler callback);		
};

void CUtils::searchJPGFiles(const char* path, bool searchSubDir, FileHandler callback){
    string tmpPath = "";
    /*...*/
    callback(tmpPath.c_str());
    /*...*/
}

Jetz möchste ich diese Funktion in eine meiner Klassen nutzen:

void CexifDataInput::inputCamModelFileHandler(const char *path){
    /*...*/
}

void CexifDataInput::inputCamModel(string &camModel){
    /*...*/
    CUtils::searchJPGFiles(((Mode2Data*)m_inputData->modeData)->source.c_str(), m_inputData->searchSubDir, inputCamModelFileHandler);
    /*...*/
}

Das funktioniert aber so nicht und es er spuckt mir folgenden Fehler aus:

cannot convert ‘void (CexifDataInput::*)(const char*)’ to ‘void (*)(const char*)’ in initialization

Wenn inputCamModelFileHandler eine normale Methode ist, und nicht zur Klasse CexifDataInput gehört, dann funktioniert der Code, sobald inputCamModelFileHandler eine Methode der Klasse ist funktioniert es nicht mehr. Wie kann ich das beheben?

MfG & Thx Bergmann

das is in der Tat ziemlich hässlich. Ich habs jetzt anders gelöst. Ich hab ein Interface geschrieben, das jede Klasse die die Suche ausführt implementieren muss. Das geht hier zum Glück, da jede Klasse nur eine Methode hat, die nach der Suche aufgerufen werden muss. Wären es mehrere Methoden dann würd ich wohl um diese hässliche Lösung nicht drum rum kommen ^^

MfG & Thx Bergmann.

Hey,

das is in der Tat ziemlich hässlich. Ich habs jetzt anders gelöst. Ich hab ein Interface geschrieben, das jede Klasse die die Suche ausführt implementieren muss. Das geht hier zum Glück, da jede Klasse nur eine Methode hat, die nach der Suche aufgerufen werden muss. Wären es mehrere Methoden dann würd ich wohl um diese hässliche Lösung nicht drum rum kommen ^^

Wieso? Dann hätte das Interface eben 2 oder mehr pur virtuelle Methoden, die implementiert werden müssen

@pumuckl: das geht eben nich so einfach, weil dann die search-Methode nicht weis, welche Methode des Interfaces sie aufrufen muss. Da müsste mann dann noch nen Parameter oder so mitgeben um das zu lösen.
@Nexus: in den Links oben hab ich nix von std:funktion gesehen, oder ich hab da noch was nich ganz verstanden ^^

MfG Bergmann.

@pumuckl: das geht eben nich so einfach, weil dann die search-Methode nicht weis, welche Methode des Interfaces sie aufrufen muss. Da müsste mann dann noch nen Parameter oder so mitgeben um das zu lösen.

Dann hab ich dich wohl missverstanden. Ich dachte du meinst, dass evtl. an mehreren Stellen in deiner Funktion callbacks aufgerufen werden müssen, also quasi Callback1 bis CallbackN. Dann müsste dein Interface eben N Methoden haben, und die Funtion sähe so aus:

void doSomething(Arg..., interface* if)
{
  /* . do 1 . */
  if->Callback1(xy);

  /* ... */
  if->Callback2(yz); 

  /* usw */
};

ne die Methode Search sucht nach JPGs im übergebenen Ordner und soll ein Callback aufrufen, wenn eine JPG gefunden wurde. Wenn jetzt eine Klasse mehrere Methoden mit Suchanfragen hat, dann brauch sie auch mehrere Methoden die die gefundenen Datein auswertet. Und dann weis die Search-Methode nicht welche Interface-Methode sie aufrufen soll...

MfG Bergmann.

ne die Methode Search sucht nach JPGs im übergebenen Ordner und soll ein Callback aufrufen, wenn eine JPG gefunden wurde. Wenn jetzt eine Klasse mehrere Methoden mit Suchanfragen hat, dann brauch sie auch mehrere Methoden die die gefundenen Datein auswertet.

Nö. sie braucht nur eine Callbackmethode, die im Zweifel anhand einer Klassenvariable rausfinden kann, welche "version" des callbacks denn grade gemeint ist.

/edit: allerdings kommt mir das Design etwas spanisch vor. Kannst du mal einen etwas weiteren Überblick geben, wer da was macht bei deinen Klassen/objekten?

ich hab ne Klasse Utils, die stellt verschiedene Methoden zur Verfügung die vom Programm in allen Bereichen heufig benutzt wird. Darunter auch die searchJPGFiles-Methode.
Dann hab ich eine Klasse die sich um die Dateneingabe kümmert, mit einem Menü. In dem Menü kann der User allerhand Sachen einstellen nach denen gesucht werden soll, bzw. welche JPGs bearbeitet werden sollen. Dort kann er z.B ein CameraModel festlegen mit der die JPGs aufgenommen wurden. Wenn er das Model aber nicht 100%tig weis, dann wird eine Suche in dem Ordner gestartet und alle CameraModele aus den exif-Daten der JPGs ausgelesen und dem User angezeigt. Die Suche übernimmt natürlich die searchJPGFiles-Methode und die Auswertung der Daten soll dann die Callback-Methode machen.
Weiterhin gibt es eine Klasse, die die gewünschten Operationen dann ausführt. Die startet auch eine Suche über die searchJPGFiles-Methode und führt in ihrer Callback-Funktion die gewünschten Aktionen aus (Datein umbennen, kopieren, verschieben, ...).
So ist die Suche universell einsetzbar und kann über die Callbacks von fast jeders Methode genutzt werden. Nur leider gingen die Callbacks nich so wie ich wollte (oder ich habs noch nich verstanden) und deshalb bin ich notgedrungen auf ein Interface umgestiegen...

MfG Bergmann.

1. searchJPGs macht genau was der Name sagt - es sucht nur die jpgs und gibt einen Container voller Dateihandles/Referenzen (in welcher Form auch immer, vielleicht auch nur filenamen) zurück.
2. Über das Ergebnis wird iteriert, und zwar mit einem Funktor, der solch ein Filehandle nimmt und damit *irgendwas* macht. Das *irgendwas* kann eben öffnen + kamerainfo lesen sein, oder öffnen + bearbeiten etc.

Wenn du Dateien immer öffnest und was tust, kannst du das erreichen, indem du den Funktor in schritt 2 aus einem Fileopener + Fileprocessor zusammensetzt - wobei der Fileprocessor wieder generisch irgendein Funktor sein kann, der auf eine geöffnete jpg-Datei losgelassen wird. Templates lassen grüßen
Auf jeden Fall würde ich die Kopplung durch das Interface und Ableitungen etc. möglichst vermeiden.

das mit den Functoren hab ich gecheckt (denke ich). Diese Seite hat mir sehr dabei geholfen: http://www.newty.de/fpt/functor.html
Jetz würd ich aber gern noch wissen wie das mit dem std:function geht. Meine IDE (CodeLite) zeigt mit das bei der Codevervollständigung an, aber der Compiler (GNU G++) sagt folgendes:

error: ‘function’ in namespace ‘std’ does not name a type

MfG Bergmann.

ich hab g++ 4:4.5.2-1ubuntu3. std::tr1::function gibts auch nicht. Sin die auch erst ab v4.6 dabei? Die Boost-Librarie wollt ich umgehen, da ich das ganze noch meinem Prof schicken muss und da sind haufen extra Libs nich so gut.
Ich werd jetzt auch die Functoren mit nem Template nutzen. Ich wollt halt nur rein Intresse halber wissen, warum er das mit den functions bei mir net findet...

MfG Bergmann.

ich hab g++ 4:4.5.2-1ubuntu3. std::tr1::function gibts auch nicht. Sin die auch erst ab v4.6 dabei? Die Boost-Librarie wollt ich umgehen, da ich das ganze noch meinem Prof schicken muss und da sind haufen extra Libs nich so gut.

die boost libraries sind fast standard. Frag am Besten deinen Prof, ob er sie hat.

das mit den Functoren un den Templates läuft jetzt. Jetzt hab ich nur noch eine kleine Frage. Und zwar hab ich den Code des Templates vollständig im Header unter gebracht. Dort arbeitet er auch. Aber sobald ich den eigentlichen Code in der .cpp unterbringen will schmeißt er mir cryptische Fehler aus.
So sieht der Code aus (.h):

class IFileHandlerFunctor{
public:
	virtual void operator()(const char* path) = 0;
	virtual void call(const char* path) = 0;	
};

template <class TClass> class TFileHandlerCallbackFunctor: public IFileHandlerFunctor{
private:
	void (TClass::*_callback)(const char*);
	TClass *_obj;

public:
	virtual void operator()(const char* path){
		(*_obj.*_callback)(path);
	};

	virtual void call(const char* path){
		(*_obj.*_callback)(path);
	};

	TFileHandlerCallbackFunctor(TClass *obj, void(TClass::*callback)(const char*)){
		_obj = obj;
		_callback = callback;
	};
};

das hab ich probiert (*.cpp, natürlich die *.h vorher angepasst):

template<class TClass> void TFileHandlerCallbackFunctor<TClass>:: operator()(const char* path){
	(*_obj.*_callback)(path);
}

template<class TClass> void TFileHandlerCallbackFunctor<TClass>::call(const char* path){
	(*_obj.*_callback)(path);
}

/**Konstruktor
 * @obj: Objekt das die Callback Methode besitzt
 * @callback: Funktionszeiger auf Callback-Methode des Objekts 
template<class TClass> TFileHandlerCallbackFunctor<TClass>::TFileHandlerCallbackFunctor(TClass* obj, void(TClass::*callback)(const char*)){
	_obj = obj;
	_callback = callback;
}

Fehler: Die Template-Klasse an sich lässt sich ohne Probleme compilieren, doch wenn ich das Ganze dann auch benutzen will kommt dieser Fehler:

undefined reference to `TFileHandlerCallbackFunctor<TexifDataInput>::TFileHandlerCallbackFunctor(TexifDataInput*, void (TexifDataInput::*)(char const*))'

So sieht der Aufruf zu dem Fehler aus:

TFileHandlerCallbackFunctor<TexifDataInput> callback(this, &TexifDataInput::searchJPGFileHandler);
TUtils::searchJPGFiles(((Mode2Data*)_inputData->modeData)->source.c_str(), _inputData->searchSubDir, &callback);

Das seltsame is, das dieser Code funktioniert, wenn ich den Code ausschließlich in die *.h schribe. Sobald ich ihn in die *.cpp packe (wo er ja normalerweise stehen sollte) meckert er rum -.-
Kann sich das jmd erklären?

MfG Bergmann.

Das seltsame is, das dieser Code funktioniert, wenn ich den Code ausschließlich in die *.h schribe. Sobald ich ihn in die *.cpp packe (wo er ja normalerweise stehen sollte) meckert er rum -.-
Kann sich das jmd erklären?

Diese Regel gilt für normale Funktionen, allerdings nicht für Templates. Bei einem Template muß der Compiler die komplette Definition sehen wenn er es instanziiert (im Hauptprogramm) - und das geht nur, wenn die Definition direkt im Header steht.
(eine Alternative wäre noch eine .impl Datei, die du am Ende des Headers einbindest)

wenn das so üblich is, dann lass ich das im Header. Ich wollt dann bloß kein Punktabzug bekommen, weil ich mich nich an die Styleregeln gehalten hab.
Damit wären dann alle Fargen beseitigt. Danke an alle für die Mühe

MfG & Thx Bergmann