4

编辑

好的，我又读了几个小时，我想我终于更好地理解了 c++ OOP（至少是基础知识）。我决定一次重写整个程序和代码并进行更多测试。我想我这次缩小了更多的错误。

命名风暴.h

#include <string>
#include <iostream>

#ifndef NAMEDSTORM_H_INCLUDED
#define NAMEDSTORM_H_INCLUDED

// NEVER use using namespce in header, use std instead.

class NamedStorm{
private:
    std::string stormName;
    std::string stormCategory;
    double maxWindSpeed;
    double stormPressure;
    static int stormCount;
public:

    // Constructor
    NamedStorm(std::string, double, std::string, double);
    NamedStorm(std::string);

    // Destructor
    ~NamedStorm();

    // Get functions
    int getStormCount();
    double getStormPressure();
    double getWindSpeed();
    std::string getStormCategory();
    std::string getName();

    // Set functions
    static void displayOutput();
    static void sortByNames();
    static void sortByWindSpeed();
    static void getAverageWindSpeed();
    static void getAverageStormPressure();
};

#endif // NAMEDSTORM_H_INCLUDED

命名风暴.cpp

// CPP => Function definition
#include <string>

#include "NamedStorm.h"

using namespace std;

// Defining static variables
int NamedStorm::stormCount = 0;

// Constructor definition
NamedStorm::NamedStorm(std::string sName, double wSpeed, std::string sCat, double sPress){
    stormName = sName;
    windSpeed = wSpeed;
    stormCategory = sCat;
    stormPressure = sPress;
    stormCount++;
}

NamedStorm::NamedStorm(std::string sName){
    stormName = sName;
    stormCount++;
}

// Destructor definition
NamedStorm::~NamedStorm(){}

// Get (Accessor) function definition
int NamedStorm::getStormCount(){
    return stormCount;
}

double NamedStorm::getStormPressure(){
    return stormPressure;
}

string NamedStorm::getStormCategory(){
    return stormCategory;
}

string NamedStorm::getName(){
    return stormName;
}

// Set (Mutator) function definition
void NamedStorm::displayOutput(){}
void NamedStorm::sortByNames(){}
void NamedStorm::getAverageStormPressure(){}
void NamedStorm::getAverageWindSpeed(){}
void NamedStorm::getWindSpeed(){}

主文件

#include <iostream>
#include <string>

#include "NamedStorm.h"

using namespace std;

NamedStorm storm[5]; // Error occurs here

int main(){
   // NamedStorm Chris("Chris", 70.0, "T.S", 990.0); 
   // storm[0] = Chris;
    return 0;
}
4

1 回答 1

4

1.删​​除构造函数定义

在您的头文件（NamedStorm.h）中，您已经定义了 NamedStorm 的默认构造函数：

NamedStorm(){};

但你真正想要的只是构造函数声明：

NamedStorm();

定义和声明的区别在于，声明只告诉编译器有某个函数（例如：NamedStorm 构造函数），而定义提供了这个函数的完整体。

请注意，如果您仅指定函数的声明，而忘记进行定义，您将收到undefined reference链接器错误。

进一步阅读：http ://www.cprogramming.com/declare_vs_define.html

2.修正第二个构造函数

NamedStorm::NamedStorm(string sName, double wSpeed, string sName, double sPress)

这是行不通的，因为您尝试传递两个具有相同名称的参数。我猜你想命名第二个sCat，因为你在构造函数定义中使用了这样的变量。正确版本：

NamedStorm::NamedStorm(string sName, double wSpeed, string sCat, double sPress)

3.运营商<<

如果您阅读了第一部分，那么您现在应该知道有什么问题了operator<<。您只提供了声明，没有提供定义。

你可以这样填写：

std::ostream& operator<<(ostream& out, NamedStorm& namedStorm)
{
    out << namedStorm.getName();
    return out;
}

请注意，声明也发生了变化——函数现在NamedStorm&取而代之const NamedStorm&，因为getName方法没有声明为const。您可以在此处阅读有关const方法的信息。

4.定义静态类变量

您在类中声明的每个静态变量（仅int stormCount在您的情况下）都必须定义。将此行放入您的NamedStorm.cpp文件中：

int NamedStorm::stormCount = 0;

应用这些更改后，您的代码应该可以正常工作。但是，您仍然可以阅读许多语言细微差别来改进您的代码。他们之中有一些是：

1. 将函数参数作为值与 const 引用传递

在这里阅读好：在 C++ 中传递值还是传递常量引用更好？

2. 返回对象副本与 const 引用的函数

这个问题也有一个很好的答案：返回一个常量引用是否更有效

3.小心“使用命名空间”

同样，SO：为什么“使用命名空间标准”被认为是不好的做法？

如果你真的想使用它，千万不要在头文件中使用它，因为它会影响所有包含它的文件。

于 2013-07-23T20:43:11.003 回答