Метод должен выполнять одно простое логически отледенное действие. Он не должен делать всё за всех. В ваших методах этот принцип сохраняется?
Вот щас очень в тему этот разговор зашёл. У меня тут говнокод под винформами, где все в одном файле form.cs (памагите). Допустим, я хочу сериализовать поля формы (текстбоксы, радиобатоны итд). За это должна форма отвечать или отдельный класс, куда надо передать форму, чтобы он все значения полей с нее собрал и потом сериализовался?