iOSアプリケーション開発のコードレビューで気をつけていること - ninjinkun's diary

日常的なコードレビューで気をつけていることリストです。GitHub会議(仮)で発表しようと思っていたのですが、日程の都合で参加できないので、書きためておいたメモを公開します。またどこかで発表するかもしれません。 AutoLayoutにできないか AutoLayout化した方がすっきりしそうならAutoLayout化する AutoLayout化できそうなものでやっていないものは、なぜコードで実装したか質問する 例えばUITableViewCell ちゃんと理由があれば別に良い。コードの方が良いことも多い UIAppearanceで解決できないか 各クラスの中にスタイルの指定が入るより、UIAppearanceでスタイル指定を分離して別クラスに書く方がデザイナーも弄りやすくて良い 3.5インチ端末が考慮されているか レイアウトが決め打ちだとここで問題が出ることが多い 着信ステータスバーが考慮さ

[ninjinkun]

id:stet そう見られるとは考えていなかったです。タイトル変更しました。

[koogawa]

「これをネタにLTとかできそうですねとか指摘する」これ良い！

[FunnyBunnyDizzy]

進んでるなー。

[yashigani_w]

「これをネタにLTとかできそうですね」使っていきたい

[paranishian]

メモメモ

[kasumani]

iOSアプリケーション開発のコードレビューで気をつけていること 日常的なコードレビューで気をつけていることリストです。GitHub会議(仮)で発表しようと思っていたのですが、日程の都合で参加できないので、書きためてお

[longroof]

気をつけまう(；´Д｀)

[efcl]

NJKレビュー。 iOSアプリのレビューの注意点について

[aretokore]

>> iOSアプリケーション開発のコードレビューで気をつけていること - ninjinkun's diary

[tarchan]

＞これをネタにLTとかできそうですねとか指摘する

[kunipon]

iOSアプリケーション開発のコードレビューで気をつけていること 日常的なコードレビューで気をつけていることリストです。GitHub会議(仮)で発表しようと思っていたのですが、日程の都合で参加できないので、書きためてお

[iriya_0624]

エッセンスが詰まってる。

[gigi-net]

Storyboardの設計とかレビューしづらそう

[mather314]

メモ

[lestructure]

チェックリスト化できるものを人力レビューするのは何故？と思ったけど、程度問題なわけやね。

[src256]

コードレビュー

[naohero22]

気をつけますー！

[UDONCHAN]

本当に有用情報

[satosssi]

こーいうの公開するのはほんと素晴らしいと思う

[sovapritz]

“Fablic”

[miso_engine]

確かにViewController閉じるときにHTTPリクエストキャンセルし忘れる……。これは便利

[stet]

Appleの中の人だと思って読んだら…"iOSアプリの"って書いた方がよくね？よくね？よくね？

[okbm]

Fablicの人！

[hush_puppy]

ちゃんとインタラクション寄りな方もレビューするのか。自分はDiffのコード断片ばかりみてしまって、そういうのできてない・・・

[ainame]

“これをネタにLTとかできそうですねとか指摘する ”

[laiso]

“自分が所属しているFablicではiOSエンジニア3人で開発しているので、ほぼレビューが溜まることがなく粛々と開発が進んで行きます。”

[oppara]

iOSのコードレビューで気をつけていること - ninjinkun's diary

[lesamoureuses]

“これをネタにLTとかできそうですねとか指摘する ”