Olá Bruno, tudo bem?
Não existe bem uma resposta certa ou errada quando falamos de refatoração de código. Isolar trechos de códigos em funções, métodos, atributos, outras classes são todas opções válidas. Só depende do contexto que está inserido.
Por exemplo, se for um valor que está se repetindo muito na classe, talvez faça sentido colocá-lo como um atributo, agora se for um comportamento, uma mesma validação que se repete em muitos métodos, pode ser que criar um novo método para isolar esse comportamento seja uma boa coisa. Uma coisa importante da refatoração é trazer um pouco mais de legibilidade ao código.