erreur undefined reference to

Version imprimable

Bonjour

Je commence le c++ et je developpe sur une virtualbox avec debian

J'ai l'erreur suivante lors de la compilation

Code:

1
2
3
4
g++ testapi.cpp -o output
/usr/bin/ld: /tmp/cciw7NXm.o: in function `main':
testapi.cpp:(.text+0xd9): undefined reference to `ApiAres::getAllZone()'
collect2: error: ld returned 1 exit status

mon fichier api.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
//Api.cpp
 
#include "api.hpp"
 
//constructeur
ApiAres::ApiAres(string url, string port)
{
    this->url = url;
    this->port = port;
};
 
// std::string Api::getAllZone()
void ApiAres::getAllZone() {
    cout << "Simulating getting all zones..." << endl;
};

mon fichier api.hpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
//Api.hpp
#include <iostream>
 
using namespace std; 
 
class ApiAres
{
    private:
        string url;
        string port;
 
    public:
        //constructeur
        ApiAres(string url, string port) {
            cout << "url : " << url << " port : " << port << endl;
        };
 
        void getAllZone();
        // std::string getAllZone();
};

mon fichier de test tesapi.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
//testapi.cpp
 
#include <iostream>
#include "api.hpp"
 
int main()
{
 
    std::string url = "https://tyradex.vercel.app/api/v1/pokemon/246";
    std::string port = "";
    ApiAres apiBdd(url,port);
    apiBdd.getAllZone();
    //std::cout << apiBdd.getAllZone() << std::endl;
    return 0;
}

auriez vous une idées svp ?

Merci

08/03/2024, 21h57
foetus
Ton code et ta ligne de commande sont mauvais :aie:

Ta ligne de commande est mauvaise. En gros, tu ne compiles pas ton unité api (et donc tu ne vois pas que ton code est mauvais)
Code:

1 2 g++ -Wall -c api.cpp g++ -Wall testapi.cpp api.o -o output
Ton code est mauvais (liste non exhaustive)
1. n'utilise pas using namespace std; et surtout pas dans 1 entête .h/ .hpp. D'autant + que tu ""ouvres"" std également dans le main via l'include.
2. tu as défini 2 fois ton constructeur. Fusionne les 2, utilise la liste d'initialisation ApiAres::ApiAres(std::string input_url, std::string input_port): url(input_url), port(input_port) { /* ... */ }
3. je ne suis pas fan de traces cout/ printf dans 1 entête .h/ .hpp à cause de l'include iostream.

bon mauvaise déclaration

api.hpp

au lieu de ca

Code:

1
2
3
4
5
       public:
            //constructeur
            ApiAres(string url, string port) {
                cout << "url : " << url << " port : " << port << endl;
            };

il faut ca

Code:

1
2
3
       public:
            //constructeur
            ApiAres(string url, string port);