2

質問の仕方がよくわからないので、質問がぎこちなくて申し訳ありません。いずれにせよ、aItem オブジェクトのベクトルを含む「aItem」というクラスの実装ファイルとヘッダーがあります。

「バッグ」というタイトルの 2 番目のファイルでは、前述のベクトルのすべての要素のリストを作成できるようにしたいと考えています。私は元々「バッグ」をヘッダーと実装ファイルに分けていましたが、2 つのファイルを結合するまで、コンパイラーは「宣言されていない参照」についてうまくいきませんでした。いずれにせよ、その部分は機能しています。

作成したアイテムの m_itemName 変数に bag.h (具体的には addItem 関数) からアクセスできるようにしたいと考えています。

aItem.cpp:

//aItem .cpp implementation file
#include "aItem.h"
#include <iostream>
#include <string>
#include <vector>
using namespace std;

//setting this up default
aItem::aItem()
{
    m_itemName = "Default Name";
    m_itemType = "Default Type";
    m_damage     = 9001;
}





void aItem::setName(string name)
{
    m_itemName = name;
}

void aItem::setType(string type)
{
    m_itemType = type;
}

void aItem::setDamage(int damage)
{
    m_damage = damage;
}

string aItem::getName()
{
    return m_itemName;
}

string aItem::getType()
{
    return m_itemType;
}

int aItem::getDamage()
{
    return m_damage;
}

aItem.h:

#ifndef AITEM_H_INCLUDED
#define AITEM_H_INCLUDED

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


class aItem
{
public:
    //constructor
    aItem();


    //Methods
    void ItemCreate(string itemName, string itemType, int damage);
    void setName(string name);
    void setType(string type);
    void setDamage(int damage);
    string getName();
    string getType();
    int getDamage();

private:

    string  m_itemName;
    string  m_itemType;
    int m_damage;


};

#endif // AITEM_H_INCLUDED

バッグ.h:

#ifndef BAG_H_INCLUDED
#define BAG_H_INCLUDED

#include "aItem.h"
#include <iostream>
#include <string>
#include <vector>
using namespace std;
class aItem;

class bag : public aItem
{
public:
    //constructor
    bag();

    //methods
    //void delItem(aItem aitem);

void addItem(aItem var)
    {
        m_items.push_back(var);
    }

void bagList()
    {
        for( vector<aItem>::size_type index = 0; index < m_items.size(); index++ )
            {
                //Makes a numerical list.
                cout << "Item " << index + 1 << ": " <<  m_items[index].m_itemName << endl;
                index+= 1;
            }
    }
private:
     vector<aItem> m_items;
};

#endif // BAG_H_INCLUDED

...そして最後に、main.cpp:

// WHYGODWHY.cpp : Defines the entry point for the console application.
//

#include "aItem.h"
#include "bag.h"
#include <iostream>
using namespace std;

int main()
{
    aItem woodSword;
    woodSword.setName("Wooden Sword");
    woodSword.setDamage(3);
    woodSword.setType("WPN");

    bag inventory;
    inventory.addItem(woodSword);
    inventory.bagList();


    cout << "Name: " << woodSword.getName() << endl;
    cout << "Type: " << woodSword.getType() << endl;
    cout << "Damage: " << woodSword.getDamage() << endl;
    //cout << "Inv: " <<
    int pause = 0;
    cout << "Pause!";
    cin >> pause;

    return 0;
}

編集: これはエラー メッセージです: C:\Users\dmarr\Documents\CSharper\Dallas\CPlusPlus\WHYGODWHYCB\aItem.h||メンバー関数内 'void bag::bagList()':| C:\Users\dmarr\Documents\CSharper\Dallas\CPlusPlus\WHYGODWHYCB\aItem.h|28|エラー: 'std::string aItem::m_itemName' は非公開です| C:\Users\dmarr\Documents\CSharper\Dallas\CPlusPlus\WHYGODWHYCB\bag.h|32|エラー: このコンテキスト内| ||=== ビルドが終了しました: 2 つのエラー、0 の警告 ===|

4

3 に答える 3

2

名前のパブリック アクセサーが既にあるので、それらを使用してみませんか?

cout << "Item " << index + 1 << ": " <<  m_items[index].getName() << endl;

アクセサーを変更して参照constを返すことを検討してください。const

const std::string& getName() const;

それとは別に、他の問題があります。特に、C++ のコンテナーは多態的なオブジェクトを保持しないため、 astd::vector<aItem>は保持のみできますaItem(オブジェクトは保持できませんbag)。完全に異なるレベルで、命名規則を再考してください。 Item の 1 つのエンティティaItemを意味するタイプ (さまざまなオブジェクトのカテゴリ) 名を使用することはほとんど意味がありません。これも名前と矛盾しています...bag

于 2012-11-20T12:37:58.550 に答える
1

getName()クラスのメンバーに直接アクセスする代わりに、メソッドを呼び出すことをお勧めします。また、なぜこの場合bagから継承するのですか? aItemあなたがやろうとしていることを正しく理解していれば、 aは abagではないaItemので、継承してはならず、aItemアクセスメソッドを使用し、直接メンバーにアクセスしないでください。別の方法として、WhozCraig の提案に従って、bag を aItem のフレンドとして宣言できます。

于 2012-11-20T12:41:56.193 に答える
1

クラスのパブリック インターフェイスを使用します。したがって、使用する代わりに

cout << "Item " << index + 1 << ": " <<  m_items[index].m_itemName << endl;

行う

cout << "Item " << index + 1 << ": " <<  m_items[index].getName() << endl;
于 2012-11-20T12:39:01.900 に答える