Ich habe ein Linker Problem. Ich habe minimal eine Klasse Network erstellt und will diese in meiner main Funktion nutzen.

Meine main Funktion:

#include <iostream>
#include "Network.h"

using namespace std;

int main(int argc, char** argv) {

Network network; // Declare an object network with type Network
cout << "Neurons: " << network._AdSize << endl;

return 0;
}

Meine Klasse Network:

Die Network.h Datei:

#ifndef NETWORK_H
#define NETWORK_H

// A class for the network topology
class Network {
  public:
   //Constructor
   Network(); 

   // Number of Neurons
   static int _AdSize;
};

#endif  // NETWORK_H

Und die Network.cpp Datei

#include "Network.h"

// Constructor for the Network class.
Network::Network() {
  _AdSize = 8;
}

Ich erhalte bei dem Befehl

g++-5 -std=c++14 -Wall -g Main.cpp -o Programm

undefined reference to `Network::Network()
undefined reference to `Network::_AdSize'
collect2: error: ld returned 1 exit status

In der Network.cpp ist der Header inkludiert und in der main Funktion auch. Wo ist der Fehler?

Danke und Grüße

int Network::_AdSize = 8; 

Network::Network() { 
}

Vermutlich sollte dieser Member auch const sein. Ausserdem sind Member-Bezeichner die mit Underscore und Grossbuchstaben beginnen problematisch, weil sie für den Compiler reserviert sind (etwas salopp ausgedrückt).

// Edit

Oder aber der Member sollte nicht static sein und du hast das nur gemacht, weil dir nicht bekannt war, was es genau bedeutet oder dir nicht klar war, wie du anschliessend darauf zugreifen kannst.

undefined reference to `Network::Network()'

bleibt leider trotzdem

Der Fehler

undefined reference to `Network::Network()'

bleibt leider trotzdem

Um diesen Fehler zu korrigieren musst du noch die Network.cpp Datei mitkompilieren.

// Edit

g++-5 -std=c++14 -Wall -g Main.cpp Network.cpp -o Programm

Estmal Danke, es läuft. Eine Fragen habe ich dazu noch:

Ich habe nun in der header Datei der Klasse Network

const int _AdSize = 4;

gesetzt und in der cpp Datei zu Network den Konstruktor leer gelassen.

In meinem Skript steht, dass man Membervariablen in der header Datei nicht einen Wert zuweisen soll, sondern die Zuweisung der Werte über den Konstruktor entsteht. Das ist ja nun im Widerspruch zu dem was ich gemacht habe. Aber es läuft... ist es trotzdem okay obige Zeile in die header Datei geschrieben zu haben? Was ist denn da guter Code, Zuweisung im Konstruktor oder in der Klasse und warum?

Grüße

Das steht das man es nicht soll. Leider ohne weitere Begründung.

Dein Programm ist nach diesem Skript aber nicht korrekt im Sinne der selbst aufgestellten Regeln.

http://www.cplusplus.com/forum/beginner/166090/#msg836782

Jetzt weist er die Variable in der Main.cpp zu. Jetzt bin ich doch verwirrt.

Geht es da evtl. darum wann der Schreiber des Codes bestimmen möchte wann Speicher zugewiesen wird? Beim Konstuktor bekommt der Compiler ja erst die Anweisung Bytes zu reservieren, wenn er zu Laufzeit aufgerufen wird und gibt sie später mit dem Destruktor wieder frei und bei einer Initialisierung von Anfang an ist der Speicher fest belegt. Geht es darum?

Und wieso wird in dem Post oben in der main Datei und nicht in dem Header initialisiert?

Danke und Grüße

constexpr int _AdSize =4;