Как использовать класс объявленный в другом файле?

Question

[Acaunt]

Создал для программы класс задача которого писать в консоль и в файл информацию, которую принимает через оператор <<

сам класс

#include <iostream>
#include <fstream>

class Logger {
public:
    Logger() {
        file.open("Logger.txt");
        if (!file.is_open()) {
            throw std::runtime_error("Fail open file \"Logger.txt\"");
        }
    }
    ~Logger() {
        file.close();
    }
    template <class T>
    Logger& operator<<(const T& value) {
        file << value;
        std::cout << value;
        return *this;
    }
    Logger& operator<<(std::ostream& (*manipulator)(std::ostream&)) {
    file << manipulator;
    std::cout << manipulator;
    return *this;
  }
private:
    std::ofstream file;
};

extern Logger logg

Любой участок программы

logg << 6 << " is six" << std::endl;

По идее это должен быть, как std::cout, но у меня возникает ошибка не разрешённый внешний символ методом проб я понял что ошибка возникает при попытке передачи информации через оператор, но если я объявлю этот класс там где он необходим то ошибок нет.

Решение:

MyClass.h

class MyClass {
    //Описание класса
}

extern MyClass myclass

MyClass.cpp

#include "MyClass.h"
MyClass myclass

Спасибо всем кто подсказал.

Comments

[Sergio]

extern лишь декларирует наличие переменной, а сама переменная должна быть определена где-то

Answer 1

[maaGames]

Сразу скажу, что это плохой код и делать так не надо.
Однако, отвечая на вопрос: весь этот код написан в .h файле. Нужно сделать .cpp файл, в котором будет сздана переменная logg. Сейчас объявлено, что переменная есть, но самого объекта нет.

Comments

[Acaunt]

Что именно плохо? И он у меня разделён на *.h и *.сpp, а здесь просто написал всё в одном месте.

[maaGames]

Acaunt, Лучще не делать глобальных переменных без использования синглтона. Неизвестно, в каком порядке они будут созданы и удалены (зависит от порядка сборки).
И реализация внутри класса не всегда хорошо - это равнозначно объявлению inline.
std::cout реализованы ещё более "неприятным" способом с созданием кучи статических переменных, но там код очень хитрый.

[Acaunt]

maaGames, я тут немного переделал это не совсем синглтон, но поведение думаю теперь более безопасное

вот

#include <iostream>
#include <fstream>
#include <filesystem>

class Logger {
public:
    Logger();
    template <class T>
    Logger& operator<<(const T& info);
    Logger& operator<<(std::ostream& (*manipulator)(std::ostream&));
private:
    static bool is_create;
};

Logger::Logger() {
    if (!is_create) {
        std::filesystem::remove("Logger.txt");
        is_create = true;
    }
}

template <class T>
Logger& Logger::operator<<(const T& info) {
    std::cout << info;
    std::ofstream file("Logger.txt", std::ios_base::out | std::ios_base::app);
    if (file.is_open()) {
        file << info;
    }
    file.close();
    return *this;
}

Logger& Logger::operator<<(std::ostream& (*manipulator)(std::ostream&)) {
    std::cout << manipulator;
    std::ofstream file("Logger.txt", std::ios_base::out | std::ios_base::app);
    if (file.is_open()) {
        file << manipulator;
    }
    file.close();
    return *this;
}

bool Logger::is_create = false;

Лучше?

[maaGames]

Acaunt, Хуже :)
Исходный вариант хороший, но только если держать в уме, что лучше так не делать, если не уверен.
Любые статические объекты создаются в неизветсном программисту порядке, установленном компилятором. А вот при испоьзовании Синглтона, объект создаётся при первом вызове instance. Плюс можно контроллировать время жизни и "оживлять" объект даже после вызова деструктора, если понадобится.

Answer 2

[0xC0000005]

Измените extern Logger logg на Logger logg;

Еще немного улучшил код:

#include <iostream>
#include <fstream>

class Logger {
public:
    Logger() {
        openFile("Logger.txt");
    }

    ~Logger() {
        closeFile();
    }

    template<class T>
    Logger& operator<<(const T& value) {
        file << value;

        std::cout << value;

        return *this;
    }

    Logger& operator<<(std::ostream& (*manipulator) (std::ostream&)) {
        file << manipulator;
    
        std::cout << manipulator;

        return *this;
    }

    void openFile(const char *path) {
        file.open(path);

        if(!file.is_open()) {
            throw std::runtime_error("Failed to open 'Logger.txt'");
        }
    }

    void closeFile() {
        file.close();
    }

private:
    std::ofstream file;
};

Logger logger;

logger << 6 << " is six" << std::endl;