Ошибка lnk1169 обнаружен многократно определенный символ один или более

Программа в с++ выдаёт: ошибку обнаружен многократно определенный символ — один или более

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

А ВЕРОЯТНАЯ причина вот какая. Вы поместили определение функции (подчеркиваю, именно определение, то есть ТЕЛО функции, а не ее декларацию) в *.h файл. Если этому файлу сделан #include только в ОДИН *.cpp файл, то линковка пройдет и линковщик ни на что не пожалуется. Но если этому файлу сделан #include в два или более *.cpp файлов, то после компиляции в каждом из них возникнет та же самая функция. Линковщик выругается, что видит больше одной функции с одним и тем же прототипом. Решения у этой проблемы два: или перенести тело функции в *.cpp файл, оставив в *.h файле только декларацию (прототип) , или оставить функцию на месте, но объявить ее inline.

Тот же самый эффект может получиться не только с функциями, но и с другими объектами. Скажем, для статических членов класса нужно давать определение в *.срр файле, за пределами определения класса. Но если вы поместите это определение в *.h файл, а затем сделаете этому файлу #include в два или более *.cpp файлов, то линковщик выругается, что он видит несколько определений одного и того же символа.

Кстати, Михаил Левин неправ: чтобы получить такую жалобу от линковщика, совершенно не обязательно сдублировать определение ИМЕННО фунции main. Любая функция проканает.

Источник

Fatal error lnk1169 обнаружен многократно определенный символ один или более

Лучший отвечающий

Вопрос

Язык программирования: C

Среда разработки: Visual Studio 2012 Express

Не могу понять следующее: я хочу связать между собой main.c и count.c(как они выглядят можно найти ниже).

В count.c определена переменная num, которая равна 10. В main.c я хочу вывести переменную num на экран.

Если определить num как static int num = 10, то всё работает, однако как только я уберу static выходят следующие ошибки:

Ошибка 1 error LNK2005: _b уже определен в count.obj c:UsersМОЁ ИМЯdocumentsvisual studio 2012Projectstest_29test_29main.obj test_29

Ошибка 2 error LNK1169: обнаружен многократно определенный символ — один или более c:usersМОЁ ИМЯdocumentsvisual studio 2012Projectstest_29Debugtest_29.exe test_29

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

Источник

фатальная ошибка LNK1169: один или несколько кратно определенных символов, найденных в игровом программировании

Я тренировался использовать объектно-ориентированное программирование в C ++, но я продолжаю получать эту ошибку:

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

Решение

Два int переменные определенный в заголовочном файле. Это означает, что каждый исходный файл, который включает заголовок, будет содержать их определение (включение заголовка является чисто текстовым). Конечно, приводит к множественным ошибкам определения.

У вас есть несколько вариантов, чтобы это исправить.

Сделать переменные static ( static int WIDTH = 1024; ). Они будут по-прежнему существовать в каждом исходном файле, но их определения не будут видны за пределами исходного файла.

Превратите их определения в декларации, используя extern ( extern int WIDTH; ) и положить определение в один исходный файл: int WIDTH = 1024; ,

Вероятно, лучший вариант: сделать переменные const ( const int WIDTH = 1024; ). Это делает их static неявно, а также позволяет использовать их в качестве констант времени компиляции, что позволяет компилятору использовать их значение напрямую, а не выдавать код для чтения его из переменной и т. д.

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

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

#pragma once просто для защиты от нескольких включений в одном исходном файле, а не от нескольких включений в нескольких исходных файлах.

Вы могли бы объявлять переменные как extern в заголовочном файле, а затем определять их в одном исходном файле. Или же вы можете объявить переменные как const в заголовочном файле, и компилятор и компоновщик будут управлять им.

Источник

Fatal error lnk1169 обнаружен многократно определенный символ один или более

Лучший отвечающий

Вопрос

Язык программирования: C

Среда разработки: Visual Studio 2012 Express

Не могу понять следующее: я хочу связать между собой main.c и count.c(как они выглядят можно найти ниже).

В count.c определена переменная num, которая равна 10. В main.c я хочу вывести переменную num на экран.

Если определить num как static int num = 10, то всё работает, однако как только я уберу static выходят следующие ошибки:

Ошибка 1 error LNK2005: _b уже определен в count.obj c:UsersМОЁ ИМЯdocumentsvisual studio 2012Projectstest_29test_29main.obj test_29

Ошибка 2 error LNK1169: обнаружен многократно определенный символ — один или более c:usersМОЁ ИМЯdocumentsvisual studio 2012Projectstest_29Debugtest_29.exe test_29

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

Источник

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

Я тренировался использовать объектно-ориентированное программирование на С++, но я продолжаю получать эту ошибку:

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

задан 20 марта ’13, 07:03

6 ответы

Эти два int переменные определенный в заголовочном файле. Это означает, что каждый исходный файл, который включает заголовок, будет содержать их определение (включение заголовка является чисто текстовым). Это, конечно, приводит к множественным ошибкам определения.

У вас есть несколько вариантов исправить это.

Сделайте переменные static ( static int WIDTH = 1024; ). Они по-прежнему будут существовать в каждом исходном файле, но их определения не будут видны за пределами исходного файла.

Превратите их определения в объявления, используя extern ( extern int WIDTH; ) и поместите определение в one исходный файл: int WIDTH = 1024; .

Вероятно, лучший вариант: сделать переменные const ( const int WIDTH = 1024; ). Это делает их static неявно, а также позволяет использовать их в качестве констант времени компиляции, позволяя компилятору использовать их значение напрямую, вместо того, чтобы выдавать код для чтения его из переменной и т. д.

Источник

I have 3 files:

SilverLines.h
SilverLines.cpp
main.cpp

At SilverLines.cpp I have:

#include "SilverLines.h."

When I don’t do:

#include "SilverLines.h"

at the main.cpp, everything is just fine. The project compiles.

But when I do:

#include "SilverLines.h"

in the main.cpp (Which i need to do), i get these 2 errors:

What’s the problem?

> edit:

As you requested, this is the entire *.h code:

#ifndef SILVER_H
#define SILVER_H

#include <iostream>
#include <algorithm>
#include <iterator>
#include <vector>
#include <string>
#include <map>

using namespace std;

typedef unsigned short int u_s_int;

map<string /*phone*/,string /*company*/> companies; // a map of a phone number and its     company
string findCompany(const string& phoneNum); //Given a phone number, the function     returns the right company

//The Abstract Client Class:

class AbsClient //Abstract Client Class
{
protected:
//string phone_number;
int counter; //CALL DURATION TO BE CHARGED
double charge;
public:
string phone_number; //MOVE TO PROTECTED LATER
void setCharge(double ch) {charge=ch;}
void setCoutner(int count) {counter=count;}
AbsClient(string ph_num): phone_number(ph_num),counter(0), charge(0.0) {} //c'tor     that must be given a phone#.
                                                                            //Initializes the counter and the charge to 0.
virtual void registerCall(string /*phone#*/, int /*minutes*/) = 0; //make a call     and charge the customer
void printReport(string /*phone#*/) const; //prints a minutes report
};

//The Temporary Client Class:

class TempClient : public AbsClient //TempClient class (NO DISCOUNT!) 2.3 NIS PER MINUTE, inherits from ABS
{
private:
string company;
public:
//string company;
TempClient(string ph_num, string cmp_name): AbsClient(ph_num), company(cmp_name) {}
virtual void registerCall(string /*phone#*/, int /*minutes*/); //make a call and     charge the customer
//virtual void printReport(string phone) const {cout << "the number of minutes: " << this->counter << endl;}
};

//The REgistered Client Class:

class RegisteredClient : public AbsClient //RegisteredClient, 10% DISCOUNT! , inherits from ABS
{
protected:
string name;
string id;
long account; //ACCOUNT NUMBER
private:
static const int discount = 10; //STATIC DISCOUNT OF 10% FOR ALL Registered Clients
public:
RegisteredClient(string ph_num, string n, string i, long acc_num):     AbsClient(ph_num), name(n) , id(i) , account(acc_num) {}
virtual void registerCall(string /*phone#*/, int /*minutes*/); //make a call and     charge the customer
//virtual void printReport(string /*phone#*/) const {cout << "the number of     minutes: " << this->counter << endl;}
};

//The VIP Client Class

class VIPClient : public RegisteredClient //VIP Client! X% DISCOUNT! also, inherits from RegisteredClient
{
private: //protected?
int discount; //A SPECIAL INDIVIDUAL DISCOUTN FOR VIP Clients
public:
VIPClient(string ph_num, string n, string i, long acc_num, int X):     RegisteredClient(ph_num,n,i,acc_num), discount(X) {}
virtual void registerCall(string /*phone#*/, int /*minutes*/); //make a call and     charge the customer
//virtual void printReport(string /*phone#*/) const {cout << "the number of     minutes: " << this->counter << endl;}
};

//The SilverLines (the company) class

class SilverLines // The Company Class
{
protected:
vector<AbsClient*> clients;
vector<string> address_book; //DELETE
public:
//static vector<AbsClient*> clients;
//SilverLines();
void addNewClient(string /*phone#*/);
void addNewClient(string /*phone#*/, string /*name*/ , string /*id*/ , u_s_int     /*importance lvl*/ , long /*account#*/);
void registerCall(string /*phone#*/ , int /*call duration*/);
void resetCustomer(string /*phone#*/); //given a phone#, clear the appropriate     customer's data ('charge' and 'counter')
void printReport(string /*phone#*/) const;

~SilverLines(){
    vector<AbsClient*>::iterator it;
    for(it = clients.begin(); it != clients.end(); ++it)
        delete(*it);
}
};

#endif

paercebal’s Answer:

Your code has multiple issues (the

using namespace std;

for one), but the one failing the compilation is the declaration of a global variable in a header:

map<string /*phone*/,string /*company*/> companies;

I’m quite sure most of your sources (main.cpp and SilverLines.cpp, at least) include this header.

You should define your global object in one source file:

// SilverLines.h

extern map<string /*phone*/,string /*company*/> companies;

and then declare it (as extern) in the header:

// global.cpp

extern map<string /*phone*/,string /*company*/> companies;