ソシャゲエンジニアの自分がコードレビュー時に重視する箇所33選 【随時追加】 - Qiita

コードレビューで土日に安寧を ソーシャルゲームは、ユーザアクセス集中と、それに伴うユーザデータ増加によって劇的に負荷が上がり、（主に土日に）サービスに影響を与えがちです。 問題があるコードは、たとえ負荷テストを行っても、作成したシナリオによっては見つけられない可能性もあります。 そういった見えない不安を払拭するという意味でも、コードレビューは重要だと思っています。 【ステキポイント】 ・ ソースを見ることにより、時限爆弾が土日に爆発するのを解除 ・ スキル共有によってメンバーがレベルアップすることにより、土日に爆発する時限爆弾の設置確率低下 まぁまとめると これに尽きます（4歳の息子談） 今は、gitのプルリクエストという強力なレビューツールもあり、敷居がかなり低くなったのでオススメです！ チェックするポイントは5つ コードレビューを行うにあたり、「どんなところをチェックすればいいのか分か

[laiso]

“今は、gitのプルリクエストという強力なレビューツールもあり”

[hatomugicha]

素晴らしい

[iekusup]

ほー。

[Rinta]

「カーディナリティが低い列にインデックスを貼っていないか？」つbitmap index。ただしインデックスは付ければそれだけ更新系が遅くなる点は注意がいる。あと、データ量が増えるかどうかもポイントかなー。

[hnw]

サラっとXAトランザクションとか書いてあるけど高負荷環境で実用できるんだ…

[raimon49]

いずれも妥当だし、こういう風に自分が何に着目してレビューしてるか言語化できるスキルって大事。見習いたい。

[Kesin]

一番やっかいなのはギャップロック

[shin0O]

「gitのプルリクエスト」 と 写真で評価ダダ下げやな

[MrX5249]

素晴らしい。明日みんなに共有しよう

[urza358]

development

[tikkss]

重点的にチェックするポイントは5点

[vanbraam]

ソシャゲ的(アクセス集中時のresponseへの考慮)な性能観点が多い;indexは張りすぎると書き込み性能が落ちるんだがその辺は気にしてないのかな?

[gratt]

コードレビュー

[efcl]

データベースとかRedisのチェックシート

[izumin5210]

チェックリストにして貼りたいレベル

[queenstage]

いずれも妥当だし、こういう風に自分が何に着目してレビューしてるか言語化できるスキルって大事。

[Steiger31226]

“妥当”

[kaipu1224]

土日に限らず平日も残業しまくる人ばかりで嫌になる。単体テストもしないで本番導入するような人ばかりだお・・・

[Peranikov]

当たり前なことも多いけど明文化大事

[sanryuu]

暗黙知を形式知化したのか。他の人をレビューするときにメモっておいたのかな。いいなぁ。

[katsuto-takeda]

“ャゲエンジニア”

[luccafort]

これはチートシートとして普通に使えそう。

[suthio]

チェックリスト作ろう

[baca-aho-doji]

具体的でわかりやすいと思った。気をつけよう。

[diveintounlimit]

だいたいあってそう

[greencoffeemaker]

こういう記事が読めるのはありがたい

[aodifaud09]

ソシャゲねえ

[uva]

運用視点のレビューできますか

[yoko_yoji]

どちらかというと、このレベルでレビューしてあげられるような、時間管理の方法をしりたい

[kikuchi1201]

インフラの知識かなあー

[perl-o-pal]

これはソシャ/Web以外の人にも有用かもだなあ。