C++コンパイル時のエラー

Question

hppファイルは1つ

#pragma once
#include <iostream>
#include <vector>
#include <string>
#include <tuple>
using namespace std;

void insertNames();
void JCtoAC(string eraName, int eraNum);
void ACtoJC(int inputNum);

extern const int startYear;
extern const int thisYear;

vector<tuple<string, int>> names;

c++ファイルは3つ

//era.cpp
#include "era.hpp"

int main()
{
  JCtoAC("Reiwa", 1);
}

//names.cpp
#include "era.hpp"

const time_t now = time(nullptr);
const tm *lt = localtime(&now);
const int startYear = 1615 - 1; //明治元年なので1を引く
const int thisYear = 2000 + lt->tm_year;
const int reiwa = (thisYear - 100) - 2019 + 1; //元号のスタートは0年ではないため1を足す

vector<string> erasNames = {
    "Genna", "Kanei", "Shouhou", "Keiwan", "Jouou", "Meireki", "Manji", "Kanbun", "Enpou", "Tenna", "Joukyou", "Genroku", "Houei",
    "Syoutoku", "Kyouhou", "Genbun", "Kanpou", "Enkyou", "Kanen", "Houreki", "Meiwa", "Anei", "Tenmei", "Kansei", "Kyouwa", "Bunka",
    "Bunsei", "Tenpou", "Kouka", "Kaei", "Ansei", "Manei", "Bunkyuu", "Genji", "Keiou", "Meiji", "Taisyou", "Syouwa", "Heisei", "Reiwa"};

vector<int> erasYears = {10, 21, 5, 5, 4, 4, 4, 13, 9, 4, 5, 17, 8, 6, 21, 6, 4, 5, 4, 14, 9, 10, 9, 13, 4, 15, 13, 15, 5, 7, 7, 2, 4, 2, 4, 45, 15, 64, 31, reiwa};

void insertNames()
{
  for (int i = 0; i < erasNames.size(); i++)
  {
    names.push_back(make_tuple(erasNames[i], erasYears[i]));
  }
}

//convert.cpp
#include "era.hpp"
void JCtoAC(string eraName, int eraNum)
{
  insertNames();
  int sumNum = 0;
  for (int i = 0; i < names.size(); i++)
  {
    string key = get<0>(names[i]);
    int value = get<1>(names[i]);
    if (eraName == key)
    {
      if (eraNum > value || eraNum <= 0)
      {
        cout << "The year " << eraName << eraNum << " doesn't exist." << endl;
      }
      else
      {
        cout << startYear + sumNum + eraNum << endl;
        break;
      }
    }
    sumNum += value - 1;
  }
}

void ACtoJC(int inputNum)
{
  if (inputNum > thisYear)
  {
    cout << "The year " << inputNum << " is the future. It may not be Reiwa Era." << endl;
  }
  else if (inputNum < startYear)
  {
    cout << "Sorry, I don't know about before" << startYear << endl;
  }
  else
  {
    int subNum = 0;
    insertNames();
    for (int i = 0; i < names.size(); i++)
    {
      string key = get<0>(names[i]);
      int value = get<1>(names[i]);
      int checkNum = inputNum - startYear - value - subNum;
      if (checkNum <= 0)
      {
        cout << key << value + checkNum << endl;
      }
      else
      {
        subNum += value - 1;
      }
    }
  }
}

これらのファイルを作成し、

$ g++ -std=c++11 -o a.out era.cpp

を実行すると

Undefined symbols for architecture x86_64:
  "JCtoAC(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, int)", referenced from:
      _main in era-ac976a.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

とエラーが出ます。

とりあえずググってincludeやタイポを確認したのですが解決策が見つかりません。コマンドが間違っているのでしょうか。
間違いが分かる方は教えてください。

Answer 1

gccを実行するさい、era.cppだけでなく、names.cppやconvert.cppもコマンドラインに空白区切りで指定してください。

Comments

[Kota.Y]

実行してみたのですが、
```
duplicate symbol '_names' in:
/var/folders/25/p6fxx4tj6sz7wxrwqsf9j9jm0000gn/T/era-1906ee.o
/var/folders/25/p6fxx4tj6sz7wxrwqsf9j9jm0000gn/T/names-3276f6.o
duplicate symbol '_names' in:
/var/folders/25/p6fxx4tj6sz7wxrwqsf9j9jm0000gn/T/era-1906ee.o
/var/folders/25/p6fxx4tj6sz7wxrwqsf9j9jm0000gn/T/convert-68d876.o
ld: 2 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
```
というエラーが出力されてしまいました。era.cppはそもそもJCtoACを実行しているだけなので定義がかぶることはないはずなのですが。。。

[Daregada]

質問変わってますが、ヘッダーファイルの中でnamesを定義してるのがまずいです。

[Kota.Y]

ありがとうございます。namesの定義をnames.cppに移したら正常に動作しました。

Answer 2

3つの .cppファイルを同時に指定してリンクさせるか、あるいは、
-c オプションを指定して個別にコンパイルだけを行い、
3つの .oファイルを同時に指定してリンクさせましょう。

追記
vector<tuple<string, int>> names; は names.cpp(または convert.cpp) の中に書いて、
era.hpp には extern vector<tuple<string, int>> names; と書きましょう。