These are chat archives for scalikejdbc/ja

19th
Feb 2015
KAWACHI Takashi
@tkawachi
Feb 19 2015 07:12
Manabu Nakamura
@gakuzzzz
Feb 19 2015 07:18
そういえば通常はクロージャ内に束縛されてるから必要なさそう。デバッグプリントとかでパラメータを書式化したときに判りやすくするようにとかですかね。
KAWACHI Takashi
@tkawachi
Feb 19 2015 07:18
デバッグプリント用。なるほど。
Manabu Nakamura
@gakuzzzz
Feb 19 2015 07:19
うそかも。この実装だと toString はオブジェクトハッシュ使われるからログにも出ないですね。
case class つくるか、 toString 実装した方が親切っぽい気がしてきた
KAWACHI Takashi
@tkawachi
Feb 19 2015 07:20
value のレゾンデートルが,,
Kazuhiro Sera
@seratch
Feb 19 2015 13:33
確かに不要ですね。ミスったな。
deprecated にして改善していくべきですね。
Manabu Nakamura
@gakuzzzz
Feb 19 2015 14:17
個人的には削除するよりtoString実装するほうがSQLのログ書き出しの所で嬉しいかなーという気がしてます。
Kazuhiro Sera
@seratch
Feb 19 2015 15:11
それはそれでもいいけど、value を impl することが必須である必要はないかなと思っています。今は必須なので。
Manabu Nakamura
@gakuzzzz
Feb 19 2015 15:13
あー確かに ParameterBinder#value は無くても良さそうですね