이렇게 리팩터링 하는게 맞는건가요?

Question

안녕하세요 개발 뉴비가 레거시 코드 수정하는 중입니다.복잡한 쿼리문에 보낼려고 파라미터가 20개 정도 되는 메소드가 있는데 파라미터 줄여볼려고 클래스 만들어볼려고 하다가그리드 보조용으로 있는 컬럼 추가를 위해 계속해서 파라미터가 추가될 수 있는 클래스가 존재해서이 클래스에다가 메소드에 필요한 파라미터 20개를 추가해서 방식으로 만들었습니다.이게 정말로 리팩터링 한 것인지 아닌지 잘 모르겠습니다.기본적으로 클린코드나 칼럼들에서는 그에 맞는 클래스를 만들어서 get-set을 통해서 전달하라 되어있는데 전 따로 담을 수 있는 다른 클래스에 담아서 넘긴건데. 잘못하고 있는건지 궁금합니다.public void DoStuff(string email, string userName, string country) {}public void DoStuff(User user){}이런식으로 바꾸도록 되어있는데.저는 다른 클래스 GreedStorage라는 친구를 써서GreedStorage gs &#61; new GreedStorage();GreedStorage.SetParameter(&#34;email&#34;, email);...public void DoStuff(GreedStorage gs){gs.GetParameter(&#34;email&#34;)...}이렇게 만들었습니다.그냥 생각없이 이게 편하네 하면서 만들다보니 이렇게 만들었는데 맞는건지 궁금합니다.(다 만들고나서 보니 이건 아니다 싶어서 질문 올립니다...)

frostyDog · Answer

Map으로 넘기는것과 같은 단점이 있겠네요

이렇게 리팩터링 하는게 맞는건가요?

1개의 답변

기타 목록