Discussion :

[gyver76370]

Bonjour

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

J'ai l'erreur suivante lors de la compilation

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[foetus]

Ton code et ta ligne de commande sont mauvais

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[gyver76370]

bon mauvaise déclaration


api.hpp

au lieu de ca

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

il faut ca

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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