alertbox를 표시하는 코드 리펙토링 (적절한 접근 방법은 무엇입니까?)

Question

저는 iOS 프로젝트에서 작업하고 있습니다. 한가지 주목할 점은 오류를 표시해야 할 때마다 경고음을 반복해서 생성한다는 것입니다. 중복을 제거하는 코드를 리팩토링하고 싶습니다. 내 질문은 :이 특정 시나리오에 대한 리팩토링에 대한 적절한 접근 방식으로 클래스를 처리하는 오류를 만드는 것입니까? 예를 들어, 나는 다음과 같은 클래스

class ErrorHandler { 

    func ShowAlertBox(Title: String, Message: String, ViewController: UIViewController) { 

      let alertController = UIAlertController(title: Title, message: Message), preferredStyle: .Alert) 
      let doneAction = UIAlertAction(title: NSLocalizedString("Cancel", comment: "Label to cancel sign-in failure."), style: .Cancel, handler: nil) 
    alertController.addAction(doneAction) 
      ViewController.presentViewController(alertController, animated: true, completion: nil) 
    } 
} 

을 만들 것입니다 그리고 다음과 같이 호출

instanceofErrorHandler.ShowAlertBox("Error","Log In Error", SignInViewController) 

Answer 1

나는이 시나리오를 처리하는 방법에 대한 다른 의견이 많이있다 생각하고 내가 어떤 문제를 볼 수 없습니다 당신의 접근 방식. 즉 말했다, 나는 그것이 확장을 만드는 것입니다 할 방법은 ViewControllerUtilities라고하고 거기에 내 일반적인 모든 기능을 넣어 :

protocol ViewControllerUtilities {} 

extension ViewControllerUtilities where Self: UIViewController { 
    func showAlert(_ title: String = "Error", message: String) { 
     let alertController = UIAlertController(title: title, message: message, preferredStyle: .alert) 
     alertController.addAction(UIAlertAction(title: "Ok", style: .default, handler: nil)) 
     present(alertController, animated: true, completion: nil) 
    } 
} 

참고로, 나는 또한 네트워크가 도달 할 수 있는지 확인하기 위해 거기에 기능을 가지고 있습니다. 그럼 난 단순히 내보기 컨트롤러에 따르는 프로토콜 목록에 ViewControllerUtilities을 추가하여 모든 기능을 얻을 :

class MyViewController: UIViewController, ViewControllerUtilities { 

    override func viewDidAppear(_ animated: Bool) { 
     super.viewDidAppear(animated) 

     showAlert("Error", message: "Sorry, had an error.") 
    } 

}