Funkcje klasy prywatnej a Funkcje w bezimiennej przestrzeni nazw

Question

Odkryłem, że nie mam prywatnych funkcji klasy. Jeśli to możliwe, wszystkie kandydatki na klasy prywatne działają raczej w nienazwanej przestrzeni nazw i przekazują wszystkie niezbędne informacje jako parametry funkcji. Nie mam wytłumaczenia dźwiękowego, dlaczego to robię, ale przynajmniej wydaje mi się to bardziej naturalne. W konsekwencji muszę ujawnić mniej wewnętrznych szczegółów w pliku nagłówkowym.

Jaka jest Twoja opinia - czy to jest właściwa praktyka?

Answer 1

W pół dużych projektach, w których zwykle pracuję (ponad 2 miliony linii kodu), zakazywałbym prywatnym klasom funkcji, gdybym mógł. Powodem jest to, że prywatna funkcja klasy jest prywatna, ale jest widoczna w pliku nagłówkowym. Oznacza to, że jeśli zmienię podpis (lub komentarz) w każdym razie, jestem nagradzany czasami pełnym rekompilacją, która kosztuje kilka minut (lub godzin w zależności od projektu).

Po prostu powiedz "nie" i ukryj to, co jest prywatne w pliku cpp.

Gdybym zaczął od nowa w dużym projekcie C++, wymusiłabym wymysł PIMPL: http://c2.com/cgi/wiki?PimplIdiom, aby przenieść jeszcze więcej prywatnych danych do pliku cpp.

Answer 2

Myślę, że to dobra praktyka. Często ma to również zaletę ukrywania struktur pomocniczych i typów danych, co zmniejsza częstotliwość i rozmiar odbudowy. Ułatwia to również podział funkcji na inny moduł, jeśli okaże się, że są one przydatne w innych miejscach.

Answer 3

Zasadniczo sprowadza się do pytania, czy dana funkcja rzeczywiście ma sens jako część klasy. Jeśli twoim jedynym zamiarem jest zachować szczegóły klasy z nagłówka, rozważam użycie idiomu pimpl.

Answer 4

Zrobiłem to w przeszłości i zawsze źle się kończyło. Nie możesz przekazywać obiektów klas do funkcji, ponieważ muszą one uzyskać dostęp do prywatnych członków, prawdopodobnie przez odniesienie (lub kończysz na zawiłych listach parametrów), więc nie możesz wywoływać publicznych metod klasy. Z tego samego powodu nie można wywoływać funkcji wirtualnych. Mocno wierzę (na podstawie doświadczenia), że jest to Zły pomysł.

Podsumowanie: To brzmi jak idea, która może zadziałać tam, gdzie "moduł" implementacji ma specjalny dostęp do klasy, ale tak nie jest w C++.