所以我在C++中有这段代码,这只是学习C++的练习:
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
int random(int max){
srand(time(NULL));
return rand() % max;
}
class player{
public:
player(){
liv = 5;
mun = 3;
}
void Rename(char name[]){
name = name;
}
void set_group(player pl[]){
pl = pl;
}
void fire(){
if (liv > 0 and mun){
mun -= 1;
int hit = random(10);
printf("%d: ", hit);
printf("%s", pl[hit].get_name());
if (hit < 5 and pl[hit].live()){
pl[hit].hit();
printf("Player %s hit player %s!", name, pl[hit].get_name());
}
}
}
bool live(){
return liv;
}
void hit(){
liv -= 1;
}
char * get_name(){
return name;
}
private:
int liv, mun;
char name[100];
player *pl;
};
main(){
player pl[5];
char str[100];
int x;
for (x=0;x<5;x++){
sprintf(str, "%d", x + 1);
pl[x].Rename(str);
pl[x].set_group(pl);
}
pl[0].fire();
}
编译器(TDM-GCC 4.9.2 32-bit
)编译代码并返回0个警告和0个错误
当我运行程序时,我收到消息:classes.exe has encountered a problem and needs to close. We are sorry for the inconvenience.
我调试了它,发现printf("%s", pl[hit].get_name());
行是一个问题(实际上它是一个调试行,之后的一行实际上是问题,但这是一样的)。我还认为访问pl是个问题。
我对C和C++(尤其是C++)还比较陌生,我的主要编程语言是Python,所以不要对我的错误笑得太多。
这实际上没有任何作用。
void Rename(char name[]){
name = name;
}
事实上,它相当于
void Rename(char *p){
p = p;
}
你想要
void Rename(char name[]){
strcpy(this->name, name);
}
但更好的做法是使用不冲突的名称并添加一些错误检查。
[EDIT]"set_group(player pl[])"也存在同样的问题——感谢@Kevin发现了这一点。
我建议您更改以下几点:
- 您不应该在每次滚动新的随机数时设置种子。试着多次调用循环中的随机函数,你会发现每次都得到相同的数字
- 使用
std::vector
而不是c样式数组 - 使用
std::string
而不是字符 - 如果使用
std::cout
而不是printf
,则不必自己处理格式化 player
构造函数未设置名称。因此,可以创建一个不完整状态的玩家(即没有名字)- 在构造函数中,使用初始值设定项列表更有效
- 球员真的必须知道自己在哪个小组吗?我建议改变它,因为自动取款机将单个玩家的身份与他们之间的互动方式混合在一起
- 如果玩家组只有一个玩家怎么办?您的代码无法处理这种情况
- 有了你的代码,玩家可以自我攻击。不确定这是否是您想要的;)
总的来说,它可能看起来像这样:
#include <time.h>
#include <string>
#include <iostream>
#include <vector>
#include <cstdlib> // for rand()
void init_random(){srand(time(0));}
int random(int max){return rand() % max;}
class player{
public:
player(std::string name) : liv(5),mun(3),name(name){}
void Rename(std::string newname){name = newname;}
bool live(){return liv;}
void hit(){liv -= 1;}
void fire(){mun -= 1;}
std::string get_name(){return name;}
private:
int liv, mun;
std::string name;
};
class player_group{
public:
void addPlayer(const player& p){players.push_back(p);}
void fire(int i){
// avoid wrong input
if (i>=0 && i<players.size() && players.size()>1){
players[i].fire();
int other = i;
// avoid player hitting himself
while (other == i){other = random(players.size());}
players[other].hit();
std::cout << "Player "
<< players[i].get_name()
<< " hits player "
<< players[other].get_name()
<< std::endl;
}
}
private:
std::vector<player> players;
};
int main() {
init_random();
player_group pg;
pg.addPlayer(player("Peter"));
pg.addPlayer(player("Paul"));
pg.fire(0);
}
由于C++11有更好的随机生成器。如果你需要高质量的随机性,你应该使用它们,而不是有一些问题的rand()
。然而,在这种情况下,imho rand()
就可以了。
还要注意,在我的代码中,玩家组"拥有"玩家。我的意思是addPlayer()
调用push_back()
,它向向量添加了一个副本。如果这不是您想要的,您可以考虑将player_group
更改为容纳std::vector<player*>
以避免复制。