Diverse developer blog

株式会社Diverse(ダイバース) 開発者ブログです。

2ヶ月間のインターンで学んだこと。

今週のお題「わたしとバレンタインデー」

はじめに

初めまして。インターン生として2ヶ月サーバーサイドの開発に携わらせて頂きました id:t-kusakabe です。 このインターンで学んだことを文字に残しておこうと思います。

今週のお題「わたしとバレンタインデー」

Poiboy

このインターンではDiverseのPoiboyというサービスの開発に携わらせていただきました。 Poiboyとは、女の子が気軽に男の子を"選ぶ"女性主導のマッチングアプリで、 表示される2人の男子から気になる方をポイしてマッチングするアプリです。 f:id:t-kusakabe:20171012163120p:plain

poiboy.jp

求めていたこと

普段僕は、アルバイト・インターンを通して新規サービスを1から作ったりPMを経験したりなど、 ある程度の実務経験を持っていました。 しかし、自分自身のスキル不足も実感しておりこのインターンでは自分自身の成長を期待していました。 とくに、テストやアーキテクチャ、ミクシィという会社から派生しているサービスは、 どのようにして作られているのかを学びたいと考えていました。

期間中にさせていただいたこと

主に以下のことを担当させていただきました。

  • 画像の差し替え
  • イベントフラグの追加
  • impression調整を管理画面から行えるように
  • SNS関連
  • 禁止ワードの正規表現対応
  • プロフィール画像検閲の機能追加

一番大きく任せて頂いたのはimpression調整に関することでした。 Poiboyではユーザー毎にどういうユーザーを表示するかのことをimpression調整と呼んでいて、今までこの調整を行うにはソースコードを直接書き換える必要がありました。 f:id:t-kusakabe:20171013134831p:plain これを管理画面から出来るようにするにあたり、設計からさせて頂くことが出来ました。 それまではシチュエーションごとに使うメソッドが限られていて汎用性が低かったのですが、調整値をDBに持たせ、メソッド自体を汎用的にすることでどこからでも使えるようにもしました。 f:id:t-kusakabe:20171012163316p:plain

実際に得たもの

この2ヶ月でコードを書く姿勢というものが大きく変わりました。 普段は、この言語のここがつらい・この言語にはこういう機能がないからという理由から言語選定をすることが多かったのですが、 今回のインターンを通してつらいところはどうすればつらくなくなるのか、どうすれば幸せになれるのかといった考え方できるようになりました。 設計思想やアーキテクチャを取り入れることで、非常に開発しやすい環境が出来ることがわかりました。
特に、関心の分離についてその大きさが感じられました。
Poiboyでは、グローバルに扱うmoduleではcontextを持たせないようにしていたり、MVCの各層で責任を明確にするすることで、結果読みやすいコード、拡張性の高いコードが生まれるようになっていました。

また、実際の開発現場に身を置くことでのメリットもありました。 周りの人たちが話している内容が耳に残りマッチングアプリ対しての知見が気づかないうちに溜まっていました。 僕はこのインターン中にも土日を利用して他企業様のインターンにも参加していたのですが、 Poiboyで培った経験から、マッチングに関して鋭い指摘をされた際、適切に応え評価されたこともありました。 こういう面でも、インターンに参加することには大きな意味があると実感しました。

最後に

大量のInputがあった2ヶ月になりました。 そして、まだまだ足りていない点、課題もたくさん見つかった2ヶ月でした。 今回得たものを自分なりの解釈に落とし込み活用していきたいです。

2ヶ月間、直接指導していただきましたメンターさん、Poiboyのみなさん、ランチの設定や各種イベントを企画していただきました人事のみなさん、 本当にお世話になりました!!!

 

夏のKotlin LT祭 でLTしてきました

id:kikuchy です。

先日開催された夏のKotlin LT祭にてKotlinJSについてお話させていただきました。

kotlin.connpass.com




GoogleがAndroidの公式開発言語にする、Springが対応を表明する、Kotlin/Nativeが発表されるなどして、Kotlin界隈もますます盛り上がってきております。
しかし、Kotlinのファーストリリース時からある機能なのにいまいちパッとしない機能があります。

Kotlin -> JavaScriptのトランスパイル機能です。

当日も会場の方々に挙手をお願いしたところ、予想通り使っている方はほとんどいらっしゃいませんでした。

どうしてパッとしないのか、実は隠れた使い勝手の良さがあるのでは、もしかしたらすぐにでも業務に投入できたりするのでは…
と思って、実際に使ってみた感じを報告するスライドになっております。


結論はスライドに書いたとおりです。

LTを聞いてKotlinについておもうこと

Androidに限定しないKotlinについての勉強会だったため、サーバーサイドの話題も多く、JVM言語としてはかなり主要な位置を占めてきはじめている言語だなと改めて感じました。

JVMで動くプログラムを作るなら、まず第一の選択肢にKotlinが上がる時代がもうすぐに来るでしょう。

私個人は自分用のコマンドラインツールやGUIのアプリなどもKotlinで書いています。
Stream関連の拡張関数も充実していますし、Delegated Propertiesを使ってPropertiesの項目をクラスメンバに割り当てると扱いが楽だったりと、便利に使えています。
趣味プロにもKotlinをぜひお使いください!

Kotlin/Nativeも盛り上がって欲しいですね。
iOSをKotlinで書く未来がくると夢見て…


DiverseではAndroid開発にKotlinを使用しています!
Kotlinを使った開発をしたい方はぜひ一度お越しください!↓

diverse-inc.co.jp

Shibuya.apk #17に登壇してきました

id:kikuchy です。
ここ半年ほどはiOSの開発をしていて、そこで一緒に仕事をしている id:devorgachem から良いアーキテクチャを教えてもらったので、それをAndroid開発に適用する方法をお話させていただきました。
とは言え、難しいことではありません。

スライド中でお見せしたサンプルのリポジトリはこちら。

github.com

要は

Activity/Fragmentにフラグを書くのをやめて、画面がとり得る状態をちゃんと列挙して管理しましょう、というお話です。

もっと言うと

状態を専用クラス(モデル)で管理し、Activity/Fragmentから切り離すととても便利です。
とり得る状態と状態遷移の仕方がわかっているので、

  1. 所定の操作をした時に、所望の状態になっているか
  2. 所定の操作をした時に、意図しない状態に陥っていないか

だけ確認できればテストは最低限のテストはできますし(Viewとかは別問題ですからね!)、一部のモデルを再利用することも可能です。


特に、APIで得られたデータを加工して表示するのが主な責務になっているアプリですと、通信状態 ≒ 画面の状態になりがちなので、通信状況を管理できるとコードの見通しが良くなります。


頂いた質問など

発表後にいくつかご質問をいただいだきましたので、共有させていただきます。

Kotlinのsealed classを使用しているのはなぜ?(enumは使わないの?)

一言で言えば、SwiftのAssociated Values付きenumと同じことをしたかったから、です。

Kotlinには特定スコープ内でのみ継承が可能になるsealed classというクラスがあります。

kotlinlang.org

アプリケーションの状態にはデータを付随させたいことが多々あります。
(例えば、「通信完了」状態には、通信の結果得られたレスポンスor失敗の原因が付随するはずです。)

やりたいことは状態を列挙して宣言しておくことなので、たしかにenumでも事足りそうな気がします。
が、JavaやKotlinにおけるenumはプログラム起動時にすべてのインスタンスが生成されてしまう(シングルトンになっている)ので、状態ごとに違う型のデータを付随させることが難しいのです。
(値をNullableで取り扱うことを許容すれば実現可能だと思います)

sealed classとdata class、それに加えてwhen式を使えば、Swiftのenumと同じことを実現できます。
data classを使えばequals()も自動的に用意されるので、テスト時の比較も簡単でSwiftよりも便利です!

LiveDataを使っている理由は?(もしくは、RxLifecycleの使用を勧めている理由は?)

モデルの生存期間がActivity/Fragmentよりも長くなりえることを想定しているためです。
(必ずしも、長くする必要があるわけではありません)

状態を抱えているモデルの生存期間は、Activity/Fragmentのライフサイクルとは独立しています。
特にDIツール(Dagger2など)を使ってシングルトンとして生成されたモデルのインスタンスなどは、Activity/Fragmentより長く生きることになります。

するとよく言われている通り、onNextのタイミングでViewが居なくなっていてNPEを引き起こしたり、Subscriberがリークする原因となるので、適宜、モデルからの通知を止めてやる必要があります。

そのために用意されているのがLiveDataなりRxLifecycleなので、それの使用をおすすめしているという訳です。


ちなみに

モデル部は通常のJava Libraryとして作れるのでJVM上でテスト可能です。(サンプルではそうしています)
テストが速攻で終わって便利です。



Diverseはこれからも新しい提案や試みを発表してゆきます°˖✧◝(⁰▿⁰)◜✧˖°

iOS Test Night #4 でLTしてきました

id:kikuchy です。
Androidをやっていたはずですが、最近はiOSをガッツリやっています。

先日開催されたiOS Test Night #4に登壇してLTをさせていただきました。


testnight.connpass.com


今回は、テストコードをアプリケーションコードと同じ(Groupの)階層に置きなおすツールを作った話をしました。

紹介しているツールはこちらになります。
github.com





きっかけは、前回 iOS Test Night #3 であった以下のツイートを見てのことでした。

ちょうどその頃、fastlaneのオプションを追いかけたりしてコードを読み、Xcodeprojというgemを知ったばかりだったのです。

github.com

このgemは、Xcodeのプロジェクト設定や含まれるファイルなどを管理する project.pbxproj を抽象化して取り扱うものです。
ビルドターゲットに含まれているファイルを取得することもできますし、ターゲットからファイルを削除したり追加することも可能。

とても便利なgemですので、Xcodeプロジェクトを操作するようなツールを作りたいと思っている方は、ぜひ一度チェックしてみてください。




発表の後に主催の方からこのようなツイートをいただきました。

勉強会で得られるものは、問題の解決方法だけではありません。
問題がある、ということ自体を教えてもらうことができます。
問題があれば、解決するチャンスです!

Diverseのエンジニアは、これからも問題も解決方法も発信してゆきます。\\ ٩( 'ω' )و //

Shibuya.apk #12に登壇してきました

こんにちは。最近はiOSの開発をしている id:kikuchy です。
iOSの開発をしていますが、Androidの新しい動きは引き続き追いかけております。


渋谷近辺のAndroidエンジニアが集まる Shibuya.apk という開発者コミュニティがあります。
先日は12回目の開催でした。

shibuya-apk.connpass.com



今回は Android Things について、AndroidエンジニアがIoTにどう関わり始めたら良さそうなのか、お話させていただきました。

Android Thingsでは普通のAndroidスマートフォン向けのコードがそのまま動くため、
あとはアナログ回路の知識とスキルがあれば、 物理的インタラクションができるAndroidアプリ を作成する事が可能です!

ということは、Androidエンジニアは既存のスキル+αで新しいジャンルに挑戦できる、ということでもあります。

ぜひこのタイミングで、Android Thingsを通じてIoTの世界に足を踏み入れてみませんか?


ちなみに、Yahoo!さんのオフィスビルがとても綺麗だったのが印象的でした。
f:id:kikuchy:20170203193433j:plain

レガシーから脱却する話をしました

f:id:lasa007:20161122215639j:plain
2016年11月22日、ヒカ☆ラボにて、ネットマーケティング様と合同で「レガシーシステム脱却秘話」をテーマに勉強会をさせていただきました。

atnd.org

弊社からはエンジニア3人が登壇いたしました。何を発表したのか、どのような思想や思いでレガシーからの脱却に取り組んでいるのかをご紹介します。




どうしてコードはレガシーになるのか

f:id:lasa007:20161122200148j:plain
id:kikuchy です。
『どうしてコードはレガシーになるのか』という題でお話をさせていただきました。



YYCチームでは昨年から今年の頭にかけてAndroidアプリをフルスクラッチでリニューアルしました。
そのきっかけ(原因)となったリニューアル前のアプリのコードのレガシーさと、その分析から得られた「レガシーコードを生み出さない環境づくり」の方法についてのお話です。

レガシーコードが生まれる原因

人間が当たり前のことを当たり前にやらないからです。
ところが、当たり前のことをやる、というのは意外と大変なものです。
しかし、人間は認知的負荷の低い行動はとりやすい(前例に倣う、など)ので、当たり前のことをきちんと実行できる環境を整備しましょう、というのが今発表の主旨でした。

整備するべき環境
  1. プロダクトがどのような構造で出来上がっているのかを、後から来た人にもわかるようにする
  2. その言語らしいコードを書く
  3. エンジニアがスキルアップを心がけるようにする(仕掛ける)
レガシーコードが生まれたら

スライドに明記していなかったのですが、このようになると思います。

  1. これ以上レガシーコードが生まれない環境を整える
  2. 地道に取り除く
  3. 取り切れる量でなかったらコードを全部捨てる

最後の「コードを捨てる」は時間もお金もかかるため、最終手段ですが……
YYC Androidアプリの場合は経営判断でOKがでたので、フルスクラッチでリニューアルを行いました。



Titaniumから脱却している話

f:id:lasa007:20161122203208j:plain

こんにちは、Diverse 結婚支援事業部 アプリエンジニアの @yuta-fujita-dvs です!

当日のスライドはこちら

YoubrideのiOSアプリをTitaniumからSwiftにするために行っていることについての発表です。

RedmineからGitlab、Jenkinsの導入、チーム横断でのコードレビュー、に開発環境を変えていったお話になります。
詳しくはスライドを参照していただければと思います。

反省点としては、

  1. デメリットにまだ自分の中で解決できていない内容を混ぜてしまっていた。
  2. LTなのに発表時間を意識せずにダラダラ話してしまった。
  3. 発表中にPCを見すぎて会場に目を向ける時間が少なかった。

といったことです。

まだまだレガシーから脱却していくためには壁があり、現在進行中で壁を破っているので、次回があれば反省点を活かしつつそういった話などもできればな・・・と思っています。




集計システムをDBからTDに移した話

f:id:lasa007:20161122205307j:plain

女性向けデーティングアプリ Poiboy でiOSエンジニアをしている中西 @cfiken です。

同様のイベントで、「集計システムをDBからTreasureData(TD)に移した話」というタイトルで、
TreasureDataを使って行ったことと、それによってチームがどのように変わったか、という話をさせていただきました。

Poiboyは今年の1月に正式リリースしたデーティングアプリで、少人数チームでの開発のため、開発効率であったり、PDCAサイクルを如何に早く回すか、という点がサービスのスケールのために重要です。
そのために必須とも言えるのがサービスの各数値の集計です。
より詳しく、より早い段階でチームメンバーが数値を追うことが出来るように、今までサービスのDBからのみ直接集計を行っていたものを、TD上に移して自動化を行いました。

前半は、具体的にどのような手続きを踏んだのか、TDの使い方、集計効率をあげるための工夫、メンバーに浸透させるための工夫などの話を。
後半は、色々と集計に関するレガシーを脱出した結果、チームメンバーの働き方にも影響があったよ、という話をさせていただきました。

まだ変化の途上ではありますが、後半の話が中心だったので具体的にいくつか紹介すると、

  • チームメンバーが以前より数値を追うようになった
  • 施策の企画から反省までの流れが確立した
  • 集計が身近になり、開発者だけでなく一部の企画などのメンバーが直接WebからTDを触るようになった

など、当初想定していなかった色々な良い影響がありました。

今までは施策の度にエンジニアがKPIを集計していたのを考えると、ただ見れる数値が増えたのみにとどまらず、大きな改善になりました。
レガシー要因を改善することで、そのレガシー部分だけでなく、想定していなかった良い影響があるかも、ということがイメージとして伝われば幸いです。

準備時間があまり取れず、時間配分や資料のクオリティがまだ甘かったところがあるので、次の機会ではもう少ししっかりと準備したいと思います。
また、タイトルにTreasureDataと入っていたことで、TDの踏み込んだ使い方やデータサイエンスに関する話を期待されていた方もいたようで、また機会があれば次はそのような話をしてみたいですね。
当日お世話になった皆さま、ありがとうございました。


今回の勉強会で、思いの外たくさんの方が古いシステムから新しいシステムへの移行について悩んだり苦労したりしていることがわかりました。

どんな新しい技術も時間が経てば陳腐化してしまい、その寿命は年々短くなっているようにも感じます。が、Diverse は、常に新しい技術を積極的に導入できるようなエンジニア文化を推進していきたいと思います。

Poiboyのインターンで非日常な体験をした話

はじめに

こんにちは!ミクシィのサマーインターンシップに参加して、Diverseでインターンしていたid:anddev68です。

f:id:kiyoponb:20161014052612j:plain

1ヶ月と短いインターンでしたが、多くの経験を得ることができました。この良き体験を自分自身で整理するため、そしてもっと多くの人に広めるためにインターン体験記&レポートを書きたいと思います。

結論からいうと、時間があるなら今すぐ参加すべきです。

mixi.co.jp

 

経緯・動機

私の今夏のコンセプトは「非日常な体験をしよう!」でした。

皆さんはデーティングサービスを利用したことがありますか?私もインターンするまでは利用したことがありませんでした。デーィングサービス*1のインターン今夏のコンセプトにぴったりではありませんか。

また、mixiのインターンに参加すると、東京で生活できる環境が提供されます。これは地方民である私にとって大きなメリットでもありました。

やったこと

大きく分けて4つあります。初週はアプリを徹底的に使い込んでバグを見つける作業でした。アプリとソースコードを合わせながらどこに何があるのか把握するという目的があります。2週目は実際にコードを触ってみたり、開発フローを学ぶために11バグの修正。3週目は、1,2週目の総まとめとしてお気に入り画面の作成。4週目は残ったタスク処理と発表会という充実したスケジュールでした。 

デバッグ

他の人が作ったアプリのデバッグをしてみて、普段の自分が作ったアプリに対するデバッグの適当さが明らかになりました。これからデバッグを行う際はさらに注意深く見る必要があると思いました。

バグ修正

既知の問題点、あるいは自分が見つけた問題点を直しました。問題点リストから1個ピックして、コードを書き、PRを出して、レビューでチェックしてもらうというのが一連の流れです。コーディング規約*2や設計に従う必要があり、思ったより時間がかかってしまいました。

お気に入りの画面作成

1週間かかる最も大きなタスクでした。画面を作成するためには、API*3や設計、画面レイアウト、非同期処理*4などを理解する必要がありました。Poiboyではクリーンアーキテクチャーという設計技法に加え、RxJavaという非同期処理用ライブラリが用いられており、苦労しましたが大変勉強になりました。

クリーンアーキテクチャーは、役割ごとにしっかりとレイヤーが分かれていて綺麗な設計でしたが、1機能の実装に4ファイル以上書き換えなければならないという点に苦労しました。また、RxJavaについては、Javaとは名ばかりで、実際には別の言語で書き方を覚えるのが大変でした。

実際に作成した画面がこちらになります。

 f:id:anddev68:20161014112047p:plain 

その他

デザイナー・エンジニアなど業種を問わずインターン生で進捗報告会が数回ありました。進捗報告会では、「VCSってなんですか?」など、共通語だと思っていたものが実は業界用語だったということがありました。自分がやっていることを専門外の人にもわかりやすく説明する技術も必要だと感じました。

 

おわりに

インターンシップは、非常に多くの体験をし、様々な知識を得ることができました。技術面、その他の面においても、今後に活かしていきたいと思います。

遠方に住んでいる方は特に、東京で働いてみる絶好の機会ですので、是非応募してみてください。

*1:女性と男性を繋ぐためのサービス, Poiboyなど。

*2:コードを書く上でのチームで決めたルール

*3:Application Programming Interface, サーバとアプリでの通信ルール

*4:画面の裏側でダウンロードするなどの処理のこと