Question

我试图编译来自 C ++的 的第4.3章的代码，其中描述了＆＃34;抽象类型＆＃34;。但是，我收到编译器错误：

$ g++ -std=c++11 abstract.cpp Vector.cpp Vector_container.cpp
/tmp/ccToFsNS.o: In function `g()':
abstract.cpp:(.text+0x9c): undefined reference to `Vector_container::Vector_container(std::initializer_list<double>)'
abstract.cpp:(.text+0xb4): undefined reference to `Vector_container::~Vector_container()'
abstract.cpp:(.text+0xc5): undefined reference to `Vector_container::~Vector_container()'
collect2: error: ld returned 1 exit status

如何修复我的代码以便编译？我是C ++的新手，我正在尽力转录Stroustrup的例子。

abstract.cpp：

#include "Vector_container.hpp"
#include <iostream>

void use(Container& c)
{
  const int sz = c.size();
  for (auto i = 0; i != sz; ++i)
    {
      std::cout << c[i] << "\n";
    }
}

void g()
{
  Vector_container vc {10, 9, 8, 7, 6};
  use(vc);
}

int main()
{
  g();
}

Vector_container.hpp：

#include "Container.hpp"
#include "Vector.hpp"
#include <initializer_list>

#pragma once

class Vector_container : public Container {
  Vector v;
public:
  Vector_container(int);
  Vector_container(std::initializer_list<double>);
  ~Vector_container();
  double& operator[](int);
  int size() const;
};

Vector_container.cpp：

#include "Container.hpp"
#include "Vector.hpp"

class Vector_container : public Container {
  Vector v;
public:
  Vector_container(int s) : v(s) {}
  Vector_container(std::initializer_list<double> lst) : v{lst} {}
  ~Vector_container() {}
  double& operator[](int i) { return v[i]; }
  int size() const { return v.size(); }
};

container.hpp：

#pragma once

class Container {
public:
  virtual double& operator[](int) = 0;
  virtual int size() const = 0;
  virtual ~Container() {}
};

Vector.hpp：

#include <initializer_list>

#pragma once

class Vector {
public:
  Vector(int);
  Vector(std::initializer_list<double>);
  ~Vector();
  double& operator[](int);
  int size() const;
  void push_back(double);
private:
  double* elem;
  int sz;
};

Vector.cpp：

#include "Vector.hpp"
#include <stdexcept>

Vector::Vector(int s) : elem{new double[s]}, sz{s}
{
  if (s < 0)
    {
      throw std::length_error{"Size must be >= 0"};
    }
  for (auto i = 0; i != s; ++i)
    {
      elem[i] = 0;
    }
}

Vector::Vector(std::initializer_list<double> lst)
  : elem{new double[lst.size()]}, sz{static_cast<int>(lst.size())}
    {
      std::copy(lst.begin(), lst.end(), elem);
    }

Vector::~Vector() {
  delete[] elem;
}

double& Vector::operator[](int i)
{
  if (i < 0 || size() <= i)
    {
      throw std::out_of_range{"Vector::operator[]"};
    }
  return elem[i];
}

int Vector::size() const
{
  return sz;
}

Answer 1

正如评论者和Sam所说，我没有正确定义Vector_container。我将Vector_container.cpp更改为：

#include "Vector_container.hpp"

Vector_container::Vector_container(int s) : v(s) {}
Vector_container::Vector_container(std::initializer_list<double> lst) : v{lst} {}
Vector_container::~Vector_container() {}

double& Vector_container::operator[](int i)
{
  return v[i];
}
int Vector_container::size() const
{
  return v.size();
}

现在程序编译并按预期运行。

Answer 2

我还没有看过那本书，但我会惊讶地发现Stroustrup会写一些东西：

1）明确依赖于gcc特定的编译指示，

2）鼓励滥用#pragma once

3）提供了一种在头文件中声明类的错误做法的示例，但是在一个完全独立的翻译单元中定义了类的方法，该翻译单元甚至不包含头文件。

完全摆脱Vector_container.hpp，然后将Vector_container.cpp重命名为Vector_container.hpp，代码应该编译并链接得很好。

您的问题是，现有Vector_container.cpp翻译单元中的任何内容实际上都不会实例化和编译类方法，以便链接器查找并解析来自其他翻译单元的引用。

您还可以使用非内联定义替换Vector_container.cpp中的内联定义。这也将实例化它们。但是这对Vector_container.hpp中的重复定义没有任何意义，我认为这是非常糟糕的做法。

C ++未定义引用`Vector_container :: Vector_container（std :: initializer_list <double>）＆＃39;

2 个答案: