Ошибки VS 2010 C++ LNK2005 при использовании #pragma Once и #ifndef

1>Deck.obj : error LNK2005: "class Card card" (?card@@3VCard@@A) already defined in Card.obj
1>PokerTester.obj : error LNK2005: "class Card card" (?card@@3VCard@@A) already defined in Card.obj
1>PokerTester.obj : error LNK2005: "class Deck deck" (?deck@@3VDeck@@A) already defined in Deck.obj
1>C:\Dev\Poker\Debug\Poker.exe : fatal error LNK1169: one or more multiply defined symbols found

Я узнал, почему эти ошибки возникают, погуглив, но я не знаю, почему они все еще происходят, когда я однажды попробовал #pragma и защиту #ifndef.

Вот моя карта.h

#pragma once

#ifndef CARD_H
#define CARD_H

#include <iostream>
#include <string>
using namespace std;

class Card
{
public:
    Card(int cardSuit = 0, int cardValue = 2); //constructor will create a two of hearts by default
    ~Card(void);
    int getSuit(); //returns the suit of the Card
    int getValue(); //returns the value of the Card
    int getColor(); //returns the color of the Card
    friend ostream& operator<< (ostream &out, Card &cCard);

private:
    int suit; //card suit
    int value; //card value
    int color; //card color
} card;

#endif

и мой Deck.h

#pragma once

#ifndef DECK_H
#define DECK_H

#include "Card.h"
#include <vector>
using namespace std;

class Deck
{
public:
     Deck(void);
    ~Deck(void);
    void newDeck(); //regenerates the full 52 card deck (e.g. cards are missing)
    void shuffle(); //shuffles the deck
    int cardsInDeck(); //returns the number of cards remaining in the deck
    Card takeTopCard(); //returns the top card and removes it from the deck
private:
    vector<Card> myDeck; //vector of 52 Card objects that make up the deck
} deck;

#endif

Наверное, это очевидно, но я не могу понять...

по запросу вот Card.cpp:

#include "Card.h"

Card::Card(int cardSuit, int cardValue)
{
card.suit = cardSuit;
card.value = cardValue;
if(cardSuit == 0 || cardSuit == 1) card.color = 0;
if(cardSuit == 2 || cardSuit == 3) card.color = 1;
}

//returns the card's color
int Card::getColor() 
{
return card.color;
}

//returns the card's suit
int Card::getSuit()
{
return card.suit;
}

//returns the card's value
int Card::getValue()
{
return card.value;
}

и вот что я написал, чтобы проверить их:

#include "Deck.h"

int main() 
{
Deck testDeck = *new Deck();
Card testCardCreation = *new Card();
Card testCard = testDeck.takeTopCard();
testDeck.shuffle();
Card testShuf = testDeck.takeTopCard();
cout << testCard << endl << testShuf << endl;

return 0;
}

c++ lnk2005
person Shmoopy    schedule 31.12.2012    source источник

Ответы (3)


arrow_upward
5
arrow_downward

Объекты card и deck определены в заголовке. Когда вы включаете заголовок в единицу перевода, он создает другое определение этого объекта. Вероятно, вам следует просто удалить card и deck из определения класса. Если вам действительно нужно, чтобы эти объекты были определены, вы должны использовать

extern Card card;
person Dietmar Kühl    schedule 31.12.2012
comment
ах так получилось. Благодарю вас! - person Shmoopy; 31.12.2012

arrow_upward
2
arrow_downward

Это ошибки компоновщика, они не связаны с кодом C++ как таковым.

Проблема в конце определений карт и колод. Поэтому у вас есть:

} card;

И:

} deck;

?

person user1610015    schedule 31.12.2012
comment
Я видел, как это делается в старом учебнике по C++, который я нашел в Интернете. Я не занимался C++ с тех пор, как прошлой весной провел первое занятие по нему, и это исправило мои ошибки. Дитмар дал мне лучший способ сделать это. - person Shmoopy; 31.12.2012

arrow_upward
0
arrow_downward

1) Если вы определили "карту класса" несколько раз, вы получите ошибку компиляции, а не ошибку ссылки.

2) По большей части ваши файлы заголовков выглядят нормально. На самом деле вам не нужны оба: одного или другого из #pragma Once или #ifndef должно быть достаточно.

3) ПРОБЛЕМА: исключите последнюю «карту» и «колоду» из определений вашего класса!

person paulsm4    schedule 31.12.2012