Kouhei Sutou
kou****@clear*****
Wed Jul 23 16:16:25 JST 2014
> + def set_tag(node_name, tag_name, value) > + new(nil, name).set_tag(tag_name, value) > + end > + > + def delete_tag(node_name, tag_name) > + new(nil, name).delete_tag(tag_name) > + end こんな風にクラスメソッドとしてどんどんショートカットを作って いくのってあんまりいい流れじゃない気がしますよ。。。 オブジェクトにするとスッキリするところなのに、その少しの手間 を嫌って無理やり組み立てていっているとき（とりあえず動く状態 ばかりを目指して、本当はどうあるべきかをおろそかにしていると き）にこんな風になっちゃう気がします。。。 後(*)で整理するもので一時的ならまぁいいとは思いますが。。。 (*) 数週間とか数ヶ月というオーダーじゃなくて、数時間とか数日のオーダー。 In <aebe283b670843f41b288eebcba369135b31da6d �� jenkins.clear-code.com> "[Groonga-commit] droonga/droonga-engine �� aebe283 [master] Add utility methods to modify tags" on Wed, 23 Jul 2014 16:10:28 +0900, YUKI Hiroshi <null+groonga �� clear-code.com> wrote: > YUKI Hiroshi 2014-07-23 16:10:28 +0900 (Wed, 23 Jul 2014) > > New Revision: aebe283b670843f41b288eebcba369135b31da6d > https://github.com/droonga/droonga-engine/commit/aebe283b670843f41b288eebcba369135b31da6d > > Message: > Add utility methods to modify tags > > Modified files: > lib/droonga/serf.rb > > Modified: lib/droonga/serf.rb (+8 -0) > =================================================================== > --- lib/droonga/serf.rb 2014-07-23 16:04:34 +0900 (0f4753e) > +++ lib/droonga/serf.rb 2014-07-23 16:10:28 +0900 (e9f006f) > @@ -70,6 +70,14 @@ module Droonga > new(nil, name).live_nodes > end > > + def set_tag(node_name, tag_name, value) > + new(nil, name).set_tag(tag_name, value) > + end > + > + def delete_tag(node_name, tag_name) > + new(nil, name).delete_tag(tag_name) > + end > + > def tag(node_name, tag_name) > node_info = live_nodes(node_name)[node_name] > node_info["tags"][tag_name]
