public class OptionsAwareObject { private Options options; public void updateOptions(String fontName, int fontSize) { options.setFontName(fontName); options.setFontSize(fontSize); } } class Options { private int foregroundColor; private int backgroundColor; private String fontName; private int fontSize; // ... }
Czy updateOptions to dobra nazwa dla tej metody? Oczywiście, że nie! updateOptions jest ogólną nazwą, która sugerowałaby całościową aktualizacji opcji. Tymczasem my aktualizujemy tylko informacje nt. czcionki. Odpowiedzialnością metody jest zatem zmiana opcji związanych z czcionką. Lepszą nazwą byłaby:
public void updateFontOptions(String fontName, int fontSize) { //...
1 komentarze:
Racja, ale czy nie lepiej byłoby zmienić nazwę klasy: Options na FontOptions, OptionsAwareObject na FontOptionsAwareObject? Wtedy wydaje mi się, że wystarczyłaby nazwa metody update.
Prześlij komentarz