使用私有的成员变量c ++

时间:2014-09-12 00:55:31

标签: c++

我添加了所有代码。

我遇到的确切问题是当我在private文件中创建成员变量game.h时,我在game.cpp文件中收到错误,其中n p h都是私有成员game.h.在Xcode

但是当我从命令行编译程序时,它编译得很好而没有错误。

我试图了解我是否做错了什么,或者这是否符合我的标准?

这是main.cpp

#include "game.h"


int main() {

   game g("Female", "Magic", true, 21, 5, 120);

    std::cout << "These are the things every game needs to be a game" << '\n';

    std::cout << g << '\n';

    return 0;
}

game.cpp

#include <iostream>

#include "game.h"

std::ostream& operator<<(std::ostream& s, const game& g) {
    return s << &g.n << ' ' << &g.p  << ' ' <<  &g.h;
}

这是我的复合类

#include <iostream>

#include "npc.h"
#include "pc.h"
#include "health.h"

class game {
private:
    npc n;
    pc p;
    health h;

public:
    game(const npc& init_n, const pc& init_p, const health& init_h):
    n(init_n),
    p(init_p),
    h(init_h)
    {}


    game(std::string gen, std::string abil, bool use, int lvl, int h, int arm) :
    n(gen, abil),
    p(use, lvl),
    h(h, arm)
    {
    }

 friend std::ostream& operator<<(std::ostream& s, const game& g) {
         g.n.output(s);
         g.p.output(s);
         g.h.output(s);
        return s;
 }

     npc get_n() { return n; }
     pc get_p() { return p; }
     health get_h() { return h; }

     void set_n(npc init_n) { n = init_n; }
     void set_p(pc init_p) { p = init_p ; }
     void set_h(health init_h) { h = init_h; }
};

这是一个班级

#include <iostream>

class health {
private:
    int hp;
    int armor;

public:
    health(int init_hp, int init_armor) :
    hp(init_hp),
    armor(init_armor)
    {
    }

public:
    void output(std::ostream& s) const { s << "Characters have this amount of hit points "<< hp << " and an armor rating of " << armor << "\n";  }

    };

这是一个班级

class pc {
private:
    bool user;
    int level;

public:
    pc(bool init_user, int init_level) :
    user(init_user),
    level(init_level)
    {
    }

public:
    void output(std::ostream& s) const { s << "A player character has at least "<< user << " user and a level of " << level << '\n';  }

};

这是一个班级

#include <iostream>

class npc {
private:
    std::string gender;
    std::string ability;

public:
    npc(std::string init_gender, std::string init_ability) :
    gender(init_gender),
    ability(init_ability)
    {
    }

public:
  void output(std::ostream& s) const { s << "A non player character has a gender of "<< gender << " and an ability of " << ability << '\n';  }

};

1 个答案:

答案 0 :(得分:3)

你犯了几个错误 - 错字是导致问题的原因。 该函数不允许访问这些成员,因为它不是该类的朋友。 朋友是(正确)std::ostream& operator<<(std::ostream& s, const game& g) 当你定义一个函数std::ostream& operator<<(std::ostream& s, const game g)时,请注意丢失的&符号。

此外,您的访问器应该是const,并返回一个const引用。 即,

 npc const& get_n() const { return n; }
 pc const& get_p() const { return p; }
 health const& get_h() const { return h; }

你的操纵者改变了错误的变量!您更改传递给函数的函数而不是该类的成员....但是,为三个私有成员添加直接操纵符是非常值得怀疑的。您必须将您的类视为一些抽象对象,并定义适用于该对象的运算符。如果您只是直接访问其所有成员,那么使用具有私有成员的类背后的对象导向理念几乎没有(这仍然比将它们公开为好十倍!)

最后,只是一个编码风格的提示。通常的做法是将CamelCase名称用于自定义类(即类Game),并且最好为私有成员添加前缀以将它们与函数参数区分开来。通常人们使用前缀m_。你真的应该使用完整的英语单词(不是缩写,更不用说单个字符了)。

这会把你的代码变成,比如......

class Game {
  private:
    Npc m_npc;
    Pc m_pc;
    Health m_health;

  public:
    Game(Npc const& npc, Pc const& pc, Health const& health) :
        m_npc(npc), m_pc(pc), m_health(health) { }