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

気に入った記事をブックマーク

気に入った記事を保存できます
保存した記事の一覧は、はてなブックマークで確認・編集ができます
記事を読んだ感想やメモを書き残せます
非公開でブックマークすることもできます

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

テクノロジーカテゴリーの変更を依頼記事元:

nein37.hatenablog.com

適切な情報に変更

エントリーの編集

エントリーの編集は全ユーザーに共通の機能です。
必ずガイドラインを一読の上ご利用ください。

このページのオーナーなので以下のアクションを実行できます

タイトル、本文などの情報を
再取得することができます

コメントを非表示にできますコメント表示の設定

ブックマークしました

ここにツイート内容が記載されますhttps://b.hatena.ne.jp/URLはspanで囲んでください

Twitterで共有

次回からTwitterへ自動リダイレクト

ONにすると、次回以降このダイアログを飛ばしてTwitterに遷移します

636usersがブックマークコメント

コメントするにはログインが必要ですブックマークを追加

ブックマークを追加

よく使うタグ

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

636 users

nein37.hatenablog.com

よく使うタグ

はてなブックマークで
関心をシェアしよう

みんなの興味と感想が集まることで
新しい発見や、深堀りがもっと楽しく

ユーザー登録

アカウントをお持ちの方はログインページへ

記事へのコメント40件

注目コメント
新着コメント

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

2014/09/29リンク

その他

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

2014/09/29リンク

その他

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

2014/09/29リンク

その他

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

2014/09/29リンク

その他

todesking厳しい

2014/09/29リンク

その他

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

2014/09/30リンク

その他

hedachi“ ”

2015/12/21リンク

その他

mslGt【Android】

2015/12/21リンク

その他

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

unclassified

2015/02/25リンク

その他

sassy_watsonわかる

Android

2014/10/05リンク

その他

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

2014/10/01リンク

その他

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

2014/10/01リンク

その他

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

Android

2014/09/30リンク

その他

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

Android

2014/09/30リンク

その他

$fractured$

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

2014/09/30リンク

その他

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

android

2014/09/30リンク

その他

luccafortWebViewはあかん子。

2014/09/30リンク

その他

aodifaud09この記事はなぜダメか。

2014/09/30リンク

その他

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

2014/09/30リンク

その他

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

2014/09/30リンク

その他

kozo002参考

2014/09/30リンク

その他

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

2014/09/30リンク

その他

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

2014/09/30リンク

その他

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

2014/09/30リンク

その他

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

unclassified

2014/09/30リンク

その他

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

2014/09/30リンク

その他

habarhabaandroid app dev大変そー

2014/09/30リンク

その他

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

2014/09/29リンク

その他

2014/09/29リンク

その他

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

2014/09/29リンク

その他

2014/09/29リンク

その他

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

2014/09/29リンク

その他

love0hate身に沁みる良記事

android

2014/09/29リンク

その他

ko-ya-matips android dev pattern

2014/09/29リンク

その他

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

2014/09/29リンク

その他

syoad007あとで読む

2014/09/29リンク

その他

コメントするにはログインが必要ですログインしてコメント

注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています

リンクを埋め込む

以下のコードをコピーしてサイトに埋め込むことができます

プレビュー

はてなブックマークボタンを作成して埋め込むこともできます

規約違反を報告

いまの話題をアプリでチェック！

バナー広告なし
ミュート機能あり
ダークモード搭載

アプリをダウンロード

usersに達しました！

さんが1番目にブックマークした記事「Androidソースコー...」が注目されています。

気持ちをシェアしよう

ツイートする

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

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