C++ - linker Probleme mit der statischen Methode

Schreibe ich ein Vector3D-Klasse, ruft eine statische Methode auf eine VectorMath-Klasse, um eine Berechnung durchführen. Wenn ich kompilieren, bekomme ich diese:

bash-3.1$ g++ VectorMath.cpp Vector3D.cpp 
/tmp/cc5cAPia.o: In function `main': 
Vector3D.cpp:(.text+0x4f7): undefined reference to 'VectorMath::norm(Vector3D*)' 
collect2: ld zurückgegeben, 1 exit status

Code:

VectorMath.h:

#ifndef VECTOR3D_H
#include "Vector3D.h"
#endif

class VectorMath {
    public:
    static Vector3D* calculatePerpendicularVector(Vector3D*, Vector3D*);
    static Vector3D* norm(Vector3D*);
    static double length(Vector3D*);
};

VectorMath.cpp

#include "VectorMath.h"
Vector3D* norm(Vector3D* vector) { //can't be found by linker
    //do vector calculations
    return new Vector3D(xHead, yHead, zHead, xTail, yTail, zTail);
}
//other methods

Vector3D.cpp

#include "Vector3D.h"
#include "VectorMath.h"
//...
//vector implementation
//...
int main(void) {
    Vector3D* v = new Vector3D(x, y, z);
    Vector3D* normVector = VectorMath::norm(v); //error here
}

Warum kann nicht der linker findet die VectorMath::norm Methode? Auf den ersten Blick würde ich denken, dass ich hätte erklären müssen-norm so:

Vector3D* VectorMath::norm(Vector3D* vector) {

aber nicht geholfen...

  • Diese: "aber nicht geholfen..." ist nicht genug Informationen. Was bedeutet das? Gleicher Fehler, andere Fehler?
InformationsquelleAutor | 2009-07-31
  1. 15

    Du bist, es fehlt:

    //VectorMath.cpp
#include "VectorMath.h"

             |
             V - here
Vector3D* VectorMath::norm(Vector3D* vector)
{
    ...
}

    Den norm Funktion ist Teil VectorMath::. Ohne, dass, Sie müssen nur eine Kostenlose Funktion.

    Dies ist mehr über Ihr design, aber warum sind Sie mit Zeigern auf alles? Dies ist viel sauberer:

    class VectorMath {
    public:
    static Vector3D norm(const Vector3D&);
};

    Nehmen Verweise, sind Sie in C++ so nicht schreiben C-code. Was passiert, wenn ich dies nennen?

    VectorMath::norm(0); //null

    Wird es entweder Abstürzen, Sie haben, um in eine Prüfung, in diesem Fall, was sollte es zurückgeben? Das ist alles bereinigt, die durch die Verwendung von Referenzen.

    Auch, warum nicht nur machen Sie diese Mitglieder des Vector3D Klasse?

    Vector3D* v = new Vector3D(x, y, z);
v->norm(); //normalize would be better, in my opinion

    Schließlich stack-Zuweisung Dinge. Dein code jetzt ein Speicherleck hat:

    int main(void) {
    Vector3D* v = new Vector3D(x, y, z);
    Vector3D* normVector = VectorMath::norm(v); 

    //delete v;
    //^ you're not deleting it!
}

    Ändern Sie diese, und verwenden Sie RAII Konzepte:

    int main(void) {
    Vector3D v(x, y, z);
    Vector3D* normVector = VectorMath::norm(v);

    //delete v;
    //^ you're not deleting it!
}

    Sowie durch die norm eine member-Funktion beenden Sie mit der sehr saubere code:

    int main(void) {
    Vector3D v(x, y, z);
    Vector3D normVector(v.norm());
}

    Keine Zeiger, keine Lecks, alle sexy.

    • OK, danke, das scheint zu funktionieren. Ich schwöre, das habe ich versucht vor und ist fehlgeschlagen - aber es funktioniert jetzt...Danke für die Zeiger über die memory-leak-Sache...ich denke, ich werde eine neue Frage stellen über diejenigen,...
    • Beeindruckende Antwort 🙂
    InformationsquelleAutor GManNickG
  2. 4

    Haben Sie nicht definiert Vector3D::norm Methode in VectorMath.cpp. Stattdessen haben Sie definiert eine Globale Funktion namens norm. Was Sie tun müssen, ist, qualifizieren den Namen der Methode, in der definition:

    Vector3D* Vector3D::norm(Vector3D* vector)
    • Du bist 24 Sekunden schneller als ich. :[
    InformationsquelleAutor Pavel Minaev
  3. 0
    Vector3D* VectorMath::norm(Vector3D* vector) { //can't be found by linker
    //do vector calculations
    return new Vector3D(xHead, yHead, zHead, xTail, yTail, zTail);
}
    InformationsquelleAutor Junier
Schreibe einen Kommentar