タグ

reviewに関するrudo108のブックマーク (2)

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

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

    Androidソースコードレビューで指摘する事が多い項目まとめ - こやまカニ大好き
  • 6画面ディスプレーの巨大デスクトップを構築する技【前編】 (2/2)

    この手のニッチな製品だと、6ヵ国語で書かれた簡易的なマニュアルしか入っていないことがままあるが、それぞれの製品に紙のマニュアルが入っており、丁寧に解説が書かれていたのはうれしかった。だいたいは感覚でわかるが、調べたい時にすぐに把握できる。また、レンチやスパナもすべてのパーツに同梱されていた。3人同時に作業できるのはうれしいところだが、ものすごい数のレンチとスパナが余ってしまったのはちょっともったいないと感じた。 設置作業は重労働でヘトヘトに 半分組み上げたディスプレーアームを2階の仕事部屋に持ち込み、いよいよ設置を開始。「モニタアーム補強プレート」をクランプに挟んで、PCデスクに装着。六角レンチで締めるのだが、支柱が2で3.2kg、アーム取り付けバーは13.1kgあるので、結構な重量になる。クランプの下に挟む小さな補強プレートは滑り止めがないので、すぐ落ちてしまい、何度もやり直しになっ

    6画面ディスプレーの巨大デスクトップを構築する技【前編】 (2/2)
  • 1