Question

我制作了一个小程序并且它有效，但我不知道它是否是正确的方式，特别是对于getter，所以这里有一些我的代码，如果你可以判断它是否正确或者它的代码会很好不，如果有更好的方法来定义setter和getter。

#include"head.h"

int main()
{
    field f("big_field", 20 , 10);
    f.print();
    cout << "Area: " << f.getArea() << endl;
    field t=f;
    t.setPlatums(20);
    t.print();
    cout << "Area: " << t.getArea()<< endl;
    return 0;
}

好吧，这似乎没问题。

#include<iostream>
using namespace std;

class field
{
    string kads;
    int garums;
    int platums;
public:
    void print();
    field(const string& , int,  int);
    int getArea();
    void setPlatums(int);
};

现在对于其他东西和吸气者：

#include "head.h"

field::field(const string&t, int a, int b)
{
    kads = t;
    garums = a;
    platums = b;
}

void field::print()
{
    cout << kads << " " << garums << " " << platums << endl;
}

int field::getArea()
{
    return garums*platums;
}

void field::setPlatums(int b)
{
    platums=b;
};

这似乎不是一个问题，因为代码正在运行，但也许我做错了，我的意思是代码并不是唯一正确的，因为它正在运行。

感谢您的快速反应，听到我正在学习正确的方法很好，因为很难重新学习错误的方法。

Answer 1

没关系。但是，您应该通过构造函数的初始化列表初始化成员：

field::field(const string&t, int a, int b) :  kads(t), garums(a), platums(b)
{
}

另一方面，您可以通过覆盖运算符print替换<<输出流：

#include <iostream>

class field
{
   .
   .
   .
   friend std::ostream& operator<<(std::ostream& os, const field& f);
};


std::ostream& operator<<(std::ostream& os, const field& f)
{
    os << kads << " " << garums << " " << platums << std::endl;
    return os;
}

另一个，尽量不要在头文件中使用using namespace std;。

最后，当您计算getArea中的区域时，您不需要班级中的成员area。除非你在setter上预先计算它并返回area。

Answer 2

#include<iostream>

using namespace std;

class field
{

public:
    ...

    // You could implement getters as const functions.
    // int getArea();                
    int getArea() const;

    void setPlatums(int);   
};

在c ++中使用getter / setter这是正确的方法吗？

2 个答案: