//einmal die Deklaration in .h 

#ifndef _CDoubleHashing 
#define _CDoubleHashing 

#include <iostream> 
class CDoubleHashing 
{ 
private: 
  CDoubleHashing(); 
  CDoubleHashing(const CDoubleHashing& other); 
  CDoubleHashing operator=(CDoubleHashing& other); 
  static CDoubleHashing m_instanz; 
public: 
  static CDoubleHashing& getInstance(); 
  unsigned int hash(unsigned int I, unsigned int J, unsigned int dict_size, unsigned int attempt); 
}; 

#endif _CDoubleHashing 

//einmal in .cpp 

#include"CDoubleHashing.h" 

CDoubleHashing& CDoubleHashing::getInstance() 
{ 
  return m_instanz; 
} 

CDoubleHashing CDoubleHashing::m_instanz; 

unsigned int CDoubleHashing::hash(unsigned int I, unsigned int J, unsigned int dict_size, unsigned int attempt) 
{ 
  int x = 2/(I+J)*(I+J+1)+J; 

  return ( x + attempt*(1+ ( x%(dict_size-2) ) ) ) % dict_size; 
}

Hier mal der genaue Fehler:
error LNK2019: Verweis auf nicht aufgelöstes externes Symbol ""private: __thiscall CDoubleHashing::CDoubleHashing(void)" (??0CDoubleHashing@@AAE@XZ)" in Funktion ""void __cdecl `dynamic initializer for 'private: static class CDoubleHashing CDoubleHashing::m_instanz''(void)" (??__E?m_instanz@CDoubleHashing@@0V1@A@@YAXXZ)".

EDIT: Konstruktor nicht definiert ...

Edit: Falsch geguckt!
Was du machen kannst, ist (seit C++11) einfach den Konstruktor als default zu markieren. Den Zuweisungsoperator musst du nicht überladen, da es dafür erstmal überhaupt zwei Objekte geben muss...

CDoubleHashing() = default; 
  CDoubleHashing(const CDoubleHashing& other);

Die Methode hash kannst du übrigens auch static machen.

Diese werden aber automatisch für dich definiert, daher ist das unnötig, lass diese Deklarationen vorerst weg.

Aber der Kopierkonstruktor und Zuweisungskonstruktor soll ich in diesem Fall deklarieren und nicht definieren, damit der Compiler nicht zwei eigene erstellt. So steht es in meinem Skript.

Es geht in dieser Klasse ums Hashing, dazu sollen I und J zu einer Zahl gemacht werden (hier: x) und dann durch diese Double-Hashing Funktion gehen, aber irgendwie funktioniert das ganze nicht... Weiß einer vllcht. wieso?

Aber der Kopierkonstruktor und Zuweisungskonstruktor soll ich in diesem Fall deklarieren und nicht definieren, damit der Compiler nicht zwei eigene erstellt. So steht es in meinem Skript.

Sone hat einen Schritt weiter gedacht.

Weiß einer vllcht. wieso?

Ist das immer noch die Ausgangsfrage, oder wie? Ich weiß nicht, warum Sone sich korrigiert hat, der Standard-ctor ist deklariert und nicht definiert, das ist nach wie vor das Problem. Also Deklaration weglassen oder Constructor auch implementieren.

Falls das nicht die neue Frage ist, gewöhn Dir ganz schnell ab Aussagen wie

irgendwie funktioniert das ganze nicht

zu machen. Handfeste Erklärungen, Fehlermeldungen, Beschreibungen sind angebracht. So was ist für alle Leser und für Dich Zeitverschwendung.

Aknayirp

Du könntest jedoch schauen, ob 2/(I+J) nicht 0 für deinen Fall ergibt. Dann wird der gesamte Summand nämlich null und x ist J. Es wird nicht automatisch zu einem Fließkommawert gecastet. Wenn Du das möchtest, solltest Du zwischendurch ein static_cast<float>(wert) einbauen oder 2.0 bzw. 2.0f statt 2 schreiben.

Hat sich erledigt Aber die 2 war das Problem... Hatte einfach die Formel nicht richtige hingeschrieben. Aber jetzt passt es.

2*(I+J+1)/(I+J)+J

(zuerst multiplizieren und dann dividieren)

Generell erscheint mir die Hashfunktion aber sehr eigenartig. Was sollen I und J denn überhaupt darstellen?

Hier aber nicht zu float casten, sondern die Berechnung entsprechend umstellen:

2*(I+J+1)/(I+J)+J

(zuerst multiplizieren und dann dividieren)

Generell erscheint mir die Hashfunktion aber sehr eigenartig. Was sollen I und J denn überhaupt darstellen?

Das ist ja auch nicht direkt die Hashfunktion, soweit ich das verstanden habe. Das ist die Cantorische Paarungsfunktion(glaub so heißt es). Aber ja wie gesagt, die zwei war einfach falsch plaziert. Jetzt funktioniert alles richtig.

Ich weiß nicht, warum Sone sich korrigiert hat, der Standard-ctor ist deklariert und nicht definiert, das ist nach wie vor das Problem.

Nein nein, ich habe zuerst gedacht, er wollte einfach alle drei Funktionen zur Verfügung stellen. Dann ist mir aufgefallen dass er ja alle private deklariert hat - Singleton...

Mal ganz blöd gefragt: Warum den ganzen Aufriss mit der Klasse? Du könntest das auch einfach als Funktion hinschreiben. CDoubleHashing hat ja keine Datenmember.

Soll so gemacht werden, da eine Aufgabe aus der Uni