Виртуальный деструктор на MinGW v4.9.2

@Хрисипп · Регистрация: 09.02.2016

Author24 — интернет-сервис помощи студентам

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

C++

class A{
  virtual ~A()=0;
};
A::~A(){cout<<"~";}
 
class B : public A {};

пишет Cmultiple definition of `A::~A()'|
в vs все работает. может где-то подкрутить нужно?

@DrOffset · 13.07.2016, 19:09

Хрисипп, а ключевое слово virtual написать?

@Хрисипп · 13.07.2016, 19:10 **[ТС]**

Сообщение от DrOffset

а ключевое слово virtual написать

это я написал, забыл в вопрос вписать

@DrOffset · 13.07.2016, 19:17

Сообщение от Хрисипп

multiple definition of `A::~A()'|

Это ошибка линкера. Если определение деструктора находится в заголовочном файле, который подключается в несколько единиц трансляции, то ошибка справедливая. MinGW здесь абсолютно прав, что это не компилирует.
Либо вынеси определение в отдельную единицу трансляции, либо сделай его inline.

@Хрисипп · 13.07.2016, 19:37 **[ТС]**

Сообщение от DrOffset

несколько единиц трансляции

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

C++

// h1
#include "h2.h"
class A
 
//h2 
#include "h1.h"
class B:A
 
// cpp
#include "h2.h"

. и наоборот

C++

// h1
#include "two.h"
#ifndef h1
#define h1
 
class A{
public:
virtual ~A()=0;
 
 
};
A::~A(){}
#endif
 
//h2
#ifndef h2
#define h2
#include "one.h"
using namespace std;
class B : A {};
 
#endif

это же подключение к одной единице трансляции

@DrOffset · 13.07.2016, 19:45

Хрисипп, поясни как ты понимаешь назначение строки номер 2 в h1? Почему он у тебя не под include-guard'ом?

Добавлено через 5 минут

Сообщение от Хрисипп

это же подключение к одной единице трансляции

Да, но у тебя проблема похоже из-за невнесенного в include-guard подключения. В итоге содержимое "расплодилось". И если объявлений в программе может быть сколько угодно, то определение сущности должно быть одно (правило одного определения).

Как ни крути - программа остается некорректной, даже если VS ее каким-то чудом собирает. Следовательно в любом случае - оставлять определение не-inline функции в заголовочном файле не стоит.

@Хрисипп · 13.07.2016, 19:48 **[ТС]**

DrOffset,

Сообщение от Хрисипп

C++

// h1
#include "h2.h"
class A
//h2 
#include "h1.h"
class B:A
// cpp
#include "h2.h"

опять ошибся при написании ответа. там

C++

// h1
class A
 
//h2 
#include "h1.h"
class B:A
 
// cpp
#include "h2.h"

Добавлено через 1 минуту

Сообщение от DrOffset

расплодилось

тогда бы он сообщил, что deep include (это имеешь в виду?)

@DrOffset · 13.07.2016, 19:53

Хрисипп, ответ я тебе дал.
* В заголовочных файлах все содержимое должно быть в обрамлено include-guard`ом.
* В заголовочных файлах определений нешаблонных и не-inline функций быть не должно.
Собственно все.

Добавлено через 3 минуты

Сообщение от Хрисипп

тогда бы он сообщил, что deep include (это имеешь в виду?)

Не факт. Зависит от того есть ли циклическое включение. У тебя его нет, насколько ты смог показать.

@Хрисипп · 13.07.2016, 20:09 **[ТС]**

Сообщение от DrOffset

циклическое включение.

а это, случаем, не оно?

Сообщение от Хрисипп

C++

// h1
#include "h2.h"
#ifndef h1
#define h1
class A{
public:
virtual ~A()=0;
};
A::~A(){}
#endif
 
//h2
#ifndef h2
#define h2
#include "h1.h"
class B : A {};
#endif
 
//main cpp
#include "h1.h"

только защищенное

@DrOffset · 13.07.2016, 20:22

Сообщение от Хрисипп

а это, случаем, не оно?

Оно. Но оно разрывается include-guard`в h2.
Я же сказал тебе как делать правильно, в чем проблема-то?

Зачем, кстати, вообще нужно циклически включать? Это в любом случае нехорошо.

@Хрисипп · 13.07.2016, 20:39 **[ТС]**

Сообщение от DrOffset

Зачем, кстати

осталось со времени, когда разбирался как устроены эти ifdef'ы. этот проект так и остался для сравнения с vs

@Хрисипп 19 / 29 / 13 Регистрация: 09.02.2016 Сообщений: 230
	13.07.2016, 20:39 [ТС]	11
	Сообщение от DrOffset Зачем, кстати осталось со времени, когда разбирался как устроены эти ifdef'ы. этот проект так и остался для сравнения с vs 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	13.07.2016, 19:09	2
	Хрисипп, а ключевое слово `virtual` написать? 0

@Хрисипп 19 / 29 / 13 Регистрация: 09.02.2016 Сообщений: 230
	13.07.2016, 19:10 [ТС]	3
	Сообщение от DrOffset а ключевое слово virtual написать это я написал, забыл в вопрос вписать 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	13.07.2016, 19:17	4
	Сообщение от Хрисипп multiple definition of `A::~A()'\| Это ошибка линкера. Если определение деструктора находится в заголовочном файле, который подключается в несколько единиц трансляции, то ошибка справедливая. MinGW здесь абсолютно прав, что это не компилирует. Либо вынеси определение в отдельную единицу трансляции, либо сделай его `inline`. 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	13.07.2016, 19:45	6
	Хрисипп, поясни как ты понимаешь назначение строки номер 2 в h1? Почему он у тебя не под include-guard'ом? Добавлено через 5 минут Сообщение от Хрисипп это же подключение к одной единице трансляции Да, но у тебя проблема похоже из-за невнесенного в include-guard подключения. В итоге содержимое "расплодилось". И если объявлений в программе может быть сколько угодно, то определение сущности должно быть одно (правило одного определения). Как ни крути - программа остается некорректной, даже если VS ее каким-то чудом собирает. Следовательно в любом случае - оставлять определение не-inline функции в заголовочном файле не стоит. 0

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	13.07.2016, 19:53	8
	Сообщение было отмечено DrOffset как решение Решение Хрисипп, ответ я тебе дал. * В заголовочных файлах все содержимое должно быть в обрамлено include-guard`ом. * В заголовочных файлах определений нешаблонных и не-inline функций быть не должно. Собственно все. Добавлено через 3 минуты Сообщение от Хрисипп тогда бы он сообщил, что deep include (это имеешь в виду?) Не факт. Зависит от того есть ли циклическое включение. У тебя его нет, насколько ты смог показать. 1

@DrOffset 18844 / 9843 / 2408 Регистрация: 30.01.2014 Сообщений: 17,285
	13.07.2016, 20:22	10
	Сообщение от Хрисипп а это, случаем, не оно? Оно. Но оно разрывается include-guard`в h2. Я же сказал тебе как делать правильно, в чем проблема-то? Зачем, кстати, вообще нужно циклически включать? Это в любом случае нехорошо. 0

Виртуальный деструктор на MinGW v4.9.2

Решение