Question

所以我在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，所以不要对我的错误大笑。

Answer 1

这实际上没什么。

void Rename(char name[]){
    name = name;
}

实际上，它相当于

void Rename(char *p){
    p = p;
}

你想要

void Rename(char name[]){
    strcpy(this->name, name);
}

但更好的方法是使用非冲突的名称并添加一些错误检查。

[编辑] “set_group（player pl []）”也存在同样的问题 - 感谢@Kevin抓住这个问题。

Answer 2

我建议你改变一些事情：

每次滚动新的随机数时都不应设置种子。尝试多次在循环内调用随机函数，每次都会看到相同的数字。
使用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开始，有更好的随机生成器。如果您需要高质量的随机性，则应使用它们而不是具有some issues的rand()。但是，在这种情况下，imho rand()就好了。

另请注意，在我的代码中，玩家组＆＃34;拥有＆＃34;玩家们。我的意思是addPlayer()调用push_back()，它会向向量添加副本。如果这不是您想要的，您可以考虑更改player_group以保留std::vector<player*>以避免复制。

类私有成员 - 数组访问失败

2 个答案: