好友功能无法访问私人模因

friend function cant access private memebers

本文关键字:访问 功能 好友      更新时间:2024-09-21

我开始学习运算符重载,起初似乎很容易,但现在在尝试创建全局功能运算符时访问私有成员时遇到问题

玩家.hpp

#ifndef _PLAYER_HPP_
#define _PLAYER_HPP_
#include <iostream>
#include <string>
#include "item.h"
class player
{
friend player operator+(player& obj, player& tem);
static int numPlayer;
float *health;
int mspeed;
int damage;
int xp;
std::string name;
public:
// Constructors
player(std::string = "player", float _health = 100, int _xp = 0);
// Copy Constructor
player(const player& obj);
// Move Constructor
player(player&& obj);
// Functions
void display();

// Friends functions
friend void test(player user);
friend player operator+(player &&obj, const item &tem);
// Diconstructors
~player();
};

#endif // _PLAYER_HPP_

播放器.cpp

#include "player.hpp"
#include "item.h"
#include <iostream>
#include <cstring>
#include <string>
int player::numPlayer = 0;
// Constructors
player::player(std::string _name, float _health, int _xp) {
numPlayer++;
this->health = new float;
*this->health = _health;
this->xp = _xp;
this->name = _name;
std::cout << "constructor for " << this->name << std::endl;
}
// Copy constructors
player::player(const player& obj) {
this->health = new float;
*this->health = *obj.health;
this->xp = obj.xp;
this->name = obj.name;
std::cout << "copy constructor for " << this->name << std::endl;
}
// Move Constructors
player::player(player&& obj) {
this->damage = 60;
this->mspeed = 50;
this->health = obj.health;
this->xp = obj.xp;
this->name = obj.name;
obj.health = nullptr;
std::cout << "Move constructor for " << this->name << std::endl;
}
void player::display() {
std::cout << "========================" << std::endl
<< this->name << std::endl
<< *this->health << std::endl
<< this->xp << std::endl
<< this->damage << std::endl
<< this->mspeed << std::endl;
}
player::~player() {
delete[] health;
std::cout << "distruction for: " << name << std::endl;
}
void test(player user) {
std::cout << user.name << std::endl;
}
player operator+(player&& obj, const item& tem) {
*obj.health += tem.health;
obj.damage += tem.damage;
obj.mspeed += tem.ms;
return obj;
}

项.h

#ifndef _ITEM_H_
#define _ITEM_H_
#include <iostream>
#include <string>
#include "player.hpp"
class item {
int damage; // Bonus damage
int health; // Bonus health
int ms; // Bonus Movement speed
std::string name; // item name
public:
//constructor
item(std::string name, int _damage = 0, int _health = 0, int _ms = 0)
: name {name}, damage {_damage}, health{_health}, ms {_ms}{}

friend player operator+(player &&obj,const item &tem);
};
#endif // _ITEM_

主.cpp

#include <iostream>
#include <string>
#include "player.hpp"
#include "item.h"
player operator+(player&& obj, const item& tem);
void test(player user);
void main(int args, char* argv) {
player a("YASOU96");
item deathSword("death Sword", 150, 0, 20);
a.display();
a = a + deathSword;
a.display();
}

我没有看到那里有错误,但它一直显示在 Visual Studio 项目类成员是私有的(无法访问 em),如果我在 player.hpp 和 item.h 标题顺序之间切换,我可以访问项目私有成员,然后我失去了对 player.hpp 私有成员的访问权限

任何帮助将不胜感激。

首先,错误 #1:

main.cpp:9:1: error: ‘::main’ must return ‘int’
9 | void main(int args, char* argv) {
| ^~~~
main.cpp:9:6: warning: second argument of ‘int main(int, char*)’ should be ‘char **’ [-Wmain]
9 | void main(int args, char* argv) {
|      ^~~~

修复很简单:

int main(int args, char* argv[])

int main([[maybe_unused]] int args, [[maybe_unused]] char* argv[])

甚至

int main()

错误#2:

In file included from player.hpp:6,
from main.cpp:3:
item.h:18:12: error: ‘player’ does not name a type
18 |     friend player operator+(player &&obj,const item &tem);
|            ^~~~~~

这更难解释。您的班级item取决于班级playerplayer取决于item。这是编译器无法完成的。溶液:

item.h替换

#include "player.hpp"

class player;

这是一个前瞻性声明。类item仅在此处使用player

friend player operator+(player &&obj,const item &tem);

也就是说,编译器只需要形成对player的引用,它不需要详细了解player实际上是什么。这是一个常见的"技巧":当类 A 仅使用指针或对 B 的引用时,B 的前向声明就完全足够了。此外,消除#include,您可以加快编译速度。

main.cpp: In function ‘int main(int, char*)’:
main.cpp:13:9: error: cannot bind rvalue reference of type ‘player&&’ to lvalue of type ‘player’
13 |     a = a + deathSword;
|         ^

不要使用你不理解的东西。或者更好的是:不要同时使用两件你不理解的东西。移动语义在移动构造函数之外很少见。在你成为专家之前,尽量不要在移动构造函数和移动operator=以外的地方使用 &&。实际上,即使你根本不会使用它们,你的程序也是完全正确的 - 不使用移动语义不会使程序不正确,它可能只会使它的运行速度比正确使用移动语义时慢一点。所以,转:

friend player operator+(player &&obj,const item &tem);

friend player operator+(player &obj, const item &tem);

此外,player中删除移动构造函数以及使用&&的任何情况,因为它不移动任何内容。你所做的只是向膝盖射击。

错误 #4

在所有这些更改之后,编译器提出了一系列类似类型的新抱怨:

player.cpp: In function ‘player operator+(player&&, const item&)’:
player.cpp:58:24: error: ‘int item::health’ is private within this context
58 |     *obj.health += tem.health;
|                        ^~~~~~
In file included from player.hpp:6,
from player.cpp:1:
item.h:11:9: note: declared private here
11 |     int health; // Bonus health

这是因为您搞砸了几乎所有的好友声明。 该修复类似于"item.hpp"中使用的修复程序。而不是

friend player operator+(player& obj,const  player& tem);

class item;

然后是真正的运算符+:

friend player operator+(player& obj, const item& tem);

错误 5*obj.health += tem.health;中删除 *

总论

  • 不要在没有编译的情况下编写大量代码。如果您正在学习一门新语言/如何编程,请写几行并编译。这样,您将始终面临 1 个,也许两个错误,通常很容易本地化。
  • 不要一次学习几件事。在这里,你已经表明你对以下内容的理解非常有限:
    • 如何处理多个源/头文件,
    • 什么是移动语义,
    • 如何使用好友声明。

我什至没有研究你的代码质量,我只是试图让它编译。

  • 如果你C++初学者,你不需要移动语义、朋友声明、运算符重载。尝试使用这些功能,您正在分散对真正重要目标的注意力。您无需成为C++专家即可使用它。循序渐进地学习,只使用你理解的工具,或者一次学习一件事。