回归班级成员是不好的做法？

Question

假设：

Class A
{
     private:
       double **CR;
     public:
       double **compute2D();
};

假设我有一个私有2D数组成员：

double **CR;

我有一个成员函数：

Double ** Compute2D()

功能computer2D将返回CR。

这是一种不好的做法吗？ 为什么？ 我应该使用getter和setter函数来返回吗？

还有一个问题：我是否正确使用垃圾收集器？

A::~A()
{
    //Delete 2D array
    for(int i = 0; i < Rows; ++i)
    {
        delete [] CR[i];
    }

    delete [] CR;
}

Answer 1

通过这样做，您可以在类之外更改私有成员值。如果它是坏的或好的，它取决于你。但在我看来，如果我有私人会员，那么我必须是唯一能改变其价值的人。

Answer 2

将指针返回给类的私有成员是很糟糕的，因为它可能会破坏封装。有人可以在没有
的情况下改变私有变量的值 告知你的班级。但是，在您的班级的字段的返回副本中没有任何不好。 当使用getter和setter时，如果没有类知道它，就无法改变类状态。所以这很好。

删除数组后，一切看起来都不错。

Answer 3

如果通过公共方法向私有数据成员返回指针或非const引用，则将其设为私有是没有意义的。您也可以公开该物业。

这基本上意味着您允许操纵类外的成员。 IMO这被认为是不良做法，甚至是反模式，因为该类不能依赖于自己的状态。这非常重要。

例如：假设你有一个指针，有人将它设置为null。即使私人方法也需要检查它，即使内部无法达到这种状态。

返回正常成员，例如类被视为不良做法，因为它涉及复制整个对象。通常最好返回引用或最好是const引用。

反过来，getters会允许你放置const个约束。在两种情况下，指针和引用。

另请注意，在这种情况下，通常会提供两种方法。 compute和get。目前，您只能通过计算来访问您的会员！

我不会建议你切换到std::vector，因为我不知道你需要什么，而且向量对一切都不好。所以坚持使用指针，这是安全的方法：

class A
{
     private:
       double **CR;
     public:
       double const * const * compute2D();
       double const * const * getCR();
};

double const * const * A::compute2D(){
 return CR;
}

double const * const * A::compute2D(){
 /*Heave CPU stuff*/
 return CR;
}

int main(){
A a;
double const* const* tmp = a.compute2D();
tmp[1][2] = 0; //this will fail to compile
tmp[1] = 0; //this will fail too

double get_test = tmp[1][2]; // this passes!
}

注意双const个限定符。保护每个级别的指针引用非常重要。

Answer 4

返回班级成员是好的。但是在你的情况下，我会略有不同。您返回指向2D矢量的指针。这样，您可以返回计算结果，而不必复制数据。并且由于它返回一个const值，您确定调用者将无法更改数据，但可以使用它来读取值。

Class B
{
     private:
       vector<vector<double> > CR; /*note space between > > is significant*/
     public:
       const vector<vector<double> >& compute2D(){ 
            /*do calculation here eg.*/
            return CR;
       }
};

由于我的示例显示了如何使用std::vector执行此操作，因此您无需删除动态保留的内存。如果仍在使用引用变量，则必须确保Class B的实例不会被破坏，例如超出范围。

Answer 5

1. 这很糟糕，因为您正在使用写入权限返回私有变量，也许它不是那么私密
2. 很糟糕，因为你要返回一个指针，在这种情况下，不清楚谁有责任释放内存：这个类是因为它是自己的成员还是用户，因为他调用了这个方法？

Answer 6

如果您希望该类成员在类外部可读，则完全依赖于您尝试完成的操作，那么对类方法进行重新编写是完全正确的。至于使用getter方法，你已经是;它是＆＃34;良好实践＆＃34;使用此类方法为您的类创建更统一，封装的设置，从而使以后更容易编辑该类。即使你的函数做了其他事情（在这种情况下，计算二维的东西），它仍然是一个getter方法。

但是，在这种特殊情况下，您不会返回值;你正在返回一个指向值的双指针。这意味着用户将能够编辑数据（我猜测是一个数组），这是一个不错的类。事情。您应该在返回数据之前执行数据的深层复制。

正确使用垃圾收集器：是的，你是。虽然它在C ++中不被称为垃圾收集器，但它简称为释放内存;垃圾收集器是自动释放内存的系统的名称。