compiler findet template fnk. nicht

solito

thx @ all

deklariere und definiere die template funktion nun im header so ist es ok.
alles andere will irgendwie nicht.

btw: wenn ich meinen post noch mal in der vorschau lese, log ich mich rel. schnell aus. kann ich das abschalten? finde keine einstellung dafür.

pumuckl

ähm google mal nach includeguards... Dein compiler liest folgendes:

#include string_fnc.h //ok machen wir
  #include string_fnc.cpp // na jut
    #include string_fnc.h // schon wieder? was solls
      #include string_fnc.cpp // ähm...
        #include string_fnc.h // ich glaube da stimmt was nicht...

du merkst es nach 6 Zeilen - Compiler sind geduldig und tun was du ihnen sagst, bis sie irgendwann (so nach 1000 includes oder mehr) nicht mehr können und das Handtuch werfen.

Die Lösung bei templates ist wie folgt:
- um den Header (wie um jeden anderen auch) einen include-guard machen
- die .cpp in der .h am ENDE includieren (sonst würde der compiler zu recht meckern, dass du die funktionen definierst bevor du sie deklariert hast)
- in der .cpp (die vielleicht wirklich besser .tpl oder so heißen sollte ums klar zu machen) KEIN include der .h

mikey

solito schrieb:

keyword ‘export’ not implemented, and will be ignored,

Welchen Compiler hast du denn?

asc

mikey schrieb:

asc schrieb:

alternativ die cpp includen...

Noch eine Alternative wäre das Schlüsselwort export, welches dem Compiler vorgaukelt, dass die Templatedefinitionen in der selben Datei stehen. Allerdings wird dieses Feature von den wenigsten Compilern unterstützt. (Von welchen, kann ich nicht sagen)

Diese Alternative habe ich nicht vorgeschlagen weil es meiner Kenntnis nach nur einen einzigen Compiler gibt der dieses Feature unterstützt: Comeau C++
(Was auch der Compiler mit der höchsten Standardkonformität sein soll)

cu André

mikey

Okay, du hast Recht mit dem Comeau C++ Compiler:
http://de.wikipedia.org/wiki/Comeau_C%2B%2B

Dann waren das also die jenigen, die stolze 5 Jahre an diesem export-Feature programmiert haben.

solito

mikey schrieb:

solito schrieb:

keyword ‘export’ not implemented, and will be ignored,

Welchen Compiler hast du denn?

gcc version 4.1.2

solito

wenn jmd noch etwas geduld hätte wäre das super
(habe die trim_str() funktion mal rausgenommen)

string_fnc.h

#include <sstream>
#include <stdexcept>
#include <string>

#ifndef string_fnc_h_INCLUDED
#define string_fnc_h_INCLUDED

template<typename T>
T to_numeric  (const std::string& data);

#include "string_fnc.cpp"

#endif

string_fnc.cpp

#ifndef string_fnc_h_INCLUDED
#define string_fnc_h_INCLUDED
#include "string_fnc.h"
#endif

using namespace std;

template<typename T>
T to_numeric(const string& data)
{
    istringstream str(data);
    T val;
    str >> val;

    if (!str)
    {
        throw invalid_argument("string to numeric failed - invalid argument");
    }

    return val;
}

es functioniert - endlich... ist doch korrekt so?

und würdet ihr die datei auch wenn sie nicht nur templates enthält nach .tpl umbenennen oder gar
eine neue .tpl datei erstellen und diese dann noch in die cpp-datei einbinden.
wird das dann nicht zu unübersichtlich falls sich doch mal jmd. anderes den code anschaut?

asc

solito schrieb:

wenn jmd noch etwas geduld hätte wäre das super
(habe die trim_str() funktion mal rausgenommen)

Ich persönlich setze Templates zwar nur in Headern ein, aber das ist Geschmackssache. Grundlegend würde ich dir aber folgendes raten:

1. using namespace gehört niemals in Dateinen die direkt includiert werden (Sprich: Header auf der einen Seite und Template-Sourcefiles).
2. nur das includieren was wirklich an der Stelle nötig ist.
3. Includeguard müssen einmalig gewählt werden

Meine Variante deines Codes (der an sich ungeprüft ist):

string_fnc.h

#include <sstream>
#include <stdexcept>
#include <string>

#ifndef string_fnc_header
#define string_fnc_header

template<typename T>
T to_numeric(const std::string& data)
{
    std::istringstream str(data);
    T val;
    str >> val;

    if(!str)
        throw std::invalid_argument("string to numeric failed - invalid argument");

    return val;
}

#endif // string_fnc_header

solito schrieb:

und würdet ihr die datei auch wenn sie nicht nur templates enthält nach .tpl umbenennen oder gar eine neue .tpl datei erstellen und diese dann noch in die cpp-datei einbinden.

Grundsätlich solltest du lieber Templates vom Rest trennen. Besonders wenn du Templates in Header und Source trennst.

Warum?
Man sollte nicht Gefahr laufen das man Teile inkludiert, die eigentlich nicht inkludiert gehören.
Zumal es auch besser lesbar ist.

Natürlich spricht nichts dagegen mehrere Templates in einer Datei zu haben (sofern diese nicht zu groß wird).

cu André

Hier

hustbaer

Wenn du Deklaration und Implementierung trennen willst kannst du auch ein ".inl" File machen, und das im ".hpp" File inkludieren.

Dabei ist es durchaus üblich dass das ".inl" File das dazugehörige ".hpp" File NICHT inkludiert, und auch keine include Guards enthält. Wenn man sich daran hält das ".inl" File immer nur im dazugehörigen ".hpp" File (ganz unten) zu inkludieren macht das auch keine Probleme.

Ansonsten pack wie vorgeschlagen wurde die Implementierung auch einfach mit ins ".hpp" File - ist sehr üblich, und auch einfacher.

darthdespotism

mikey schrieb:

Okay, du hast Recht mit dem Comeau C++ Compiler:
http://de.wikipedia.org/wiki/Comeau_C%2B%2B

Dann waren das also die jenigen, die stolze 5 Jahre an diesem export-Feature programmiert haben.

IMO gibt's die export Implementierung von der EDG und steht auch im neuen Borland Compiler zur Verfügung

Braunstein

Die derzeit aktuelle version des BCB2007 unterstützt export nicht.