go代码演示:重构 改善既有代码的设计(第2版)重构第一版

重构前代码:https://www.jianshu.com/p/e5a3ba308aca

theatre.go

type Play struct {
    Name string
    Type string
}

type Plays map[string]Play

type Performance struct {
    PlayID   string
    Audience int
}

type Invoices struct {
    Customer     string
    Performances []Performance
}

func statement(invoices Invoices, plays Plays) (result string) {
    param := NewStatementParam(invoices, plays)
    data := NewStatementDate(param)
    return data.packTxt()
}

theatre_data.go

type statementDate struct {
    Customer     string
    PerfDataList []PerformanceData
    TotalAmount  float64
    TotalCredits int
}

func NewStatementDate(param *statementParam) *statementDate {
    s := &statementDate{}
    s.Customer = param.invoices.Customer
    s.PerfDataList = param.buildPerfDataList()
    s.TotalAmount = s.totalAmount()
    s.TotalCredits = s.totalCredits()
    return s
}

func (s *statementDate) totalAmount() (totalAmount float64) {
    for _, perf := range s.PerfDataList {
        totalAmount += perf.ThisAmount
    }
    return totalAmount
}

func (s *statementDate) totalCredits() (totalCredits int) {
    for _, perf := range s.PerfDataList {
        totalCredits += perf.ThisCredits
    }
    return totalCredits
}

type PerformanceData struct {
    PlayName    string
    ThisAmount  float64
    ThisCredits int
    Audience    int
}

func NewPerformanceData(playName string, thisAmount float64, thisCredits, audience int) PerformanceData {
    return PerformanceData{
        PlayName:    playName,
        ThisAmount:  thisAmount,
        ThisCredits: thisCredits,
        Audience:    audience,
    }
}

theatre_pack.go

func (s *statementDate) packTxt() (result string) {
    result = fmt.Sprintf("Statement for %s\n", s.Customer)
    for _, data := range s.PerfDataList {
        result += fmt.Sprintf("    %s: $%.2f(%d seats)\n", data.PlayName, data.ThisAmount, data.Audience)
    }
    result += fmt.Sprintf("Amount Owed is $%.2f\n", s.TotalAmount)
    result += fmt.Sprintf("You earned %d credits\n", s.TotalCredits)
    return result
}

func (s *statementDate) packHTML() (result string) {
    return result
}

func (s *statementDate) packXML() (result string) {
    return result
}

theatre_param.go

type statementParam struct {
    invoices Invoices
    plays    Plays
}

func NewStatementParam(invoices Invoices, plays Plays) *statementParam {
    return &statementParam{
        invoices: invoices,
        plays:    plays,
    }
}

func (s *statementParam) buildPerfDataList() []PerformanceData {
    perfDataList := make([]PerformanceData, 0, len(s.invoices.Performances))
    for _, perf := range s.invoices.Performances {
        perfDataList = append(perfDataList, s.buildPerformanceData(perf))
    }
    return perfDataList
}

func (s *statementParam) buildPerformanceData(perf Performance) PerformanceData {
    var (
        playName    = s.NameFor(perf)
        thisAmount  = s.thisAmountFor(perf)
        thisCredits = s.thisCreditsFor(perf)
        audience    = perf.Audience
    )
    return NewPerformanceData(playName, thisAmount, thisCredits, audience)
}

func (s *statementParam) thisAmountFor(perf Performance) float64 {
    var thisAmount int
    switch s.TypeFor(perf) {
    case "tragedy":
        thisAmount = 40000
        if perf.Audience > 30 {
            thisAmount += 1000 * (perf.Audience - 30)
        }
        break
    case "comedy":
        thisAmount = 30000
        if perf.Audience > 20 {
            thisAmount += 10000 + 500*(perf.Audience-20)
        }
        thisAmount += 300 * perf.Audience
        break
    default:
        panic(fmt.Sprintf("unknow type: %s", s.TypeFor(perf)))
    }
    return float64(thisAmount) / 100
}

func (s *statementParam) thisCreditsFor(perf Performance) (thisCredits int) {
    thisCredits = util.Max(perf.Audience-30, 0)
    if s.TypeFor(perf) == "comedy" {
        thisCredits += perf.Audience / 5
    }
    return thisCredits
}

func (s *statementParam) TypeFor(perf Performance) string {
    return s.plays[perf.PlayID].Type
}

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

推荐阅读更多精彩内容