Почему пишет неопределенный тип?

Question

[Савва Насыров]

Код класса:

#include "EventArgs.h"
#include <vector>
#include <string>

using namespace std;

class EventArgs {
public:
    static const EventArgs Empty;
};

const EventArgs EventArgs::Empty = EventArgs();

class CmdEventArgs : public EventArgs
{
public:
    enum ReturnTypes{
        Nothing,
        Args
    };

    ReturnTypes content_type;

    vector<string> args;

    CmdEventArgs(ReturnTypes type)
    {
        content_type = type;
    }

    CmdEventArgs(ReturnTypes type, vector<string> args)
    {
        content_type = type;
        this->args = args;
    }

    static const CmdEventArgs Empty;
};

const CmdEventArgs CmdEventArgs::Empty = CmdEventArgs(CmdEventArgs::Nothing);

По неопределенной причине Empty инициализирует прекрасно.

И на будущее посоветуйте пожалуйста как решать эту ошибку, ведь она возникает по самым разным проблемам.

Comments

[GavriKos]

На будущее советую правильно теги ставить. Джава и c# тут причем?

[Савва Насыров]

GavriKos, извините писал другой вопрос, но решение пришло

[#]

Савва Насыров, правьте теги, а если решили - публикуйте своим ответом. если уверены - отмечайте ответом. может кому еще пригодится ))

[Савва Насыров]

#, я понимаю, но там банальщина. гуглится за 3 минуты. оно не стоит сего. теги сразу исправил

[#]

Савва Насыров, тут много вопросов от новичков ))
если уж задали вопрос и решили, абсолютно нормально дать решение для других ))

[Владислав Лысков]

Савва Насыров,

я понимаю, но там банальщина. гуглится за 3 минуты. оно не стоит сего. теги сразу исправил

по секрету: пока ответов на вопрос нету, его можно удалить, раз уж вы не хотите делиться решением

[Wataru]

Савва Насыров, Покажите код хедера.

[Савва Насыров]

Wataru, maaGames помог мне с неопределенным типом. Сейчас обсуждаем другие ошибки под ответом

Это пофикшеный код

//
// Created by SavvaNasyrov on 14.12.2023.
//

#ifndef TERMINALHELPER_EVENTARGS_H
#define TERMINALHELPER_EVENTARGS_H


#include <vector>
#include <string>

using namespace std;

class EventArgs {
public:
    static const EventArgs Empty;
    
    virtual ~EventArgs();
};

class CmdEventArgs : public EventArgs
{
public:
    enum ReturnTypes{
        Nothing,
        Args
    };

    ReturnTypes content_type;

    vector<string> args;

    CmdEventArgs(ReturnTypes type);

    CmdEventArgs(ReturnTypes type, vector<string> args);

    static const CmdEventArgs Empty;
};


#endif //TERMINALHELPER_EVENTARGS_H

[#]

Савва Насыров, так или делитесь решением, либо удаляйте вопрос )).. не я один настаиваю ;)

[Савва Насыров]

#, в текущем ответе на вопрос все подробно расписано. На этот вопрос есть решение. Хотя по моему мы друг друга не понимаем

[Савва Насыров]

#, если вы имеете ввиду то, что мы обсуждаем другие ошибки под ответом, то еще не обсудили чтобы что-то выкладывать

[#]

Савва Насыров, не, все норм.. не подписывался, по тому и пропустил ответ ниже ))

Answer 1

[maaGames]

Судя по вот этому: "EventArgs.h", в вопросе показан код из срр файла. А в другом срр файле происходит создание объектов и там включен только "EventArgs.h" с forward declaration этих классов. Собственно, об этих классах там известно только то, что они есть и можно только указатель на них создать и всё. Определение класса олжно быть в хэдере и этот хедер надо включить во все файлы, которые этот класс используют.
И сделай виртуальный деструктор. Но это уже следующая ошибка, про которую спросишь.

Comments

[Савва Насыров]

зачем тогда нужны сами cpp файлы, если код классов пишется в заголовке?

[maaGames]

Савва Насыров, Код классов не пишется в заголовке. Это ты так написал. В хэдере нужно писать определение класса, а реализацию в срр файле. То есть в хэдере только объявление функций должно быть.

[Савва Насыров]

я вас понял. теперь задаю вопрос возможно связанный с виртуальным деструктором.

#include "Terminal.h"
#include "Types/EventArgs.h"

#include "string"

using namespace std;

void BasicTerminal::AddNewCommand(string command, TFunction exe_function, string description)
{
    functions.push_back(exe_function);
    commands.push_back(command);
    descriptions.push_back(description);
}

EventArgs BasicTerminal::Execute(string input_command, CmdEventArgs args)
{
    int index;

    for(int i = 0; i < commands.size(); i++)
    {
        if (input_command == commands[i]) index = i;
    }

    return functions[index](args);
}

BasicTerminal term;

EventArgs Hello(CmdEventArgs args)
{
    cout << "hello" << endl;
    return EventArgs::Empty;
}

int main()
{
    term.AddNewCommand("hello", Hello, "prints hello");
    term.Execute("hello", CmdEventArgs::Empty);
}

D:\Games\CLion 2023.3\bin\mingw\bin/ld.exe: C:\Users\SAVVAN~1\AppData\Local\Temp\ccrPdaKh.o:Terminal.cpp:(.text+0x3d4): undefined reference to `CmdEventArgs::CmdEventArgs(CmdEventArgs::ReturnTypes)'
D:\Games\CLion 2023.3\bin\mingw\bin/ld.exe: C:\Users\SAVVAN~1\AppData\Local\Temp\ccrPdaKh.o:Terminal.cpp:(.rdata$.refptr._ZN12CmdEventArgs5EmptyE[.refptr._ZN12CmdEventArgs5EmptyE]+0x0): undefined reference to `CmdEventArgs::Empty'
collect2.exe: error: ld returned 1 exit status

[Савва Насыров]

class EventArgs {
public:
    static const EventArgs Empty;
    
    virtual ~EventArgs();
};

[maaGames]

Савва Насыров, деструктор теперь правильный!
static переменные надо в срр файл добавить. В хэдере только их объявления в классе. Инициализацию (вне класса) надо в срр файл перенести.

[Савва Насыров]

maaGames, по моему у меня так и сделано:
.h

class EventArgs {
public:
    static const EventArgs Empty;
    
    virtual ~EventArgs();
};

.cppconst EventArgs EventArgs::Empty = EventArgs();
---
в cmd наследнике все также

[maaGames]

Савва Насыров, а это тоже в срр?
const CmdEventArgs CmdEventArgs::Empty = CmdEventArgs(CmdEventArgs::Nothing);

[maaGames]

Савва Насыров, хотя больше похоже, что срр файлы просто не добавлены к проекту и не компилируются

[Савва Насыров]

CMakeLists.txt

cmake_minimum_required(VERSION 3.27)
project(TerminalHelper)

set(CMAKE_CXX_STANDARD 17)

add_library(TerminalHelper STATIC Terminal.cpp
        Types/EventArgs.cpp
        Types/EventArgs.h
)

@maaGames,

[Савва Насыров]

maaGames, в h нет инициализации как в cpp

[maaGames]

Савва Насыров, "Types/EventArgs.h" этой строчки точно в файле быть не должно

[Савва Насыров]

maaGames, насколько я понял, это ничего не меняет. ошибка одна:
g++.exe D:\Projects\TerminalHelper\Terminal.cpp -o Terminal
D:\Games\CLion 2023.3\bin\mingw\bin/ld.exe: C:\Users\SAVVAN~1\AppData\Local\Temp\ccbgWMrS.o:Terminal.cpp:(.text+0x2d4): undefined reference to `EventArgs::~EventArgs()'
D:\Games\CLion 2023.3\bin\mingw\bin/ld.exe: C:\Users\SAVVAN~1\AppData\Local\Temp\ccbgWMrS.o:Terminal.cpp:(.text$_ZN12CmdEventArgsC1ERKS_[_ZN12CmdEventArgsC1ERKS_]+0x64): undefined reference to `EventArgs::~EventArgs()'
D:\Games\CLion 2023.3\bin\mingw\bin/ld.exe: C:\Users\SAVVAN~1\AppData\Local\Temp\ccbgWMrS.o:Terminal.cpp:(.text$_ZN12CmdEventArgsD1Ev[_ZN12CmdEventArgsD1Ev]+0x32): undefined reference to `EventArgs::~EventArgs()'
D:\Games\CLion 2023.3\bin\mingw\bin/ld.exe: C:\Users\SAVVAN~1\AppData\Local\Temp\ccbgWMrS.o:Terminal.cpp:(.rdata$.refptr._ZN12CmdEventArgs5EmptyE[.refptr._ZN12CmdEventArgs5EmptyE]+0x0): undefined reference to `CmdEventArgs::Empty'
D:\Games\CLion 2023.3\bin\mingw\bin/ld.exe: C:\Users\SAVVAN~1\AppData\Local\Temp\ccbgWMrS.o:Terminal.cpp:(.rdata$.refptr._ZN9EventArgs5EmptyE[.refptr._ZN9EventArgs5EmptyE]+0x0): undefined reference to `EventArgs::Empty'
D:\Games\CLion 2023.3\bin\mingw\bin/ld.exe: C:\Users\SAVVAN~1\AppData\Local\Temp\ccbgWMrS.o:Terminal.cpp:(.rdata$.refptr._ZTV9EventArgs[.refptr._ZTV9EventArgs]+0x0): undefined reference to `vtable for EventArgs'
collect2.exe: error: ld returned 1 exit status

но я не понял почему она изменилась(я убрал строку и ошибка поменялась, но когда добавил обратно ничего не изменилось

[Савва Насыров]

maaGames, если это чем-то поможет, я могу скинуть архив с проектом

[maaGames]

Савва Насыров, Я не пользуюсь mingw, не знаю тонкостей сборки. По кодам ошибки видно, что срр файлы с EventArgs не компилировались. А судя по первой строчке, компилируется только файл terminal.cpp. А надо, чтобы компилировались все срр файлы.

[Савва Насыров]

maaGames, мне, как новичку, не принципиально каким компилятором пользоваться. Давайте синхронизируемся. Может так понятней станет

[maaGames]

Савва Насыров, Рекомендую испоьзовать какую-нибудь IDE, а не через командную строку собирать. Собирать проекты вручную вам потребуется примерно ноль раз в жизни, не в девятнадцатом веке живём, всё-таки. Под виндоус проще всего в Microsoft Visual Studio. Community Edition бесплатная с офф.сайта скачивается. Там любой пример или туториал посмотри и будет понятно, как добавлять файлы в процект, как компилировать, отлаживать и прочее.

[maaGames]

Савва Насыров, И ещё. на будущее. Ни в коем случае нельзя писать "using namespace std;" в хэдерах. В файлах срр можно.

[Савва Насыров]

maaGames, я использую clion, он сам по себе использует mingw, но я поменял на vs компилятор: вот ошибки

Terminal.obj : error LNK2019: ссылка на неразрешенный внешний символ "public: virtual __cdecl EventArgs::~EventArgs(void)" (??1EventArgs@@UEAA@XZ) в функции main.
Terminal.obj : error LNK2019: ссылка на неразрешенный внешний символ "public: static class EventArgs const EventArgs::Empty" (?Empty@EventArgs@@2V1@B) в функции "class EventArgs __cdecl Hello(class CmdEventArgs)" (?Hello@@YA?AVEventArgs@@VCmdEventArgs@@@Z).
Terminal.obj : error LNK2019: ссылка на неразрешенный внешний символ "public: static class CmdEventArgs const CmdEventArgs::Empty" (?Empty@CmdEventArgs@@2V1@B) в функции main.
Terminal.exe : fatal error LNK1120: неразрешенных внешних элементов: 3

[Савва Насыров]

cmake_minimum_required(VERSION 3.27)
project(TerminalHelper___other_compiler)

set(CMAKE_CXX_STANDARD 23)

add_library(TerminalHelper___other_compiler STATIC Terminal.cpp
        Types/EventArgs.cpp)

[maaGames]

Савва Насыров, речь не про компилятор, а про IDE. Это разное. А если прям выпендриваться, то это ошибки компоновщика, а не компилятора. Ошибки из-за того, что срр файлы с классами EventArg не компилируются вообще, либо объектные файлы куда-то в другое место скомпилировались. Через CMake генерируете файл проекта, открываете его в IDE и видите все свои ошибки создания проекта, каких файлов не хватает и почему не компилируется.