Является ли небольшое количество участников в классе, который считается запахом кода?

Question

В настоящее время я делаю простую программу списка, используя шаблон MVC, и, таким образом, имею модельный класс для ноутбука. Однако что-то чувствует себя «выключенным», так как у него очень мало членов.

Ноутбук состоит из категорий, которые состоят из списков дел, которые состоят из элементов.

То, что я не могу разместить, является ли это плохим анализом (например, есть больше членов и обязанностей, которые я просто пропустил.) Или, возможно, запах кода, что класс не нужен (в этом случае я не знаю, что делать, поскольку я могу просто иметь список категорий в этом контроллере, но тогда у меня нет модели ноутбука, которая тоже кажется неправильной).

Ниже очень простой класс у меня есть:

class Notebook 
{ 
    private String title; 
    private List<Category> categories; 

    public Notebook(String title, List<Category> categories) 
    { 
    } 

    public void setCategories(List<Category> categories) 
    { 
    } 

    public List<Category> getCategories() 
    { 
    } 
} 

Я часто этот вопрос, где он чувствует, как я делаю уроки ради него, и они имеют очень число членов/обязанностей, так было бы неплохо узнать, не подчеркиваю ли я ни по какой причине или нет.

Answer 1

Не обязательно, существует концепция в Domain Driven Design того, что называется «стандартным типом». Это действительно базовый примитив, завернутый в класс объектов. Идея состоит в том, что примитив не содержит информации о том, какую информацию он содержит, это просто строка/int/что угодно. Таким образом, говоря, что объект, который окружает примитив и гарантирует, что он всегда действителен, гарантирует, что объект имеет значение далеко за пределы только примитива, который он содержит, например. Имя - это не просто строка, это имя.

Вот пример, взятый из комментариев Velocity

public class Velocity 
{ 
    private readonly decimal _velocityInKPH; 

    public static Velocity VelocityFromMPH(decimal mph) 
    { 
     return new Velocity(toKph(mph)); 
    } 

    private Velocity(decimal kph) 
    { 
     this._velocityInKPH = kph; 
    } 

    public decimal Kph 
    { 
     get{ return this._velocityInKPH; } 
    } 

    public decimal Mph 
    { 
     get{ return toMph(this._velocityInKPH); } 
    } 

    // equals addition subtraction operators etc. 

    private static decimal ToMph(decimal kph){ // conversion code } 
    private static decimal ToKph(decimal mph){ // conversion code } 
}