data直接赋值到template??!!

[smithfox]

一个template可能对应同时N次的Execute, 这个设计肯定有问题。

[smithfox]

而且你说的 data 所起的作用, 官方已经有这个功能了, 用 {{$}}

[smithfox]

官方文档如是说: When execution begins, $ is set to the data argument passed to Execute, that is, to the starting value of dot.

[achun]

你说的是 data() interface{} 这个函数么?
这个函数的应用场景是在多个 template 中使用的.
请看下这个 http://play.golang.org/p/Y3bEyqp6ez .
也就是说调用子template 时如果传递的不是原始 data 那么以后就无法获取原始data了.
虽然现实中可以使用总是传递原始data的方法解决这个问题, 不过这个问题确实是存在的.
data() interface{} 就是为了满足这种需求的.

[smithfox]

我理解你的意思了，但还是没有明白这样设计的好处是什么？
tpl.Parse({{template "sub" .key1}})
直接写成
tpl.Parse({{template "sub" .}})

有什么不好吗？ 既然你都要用到 root, 为什么不直接传 . 呢

[smithfox]

而且你将运行时的 data, 赋值给 template 静态数据。
多线程 同时调用怎么办？
thread1: templateObj.Execute(data1)
thread2: templateObj.Execute(data2)
thread3: templateObj.Execute(data3)

[achun]

其实这个rep的目的不仅仅是为了让大家直接import的,而是要说明,最简单便捷的使用template的方法就是用闭包.
可以确定的是,闭包是在运行期及时 new template对象, 本身就是在独立的线程中的.
能优化的地方在于模板的解析. 可以预先把公用的 template file 都 parse 了,得到一个基础全局template 对象 baseTpl. 然后在线程中使用 baseTpl.Clone() 得到副本template 对象.这样就不必每次都 parse, data 绑定也各自有独立的作用域了.
事实上这个rep自打设计出来到现在, 还没在真正的应用项目环境中检验.
不过, 你通过这个rep的设计方法应该可以感觉到是可以安全用于多线程的.本来就是完全模仿官方包的设计嘛. 仅仅是提出了与闭包结合的好处而已.
这一段时间我忙着其他的基础rep,还没有把这些rep完全串起来使用.
共同实践吧.

至于
tpl.Parse({{template "sub" .}})
写法的问题,不同的子模板都有各自的root的话, 就不必考虑协调key名称的问题了, 可以预见当子模板数量多了以后,维护key名称是很烦人的.也可以预见, 总会有那么一些子模板需要访问原始root的.
这个设计就是出于这个假定的场景. 当然这只是闭包的一个应用例子, 现实中是否要用完全决定于使用者.

[achun]

另外,通过和您的讨论,发现
https://github.com/achun/template/blob/master/template.go#L139
改变了原始 data, 现在我还没考虑清楚相互影响, 等实践后再做决定,看看到底怎样更符合使用习惯.

[smithfox]

每次exeucte, template 都 clone 一份, 感觉为了这个设计代价太大了。

beego, revel 都没有这样做, 我在production用 beego ,不clone ，一直没有问题，应该官方默认的template是线程安全的。

[smithfox]

值得一提的是： Martini render 中 的 yeild 的实现思路, 就是用的闭包。

[smithfox]

https://github.com/codegangsta/martini-contrib/blob/master/render/render.go#L258

[achun]

每次exeucte, template 都 clone 一份 这不是必须的啊, 只有用到 data()这个接口的时候才需要.
但是你确实提醒了我, data()的内部实现方法确实不科学, 在没有找到更好的方法前,这个接口应该废弃,直接用闭包的方法更好.
至于beego和revel都没有这样做,可他们也没有提供data()这样的接口啊.

[achun]

谢谢, 那些内建接口确实有线程安全问题, 用 Clone() 的方法可能大家不太容易接受, 不过我感觉 Clone的代价不是很大啊,你看下代码.
https://github.com/achun/template/blob/master/template.go#L53
应该就是一些字符串和 t.tpl 的复制 , t.tpl就是一个 interface{} ,是个指针, 代价大么?
ps:晕,脑子乱的很, Clone 代价很大. 确实要再考虑考虑内部实现方法.

func (t *Template) Clone() (*Template, error) {
    return t.clone(), nil
}

我自己先试试用这样的伪 Clone, 就是这名字不合适啊,和官方包的用法不一致了,名不副实啊.

[achun]

https://github.com/martini-contrib/render
martini-contrib 已经独立了.
render 这个rep很好, 不过和我考虑的方向不同.
我主要关注的是数据访问问题. 其他的rep多考虑模板管理问题.
当然如果能结合的话就更好了.