Это хорошая идея использовать частное наследование, чтобы скрыть реализацию?

например

// Implementation.
struct PrivatePoint {
void SomePrivateMethod();

double x;
double y;
}

struct Point : private PrivatePoint {
double DistanceTo(const Point& other) const;
}

Это похоже на Идиома. Это имеет два преимущества, которые мне действительно нравятся:

1. SomePrivateMethod является тестируемым. Если бы SomePrivateMethod вместо этого был объявлен закрытым в Point, вы бы не смогли вызвать его из тестов. Если вы объявите его как общедоступный или защищенный в Point, тесты смогут его вызывать, как и обычные пользователи Point.
2. Доступ к частным данным легче читать и писать по сравнению с тем, как вы делаете это в идиоме Pimpl, потому что вам не нужно проходить через указатель, например

.

Point::DistanceTo(const Point& other) {
SomePrivateMethod();

double dx = other.x - x;
double dy = other.y - y;
return sqrt(dx * dx + dy * dy);
}

против

Point::DistanceTo(const Point& other) {
ptr->SomePrivateMethod();

double dx = other.ptr->x - ptr->x;
double dy = other.ptr->y - ptr->y;
return sqrt(dx * dx + dy * dy);
}