Вот решил перейти Visual C++ c CBuilder'a стал класс описывать, компилировал ..., вроде работало, а потом в дебагере получил:
ClassTMas error LNK2005: "public: __thiscall TMasV::TMasV(int)" (??0TMasV@@QAE@H@Z) already defined in ClassTMas.obj
ClassTMas error LNK2005: "public: __thiscall TMasV::~TMasV(void)" (??1TMasV@@QAE@XZ) already defined in ClassTMas.obj
ClassTMas error LNK2005: "public: void __thiscall TMasV::FillMasV(void)" (?FillMasV@TMasV@@QAEXXZ) already defined in ClassTMas.obj
ClassTMas error LNK2005: "public: void __thiscall TMasV::PrintMasV(void)" (?PrintMasV@TMasV@@QAEXXZ) already defined in ClassTMas.obj
ClassTMas fatal error LNK1169: one or more multiply defined symbols found
Помогите пожалуйста.

Код в студию
Посмотри по хелпу

Цитата

symbol already defined in object The given symbol, displayed in its decorated form, was multiply defined. Possible causes Accidentally linking with both the single-threaded and multithreaded libraries. Ensure that the application project file includes only the appropriate libraries and that any third-party libraries have appropriately created single-threaded or multithreaded versions. The symbol is a packaged function (created by compiling with /Gy) and was included in more than one file but was changed between compilations. Recompile all files that include symbol. The symbol is defined differently in two member objects in different libraries, and both member objects were used. An absolute is defined twice, with a different value in each definition. A header file declared and defined a variable. Possible solutions include: Declare the variable in .h: extern BOOL MyBool; and then assign to it in a .c or .cpp file: BOOL MyBool = FALSE;. Declare the variable static. Declare the variable selectany. If you use uuid.lib in combination with other .lib files that define GUIDs (for example, oledb.lib and adsiid.lib). For example: oledb.lib(oledb_i.obj) : error LNK2005: _IID_ITransactionObject already defined in uuid.lib(go7.obj) To fix, add /FORCE:MULTIPLE to the linker command line options, and make sure that uuid.lib is the first library referenced.

Help'a нету!

Код

/* Project ClassTMas */ #include "stdafx.h" int main() {     int n;     std::cout << "n: ";     std::cin >> n;     TMasV vec(n);          vec.FillMasV;     vec.PrintMasV;         getch();     return 0; }

Код

class TMasV {     public:         const int n; //Размерность массива         int *MasV;   //Указатель на массив         TMasV(int t);         ~TMasV();         void FillMasV();   //Заполнение массива         void PrintMasV();  //Печать массива         int FindMaxMasV(); //Поиск max в массиве }; TMasV::TMasV(int t): n(t) {     MasV = new int [n]; }; TMasV::~TMasV() {     delete[] MasV; }; void TMasV::FillMasV() {     for(int i=0; i<n; i++)         MasV[i]=i; }; void TMasV::PrintMasV() {     for(int i=0; i<n; i++)         std::cout << MasV[i] << std::endl;       }; int TMasV::FindMaxMasV() {     return 0; };

0. код который ты написал не обязаб работать так как в файле имеющем main() нет никакого отношения к классу TMasV. на среду разработки нельзя полагаться.

1. нет разделения на интерфейс-реализацию. нужен файл - интерфейс (заголовок) с объявлением класса и его методами, и другой с их реализациями. обычно делают так:
а) файл-заголовок ( interface )

Код

// filename: myclass.hh  #ifndef _MYCLASS_H_ #define _MYCLASS_H_ class MyClass{       public:               int myint;               MyClass();               double GetSecret() const;       private:               double secret; }; #endif

b) файл-реализация всех или части методов класса (implementation)

Код

// filename: myclass.cc #include "myclass.hh" #include <iostream> using namespace std; MyClass::MyClass(){             secret=-0.1L;             myint=3; } double MyClass::GetSecret() const{             cout << "My Secret is: " << secret << endl;             return secret; }

c) теперь используем класс MyClass в proc.cc:

Код

// filname: proc.c #include "myclass.hh" int main(){       MyClass mc;       mc.GetSecret();       return 0; }

2. у тебя ";" после определения ( а не объявления ) всех функций. это необязательно.
3. в главной программе у тебя написано:

Код

vec.FillMasV;     vec.PrintMasV;

ты скобочки забыл.

удачи.

Это сообщение отредактировал(а) bilbobagginz - 10.9.2005, 11:25

bilbobagginz спасибо. Сделал как ты показал, правда выдал ошибку на пункте b:
ClassTMas fatal error C1010: unexpected end of file while looking for precompiled header directive
затем в нем подключил "stdafx.h" и все заработало.
А поподробней нельзя про:

Код

#ifndef _MYCLASS_H_     #define _MYCLASS_H_ ... #endif

Это защита от повторного включения заголовочного файла. То есть, если большой проект и много заголовочных файлов, может случится, что какой-то хидер-файл включили несколько раз. Тогда возникают ошибки типа class redefinition. С этими директивами такое не пройдет.
Способ, кажется, устаревший Лучше используй

Код

#pragma once

Цитата

Specifies that the file will be included (opened) only once by the compiler in a build. This can reduce build times as the compiler will not open and read the file after the first #include of the module.

kometa_triatlon

Цитата(kometa_triatlon @ 10.9.2005, 17:01)

Способ, кажется, устаревший

Не способ устаревший, это просто Майкрософт опять своими нововведениями кормит. Им ведь не выгодно, чтобы код был портируемым

En_t_end
Во всяком случае, короче

Есаул. используй #ifdef, #define и #include
Есссно, при нужной ситуации.
a на самом деле почитай про пре-процессор.

ЗЫ
кстати... откуда пошло это "хидер"... вроде "хедер" .. если с английского.[плиз без обид]

Вот опять ошибка :
Project(Key) error C2512: 'ClassLock' : no appropriate default constructor available

ClassLock.h:

Код

/* Класс ClassLock класс мазаичного изображения замка */ #ifndef CLASSLOCK #define CLASSLOCK     class ClassLock {     public:         const int n; //Размерность матрицы         int **Matr;  //Указатель на матрицу         ClassLock(int t);         ~ClassLock();         void GenMazLock();   //Создание маз. изображения замка         void PrintMatr();    //Вывод маз. изображения замка }; #endif

ClassLock.cpp

Код

#include "stdafx.h" #include "ClassLock.h" using namespace std; ClassLock::ClassLock(int t): n(t) {     Matr = new int *[n];     for(int i=0;i<n;i++)         Matr[i]=new int[i]; } ClassLock::~ClassLock() {     delete[] Matr; } void ClassLock::GenMazLock() {     for(int i=0; i<n; i++)         for(int j=0; j<n; j++)             Matr[i][j]=rand()%2; } void ClassLock::PrintMatr() {     for(int i=0; i<n; i++)     {         for(int j=0; j<n; j++)             cout << Matr[i][j] << ' ';                     cout << endl;     } }

ClassKey.h

Код

/* Класс ClassKey класс подбора ключей */ #include "ClassLock.h" #ifndef CLASSKEY #define CLASSKEY     class ClassKey: public ClassLock {     public:         const int n; //Размерность матрицы         ClassKey(int t);         ~ClassKey();         void CopyMazLock(); //Создание маз. изображения замка         void TurnKey();     //Создание маз. изображения замка }; #endif

Ошибка сдесь.
ClassKey.cpp

Код

#include "stdafx.h" #include "ClassKey.h" using namespace std; ClassKey::ClassKey(int t): n(t) {/*<-Тут выдает ощубку */     Matr = new int *[n];     for(int i=0;i<n;i++)         Matr[i]=new int[i]; } ClassKey::~ClassKey() {     delete[] Matr; } void ClassKey::CopyMazLock() {     for(int i=0; i<n; i++)         for(int j=0; j<n; j++)             Matr[i][j]=rand()%2; } void ClassKey::TurnKey() {     for(int i=0; i<n; i++)     {         for(int j=0; j<n; j++)             cout << Matr[i][j] << ' ';                     cout << endl;     } }

Могу и ошибится , но зачем 2 раза объявлять

Цитата(Esaul @ 13.9.2005, 06:42)

const int n; //Размерность матрицы

в двух классах, тем более наследуемых?

Esaul
обшибка в том, что в конструкторе производного класса нужно явно вызывать конструктор базового, ибо no appropriate default constructor available (у базового класса нет конструктора по-умолчанию).
Т.е. должно быть так:

Код

ClassKey::ClassKey(int t):     ClassLock(t),    n(t) {    ...  }

Ну и Romikgy прав, разумеется. Это переопределение n, во-первых, очень нехорошее и может вызвать проблемы, во-вторых, нафиг никому не нужное.

Earnest
Спасибо за помощь. А я то думал, что он к конструктору по умолчанию прикапывается.
Кстати если "n" повторно необъявить он выдаёт:
Project(Key) error C2614: 'ClassKey' : illegal member initialization: 'n' is not a base or member
С "n" вроде все работает.
И маленький вопрос:
Конструкторы не наследутся, а деструкторы наследуются?

Конструкторы, как и деструкторы, не наследуются.

Конструктор базового класса просто вызывается до конструктора потомка, а деструктор базового после соответственно деструктора потомка.

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

З.Ы. Последнее можно делать всегда, независимо от того, есть ли в базовом классе конструктор по умолчанию (не закрытый).

Цитата(Esaul @ 13.9.2005, 19:04)

Кстати если "n" повторно необъявить он выдаёт: Project(Key) error C2614: 'ClassKey' : illegal member initialization: 'n' is not a base or member С "n" вроде все работает.

А ты его просто убери из инициализации: так инициализировать можно только члены данного класса, но не предков. n инициализируется в конструкторе базового класса. Если не уберешь n из производного класса, у тебя будут 2 различных n, а это нехорошо.
Добавлено @ 17:28

Цитата(Esaul @ 13.9.2005, 19:04)

Кстати если "n" повторно необъявить он выдаёт: Project(Key) error C2614: 'ClassKey' : illegal member initialization: 'n' is not a base or member С "n" вроде все работает.

А ты его просто убери из инициализации: так инициализировать можно только члены данного класса, но не предков. n инициализируется в конструкторе базового класса. Если не уберешь n из производного класса, у тебя будут 2 различных n, а это нехорошо.