1

我是一个完整的 C++ 初学者。我正在尝试做一些看起来很基本的事情：创建某个类的对象，将其存储在向量中，然后输出该对象的值。目前，无论我存储什么字符，它都会输出一个矩形字符。这是代码：

#include <iostream>
#include <string>
#include <cstdlib>
#include <conio.h>
#include <time.h>
#include <windows.h> 
#include <vector>
#include <algorithm>
#include <map>
using namespace std;

class terrainType
{
    public:
        string name;
        char symbol;
        int freq;
        terrainType(string,char,int);
};
terrainType::terrainType(string name, char symbol, int freq)
{
    name=name;
    symbol=symbol;
    freq=freq;
}

int main()
{
    vector<terrainType> terrainTypes;
    terrainType dirt("dirt",'.',1);
    terrainTypes.push_back(dirt);
    cout << terrainTypes[0].symbol;
    return 0;
}

任何建议或背景信息表示赞赏。谢谢！

4

1 回答 1

5

您在构造函数中的三个分配实际上是无操作的（您将每个变量分配给它自己）：

terrainType::terrainType(string name, char symbol, int freq)
{
    name=name;
    symbol=symbol;
    freq=freq;
}

问题是您有两个称为name的东西，并且您希望编译器找出name=name左侧指的是其中一个，而右侧指的是另一个。

解决此问题的最简洁方法是更改​​为构造函数，如下所示：

terrainType::terrainType(string name, char symbol, int freq)
: name(name),
  symbol(symbol),
  freq(freq)
{
}

语言的规则是这样的，这将具有预期的含义。

另一种选择是避免使用相同的标识符来引用成员和函数参数：

terrainType::terrainType(string name_, char symbol_, int freq_)
{
    name=name_;
    symbol=symbol_;
    freq=freq_;
}

另一种选择是使用前缀成员访问this->：

terrainType::terrainType(string name, char symbol, int freq)
{
    this->name=name;
    this->symbol=symbol;
    this->freq=freq;
}
于 2012-04-28T20:49:25.723 回答