ошибка множественного включения

[Antej]

есть хедер с описанными функциями. хедер включается в 2х местах. хедер целиком стандартного вида:

#ifndef OPERATORS_H
#define OPERATORS_H
//функции вида
double Mnogennja(double l, double r)
{return l*r;}
#endif

при компиляции проекта получаю ошибку:

main.o: In function `Dodavannja(double, double)':
main.cpp:(.text+0x0): multiple definition of `Dodavannja(double, double)'
ExpressionTree.o:ExpressionTree.cpp:(.text+0x0): first defined here

IDE (QtCreator 1.3.1) в обоих случаях приводит в одно итоже место.- в файл Operators.h, так для каждой обьявленной в нем функции.
что я делаю не так? если надо - приложу код...

[pproger]

2Antej

#ifndef OPERATORS_H
#define OPERATORS_H
//функции вида
inline static double Mnogennja(double l, double r)
{return l*r;}
#endif

[ROD]

Компилятор почему-то, иногда, не пересоздает объектные файлы после вот таких вот изменений.
Очистите проект (должны удалиться все moc и .o файлы) и попробуйте снова. Делается это через меню "Сборка" (или "Проект", почему-то на винде и линуксе перевод отличается) -> "Очистить проект %project_name%".


2pproger, не делать же каждую функцию inline, да и с класс нельзя inline сделать (а с ним будет такая же ошибка, при множественном определении)

[pproger]

2ROD

Очистите проект (должны удалиться все moc и .o файлы) и попробуйте снова. Компилятор почему-то, иногда, не пересоздает объектные файлы после вот таких вот изменений.

он определил функцию с внешним связыванием в заголовочном файле. не поможет твой совет

2pproger, не делать же каждую функцию inline ?

такие можно

[ROD]

такие можно

Ну такую да, а если что-то будет более громоздкое? (хотя скорее всего компилятор ее, на самом деле, не сделает inline)

О, к стати... не обратил внимания сразу. В .h файлах лучше держать прототип функций, а сами функции реализовывать в .cpp файлах. Пример:

Файл operators.h

#ifndef OPERATORS_H
#define OPERATORS_H
//функции вида
double Mnogennja(double l, double r);
#endif

Файл operators.cpp

#include "operators.h"
double Mnogennja(double l, double r)
{return l*r;}

При этом вы везде включаете только operators.h. Файл operators.cpp никуда включать не надо (ну он только в самом проекте должен быть)

[pproger]

2ROD

Ну такую да, а если что-то будет более громоздкое? (хотя скорее всего компилятор ее, на самом деле, не сделает inline)

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

[Antej]

Большое спасибо, комрады.
Пересобрать проекте посредством make clean, make я пробовал. встраиваемые сработали. Насчет разделения реализации обьявления - я так полагая это и будет и стилистически правельнее, или без разницы?

[pproger]

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

Большое спасибо, комрады.
Пересобрать проекте посредством make clean, make я пробовал. встраиваемые сработали. Насчет разделения реализации обьявления - я так полагая это и будет и стилистически правельнее, или без разницы?

http://stackoverflow.com/questions/1...-not-to-use-it

[Antej]

pproger, большое спасибо еще раз.

[_Bers]

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

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

Неужели?

#ifndef OPERATORS_H
#define OPERATORS_H

struct Test
{
   static double Mnogennja(double l, double r){return l*r;}
};
#endif

Все методы определенные в декларации класса по дефолту являются inline