네이밍을 어떻게 하는 게 좋을지 여쭤보려고 합니다!

Question

경력 0.5년 정도 된 신입입니다. 코드를 좀 더 알아보기 쉽게 중복코드에 대해서 간단한 메서드 추출을 해 보고 있는데요. 이런 코드가 있습니다. private void modifyMultiSendFlagPre(String tax_no, URLConnection connection, String send_flag) throws ParserConfigurationException, SAXException, IOException { TaxCheckBO taxCheckBO &#61; new TaxCheckBO(); Element root &#61; SaleUtils.getDocumentElementForXml(connection); String ret &#61; SaleUtils.getRetFrom(root); String msg &#61; SaleUtils.getMsgFrom(root); String dockey &#61; &#34;&#34;; String subkey &#61; &#34;&#34;; if (ret.equals(&#34;OK&#34;)) { dockey &#61; SaleUtils.getDockeyFrom(root); subkey &#61; SaleUtils.getSubkeyFrom(root); } taxCheckBO.modifyMultiSendFlag(tax_no, ret, msg, dockey, subkey, send_flag); } 위에 계산해 주는 내용들은 전부 맨 아래의 메서드를 실행하기 위한 파라미터를 생성해 주기 위한 용도인데요.그래서 이 내용들을 하나로 묶어도 되겠다 싶어서 메서드 추출을 했는데 네이밍을 어떻게 하는 게 좋을지 잘 모르겠습니다.그냥 modifyMultiSendFlag로 하자니 메서드명이 겹쳐서 헷갈릴 수 있겠다고 생각했습니다. 오버로딩 같은 개념이 아니라 그 차원?위상?이 다른 메서드이니까... 일단 저는 메서드명을 뒤에 'Pre'를 붙이는 이런 식으로 지었는데요. 혹시 이런 경우에 대해서 많이 쓰이는 네이밍 방법이 있을까 하여 여쭤보려고 합니다! 감사합니다!

LichKing · Answer

단순히 저것만보고 함부로 말하긴 좀 힘들지만일단 저라면 taxCheckBO.modifyMultiSendFlag 메서드의 인자들을 모아놓은 클래스를 하나 만들고저 파라미터 초기화하는 코드를 전부 생성자로 넣을것 같네요.음...써놓고보니 질문 내용이랑은 조금 다른 답변이 되버린것같군요..;;

seychelles · Answer

// LichKing넵 저도 그러고 싶지만 그러기엔 코드가 이미 너무 먼 길을 와있는것 같아요ㅋㅋvo도 안쓰고 map도 안쓰고 그냥 파라미터 줄줄이 써붙이는게 일상화되어 있는 프로젝트입니다. 이게 한 10년 전에 만들어졌다고 알고 있는데요. 저것까지 하기에는 먼저 해야 할 메서드 추출 작업들이 너무 많아서요ㅎㅎ

네이밍을 어떻게 하는 게 좋을지 여쭤보려고 합니다!

2개의 답변

#naming 목록