함수의 분리는 어디까지 가는지 궁금해요!

[youz2me]

최근 주석을 너무 많이 남기는게 좋지 않다는 이야기를 듣기도 했고 함수 하나의 길이가 너무 길어지면 가독성이나 코드 분리 측면에서 좋지 않다는 생각도 들어서 #277 에서 중복되는 코드 수정과 함께 함수 분리를 중점적으로 가져가보려고 했는데요!

아무리 함수 분리가 ASAP이라고 하지만 어디까지 분리해야할지도 잘 모르겠고 ... 나름 제가 가지고 있던 생각은 한 함수는 한 기능만 수행한다였는데 이번 리팩토링을 하면서 이게 맞나 생각이 들어서 올려봅니다.

CreateMeeting>ViewController>CreateMeetingViewController 에서 기존 presentButtonDidTapped() 함수 안 내용이 너무 길어서 함수를 조금 분리해봤어요!

리팩토링 이전

@objc 
    func presentButtonDidTapped() {
        let inviteCodePopUpViewController = InvitationCodePopUpViewController(
            invitationCode: createMeetingViewModel.inviteCode.value
        )
        
        inviteCodePopUpViewController.modalPresentationStyle = .overFullScreen
        inviteCodePopUpViewController.modalTransitionStyle = .crossDissolve
        inviteCodePopUpViewController.view.backgroundColor = .black.withAlphaComponent(0.7)
        
        inviteCodePopUpViewController.rootView.copyButton.addTarget(
            self,
            action: #selector(copyButtonDidTapped),
            for: .touchUpInside
        )
        inviteCodePopUpViewController.rootView.inviteLaterButton.addTarget(
            self,
            action: #selector(inviteLaterButtonDidTapped),
            for: .touchUpInside
        )
        
        inviteCodePopUpViewController.rootView.removeGestureRecognizer(
            inviteCodePopUpViewController.rootView.gestureRecognizers![0]
        )
        
        createMeetingViewModel.createMeeting(
            name: createMeetingViewModel.meetingName.value
        )
        
        DispatchQueue.main.asyncAfter(deadline: .now() + 0.5) {
            inviteCodePopUpViewController.rootView.setInvitationCodeText(
                self.createMeetingViewModel.inviteCode.value
            )
            
            self.present(inviteCodePopUpViewController, animated: true, completion: nil)
        }
    }

리팩토링 이후

@objc 
    func presentButtonDidTapped() {
        let inviteCodePopUpViewController = InvitationCodePopUpViewController(
            invitationCode: createMeetingViewModel.inviteCode.value
        )
        
        setupPopUpViewController(viewController: inviteCodePopUpViewController)
        setupPopUpAction(view: inviteCodePopUpViewController.rootView)
        removeDismissGesture(view: inviteCodePopUpViewController.rootView)
        createMeetingViewModel.createMeeting(name: createMeetingViewModel.meetingName.value)
        
        DispatchQueue.main.asyncAfter(deadline: .now() + 0.3) {
            inviteCodePopUpViewController.rootView.setInvitationCodeText(
                self.createMeetingViewModel.inviteCode.value
            )
            
            self.present(inviteCodePopUpViewController, animated: true)
        }
    }
    
    private func setupPopUpViewController(viewController: BaseViewController) {
        viewController.modalPresentationStyle = .overFullScreen
        viewController.modalTransitionStyle = .crossDissolve
        viewController.view.backgroundColor = .black.withAlphaComponent(0.7)
    }
    
    private func setupPopUpAction(view: InvitationCodePopUpView) {
        view.copyButton.addTarget(
            self,
            action: #selector(copyButtonDidTapped),
            for: .touchUpInside
        )
        
        view.inviteLaterButton.addTarget(
            self,
            action: #selector(inviteLaterButtonDidTapped),
            for: .touchUpInside
        )
    }
    
    private func removeDismissGesture(view: BaseView) {
        if let gesture = view.gestureRecognizers?.first {
            view.removeGestureRecognizer(gesture)
        }
    }

제가 문제라고 생각했던 부분은

private func setupPopUpAction(view: InvitationCodePopUpView) {
        view.copyButton.addTarget(
            self,
            action: #selector(copyButtonDidTapped),
            for: .touchUpInside
        )

이 부분이었는데요. 매개변수로 InvitationCodePopUpView 타입을 받고 있습니다. 저 타입을 BaseView로 설정하게 되면 BaseView가 copyButton 프로퍼티를 가지고 있지 않아서 오류가 발생합니다 ... 근데 타입을 저렇게 받게 되면 재사용성 측면에서 올바르지 않는 코드가 될 것 같다는 생각이 들었습니다. 한번 그런 생각이 드니까 함수에 매개변수가 들어있는 게 맞는지, 또 함수를 얼마나 분리해야 하는지에 대해서 생각이 많아지더라구요.
다른 분들은 어떻게 생각하시는지 궁금합니다!

[JinUng41]

SRP 관점에서 함수가 하나의 일만 하고 있는지 항상 생각해 보면 리팩토링의 방향성이 보일 것 같아요.
다시 말해서 너무 많은 일을 하고 있지는 않은지 생각해 보면 좋겠죠?
재사용성과 SOLID 등의 내용은 그런 것은 본인의 생각하에,,
저는 누가 답을 내려주길 원치 않습니다.
항상 끝까지 탐구할 수 있도록 길만 제시하고 싶어요. 그래야 개발자가 될 수 있다고 생각합니다.