Diverse Advent Calendar の18日目です。 今までまともにコードレビューをしたことがなかったエンジニアが、半年前からレビューを経験し始めて、チームの実力を底上げした（と思っている）レビューの仕方をご紹介します。 ここで説明すること、しないこと ここでは以下のことについて説明します。 master のコードの質を上げるためのチェックポイント 自分のレベルを上げるコードリーディングの方法 以下のことは説明しません。昨日の mixi Advent Calendar で記事にしましたのでそちらをご参照ください。 コードレビューの雰囲気を良くする（良く保つ）方法 自分やチームメンバーの成長を促すための心がけ なぜ見るべきところとそうでないところがあるのか 効率的にレビューを進める必要があるためです。 レビューに時間をかけすぎてもあまり良いことがないのです。 レビューワーの仕事は

クラス名編をつくりました あるメソッドを定義しようとするとき、そのメソッドを使う人達が名前からどんなことをするか理解できるようにするには、メソッドの内容に応じて適切な情報量の命名が求められます。 この記事では、メソッド名に用いることでどのような情報が提供できるかを見ていきたいと思います。 真偽値を返すメソッド 場所 単語 意味 例

※この記事は元々「Gitのこれやめて！リスト」として2015年11月に投稿したものを改訂したものです。 この記事について 私が個人的にgitとプルリクエストについて、「こうして欲しい」とか「これはやらないで！！」とか思っていることをまとめたものです。 元々は2015年に私がコードレビューをしてる時に気になったことを、あまり推敲もせず思うがままに書いた記事でした。今改めて読み返すと稚拙な文章なのと、他に思うところとがあったりしたので、改めて書き直しました。いいね貰ってるのに書き直すのに若干後ろめたさがあるのですが、よりいい内容にできればと思います。 コミットログがきれいだとレビューしやすい 一人で開発するときはgit使っててもブランチ切らないし、プルリクもださないしで、コミットログも"First Commit"の次が"Second Commit"とかでも支障はないです。しかし、チームで開発す

JavaでのテストはJUnit4が使われていると思いますが、自分としては、それに加えてAssertJをオススメします。 AssertJ AssertJが使いやすい理由 JUnit4のassertThatと比べてAssertJが使いやすい理由は2つあります。 流れるようなインターフェース AssertJは「Fluent assertions for java」とトップページに大きく書かれているように、流れるようなインターフェースが最大の特徴です。いちいちドキュメントを調べなくても、IDEの補完機能で適切なメソッドを調べられるので、JUnit4のassertThatに比べて書きやすいです。 拡張がMatcherに比べて遥かに楽 Matcherの拡張対象は「比較方法」で、AssertJの拡張対象は「クラス」なので比較するのは適切ではないかもしれませんが、Matcherの拡張が靴の上から足を掻く感

普段Javaで書いている部分をScalaで書けるようになった。 Javaだと、文字列や複雑なデータ構造を処理したり、関数プログラミングがしづらい。lambda無いし。そういう部分だけでもScalaでやりたい。 https://github.com/pfn/android-sdk-pluginというフレームワークを使って、既存のAndroidアプリのプロジェクトでScalaとJavaを混ぜこぜに書けるようにする方法を説明する。 全体の簡単な手順 homebrewでscalaとsbtをインストールする Androidのプロジェクトを普通に作る or 既存のAndroidプロジェクトを用意する sbtで使う設定ファイル3つを配置する sbt初回起動時に必要なライブラリがインストールされて、Androidプロジェクトとsbtプロジェクトが共存した状態になる sbtでandroid:packageす

Git Advent Calendar / Jun. 最終日(30日目)の記事です．29日目は「いざという時のためのgit reflog」でした． Git Advent Calendar最後なので，git操作でやりがちなミスからどう回復するかをまとめます．他にもあればコメントもらえるとマージしていきます． ブランチを切り忘れてmasterでコミットしてしまった その時点でブランチを切る&reset --hardで間違ったコミットたちをmasterから消す $ git checkout -b new-branch # masterの最新コミットを消す $ git checkout master && git reset --hard HEAD~

以前gitで一度行った変更をなかったことにする方法4つを紹介しましたが、 日常的に git を使用していると他にも様々な 「なかったことにしたい」「元に戻したい」 という状況に遭遇します。 そのひとつひとつについて対処方法を紹介していきます。 目次 問題1: ライブラリの新機能を試すためにあれこれ適当なコードを書いてみた。でももう要らない。問題2: トピックブランチをマージしたけど実はまだ不完全だった。マージをやり直したい。問題3: リリース後に発覚したバグ。原因は30日前に自分が行ったコミットだった。なかったことにしたい。問題4: 新しいコミットしようとして間違えてgit commit –amendで書き換えてしまった。元に戻したい。問題5: 色々作業していたら作業ディレクトリの内容が混沌としてきた。一度綺麗な状態にしたい。問題6: 作業ディレクトリにゴミファイルが溜まってきた。一度綺麗

gitは、とにかくトピックブランチを作成して作業する。だいたい機能追加とかバグ修正とかの単位でブランチを作って作業します。（ちゃんとやってますよね？） なので、作業の途中で別の修正を優先してお願いっ！なんて言われたときは、別のブランチに切り替えて作業をする必要がでてくる。そんな時に変更を一時的に退避しておくことのできる機能、それがstashである。 では、早速使い方 まだcommitしていない状態の変更ファイル（addしてる or add していない)が存在する状況で、次のコマンドを実行すると変更ファイルを退避することができる。 ※saveは、省略することもできる。 これでファイルの退避完了！git statusとか見てみると変更状態であったファイルがなくなっている。この状態なら安心してブランチを切り替えることもできる。めでたしめでたし。 と、退避だけならこれで終わりだけど、次につかうとき

まずはmecab-rubyを使うためにmecab関連のパッケージを検索 $ brew search mecab mecab mecab-ipadic 調べてみると上で出てきた２つは必要らしいので２つともインストール $ brew install mecab $ brew install mecab-ipadic 以上でterminal上でmecabは使えるようになった。 $ mecab 何て日だっ！ 何 名詞,代名詞,一般,,,,何,ナニ,ナニ て 助詞,格助詞,連語,,,,て,テ,テ 日 名詞,非自立,副詞可能,,,,日,ヒ,ヒ だっ 助動詞,,,,特殊・ダ,連用タ接続,だ,ダッ,ダッ ！ 記号,一般,,,,,！,！,！ EOS 次にrubyで使えるようにmecab-rubyをインストールします。 rubyのversion確認 $ ruby -v ruby 1.9.3p362 (2012

皆さんWeb開発してますか？ 開発者にとってデザインはいつの日も悩みの種。 そんな時に便利なのがTwitter Bootstrapですよね。何となくカッコよくしてくれるということで猫も杓子もBootstrapですよ。 っでどうなったかというと、巷にBootstrapなデザインが溢れかえるわけです。人の欲望は無尽蔵。今度はBootstrapを使ってどれだけ「Bootstrapっぽくない」デザインをするかということになるわけです。 そこで昨日、突然話題？となったGoogle Bootstrapを使ってみました。 使うのに色々と面倒な手順を踏まないといけないので、メモとして。 っで最初に躓くの何故かGoogle Bootstrapの「Download」ボタンを押すとgithubの404？？？ でソースを探すとここではないかと。 https://github.com/todc/google-boo

以下のようなJSONデータがあったとします。 {"json":[{"id": "00001","category": "カテゴリー1","title": "テスト1"},{"id": "00002","category": "カテゴリー2","title": "テスト2"}]} String json_data = "{\"json\":[{\"id\": \"00001\",\"category\": \"カテゴリー1\",\"title\": \"テスト1\"},{\"id\": \"00002\",\"category\": \"カテゴリー2\",\"title\": \"テスト2\"}]}"; JSONArray jArray; StringBuilder sb = new StringBuilder(); try { jArray = new JSONObject( json_d

こないだGoogle App Engine + slim3 + JSONICで作ったサンプルをAndroidで表示するアプリを作ってみたのでメモ。 とりあえず実現したい事は、JSONをHTTPのgetで受け取ってListViewで表示までを目標に。 で、下記のように実装。 （URIベタ書きですが...、とりあえず動かしたいので。） import java.io.IOException; import java.util.ArrayList; import java.util.List; import org.apache.http.HttpEntity; import org.apache.http.HttpResponse; import org.apache.http.HttpStatus; import org.apache.http.ParseException; import o

本日、ParseというBaaSサービスプロバイダーがベータを終え、正式サービスに移行したというニュースがTLを賑わしていました。 モバイルアプリのサーバサイド/バックエンドを肩代わりする人気のParseがベータを卒業 個人的にBaaSの事が少し前から気になっており、早速どんなものか試しに使ってみました。 BaaSってなに？ BaaSとは”Backend as a Service”の略称で、モバイルアプリと連携するクラウドのバックエンドに求められる機能、例えばプッシュ通信、ソーシャル連携、ロケーション、写真のクラウドへのストア、など本来サーバー側の実装が必要だった機能群を、アプリ側からAPIを呼び出すだけで完結出来てしまう、というサービスです。 少し前にTitanium Mobileを提供するAppceleratorがBaaSプロバイダのCocoafishを買収した事も話題になっていました。

AndroidプログラミングのTOPへ 重要なお知らせ： この記事で公開した情報は，AndroidのMVCフレームワーク「Android-MVC」の機能の一部として取り込まれました。 より正確な設計情報や，動作可能な全ソースコードを閲覧したい場合，「Android-MVC」の公式ページより技術情報を参照してください。 AndroidのMVCフレームワーク - 「Android-MVC」 http://code.google.com/p/android-mvc-... Androidアプリ開発時に，HTTP通信でWebサーバに対しPOSTリクエストをするサンプルコード。 ここでPOSTパラメータは，UTF8などのエンコードで日本語のマルチバイト文字列を含むとする。 この通信処理を，文字化け無しで正しく扱う。 送信するリクエストヘッダも，サーバから受信するレスポンス内容も，指定した文字コード通