These are chat archives for scalikejdbc/ja

1st
Mar 2016
Manabu Nakamura
@gakuzzzz
Mar 01 2016 05:54
ほりうちさんの修正で良さそうな気配。 2.11.8 でもコンパイル通りましたね。
scalikejdbc/scalikejdbc#423
kenji yoshida
@xuwei-k
Mar 01 2016 07:14
え?あっ、というかそもそも参照してた2.11.8のsnapshot消えてるな
https://twitter.com/xuwei_k/status/703166101153185792 ちなみに、自分がデグレ見つけたせいで2.11.8遅れてます(sonatypeのstageまでいったけど、dropして出し直しかも?)
Manabu Nakamura
@gakuzzzz
Mar 01 2016 07:15
あれ? さっき取ってきたのは何だったんだろう? 2.11.7 でコンパイルしてたのかな?
なるほど
kenji yoshida
@xuwei-k
Mar 01 2016 07:15
まぁ 2.11.7 と 2.11.8 で変わらない気もするので、(もしほりうちさんのやつで直るなら) 2.11.7 で直る気がするけど、
ほりうちさんのやつで直る理屈がよくわかってない。単に自分が勘違いしてるのか、がくぞーさんが確認方法間違ったのか
Manabu Nakamura
@gakuzzzz
Mar 01 2016 07:16
2.11.7 で無事 test:compile まで通りますね。テスト実行で失敗してるけど
kenji yoshida
@xuwei-k
Mar 01 2016 07:17
なるほど?あれ、じゃあ 2.11.7 のコンパイル時に 2.10.x の何かがpathに入っていたのだろうか?(単に2.10のビルドをサボっていただけで2.11は関係ないかと思ってたけど)
Manabu Nakamura
@gakuzzzz
Mar 01 2016 07:19
一部 依存の指定で % "foo_2.11" てのがあったからそれで 2.10 の時おかしかったっぽい?
しかしこれ、 SQLSyntax.value が何の意味を持たなくなってちょっとそれはそれで問題っぽい……
kenji yoshida
@xuwei-k
Mar 01 2016 07:22
一部 依存の指定で % "foo_2.11" てのがあったからそれで 2.10 の時おかしかった もしそれだけだとしたら、やっぱり2.11でコンパイラ落ちてたのおかしいですよね?
Manabu Nakamura
@gakuzzzz
Mar 01 2016 07:39
確かに……
Manabu Nakamura
@gakuzzzz
Mar 01 2016 08:34
ところで話変わりますが、ScalikeJDBC の scalariform の設定、
alignArguments と alignSingleLineCaseStatements を true にしませんか?
kenji yoshida
@xuwei-k
Mar 01 2016 08:58
どう変わるやつですか?
kenji yoshida
@xuwei-k
Mar 01 2016 09:18
なるほど、個人的にはどっちでもいいかな
Manabu Nakamura
@gakuzzzz
Mar 01 2016 09:28
リーダブルコードで縦のalignは揃えろというのがあって、なるほどなーってなった感じです。
テストの修正はこんな方向性でいいのだろうか……
https://github.com/scalikejdbc/scalikejdbc/commit/2d2329a04e4c19968e2a794869dfa811c265ff4f#diff-c58ad5327ff9d187fe362822dda2c546R26
それとも SQLSyntax 側に map 処理した後の値を返す parameters改 みたいなメソッド足した方がいいのか。 。。
Kazuhiro Sera
@seratch
Mar 01 2016 10:56

ところで話変わりますが、ScalikeJDBC の scalariform の設定、alignArguments と alignSingleLineCaseStatements を true にしませんか?

いいんじゃないですかね。

kenji yoshida
@xuwei-k
Mar 01 2016 11:05
リーダブルコードで縦のalignは揃えろ リーダブルかもしれないけど、書くの面倒とか差分出るの多少嫌とかあるけど、まぁ scalariform で勝手に揃うならいいかな、程度の
this is a scalac bug ウケるw
やっぱりバグ報告しないと・・・
Manabu Nakamura
@gakuzzzz
Mar 01 2016 11:15
リーダブルコードの主張としては書くの面倒だけど書く回数より読む回数の方が多いんだから読む方が楽になるようにしとけという事でした。まぁ scalariform が自動でやるのでその辺の手間も無いですしね。
差分の問題は確かに過去分と比較したいときに面倒ですね
そして contramap が壊れていた……
Manabu Nakamura
@gakuzzzz
Mar 01 2016 13:40
そもそもなんで ParameterBinder と ParameterBinderFactory と両方つくったんだっけ……
そういえば API 変えるかどうかの話だと https://gitter.im/scalikejdbc/ja?at=55af656544064af0505c827a の一連の話もあったような
kenji yoshida
@xuwei-k
Mar 01 2016 13:46
そんな話もあったなぁー(自分で言い出しておいて、互換性が微妙にアレでいい方法思いつかないので面倒になったまま)