C++ Как делить константы с экстерном между кастами - Ошибка: классом хранения указано

Question

Я заголовочный файл, в котором я декларировать некоторые константы с ехЬегпом:

#ifndef CONSTANTS_H 
#define CONSTANTS_H 

#include <string> 

class Constants 
{ 
    public: 
     Constants(){} 
     extern const std::string DELIMITER; 
     extern const std::string FILENAME; 
     extern const int BLUCAR; 
     extern const int REDCAR; 
     extern const int EMPTY; 
     extern const int SPARSE_LIM; 
    protected: 
    private: 
}; 

#endif // CONSTANTS_H 

Тогда в источнике я определяю их следующим образом:

#include "constants.h" 

extern const std::string DELIMITER = ","; 
extern const std::string FILENAME = "cars.csv"; 
extern const int BLUCAR = 1; 
extern const int REDCAR = 2; 
extern const int EMPTY = 0; 
extern const int SPARSE_LIM = 5; 

Почему компилятор дает мне ошибку: класс хранения указан для «DELIMITER»?

Answer 1

Во-первых, они, похоже, не являются членами класса. То, как вы используете extern, похоже, вы предназначались для того, чтобы они были бесплатными. Возможно, в пространстве имен. Выведите их из этого класса.

Затем, определив их, оставьте extern.

В этом контексте это означает «найти эту переменную в другом месте». Вы не хотите его искать в другом месте. Это здесь!

// Header 
namespace Constants { 
    extern const std::string DELIMITER; 
    extern const std::string FILENAME; 
    extern const int   BLUCAR; 
    extern const int   REDCAR; 
    extern const int   EMPTY; 
    extern const int   SPARSE_LIM; 
} 


// Source 
namespace Constants { 
    const std::string DELIMITER = ","; 
    const std::string FILENAME = "cars.csv"; 
    const int   BLUCAR  = 1; 
    const int   REDCAR  = 2; 
    const int   EMPTY  = 0; 
    const int   SPARSE_LIM = 5; 
} 

Помните, что вы сделали бы то же самое для определения объектов static!

Answer 2

Я думаю, вы либо хотите использовать статические члены в своем классе, используйте пространство имен вместо класса, либо поместите свои константы в глобальное пространство имен.

Для статических членов:

class Constants 
{ 
    public: 
     Constants(){} 
     static const std::string DELIMITER; 
     static const std::string FILENAME; 
     static const int BLUCAR; 
     static const int REDCAR; 
     static const int EMPTY; 
     static const int SPARSE_LIM; 
    protected: 
    private: 
}; 

и в исходном файле

const std::string Constants::DELIMITER = ","; 
const std::string Constants::FILENAME = "cars.csv"; 
const int Constants::BLUCAR = 1; 
const int Constants::REDCAR = 2; 
const int Constants::EMPTY = 0; 
const int Constants::SPARSE_LIM = 5; 

Однако, это выглядит как ваш класс больше похож на пространство имен, чем класс, так как не было бы никакого смысла создание экземпляров.

Но есть и другой вариант. Вы можете даже не хотите использовать класс или пространство имен, а просто их всех в глобальном пространстве имен:

// header 
extern const std::string DELIMITER; 
extern const std::string FILENAME; 
extern const int BLUCAR; 
extern const int REDCAR; 
extern const int EMPTY; 
extern const int SPARSE_LIM; 

// source 
const std::string DELIMITER = ","; 
const std::string FILENAME = "cars.csv"; 
const int BLUCAR = 1; 
const int REDCAR = 2; 
const int EMPTY = 0; 
const int SPARSE_LIM = 5;