Question

好的如果需要，我会添加代码，但通常我在找出错误时很不错。然而，我得到的这个非常令人困惑。我正在使用三个文件。驱动程序代码citysim.cpp，以及两个支持文件utils.cpp和utils.h。我猜这是因为我在协调它们时做错了。

错误如下

g++ -g -Wall -o citysim citysim.o utils.o
utils.o: In function `map_of_cities::read(std::basic_string<char, std::char_traits<char>, std::allocator<char> >)':
/home/rryder1/cs302/labs/lab4/utils.cpp:95: multiple definition of `map_of_cities::read(std::basic_string<char, std::char_traits<char>, std::allocator<char> >)'
citysim.o:/home/rryder1/cs302/labs/lab4/utils.h:61: first defined here
citysim.o: In function `map_of_cities::read(std::basic_string<char, std::char_traits<char>, std::allocator<char> >)':
citysim.cpp:(.text+0xcb): undefined reference to `city::city()'
citysim.cpp:(.text+0x244): undefined reference to `city::~city()'
citysim.cpp:(.text+0x2a3): undefined reference to `city::~city()'
citysim.o: In function `std::vector<city, std::allocator<city> >::_M_insert_aux(__gnu_cxx::__normal_iterator<city*, std::vector<city, std::allocator<city> > >, city const&)':
citysim.cpp:(.text._ZNSt6vectorI4citySaIS0_EE13_M_insert_auxEN9__gnu_cxx17__normal_iteratorIPS0_S2_EERKS0_[std::vector<city, std::allocator<city> >::_M_insert_aux(__gnu_cxx::__normal_iterator<city*, std::vector<city, std::allocator<city> > >, city const&)]+0xd2): undefined reference to `city::~city()'
citysim.cpp:(.text._ZNSt6vectorI4citySaIS0_EE13_M_insert_auxEN9__gnu_cxx17__normal_iteratorIPS0_S2_EERKS0_[std::vector<city, std::allocator<city> >::_M_insert_aux(__gnu_cxx::__normal_iterator<city*, std::vector<city, std::allocator<city> > >, city const&)]+0x27d): undefined reference to `city::~city()'
citysim.o: In function `__gnu_cxx::new_allocator<city>::destroy(city*)':
citysim.cpp:(.text._ZN9__gnu_cxx13new_allocatorI4cityE7destroyEPS1_[__gnu_cxx::new_allocator<city>::destroy(city*)]+0x18): undefined reference to `city::~city()'
citysim.o:citysim.cpp:(.text._ZSt8_DestroyI4cityEvPT_[void std::_Destroy<city>(city*)]+0x14): more undefined references to `city::~city()' follow
utils.o: In function `map_of_cities::read(std::basic_string<char, std::char_traits<char>, std::allocator<char> >)':
utils.cpp:(.text+0xacd): undefined reference to `city::city()'
utils.cpp:(.text+0xc68): undefined reference to `city::~city()'
utils.cpp:(.text+0xcc7): undefined reference to `city::~city()'
collect2: ld returned 1 exit status

它可能源于的一件事是我不知道如何将utils.cpp与utils.h或我需要链接。我习惯于在一个cpp文件中编写所有内容。 utils.cpp和citysim.cpp都包含utils.h的include语句。 utils.h基本上只有类定义和函数头。 utils.cpp包含所有函数定义，citysim.cpp包含main函数。

这是我的utils.h文件

#ifndef __CITY_DEFS_H__
#define __CITY_DEFS_H__

#include <string>
#include <fstream>
#include <iostream>
#include <cstdlib>
#include <algorithm>
#include <cmath>
#include <vector>
#include <iomanip>
using namespace std;

//#include ALL HEADER FILES NEEDED FOR THIS HEADER FILE

class city {
  friend ostream &operator<<(ostream &, const city &);
  //OTHER FRIENDS
  friend class map_of_cities;
  public:
    city();
    ~city();

    // MEMBER FUNCTIONS
  protected:
    // MEMBER DATA

    string Name, type;
    float latitude, longitude;
    int population, listlocation;


    // MEMBER FUNCTIONS
};


ostream &operator<<(ostream &, const city &);

//OTHER CLASS DEFINITIONS
class map_of_cities {
    public:
        map_of_cities();
        ~map_of_cities();

        void read(string fname);
        float distance(city one, city two);
        void populatelists();
        void print();
        int listpop(vector<city> list);

    private:
        vector<city> cities;
        vector<city> local;
        vector<city> regional;
        vector<city> global;
};

#endif

如果我需要包含其他文件，我可以，但utils.cpp非常大，只是乱七八糟的筛选。这是我的问题吗？

Answer 1

这不是编译器错误，而是链接错误。您没有要链接的“city”类的构造函数或析构函数。确保在运行c ++命令时包含两个cpp文件，或者在编译后将链接阶段分开。

构建C ++项目涉及几个阶段。两个重要的阶段是“编译”和“链接”。编译器获取每个C ++文件并生成“对象”文件。目标文件包含所有已定义的符号（类，函数，方法，全局变量等）。目标文件的某些部分可能引用不在原始目标文件中的其他符号。

在您的情况下，您将拥有util目标文件，该文件引用city::~city()和city::city()。

在编译阶段，源代码还可以引用将在另一个对象中的符号。通常.h文件用于提供有关符号的信息，但通常它们不包括实际符号本身。在你的情况下util.h拼写出“这个程序作为城市构造函数和析构函数”，但它实际上并没有提供一个。

正如您在评论中所说，您确实需要在“util.cpp”中添加它们，以提供链接器可以找到的实际实现。

令人困惑的编译错误

1 个答案: