Ja chyba bym zrobił
if (IsColumnNotEmpty()) {
tooltip.Show(mousePosition);
}
boolean IsColumnNotEmpty() {
return IsCommentColumn() && !string.IsNullOrWhiteSpace(_abc.Comment);
}
Ale tak na prawdę żeby wymyślić dobrą nazwę musisz wiedzieć:
a. jaki jest problem który próbujesz rozwiązać?
b. po co?
c. jak?
Bierzesz te trzy rzeczy pod uwagę, i wychodzi nazwa która to opisuje. @still.still. Czytam ten kod i domyślam się, że user najedzie myszką nad komentarz, to ma się pokazać tooltip, chyba że komentarz jest pusty, jak rozumiem?
Nie i w sumie nie wiem co to ma to clean code za bardzo. Ludzie rozstrząsają takie pierdoły jak im się za bardzo nudzi. Dla mnie nieistotny szczegół, a przeniesienie do metody ma ten minus, że trzeba jej wymyślić sensowną nazwę, inną niż IsCommentNullOrWhitespace(Abc abc)
.
Minus?
Moim zdaniem właśnie to jest plus, bo jeśli nie wiesz jak nazwać pod-metodę, to to jest silny sygnał że nie do końca rozumiesz jak kod ma działać i jakie zadanie spełnia. Więc to jest bardzo dobra okazja, żeby samemu się nauczyć więcej, wymyślając nazwę. Jeśli nazwę jest trudno wymyślić to może to być sygnał że design kodu jest zły, i należy go poprawić.