对optional 属性的思考

我们现在假设有一个API需求一个类叫做联系人,那么我们建一个:

struct Contact {
    let id:String
    let firstName:String
    let lastName:String
    var type:ContactType
    //phoneNumber is optional only for FaceBook type
    var phoneNumber: String?
    //userName is optional only for AddressBook type
    var userName:String?
    
}

enum ContactType{
    case AddressBook
    case FaceBook
}

表面上看起来不错,完美实现了联系人这个类的需求,但是这个类存在一个比较大的问题在于,这个类的使用者是存在潜在可能性给这个类传一个错误的值得。尽管我们为userName这个属性写了documentation表示如果你想传给我一个来自facebook的联系人的话,那么这个属性并不是optional的。然而,作为一个Dev你自己也知道documentation是写给经理和security team看的,并不会有什么开发者真的会去看documentation。那么这样的话,API在接受了错误的类以后肯定会返回错误的。那么这个时候,你这个类的使用者多半是会骂娘的,甚至严重一点的会对你们公司产生一些负面的评价类似什么垃圾公司出的SDK连最基本的binding都不做。看起来毫无道理,但是现实却是大部分程序员都觉得自己才是最屌的,出了问题肯定都是别人傻逼,然后跟上面汇报的时候只会说你看出这个SDK的公司连最基本的安全性检查都没有,现在App出问题了。

那么既然it happens,只能自己解决这样的问题,现在先看一种最直觉和最容易想到的解决方案:

enum Contact{
    case AddressBook(AddressBookContact)
    case FaceBook(FaceBookContact)
}

extension Contact {
    var id:String {
        switch self {
        case .AddressBook(let contact):
            return contact.id
        case .FaceBook(let contact):
            return contact.id
        }
    }
    
    var firstName:String {
        switch self {
        case .AddressBook(let contact):
            return contact.firstName
        case .FaceBook(let contact):
            return contact.firstName
        }
    }
    
    var lastName:String {
        switch self {
        case .AddressBook(let contact):
            return contact.lastName
        case .FaceBook(let contact):
            return contact.lastName
        }
    }
}

struct FaceBookContact {
    let id:String
    let firstName:String
    let lastName:String
    var userName:String
}

struct AddressBookContact {
    let id:String
    let firstName:String
    let lastName:String
    var phoneNumber: String
}

So far so good, 沾沾自喜五分钟以后觉得可以收工回家了,觉得今晚可以睡个好觉了。表面上来看,的确是这样的,问题解决了,暗坑也被填平了,共有属性也可以直接访问了,完美。
然而不幸的事情发生了,过了几天,产品跑过来跟你说,是这样的,我需要给联系人增加几个共有属性,于是你发现,卧槽我竟然在不断重写同样的东西!这样一想,岂不是会被end dev笑死?不行,如果作为一个程序员最基本的权利-懒惰都无法保证,那跟咸鱼有什么区别?显然,要解决的是Contact enum中每一个共有属性都需要写一次switch表达式的问题:

enum Contact{
    case AddressBook(AddressBookContact)
    case FaceBook(FaceBookContact)
}

extension Contact {
    private func value(of name:String)->Any? {
        let reflectedSelf = Mirror(reflecting:self)
        let propertys = Mirror(reflecting: reflectedSelf.children.first!.value)
        for child in propertys.children {
            if child.label == name {
                return child.value
            }
        }
        return nil
    }
    var id:String{return value(of: #function) as! String}
    var firstName:String{return value(of: #function) as! String}
    var lastName:String{return value(of: #function) as! String}
}

上面这段代码唯一值得解释一下的应该是#function,实际上呢,这个就是外层函数的函数名,比如:

func method1(_ name:String){}
func methods(){
    //这个时候#function就是methods
    method1(#function)
}

在swift之中,closure和function是没有本质区别的,或者说closure是一种特殊的function。 因此,对于 var id {} 这种结构,id就变成了closure的名字,也就是外层函数的名字。

这个解决方案到这里,的确是解决了所有的问题,但是这个方案显得很不自然,可读性很差,可维护性也不高。那么反思一些是不是走错了方向?毕竟如果开始的方向就不对,那么走再远也是没有意义的。其实这个问题用POP是可以很轻松的解决的:

protocol ContactCore {
    var id:String{get}
    var firstName:String{get}
    var lastName:String{get}
}

protocol AddressBoonContact:ContactCore{
    var phoneNum:String{get}
}

protocol FaceBookContact:ContactCore {
    var userName:String{get}
}

//用法范例
var contacts:[ContactCore] = []

for contact in contacts {
    if contact is AddressBoonContact {
        
    }else if contact is FaceBookContact {
        
    }
}
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,793评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 87,567评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,342评论 0 338
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,825评论 1 277
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,814评论 5 368
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,680评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,033评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,687评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 42,175评论 1 300
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,668评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,775评论 1 332
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,419评论 4 321
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,020评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,978评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,206评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,092评论 2 351
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,510评论 2 343

推荐阅读更多精彩内容

  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,596评论 18 139
  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 171,454评论 25 707
  • 以下翻译自Apple官方文档,结合自己的理解记录下来。翻译基于 swift 3.0.1 原文地址 Closure...
    艺术农阅读 1,511评论 0 3
  • 在众多主流浏览器之间,老版本的IE依旧是个“神话”,前端开发者们无不为之撕心裂肺,高歌呐喊。老版本IE的兼容性问题...
    流浪的三鮮餡阅读 989评论 0 0
  • 致自己……起早摸黑,忙忙碌碌,就这样一天一天的过着重复的生活……你会发现,只会有人问你挣多少钱?却很少有人问你累不...
    一个意外的结尾阅读 174评论 2 0