Dwie funkcje lub parametr boolowski?

Question

Czy istnieje jakaś zasada, kiedy używać dwóch funkcji lub kiedy przekazywać parametr boolowski.

Dzięki

Answer 1

Minęło trochę czasu odkąd ostatni re-read Code Complete, ale niejasno przypominam McConnell adresowania tego, a wyrazy „dysjunktywny conherence” pop w mojej głowie. Krótko,

void f(int x, int y, bool b) 

kontra

void f1(int x, int y) 
void f2(int x, int y) 

jest często wybór, w zależności od tego, jak podobne lub różne f będzie zachowywać się pod true kontra false może sensu podzielić go na dwie funkcje i dać im różne nazwy. Często trzeci wybór jest lepszy, czyli zmiana bool na enumę dwuwartościową, gdzie nazwa wyliczeniowa daje wyraźne rozróżnienie.

Kluczem jest spojrzenie na strony z ogłoszeniami i sprawdzenie, czy znaczenie jest jasne po przeczytaniu kodu. Jeśli jesteś skłonny umieścić komentarz na każdej logicznej call-site:

f(3, 4, true /* absoluteWidgetMode */) 

i call-strony zazwyczaj zadzwonić z logicznych stałych, to silny zapach, że należy podzielić ją na kilka funkcji.

Answer 2

logiczne parametry są znaczeniawiększość czasu, w zasadzie taką samą krytykę zasługuje magiczne numery zrobienia. Nie masz szans na niezauważenie, co się dzieje po prostu patrząc na wywołanie funkcji.

Nawet jeśli wygodnie jest mieć parametr boolowski dla bardzo podobnych kodów (dodawanie/nadpisywanie pliku), zachowaj go wewnętrznie, prywatnie i nie pozwól, aby był widoczny w interfejsie.

Zamiast zawsze zmusić programatora być wyraźny:

Zastosowanie wyliczenia dać sensowne opisy rozróżnienia lub po prostu użyć oddzielne funkcje.

Porównaj:

WriteFile(path, "Hello, World", true) 

z

WriteFile(path, "Hello, World", FileMode.Append) 

lub po prostu

AppendFile(path, "Hello, World")