All we do is looking for some way to fulfill our needs.

piątek, 25 lutego 2011

Tagged under: , , ,

Jaka nazwa dla tej metody?

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:

Maciek Rogoziński pisze...

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.