how unext took in eclipse collections in fw

Eclipse Collectionsを
現場に浸透させるための
フレームワーク対応
U-NEXT 技術顧問
久保雅彦 ( jflute )
2018.5.26 JJUG CCC 2018 Spring

まずは自己紹介
久保雅彦 ( jflute )
OSS開発
LastaFlute : WEB Framework
DBFlute : DB管理支援ツール & O/R Mapper
U-NEXTでは、サーバーサイド技術顧問として
現場フォローを行っている
Twitter : @jflute
Blog : http://d.hatena.ne.jp/jflute/

Eclipse Collectionsのちょこっと紹介

さっそくライヴコーディング
jflute が Eclipse で Eclipse Collections 書くよ、バババッ

Eclipse Collections自体の詳しい話は、
前回JJUGで発表された、
U-NEXT ながほりさんのスライドを！
http://slides.com/nashcft/jjug_ccc_2017_fall/#/

ずばりジレンマ
「Java組み込みじゃないので、
フレームワークが対応してない」

java.util.List と ImmutableList の往復修行

Eclipse Collections便利だけれど...
最後は結局 java.util.List
最初は結局 java.util.List

Gsonで ImmutableList 対応
JSONクラスのプロパティの型でも
ImmutableList 使いたいよぉ
from 現場

(> <

まあ、確かに.....
Now Live Coding

さあ、Gsonさんを拡張するぞー

LastaFlute 経由で、できるようになった！
Now Live Coding

独自のコレクション型を設定できるように
Now Live Coding

実は、思ったより大変だったぞぅ（＞＜

さあ、何が大変だったか？
どう実現しているか？
Let's ソースコードリーディング !

GsonBuilder の registerTypeAdapterFactory() に到着
Now Reading Source Code

TypeAdapterFactory を継承したクラスを作って登録

LaYourCollectionTypeAdapterFactory…
La は、LastaFlute の La だよね？
でも、パッケージが com.google.gson.internal.bind
だぞぅ。
なぜなぜ？

TypeAdapterRuntimeTypeWrapper が
package スコープでした（＞＜
呼べない.... けど、中で利用したいクラスだし、
重要そうな処理なので、あまり自作したくないので呼びたい。

そもそも組み込みの
CollectionTypeAdapterFactory を継承して...
とか考えたけど、final 付いてるし private も多いので
参考するだけ。
それなりに Gson 内部実装を
まんま自分でも実装しないといけなかった !

でもまあ、できたはできた！（＾＾
Now Live Coding

Hibernate Validator で ImmutableList 対応

Hibernate Validator が
ImmutableList で効かないときがあるよぉ
from 現場
Hibernate Validatorで ImmutableList 対応

(> <

実はわりとそのまま使えるのもある
List 関係ないアノテーションなど

NotEmpty, Length, Size などだと...

でも、LastaFlute 提供の Required は効く...おや!?なぜ？

おおぉ、instanceof 先生...んっ？
ImmutableList も Collection のインスタンス？

いま一度継承関係をおさらい
ImmutableList (implements Iterable)
┗ AbstractImmutableList (implements List)
┗ ImmutableArrayList

ImmutableList のインスタンスは java.util.List である！
なので、instanceof で判定してるものは動く

ただ、組み込みのアノテーションは、
(恐らく) 大抵宣言されている型で判断してるっぽいので...
とりあえず現場から要求のあった Size を対応
(Required があったので、NotEmpty はあまり必要とされず)

LastaFluteで、このようにすると対応できる１

LastaFluteで、このようにすると対応できる２

LastaFluteで、このようにすると対応できる３

つまり、Hibernate Validator の...
ImmutableListのSize対応のConstraintValidatorを作って、
DefaultConstraintMappingで定義して、そしてそれを、
HibernateValidatorConfiguration@addMapping()
で、登録する。

まったく同じ要領で、
NotEmpty や Length も対応できるはず
(必要になってないので、まだやってないけど... )

めでたしめで...

まだ冒険あります。
Hibernate Validator を 5.x から 6.x にアップグレードしたら、
ImmutableList の要素のクラスの
バリデーションのエラーメッセージが出なくなった。
(若干記憶が曖昧ですが、確かそのような現象だったような)

Hibernate Validator から戻ってくるパス文字列が、
5.x: sea[2].mystic
6.x: sea[].mystic
… 消えてる！
というか、もともとなんで動いてた？

深く追えてないのですが、恐らく...
ImmutableList (implements Iterable)
┗ AbstractImmutableList (implements List)
┗ ImmutableArrayList
インスタンスの型で、たまたま「動いちゃってた」って
感じなのかもしれない。そこに頼らないほうがいいかも!?

まあ、とにかく LastaFlute でこのようにして対応１
[App]ActionAdjustmentProvider :

まあ、とにかく LastaFlute でこのようにして対応２
[App]ActionAdjustmentProvider :

つまり、Hibernate Validator の...
ImmutableList専用の ValueExtractor クラスを作成して
HibernateValidatorConfiguration@addValueExtractor()で、登録する。

ふぅ

Formの受け取りで ImmutableList 対応

普通のHTMLリクエスト受け取る
FormのプロパティでImmu…
わかったわかった、もうわかった
from 現場
Form の受け取りで ImmutableList 対応
from jflute

セッション残り時間があったら、
コードを読みます！

これは完全に LastaFlute だけの話で、
該当箇所は ActionFormMapper クラス。
[App]ActionAdjustmentProvider で、
ImmutableList 情報を登録します。
(要領は今までのとほぼ同じ)

とにかく感想は...
ImmutableList を使う方はImmutableで嬉しいけど、
ImmutableList に値をマッピングするフレームワークの方は、
「追加でっきーーん」
...ってまあ、しょうがないしょうがない（＾＾）
とりあえず対応できたのでよし！

(> <
DBFluteで ImmutableList 対応

難易度桁違い...断念いや冒険半ば
ということで、DB側では手動変換はまだ発生
※ただ、手動変換も少し楽になる施策も

(DBFlute関係ないけど)
static import 補完を前提とした toImmutable()
これは現場の方が自ら作成して促進 (Good Job!)
※案外、これでまあまあ楽に。検索したらすぐ toImmutable() !

stream()と混ざるのどうする？

あーあ、まじめな話になっちゃった
Stream() と混ざるのどうする？

ジレンマ
どっちでもいいようにする？
混ざると後から読む人つらくない？
そもそも存在を知らないで書いちゃう人いるかも？

ひとまず、やはり Eclipse Collections に寄せることに
日々成長するサービスなので、積み上げ式の開発。
他の人のコードを読んで実装が日常茶飯事。
そもそも Java8 経験者も少ないのでこれから学べば良い。

それならそれで、割り切ってちゃんと促進しよう。
投げっぱなしはダメ
「それができないのなら、きっと誰も使わなくなる」

stream()を自動検知するチェックプログラム
java.util.Listのプロパティ検出するとUnitTest落ちる
...List.stream() を検出するとUnitTest落ちる
(UTFlute の PoliceStory を活用)
※ただ、ちょっと検出タイミングが遅いのが難点...

成長真っ只中、五月雨に人が増える開発現場だと...
「えっ、Eclipse Collections を使うって
知らずに stream() で書いちゃった」
「そもそも、Eclipse Collections 知らない」

入社時のオリエンテーションで
アーキテクチャ講義 (後述)
☆ なんだかんだこれが一番

習得するための研修制度

サーバーサイドチームに配属されたら、
オリエンテーションと約二週間の研修！

オリエンテーションでは、
現場のアーキテクチャやそのコンセプト、
開発の歴史などを、1on1でしっかり伝える。
(基本的に、jfluteが講師やっています)

約２週間 DFluteハンズオンをやりつつ、
その中で Eclipse Collections にも慣れてもらう。
ちなみに AssertJ も同様。
(基本的に、jfluteが猛烈レビューやっています)

「中途やパートナーでも、いきなりチケットを振るのではなく、
じっくり研修されて配属されるので、受け入れる方もすごく楽」
(現場の方の声、頂きました！)

現場のコードは、いたるところに
Eclipse Collections !!!
その結果

U-NEXT のオープンソース貢献

「オープンソースは使いながら鍛えていく」
という思想を持つ素敵な方々ばかり。
( 現場の積極的なフィードバックでフレームワークも成長！)

社内で作った汎用クラスはどんどん
オープンソースにコントリビュート！
「他でも使えるんじゃない？」が合言葉

KVSFlute, SolrFlute, Lasta RemoteApi などに加え、
一クラス単位でも汎用的な使えそうなものは、
Exampleコードとして寄与 (ありがとうございます！)
U-NEXTさんからSolrFlute,KVSFluteコントリビュート頂きました
U-NEXTさんから Lasta RemoteApi をコントリビュート頂きました

という訳でちょっとだけ宣伝

定額制動画配信サービス
実は動画配信における 10年の実績
実は日本の定額制動画配信市場においてシェア3位
(金額ベース・2017年・GEM Partners株式会社調べ)
配信動画本数 120,000本以上

U-NEXT では
サーバサイドエンジニアを
募集しています。
今注目されている動画配信市場で
一緒にサービスを作ってみませんか？
採用ページ : https://inside.unext.jp/
エンジニア募集中
開発環境
エンジニアが働きやすい環境が整っています。
定期的に開催される勉強会
フロントやビジネスなど別チームを
超えた勉強会を開催、自由に参加しています。

定期的に開催される勉強会
例えば....
DBFlute, LastaFlute (by jflute)
暗号化の仕組み, JWT
Docker, Kurbernetes
GraphQL
React, Redux
映像著作権
働きやすい開発環境
例えば....
フレックスタイム制
希望のPC貸与 (例:MacBookPro メモリ16GB)
快適なイス
大きなモニターとセカンドモニター利用可
ガジェット、ソフトウェア、書籍、セミナー等
費用全額会社補助
スーツ着用不要
フリースナック、フリードリンクあり
エンジニア同士の360度評価 (年2回)

突貫でスライドを作ってくれた、
U-NEXT なかのさん、ありがとう！
(前日深夜、見た目の調整してくれた、いのうえさん、ありがとう)

本日、利用したJavaプロジェクトは、
Githubで公開されています。
https://github.com/lastaflute/lastaflute-test-fortress

ご静聴ありがとうございました。

how unext took in eclipse collections in fw

More Related Content

More from Masahiko Kubo

how unext took in eclipse collections in fw