重构Switch-Statement

时间:2016-02-23 08:44:07

标签: swift switch-statement refactoring global-variables

有没有办法以尽可能少的重复我自己的方式重构这个功能?

我有四个ViewControllers,每个ViewControllers都包含一个变量eventData,它是一个自定义对象。

我考虑过将eventData作为全局变量。这样做有什么缺点吗?

这是我现在正在使用的代码:

override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?)
{
    navigationItem.title = nil

    let segues = UIStoryboardSegue.Segue.self

    switch segue.identifier! {

    case segues.EventOverviewToGuestList.rawValue:

        let destinationVC = segue.destinationViewController as! GuestListVC

        destinationVC.eventData = eventData


    case segues.EventOverviewToAnalytics.rawValue:

        let destinationVC = segue.destinationViewController as! AnalyticsVC

        destinationVC.eventData = eventData


    case segues.EventOverviewToMessaging.rawValue:

        let destinationVC = segue.destinationViewController as! MessagingVC

        destinationVC.eventData = eventData


    case segues.EventOverviewToInformation.rawValue:

        let destinationVC = segue.destinationViewController as! InformationVC

        destinationVC.eventData = eventData

    default: break

    }

}

seguesenum

extension UIStoryboardSegue {

    enum Segue : String
    {

        case EventListToEventOverview
        case EventOverviewToGuestList
        case EventOverviewToAnalytics
        case EventOverviewToMessaging
        case EventOverviewToInformation

    }

}

3 个答案:

答案 0 :(得分:2)

是的,您可以拥有一个具有此eventData成员的基类。我们说 -

class BaseViewController : UIViewController{
    var eventData : SomeCustomClass?
}

派生所有需要事件数据的视图控制器。

class InformationVC : BaseViewController{
}

class AnalyticsVC : BaseViewController{
}

class GuestListVC:BaseViewController{
}

现在您的案例可以合并:

switch segue.identifier! {

  case segues.EventOverviewToGuestList,
     segues.EventOverviewToAnalytics,
    segues.EventOverviewToMessaging,
     segues.EventOverviewToInformation:

  let destinationVC = segue.destinationViewController as! BaseViewController
  destinationVC.eventData = eventData

  default: break

}

答案 1 :(得分:1)

您可以在ViewControllers上使用EventDataProtocol来设置eventData。任何使用EventDataProtocol的视图控制器都将始终具有Segue设置的eventData。例如:

protocol EventDataProtocol {
    var eventData: Int? { get set } //Change to your actual type
}

class EventData: UIViewController, EventDataProtocol {
    var eventData: Int?
}

然后在你的prepareForSegue方法中:

override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
    super.prepareForSegue(segue, sender: sender)

    if var eventDataVC = segue.destinationViewController as? EventDataProtocol {
        eventDataVC.eventData = 5
    }
}

答案 2 :(得分:1)

鉴于

struct EventData {}

定义以下协议

protocol Module0 { var eventData: EventData? { get set } }

并使其符合您的视图控制器

class GuestListVC: UIViewController, Module0 { var eventData: EventData? }
class AnalyticsVC: UIViewController, Module0 { var eventData: EventData? }
class MessagingVC: UIViewController, Module0 { var eventData: EventData? }
class InformationVC: UIViewController, Module0 { var eventData: EventData? }

在枚举中添加属性

extension UIStoryboardSegue {

    enum Segue: String {
        case EventListToEventOverview
        case EventOverviewToGuestList
        case EventOverviewToAnalytics
        case EventOverviewToMessaging
        case EventOverviewToInformation

        static var allRawValues: [String] {
            return
                [
                    Segue.EventListToEventOverview.rawValue,
                    Segue.EventOverviewToGuestList.rawValue,
                    Segue.EventOverviewToAnalytics.rawValue,
                    Segue.EventOverviewToMessaging.rawValue,
                    Segue.EventOverviewToInformation.rawValue
            ]
        }
    }

}

现在它很简单

class ViewController: UIViewController {
    var eventData = EventData()
    override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
        navigationItem.title = nil

        if var destinationVC = segue.destinationViewController as? protocol<Module0> where UIStoryboardSegue.Segue.allRawValues.contains(segue.identifier!) {
            destinationVC.eventData = eventData
        }
    }
}