Page 1
「ふつうのPHP」が
になるまで
pixiv.inc
USAMI Kenta @tadsan
2018-07-14 PHPカンファレンス関西 #php
Page 7
pixiv.git
- pixivとデータベースを共有するコードは一個のリポジトリに含まれる
- それ以前はサービス横断のgit submoduleで共有されてたが効率悪かった
! accounts.pixiv.net! admin! app-api.pixiv.net! batch! bin! bungei-api.pixiv.net! comic-api.pixiv.net! embed.pixiv.net! fanbox.pixiv.net! m.pixiv.net! me.pixiv.net" oauth.pixiv.net! pixiv-lib! public-api.pixiv.net! rpc.pixiv.private! sensei.pixiv.net! source.pixiv.net! spotlight.pics! ssl.pixiv.net! tests! touch.pixiv.net! util! www.pixiv.net" www.pixivision.net
Page 8
https://niconare.nicovideo.jp/watch/kn1259
Page 11
お断り
数々の変更は@tadsanが直接行ったものではなく、歴代の開発者が改善を積み重ねてきたものです(むしろ恩恵を受けてきた立場)
Page 14
お断り
今回の発表で言及されないものは概ね時間の制約で取捨選択されたか忘れてるだけなので、懇親会やTwitterで@tadsanにきいて
Page 17
2012年のコードのイメージ
include_onceいっぱい
エラーハンドリング(してないページもあった)
<?php // www.pixiv.net/htdocs/hoge.php require_once __DIR__ . '/../inc/bootstrap.php';include_once INC_PATH . '/Hoge/Fuga.php';include_once INC_PATH . '/Hoge/Piyo.class.php';try {display()} catch (Exception $e) {error::exception_error($e);}
この下にもいっぱい
ファイルローカルな
function display() {// ...
グローバル関数
Page 18
2015年のコードのイメージ
<?php // www.pixiv.net/htdocs/hoge.php require_once __DIR__ . '/../inc/bootstrap.php';
include_once一個だけ
AppRunner::execute(new Www_HogeController);
このクラスは自動ロードされる
Page 19
AppRunner (名前の由来は知らない)
<?php
whoops
final class PCAppRunner {public static function execute(Controller $controller) {try {Controller_Util::turnOnWhoops();Controller_Util::redirectToHttps();
パソコン版
$controller->main();} catch (\Throwable $exception) {PCAppRunner::setHttpStatus(500);Controller_Util::displayWhoops($exception);
Page 20
AppRunnerとエラーハンドリング
- サービスごとの例外ハンドリングはAppRunnerが受け持つ
- whoops! はエラー処理用のフレームワークなので、これだけを使って
例外／エラー処理を完結させることも可能ではあるが、割と複雑になるので、開発環境でエラー画面を表示するだけの目的に専念してる
- エラーログはPHPの標準機能は利用せずfile_put_contents()でログファイル
に書き込んでる
- whoops! とエラーログの話はWEB+DB PRESS Vol.96 に書きました
Page 21
2018年のコードのイメージ
htdocs/hoge.phpは消滅
URLとファイルは切り離され
マップの一要素に
全URLが1ファイルに
final class Controller_WwwRoutes {public static function getRoutes() {$route_map = ['/' => ['action' => function () {Www_IndexController::main();},],'/hoge.php' => ['action' => function () {Www_HogeController::main();},
Page 27
クエリパラメータの問題
$id = $_GET['user_id'];if (is_numeric($id)) User_Common::getById($id);else error("不正な入力です");
- こういうコードを書いてはいけない
- クエリパラメータは数字ではない値が入ってくる可能性
- 入力が空、入力が任意の文字列、入力が不正な数字列、入力が配列
- filter_input() をかけると安全にはなるが、それでも面倒
Page 28
$id = ParamHelper::getPositiveInt('id');User_Common::getById($id);
- クエリパラメータ (またはform) から値を取り出すヘルパー
- 未入力や不正な入力があると例外を投げる
- 前述のAppRunnerでキャッチしてエラー画面を描画する
Page 29
$mode = ParamHelper::getEnum('mode', ['hoge', 'fuga'], ['post' => 'only']);
- ?mode=hoge または ?mode=fuga のみを期待するようなパターン
- 不正な形式や ?mode=piyo のような期待しない入力で例外を投げる
Page 30
テンプレートとURL生成の問題
<a href="{$smarty.const.SYSTEM_URL_WWW|escape}member_illust.php?id={$user.id|escape}">{$user.name|escape}</a>
- こういうコードを書くと事故が入り込みやすい
- 純粋にtypoの危険性
- idなどのパラメータに不正な値を入れるリスク
Page 31
<a href="{reverse_route page='fullWwwMemberProfile' id=$user.id}">{$user.name|escape}</a>
- あるページに名前をつけて、reverse_route関数に page 引数で渡す
- 生成結果は変更前と同じ
- ルーティングの逆関数にあたるので、一部のフレームワークは
ReverseRoutingやURLヘルパーなどの名前でサポートしてる
Page 32
/*** @route\example https://www.pixiv.net/member.php?id=12345 {id: 12345}* @route\example https://www.pixiv.net/member.php?id=12345&utm_source=xxxxx{id: 12345, utm_source: "xxxxx"}*/public static function fullWwwMemberProfile(array $params){Util_Assert::num($params['id']);
return ReverseRoute::buildUrl(SYSTEM_URL_WWW, '/member.php', ['id'],$params);}
Page 33
blogに書きました
https://devpixiv.hatenablog.com/entry/2016/10/25/093000
Page 34
PDOの問題
でもPDOで複雑なクエリを書こうとすると文字列結合が避けられない…
- PDOで書きにくいクエリの代表例: IN
- もう ? を並べてインデックスをインクリメントしながらbindValueはやだお
- PDOのクエリのplaceholderに ? を書くか :hoge で書くか問題
- 型チェックできるように書くのもちょっとめんどう
Page 38
新機能をリリース時にどうする？
- A: 新機能が実装されたブランチをリリースするときマージする
- B: リリース前にあらかじめマージして、実行されないようにしておく
- よくある問題
- Aは機能の規模が大きいとビッグバンマージを引き起こし、
予期せぬ問題を引き起こすことがある
- 理想としてはBで、一般ユーザー向けに実行されないとしても
鮮度のよいうちにマージしておきたい (リリース時の変更を最小限に)
Page 39
A/Bテスト／機能の有効化／無効化
if ($_SERVER['REMOTE_ADDR'] === OFFICE_IP) {hogehoge();}
- 新しい機能をサービスに実装して、社内でだけ有効化したいとする。
- 無造作にこんなコードを書かれると後から意味不明になる
- そもそも社外と社内での挙動が別物になるので嫌な予感しかしない
Page 40
DevToggle (A/Bテストフレームワーク)
if (ABTest_DevToggle::isEnabledDevToggle('hogeFunc')) {hogehoge();}
- 専用のコンソール画面でボタンを押すことで有効化／無効化される
- 設定を一行足すだけで一般ユーザーに対してリリースできる
- リリース後に障害などが発生しないことが確認できたらifを消す
- 元はA/Bテストのための機構だった (一般ユーザーに対して確率で適用)
Page 42
何もない野原に秩序をつくる
- べんり機能を導入すると、部分的に生産性が体感で数倍になったりする
- 一からサービスを作るとしたら同じことを繰り返せるかは、悩む
- いまだったらLaravelを入れるかもしれない
- ただ、それは2018年のチート未来人だからできる発言
- サービスの価値はコードではなく、その結果のユーザー体験
- フレームワークがなかったとしても、
pixivが2007年からユーザーに価値を提供してきた結果は変らない
