Ошибка c2084 функция уже имеет текст реализации - Не ошибается лишь тот, кто ничего не делает!

Делаю одну программу для эксперимента и возникла одна ошибка.
Сделал новый проект и оставил ту же функцию и те же свойства, а ошибка так и осталась, в чем может быть дело?
Ошибка: функция «int main(void)» уже имеет текст реализации (C2084)

Source.cpp

#include <iostream>
#include <Windows.h>
#include "func.h"

using namespace std;

void Interface();

int main() {
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    Interface();
}

func.h

#pragma once
#include "Source.cpp"

void Interface() {
    int quest;
    while (true) {
        cout << "1. Открыть базу" << endl;
        cout << "2. Закрыть программу" << endl;
        cout << "Номер пути _b";
        cin >> quest;
        if (quest = 1) {
            cout << "Открыто!";
        }
        else if (quest = 2) {
            cout << "Закрыто!";
        }
    }
}

задан 1 янв 2018 в 19:25

У вас неверное понимание, что должно находиться в заголовочном файле, а что — в .cpp.

В заголовочном файле располагайте объявления, а определения — в cpp-файле. В заголовочном файле располагаются также inline-реализации, шаблоны и т.п. вещи, но в вашем случае все, что следует разместить в func.h — это

void Interface();

Все остальное — в .cpp-файлах, и не включать .cpp-файлы с помощью директивы #include — иначе вы получаете нарушение правила одного определения.

ответ дан 1 янв 2018 в 19:50

HarryHarry

215k15 золотых знаков117 серебряных знаков228 бронзовых знаков

Источник

Dmitriy1342

1 / 1 / 0

Регистрация: 07.02.2012

Сообщений: 37

Ошибка: Функция уже имеет текст реализации.

13.02.2012, 14:54. Показов 21757. Ответов 7

Метки нет (Все метки)

Вылезает данная ошибка error C2084: функция «double hypot(double,double)» уже имеет текст реализации. Это мой вариант программы, пробовал скопировать текст c учебника — не помогло. Что делать?

C++

#include <iostream>
#include <cmath>
#include <locale>
#include <conio.h>
 
using namespace std;
double hypot (double a, double b); 
 
int main()
{
    setlocale (LC_ALL, "Russian");
 
    double a, b;
 
    cout <<"Введите значение первого катета: ";
    cin >>a;
    cout <<"Введите значение второго катета: ";
    cin >>b;
    cout <<"При катетах "<<a<<" и "<<b<<" гипотенуза равна "<<hypot (a, b);
    getch();
    return 0;
}
 
double hypot (double a, double b)
{
    return sqrt ((a*a)+(b*b));
}

Комментарий модератора

Используйте теги форматирования кода!

5054 / 3115 / 271

Регистрация: 11.11.2009

Сообщений: 7,045

13.02.2012, 15:00

Dmitriy1342, переименуйте функцию.

return (true);

1976 / 1111 / 221

Регистрация: 19.04.2011

Сообщений: 2,345

13.02.2012, 15:02

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

уже имеет текст реализации

Русским по белому написано, функция с таким именем и насколько помнится с таким же содержанием уже описана в math.h
Назовите свою функцию по другому или воспользуйтесь стандартной

~~-=ЮрА=-~~

Заблокирован

13.02.2012, 17:13

Dmitriy1342, а можете не менять название а просто записать функцию в вашем собственном namespace ИМХО все так любят ставить std:: почему бы не воспользоваться рефакторингом в свою пользу!

C++

#include <iostream>
#include <cmath>
using namespace std;
 
namespace my
{
    double hypot (double a, double b)
    {
            return sqrt ((a*a)+(b*b));
    } 
};
 
int main()
{
    //setlocale (LC_ALL, "Russian");
    system("chcp 1251");
    double a, b;
    cout <<"Введите значение первого катета: ";
    cin >>a;
    cout <<"Введите значение второго катета: ";
    cin >>b;
    cout<<"При катетах "<<a<<" и "<<b
        <<" гипотенуза равна "
        <<my::hypot (a, b)//Вот использование рефакторинга во всей красе
        <<endl;
    system("pause");
    return 0;
}

PS:Раз уж мы хотим стать «продвинутыми дядьками», то почему бы по продвинутому не действовать?

Миниатюры

Dmitriy1342

1 / 1 / 0

Регистрация: 07.02.2012

Сообщений: 37

13.02.2012, 17:35

[ТС]

Сообщение от -=ЮрА=-

C++

#include <iostream>
#include <cmath>
using namespace std;
 
namespace my
{
    double hypot (double a, double b)
    {
            return sqrt ((a*a)+(b*b));
    } 
};
 
int main()
{
    //setlocale (LC_ALL, "Russian");
    system("chcp 1251");
    double a, b;
    cout <<"Введите значение первого катета: ";
    cin >>a;
    cout <<"Введите значение второго катета: ";
    cin >>b;
    cout<<"При катетах "<<a<<" и "<<b
        <<" гипотенуза равна "
        <<my::hypot (a, b)//Вот использование рефакторинга во всей красе
        <<endl;
    system("pause");
    return 0;
}

PS:Раз уж мы хотим стать «продвинутыми дядьками», то почему бы по продвинутому не действовать?

Хм, а что такое system(«chcp 1251»); ?

~~IFree Host~~

Заблокирован

13.02.2012, 17:39

chcp — изменение кодовой страницы cmd.exe

Очень часто нужно, чтобы вместо кодовой страницы cp866 (заданной по-умолчанию) данные были в cp1251 (команда chcp 1251) или в utf8 (chcp 65001).

~~-=ЮрА=-~~

Заблокирован

13.02.2012, 17:41

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

Хм, а что такое system(«chcp 1251»); ?

— да єто руссификация на старых компиляторах, вместо неё поставь + в хедеры #include <locale>

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

setlocale (LC_ALL, «Russian»);

видишь же я закоментил её…

1 / 1 / 0

Регистрация: 07.02.2012

Сообщений: 37

13.02.2012, 17:42

[ТС]

А, понятно, спасибо.

Источник

У вас неверное понимание, что должно находиться в заголовочном файле, а что — в .cpp.

void Interface();

Источник

Перейти к контенту

Source.cpp

#include <iostream>
#include <Windows.h>
#include "func.h"

using namespace std;

void Interface();

int main() {
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    Interface();
}

func.h

#pragma once
#include "Source.cpp"

void Interface() {
    int quest;
    while (true) {
        cout << "1. Открыть базу" << endl;
        cout << "2. Закрыть программу" << endl;
        cout << "Номер пути _b";
        cin >> quest;
        if (quest = 1) {
            cout << "Открыто!";
        }
        else if (quest = 2) {
            cout << "Закрыто!";
        }
    }
}

задан 1 янв 2018 в 19:25

У вас неверное понимание, что должно находиться в заголовочном файле, а что — в .cpp.

void Interface();

ответ дан 1 янв 2018 в 19:50

HarryHarry

210k15 золотых знаков114 серебряных знаков224 бронзовых знака

Remove From My Forums

Question
Hi All,

I’m migrating an existing project from vs 6.0 to vs 2008. I get the below error,

» error C2084: function ‘tstring::tstring(void)’ already has a body «

Error Code in tstring.cpp file:
```
tstring::tstring()
{
}
```
As the error message suggest there is another constructor for the same function in the header file tstring.h:
```
// Constructors
inline tstring::tstring() : base_class()
{
}
```
I don’t understand why «tstring::tstring()» constructor is there twice in cpp and h file and how to fix this error. Shall I remove tstring::tstring(){} completely?, that way a duplicate entry is removed.. Any help to fix this issue is
greatly appreciated. Thanks in advance!

P.S: This error does not occur when compiled in vs 6.0

Regards,

Ankush

Answers

On 19/02/2014 12:29, ankushkumar wrote:
» error C2084: function ‘tstring::tstring(void)’ already has a body/ /»
Error Code in tstring.cpp file:
```
tstring::tstring()
{
}
```
As the error message suggest there is another constructor for the same function in the header file tstring.h:
```
// Constructors
inline tstring::tstring() : base_class()
{
}
```
I don’t understand why «tstring::tstring()» constructor is there twice in cpp and h file and how to fix this error. Shall I remove tstring::tstring(){} completely?, that way a duplicate entry is removed..
You may want to remove one of the two. Considering that the body is just empty, it seems a good candidate to be inlined, so I’d just use this in the header file:
```
inline tstring::tstring()
{
}
```
I’m not sure about the base_class() initialization… should it be just automatic?

P.S: This error does not occur when compiled in vs 6.0

Note that the C++ compiler that ships with VS2008 is better than the one in VC6 (VS2008’s C++ compiler conforms to the C++98/03 standard, VC6 compiler doesn’t).
So, it’s very possible that the C++ compiler that comes with VS2008 emits several errors that the VC6 compiler ignored.

Giovanni
- Marked as answer by
  Tuesday, February 25, 2014 8:34 AM

I receive the following error:

1> resistor.cpp(7): error C2084: function 'Resistor::Resistor(int,std::string,double,int [])' already has a body
1>          resistor.h(25) : see previous definition of '{ctor}'

With every single one of my class functions, even though in resistor.h I have don’t have any empty implementations:

Resistor.h:

class Resistor
{
private:
   int rIndex;
   double resistance; // resistance (in Ohms)
   string name; // C++ string holding the label
   int endpointNodeIDs[2]; // IDs of nodes it attaches to

public:    
   Resistor(int rIndex_,string name_,double resistance_,int endpoints_[2]);

}

Resistor.cpp:

Resistor::Resistor(int rIndex_,string name_,double resistance_,int endpoints_[2])
{
    if (nodeArray[endpoints_[0]].addResistor(rIndex_) && NodeArray[endpoints_[1]].addResistor(rIndex_))
{
    rIndex = rIndex_;
    name = name_;
    resistance = resistance_;
    endpointNodeIDs[0] = endpoints_[0];
    endpointNodeIDs[1] = endpoints_[1];
}

return;
}

etc. for each of my class functions

Can anybody help me?

p.s. I also receive the following error, once again for every function in resistor class (except for the constructor, mysteriously):

1>rparser.cpp(301): error C2264: 'Resistor::setIndex' : error in function definition or declaration; function not called

I receive the following error:

1> resistor.cpp(7): error C2084: function 'Resistor::Resistor(int,std::string,double,int [])' already has a body
1>          resistor.h(25) : see previous definition of '{ctor}'

With every single one of my class functions, even though in resistor.h I have don’t have any empty implementations:

Resistor.h:

class Resistor
{
private:
   int rIndex;
   double resistance; // resistance (in Ohms)
   string name; // C++ string holding the label
   int endpointNodeIDs[2]; // IDs of nodes it attaches to

public:    
   Resistor(int rIndex_,string name_,double resistance_,int endpoints_[2]);

}

Resistor.cpp:

Resistor::Resistor(int rIndex_,string name_,double resistance_,int endpoints_[2])
{
    if (nodeArray[endpoints_[0]].addResistor(rIndex_) && NodeArray[endpoints_[1]].addResistor(rIndex_))
{
    rIndex = rIndex_;
    name = name_;
    resistance = resistance_;
    endpointNodeIDs[0] = endpoints_[0];
    endpointNodeIDs[1] = endpoints_[1];
}

return;
}

etc. for each of my class functions

Can anybody help me?

p.s. I also receive the following error, once again for every function in resistor class (except for the constructor, mysteriously):

1>rparser.cpp(301): error C2264: 'Resistor::setIndex' : error in function definition or declaration; function not called

Forum
Beginners
error c2084

error c2084

#include <Windows.h>
#include <iostream>
#include<conio.h>
#include<iomanip>
using namespace std;

void classRec(); void end(); void editRec(); void gotoxy(int x,int y);
;
void menu()
{
char ch;
do
{ cout<<«nnntMAIN MENU»;
cout<<«nnt1. CLASS RECORD»;
cout<<«nnt2. EDIT RECORDS»;
cout<<«nnt3. HELP»;
cout<<«nnt4. EXIT»;

cout<<«nntPlease Select Your Option (1-3): «;
k:
cin>> ch;

switch(ch)
{
case ‘1’: classRec();

break;
case ‘2’: editRec();
break;
case ‘3’: end();
break;
default :
gotoxy(8,15);
cout<<«Please enter a valid choice: «;
goto k;

}
}while(ch!=’3′);
system(«cls»);
system(«pause>0»);

}

I can’t run my program. It says that there’s an error,error C2084: function ‘void menu(void)’ already has a body. How can I solve this? Please help me. Thank you. =)

It would be more helpful if you showed the exact error the compiler produced. It will name one or more files, so we’ll need to those too (with their names).

Error 1 error C2084: function ‘void menu(void)’ already has a body c:userspaulinedocumentsvisual studio 2010projectsclassrecordclassrecordmenu.h 11

Please help me. I’m just a beginner in programming and I badly needed to finish this program. Thank you so much. =)

And what’s in menu.h?

And why is there no reference to menu.h in the posted code?

Last edited on

Topic archived. No new replies allowed.

Вопрос:

Я не мог понять, что мне нужно сделать, чтобы исправить эту ошибку или найти что-либо на этом веб-сайте. В основном я получаю ошибку C2084: функция “Калькулятор :: GUI :: GUI (void)” уже имеет тело. Все, что у меня есть, – это форма окна, называемая GUI, добавленная в приложение Win32, калькулятор.

В GUI.h:

#pragma once

namespace Calculator {

using namespace System;
using namespace System::ComponentModel;
using namespace System::Collections;
using namespace System::Windows::Forms;
using namespace System::Data;
using namespace System::Drawing;

/// <summary>
/// Summary for GUI
/// </summary>

public ref class GUI : public System::Windows::Forms::Form
{

void AddControls();
public:
GUI()
{
InitializeComponent();
//
//TODO: Add the constructor code here
//


}

и в GUI.cpp

#include "GUI.h"

namespace Calculator {

GUI::GUI()
{

}

void DrawButtons();
void DrawLabels();

void GUI::AddControls()
{
DrawButtons();
DrawLabels();
}

Я получил то, что хотел работать, поместив все в файл GUI.h, но хотел иметь код метода внутри.cpp файла.

Лучший ответ:

Измените заголовок следующим образом:

public ref class GUI : public System::Windows::Forms::Form
{

void AddControls();
public:
    GUI();
}

Вы видите, что заголовок должен содержать только декларации и вносить реализацию в cpp.

Источник

Форум программистов Vingrad

Модераторы: bsa

Поиск:

Несрабатывает класс

Опции темы

stat007

Дата 22.11.2009, 15:46 (ссылка)

(нет голосов)

Загрузка …

Опытный

Профиль
Группа: Участник
Сообщений: 521
Регистрация: 9.10.2008

Репутация: -2
Всего: -4

Подскажите плиз, почему не хочет компилироваться привидённый ниже код?
Проект написан в среде: Microsoft Visual C++ 2008 Express Edition……
Принаписании проекта я создал 3 файла: game.h, game.cpp и main.cpp…. а так же при создании автоматически в этой среде создаются ещё несколько файлов. Такие как: stdafx.h, targetver.h, stdafx.cpp… Ну вообщем кто работает в этой среде, тот знает…
Вообщем какие только я #include не писал — непомогает. Подскажите что нужно сделать??

Код


//Файл game.h
#include "stdafx.h"
#ifdef GAME_H
#define GAME_H
class Game
{
public:
    bool run(void);
protected:
    bool getInput(char *c);
    void timerUpdate(void);
};
#endif

Код


//Файл game.cpp 
#include "stdafx.h"
#include "game.h"
#include<iostream>
using namespace std;
bool Game::run(void)                     [color=red]//ОШИБКА Тут[/color]
{
    return true;
}

Код


//Файл main.cpp
#include "stdafx.h"
#include<iostream>



  


                       
  




int _tmain(int argc, _TCHAR* argv[])
{
    system("pause");
    return 0;
}

После компиляции выдаёт одну ошибку (я указал где), и пишут:
error C2653: Game: не является именем класса или пространства имен

Alca

Дата 22.11.2009, 16:09 (ссылка)

(нет голосов)

Загрузка …

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 3993
Регистрация: 14.6.2006

Репутация: 2
Всего: 50

Код


#ifndef GAME_H
#define GAME_H

———————

C++ cross-platform library

zim22

Дата 22.11.2009, 16:10 (ссылка)

(нет голосов)

Загрузка …

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 29
Всего: 69

поменяй #ifdef GAME_H на
#ifndef GAME_H
***
Alca, smile

Это сообщение отредактировал(а) zim22 — 22.11.2009, 16:11

———————

C++ FAQ Lite

stat007

Дата 22.11.2009, 19:38 (ссылка)

(нет голосов)

Загрузка …

Опытный

Профиль
Группа: Участник
Сообщений: 521
Регистрация: 9.10.2008

Репутация: -2
Всего: -4

Спасибо, с этим разобрался, но теперь появилась ещё одна ошибка в той же строке, компилятор пишет:
error C2084: функция «bool Game::run(void)» уже имеет текст реализации

mes

Дата 22.11.2009, 22:16 (ссылка)

(нет голосов)

Загрузка …

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 79
Всего: 250

Цитата(stat007 @ 22.11.2009, 18:38

)

error C2084: функция «bool Game::run(void)» уже имеет текст реализации

ну так ищите где Вы два раза ее определили.. В приведенном фрагменте кода этого нет..
Вы случайно «.cpp» нигде не включили (#include) ?

———————

http://opendots.net

stat007

Дата 23.11.2009, 11:51 (ссылка)

(нет голосов)

Загрузка …

Опытный

Профиль
Группа: Участник
Сообщений: 521
Регистрация: 9.10.2008

Репутация: -2
Всего: -4

mes,
да нет!! Всё точ в точ как на листингах…

mes

Дата 23.11.2009, 16:34 (ссылка)

(нет голосов)

Загрузка …

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 79
Всего: 250

Цитата(stat007 @ 23.11.2009, 10:51

)

да нет!! Всё точ в точ как на листингах…

сильно сомневаюсь..чтоб убедиться выложите тогда текущее содержимое файла game.h

———————

http://opendots.net

bsa

Дата 23.11.2009, 16:44 (ссылка)

(нет голосов)

Загрузка …

Эксперт

Профиль
Группа: Модератор
Сообщений: 9185
Регистрация: 6.4.2006
Где: Москва, Россия

Репутация: 85
Всего: 196

stat007, кстати, stdafx.h нужно подключать в *.cpp, а не в хидерах.

———————

Правильно заданный вопрос — половина ответа
Отзывы о работе модератора.

stat007

Дата 23.11.2009, 22:47 (ссылка)

(нет голосов)

Загрузка …

Опытный

Профиль
Группа: Участник
Сообщений: 521
Регистрация: 9.10.2008

Репутация: -2
Всего: -4

mes,

Код


//Файл game.h
#ifndef game_dot_h
#define game_dot_h 1
class Game
{
public:
    bool run(void);
protected:
    bool getInput(char *c);
    void timerUpdate(void);
};
#endif

Это сообщение отредактировал(а) stat007 — 23.11.2009, 22:47

zim22

Дата 24.11.2009, 08:45 (ссылка)

(нет голосов)

Загрузка …

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 29
Всего: 69

Цитата(stat007 @ 23.11.2009, 21:47

)

#define game_dot_h 1

единица в конце зачем?

———————

C++ FAQ Lite

mes

Дата 24.11.2009, 11:17 (ссылка)

(нет голосов)

Загрузка …

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 79
Всего: 250

Цитата(zim22 @ 24.11.2009, 07:45

)

единица в конце зачем?

имхо, добавил просто потому, что увидел, что так делают другие.

———————

http://opendots.net

mes

Дата 24.11.2009, 11:51 (ссылка)

(нет голосов)

Загрузка …

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 79
Всего: 250

stat007, закомментируйте в cpp файле функцию /* bool Game::run(void) {…} */
и попробуйте еще раз откомпилить..

———————

http://opendots.net

zim22

Дата 24.11.2009, 11:55 (ссылка)

(нет голосов)

Загрузка …

depict1

Профиль
Группа: Завсегдатай
Сообщений: 2682
Регистрация: 15.1.2009
Где: Украина

Репутация: 29
Всего: 69

stat007, прикрепите к сообщению сам проект. только без файлов .ncb, .obj, .exe. только самое необходимое для его открытия.

———————

C++ FAQ Lite

stat007

Дата 24.11.2009, 23:06 (ссылка)

(нет голосов)

Загрузка …

Опытный

Профиль
Группа: Участник
Сообщений: 521
Регистрация: 9.10.2008

Репутация: -2
Всего: -4

mes, Закаментировал, так ошибок нет!!!
Дело в том, что если всё это написать в едином .cpp файле, то всё срабатывает, а не срабатывает именно когда это всё в нескольких файлах…

zim22, Вот сам проект, прикрепил…

Присоединённый файл ( Кол-во скачиваний: 4 )

Game.rar 100,19 Kb