Переопределение типа класс ошибка

Так выглядит declaration класса:

class Player {
private:
  int health;
  int strength;
  int agility;
public:
  void move();
  void attackMonster();
  void getTreasure();
};

Т.е. это просто описание интерфейса класса.

Понятное дело, что оно должно присутствовать в файлах ОДИН раз, но в каждом cpp, где этот класс используется. Поэтому определение класса выносят в h-шник.

Если же у Вас определение класса есть и в h-файле, и во включающем его cpp-файле, то естественно, что компилятор заругается. Поэтому если Вы собираетесь использовать этот класс в нескольких файлах, то разумно в cpp-файл вынести исключительно реализацию функций-членов (т.е. function definition). Т.е. в cpp-файле будет:

#include <player.h> // наш h-файл
void Player::move
{
// код ф-ции
}
void Player::attackMonster()
{
// код ф-ции
}
void Player::getTreasure()
{
// код ф-ции
}

Шаблонные классы вместе со своими ф-циями удобно целиком описывать в h-нике. Все равно они потом будут подставляться по месту использования компилятором.

Ну, и не забывайте в своих h-файлах делать блоки защиты от повторного включения:

#ifndef MY_SYMBOL_H
#define MY_SYMBOL_H
// здесь идет собственно тело h-ника
#endif //MY_SYMBOL_H

The problem is that in MyClass.h you define a class MySecondClass as an empty class. When you the define your class in MyClass.cpp you give a different definition, which contains some new members. This infringes the One Definition Rule (ODR).

Solution 1

remove {} in the header. This will tell the compiler that you declare that such a class exists but that it will be defined later. Your code would compile. Unfortunately if you’d include the header in other cpp, these could make only a very very limited use of MySecondClass.

Solution 2

define in the header the class with all its members (but without providing the implementation of the member functions:the signature is sufficient). This would allow the class to be used in whichever cpp that
would include it:

// MyClass.h
#pragma once
namespace MyNamespace {
    class MySecondClass {
    public:
        void print(const char* msg);
    };
}

You’d then define the members of the class in its cpp in the appropriate namespace:

// MyClass.cpp
#include <iostream>
#include "MyClass.h"

using namespace std;

namespace MyNamespace {
    // member functions
    void MySecondClass::print(const char* msg)
    {
        cout << msg << endl;
    }
}

Remark: the include sequence in the cpp should first include the standard library headers, then only your own headers. It makes no difference in your simple example, but better get used the good practices immediately.

The problem is that in MyClass.h you define a class MySecondClass as an empty class. When you the define your class in MyClass.cpp you give a different definition, which contains some new members. This infringes the One Definition Rule (ODR).

Solution 1

remove {} in the header. This will tell the compiler that you declare that such a class exists but that it will be defined later. Your code would compile. Unfortunately if you’d include the header in other cpp, these could make only a very very limited use of MySecondClass.

Solution 2

define in the header the class with all its members (but without providing the implementation of the member functions:the signature is sufficient). This would allow the class to be used in whichever cpp that
would include it:

// MyClass.h
#pragma once
namespace MyNamespace {
    class MySecondClass {
    public:
        void print(const char* msg);
    };
}

You’d then define the members of the class in its cpp in the appropriate namespace:

// MyClass.cpp
#include <iostream>
#include "MyClass.h"

using namespace std;

namespace MyNamespace {
    // member functions
    void MySecondClass::print(const char* msg)
    {
        cout << msg << endl;
    }
}

Remark: the include sequence in the cpp should first include the standard library headers, then only your own headers. It makes no difference in your simple example, but better get used the good practices immediately.

#include <Ogre.h>

using namespace Ogre;

class PlaneMovement
{
public:
PlaneMovement(Degree startingAngle, Real velocity = 2, Real gravity = 2);
Vector2 updateMovement(const FrameEvent& evt);
private:
Degree currentAngle;
Real currentVelocityX;
Real currentVelocityY;
Real gravity;
bool top;
};

Решение

#ifndef FILE_H
#define FILE_H

//file contents here

#endif

#pragma once

Другие решения

namespace MyNamespace { enum NotActuallyAnEnum; }

class NotActuallyAnEnum

{
...
}

error C2011: 'enum' type redefinition

namespace MyNamespace { class NotActuallyAnEnum; }

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

Предположим, у вас есть две разные ветки вашего решения, называемые Project1 и Project2 и вы разрешили символическую ссылку под названием Project указать на любой Project1 or Project2.

Идея состоит в том, что вы можете переключаться между ветвями, и проект всегда будет казаться Project к вашему приложению и, возможно, некоторым другим инструментам, которые ожидают его там.

Дисклеймер: да, контроль версий может переключаться между ветками, но таким образом вам не придется пересобирать все приложение каждый раз, когда вы переключаете ветки. Кроме того, обе ветки могут по-прежнему находиться под контролем версий.

Ну так открывай Project откроет либо Project1 or Project2 в зависимости от симлинка. Символическая ссылка может быть удалена/создана простым mklink_1 и mklink_2 как файлы сценариев.

Вот и подводный камень:

Если вы не обратите внимание и напрямую откроете решение в расположении 1 или 2 напрямую (вместо того, чтобы следовать символической ссылке каталога с Visual Studio), препроцессор может быть обманут в смешивании Project1MyHeader.h (или MyProject2MyHeader.h) С MyProjectMyHeader.h!

Даже если технически это один и тот же файл, препроцессор не знает о символической ссылке. Так вот #pragma once тебя бы не спас!

Нравится ресурс? Помоги проекту!

[ Script execution time: 0,0439 ]   [ 16 queries used ]   [ Generated: 30.01.23, 19:25 GMT ]