Androidソースコードレビューで指摘する事が多い項目まとめ - こやまカニ大好き

業務でソースコードレビューを行う機会が増えたので、複数回指摘した項目や気になった実装などをまとめてみました。 こういう観点をできる人と共有できるといいなあ…。 2014/09/29 23:00 一部修正しました。 業務上ソースコードレビューの名目で仕様・デザインまで見ることになっていたためこれらを先頭に書いていましたが、わかりづらかったため最後にまとめました。 Fragment関連 FragmentとActivityの密結合 Fragmentが特定のActivityから呼ばれることを想定して書かれている場合、そのFragmentとActivityは密結合である場合が多いです。 具体的には、以下の様な実装です。 ActivityのViewを参照する Activityのメソッドを直接呼び出す なぜダメか Fragmentの利点のひとつは優れた再利用性にあります。 Fragmentが特定のAct

[karasu9113]

コールバックはホームボタン押下から戻って来た時、参照がなくなってることある、getActivity()も参照消えてることあるよ。気をつけてね。

[typex2]

JSSECのセキュアコーディングガイドを鑑みても、アプリ連携する際の情報に応じて明示的Intentが推奨されるべきかな。

[americanboss]

タイトルがコードレビューのまとめなのに最初が「画面構成がiOS風」な時点で、後に書いてある有益な記事も「残念」という先入観を持って見てしまう。WebViewは明示的に止めるし、セキュリティも大変だから気をつけよう

[junpeso]

”画面構成がiOS風”　ソースコードレビューだよね？

[sugawara1991]

"AndroidのUIガイドラインに従え"という極々当たり前の指摘をどうこうするコメになんでスターが集まるのか/"ソース"という言葉に過剰反応しすぎ。CSSはソースに含まないのかな

[todesking]

厳しい

[s17er]

FragmentとActivityの密結合はなかなか頭の痛い問題

[hedachi]

“ ”

[mslGt]

【Android】

[ginzi009]

Androidソースコードレビューで指摘する事が多い項目まとめ

[sassy_watson]

わかる

[yujin_kyoto]

具体的すぎてロートルにはよく解らないけどいいこと書いてる感じ。

[kimutansk]

Get～で取ってくる場合、Androidだと時々nullになっているので、そのあたりとのバランスは必要そうですが、参考になります。

[tumo300-500]

「Android アンチパターン」あったら面白そうっすね

[d_animal141]

Androidソースコードレビューで指摘する事が多い項目まとめ

[fractured]

かつてinclude方式の構造で初版をリリースしたら、親切な人が『【ソースコードによるレイアウトの共通化】したから個別画面は気にしなくていいですよ！』と「改善」してくれたので、後者が一般的かと思っていました。

[youfzy]

“したい場合、暗黙的Intentを発行すれば良くなります。 ユーザの利便性を考慮し、ShareActionProviderを利用するこ”

[luccafort]

WebViewはあかん子。

[aodifaud09]

この記事はなぜダメか。

[nakag0711]

webview.onDestroy();のところはwebview.destroy();の間違いかな?

[kozo002]

参考

[swordbean]

よくわかってないけど、ネイティブでプログラムするかもしれないのでとりあえずブクマ

[nilab]

Androidソースコードレビューで指摘する事が多い項目まとめ - nein37’s diary

[kasumani]

Androidソースコードレビューで指摘する事が多い項目まとめ 業務でソースコードレビューを行う機会が増えたので、複数回指摘した項目や気になった実装などをまとめてみました。 こういう観点をできる人と共有できるとい

[komz]

Androidソースコードレビューで指摘する事が多い項目まとめ

[morioka]

Androidわからないけど、そのうち読むことにはなるだろう（書かないかも）。

[habarhaba]

android app dev大変そー

[taniai114]

Androidソースコードレビューで指摘する事

[Rewish]

まるでエロサイトのような大量無言ブクマっぷりですね！

[love0hate]

身に沁みる良記事

[ko-ya-ma]

tips android dev pattern

[longroof]

ふむふむ( •́ .̫ •̀ )

[syoad007]

あとで読む