[C++] Undefined referenced bij aanroep static method.

[Verwijderd]

Hallo,

ik heb de volgende code en makefile. Het probleem is dat ik het niet geompileerd krijg wanneer ik in de main de static method van Test aanroep; Test::setInt(1); dus. Ik krijg dan steeds de volgende error:
g++ -c -g -Wall test.h main.cpp
g++ -g -o test test.o main.o
main.o(.text+0x12e): In function `main':
/home/joren/temp/main.cpp:10: undefined reference to `Test::setInt(int)'
collect2: ld returned 1 exit status
make: *** [test] Error 1

Het lijkt me aan de makefile te liggen, dus hopelijk kan hier iemand zeggen wat er ontbreekt of verkeerd staat in de makefile.

test.h

code:

#ifndef MYTEST
#define MYTEST

#include <cstdio>
#include <iostream>

class Test
{
public:
    Test(){}
    ~Test(){}
    static void setInt(int i);
private:
    static int m_int;   
};

test.cpp

code:

#include "test.h"

main.cpp

code:

#include <cstdio>
#include <iostream>
#include "test.h"

using namespace std;

int main(int argc, char *args[])
{
    printf("blabla\n");
    Test::setInt(1);
    return(0);
}

makefile

code:

#Options for development
CC=g++
CFLAGS=-c -g -Wall 

test: main.o test.o
    $(CC) -g -o test test.o main.o

main.o: test.h main.cpp
    $(CC) $(CFLAGS) test.h main.cpp

test.o: test.h test.cpp
    $(CC) $(CFLAGS) test.cpp

clean: 
    rm -f *.o *.tab.c *.tab.h *.output *.yy.cpp *~ *.output y.tab.* parse

[Daspeed]

Ik kan het verkeerd hebben, maar mis je geen #endif in je headerfile?
Edit: en owja waar is je implementatie van de class Test en dan met name de methode setInt

[ Voor 40% gewijzigd door Daspeed op 22-05-2005 11:14 ]

[Super_ik]

je moet eerst iets aanmaken als Test volgens mij

code:

Test test;
test.setInt(1)

in je main ?

[Creepy]

Er mist een #endif in je test.h, of ben je dat vergeten hier neer te zetten? Ook zie ik in je compiler output niet dat test.o wordt gecompileerd. Ik zou erop gokken dat dat komt doordat je nu een test.o hebt en test als programma. Kijk eens of het werkt door je programma een andere naam te geven.

code:

testprog: main.o test.o
    $(CC) -g -o testprog test.o main.o

[whoami]

Super_ik schreef op zondag 22 mei 2005 @ 11:16:
je moet eerst iets aanmaken als Test volgens mij

code:

Test test;
test.setInt(1)

in je main ?

Nee, want het is een static.

[Creepy]

DrClearbottom schreef op zondag 22 mei 2005 @ 11:12:
Edit: en owja waar is je implementatie van de class Test en dan met name de methode setInt

Daar keek ik nog ff overheen. Vergeet m'n vorige post maar

[ Voor 7% gewijzigd door Creepy op 22-05-2005 11:21 ]

[Jrz]

waar is je setInt() code?

[MSalters]

Test is een erg onhandige naam in Linux shells.

[Super_ik]

MSalters schreef op zondag 22 mei 2005 @ 11:29:
Test is een erg onhandige naam in Linux shells.

want?

_{* Super_ik is nieuwsgierig}

edit:

nevermind, man test is duidelijk genoeg

[ Voor 15% gewijzigd door Super_ik op 22-05-2005 11:36 ]

[MrBucket]

Jrz schreef op zondag 22 mei 2005 @ 11:25:
waar is je setInt() code?

Precies, in je test.cpp moet je nog iets hebben staan als

C++:

void Test::setInt(int i){
    m_int = i;
}

[ Voor 2% gewijzigd door MrBucket op 23-05-2005 13:12 . Reden: 'void' vergeten :X ]

[Verwijderd]

Hallo,

dank je voor jullie reacties!! Maar het probleem zit hem neit in de implementatie van de setInt(int i), dat is gewoon iets wat ik ff vergeten had, die implementatie zit in de h file, moet dus zijn:

[code]
static void setInt(int i) { m_int = i; }
[code/]


De fout zit dus ergens in de makefile vlg mij, geen implementatie fout volgens mij... Probeer het desnoods zelf eens te compileren.... als jullie tijd hebben :-)
Het probleem dat ik heb is puur bij het aanroepen van de static method...;

Groeten.

[NMe]

Waar in de h-file staat die implementatie? In de klassedefinitie of daaronder?

C++:

class Blaat {
  public:
    static void setInt(int i) { m_int = i; }
};

of:

C++:

class Blaat {
  // .....
};

static void setInt(int i) { m_int = i; }

In het laatste geval is het natuurlijk fout. Bij mijn weten moet je daar wel de klassenaam voor zetten:

C++:

static void Test::setInt(int i) { m_int = i; }

[H!GHGuY]

ik had overlaatst ook een erg verwant probleem:

C++:

class Test
{
  static void setInt( int i );
public: // methodes
private: // nog wat meuk
  static int getal;
};

void Test::setInt(int i) { getal = i };

de fout die ik kreeg: undefined external int Test::getal

[MSalters]

-NMe- schreef op zondag 22 mei 2005 @ 21:20:
Waar in de h-file staat die implementatie? In de klassedefinitie of daaronder?

C++:

class Blaat {
  public:
    static void setInt(int i) { m_int = i; }
};

of:

C++:

class Blaat {
  // .....
};

static void setInt(int i) { m_int = i; }

In het laatste geval is het natuurlijk fout. Bij mijn weten moet je daar wel de klassenaam voor zetten:

C++:

static void Test::setInt(int i) { m_int = i; }

Nee, static mag alleen op de in-class declaratie. Overigens mag je niet zomaar een functie definieren in een header.

De code boven doet allebei fout, met als gevolg dat je iets heel anders "legaals" krijgt: 1 functie ::setInt per .cpp file die de .h include. Die functies zijn allemaal verschillend, dus &::setInt != ::setInt. Omdat het allemaal verschillende definities zijn, heb je dus ook geen multiple definition linker fout.

[.oisyn]

HIGHGuY schreef op maandag 23 mei 2005 @ 10:27:
ik had overlaatst ook een erg verwant probleem:

C++:

class Test
{
  static void setInt( int i );
public: // methodes
private: // nog wat meuk
  static int getal;
};

void Test::setInt(int i) { getal = i };

de fout die ik kreeg: undefined external int Test::getal

Omdat je 'getal' alleen hebt gedeclareerd maar nergens gedefinieerd. Je zult toch ruimte moeten reserveren voor die int in een bepaalde translation unit, net als met globale variabelen.

[MrBucket]

Voor de volledigheid, zo hoort het dus:

MyClass.h:

C++:

class MyClass{
  static int s_iValue;
  public:
    static void setValue(int i);
};

MyClass.cpp:

C++:

include "MyClass.h"

int MyClass::s_iValue = 0;

void MyClass::setValue(int i){
  s_iValue = i;
}

[H!GHGuY]

een heele luide "aaaaaahhh"

zo hoort dat dus in c++ met die static variabelen in classes thx

[Verwijderd]

Hallo,

Thx. De laatste reply is volgens mij wat ik zoek, ik zal het vanavond eens proberen en hopelijk krijg ik het dan gecompileerd.

Thx voor alle replies.