はじめに

こんにちは。星です。

弊社では、お客様の基幹システム構築をする際、Java言語を採用することが多いのですが、2015年4月末にJava 7のサポート切れになったことを受けて、昨年よりJava 8で開発をしています。

弊社でもそれなりの規模の案件になると、社員やパートナーの皆様を合わせて、数百人が同時に開発することも珍しくありませんので、私の所属する技術部隊でコーディング規約をはじめとして、開発をするにあたってのガイドラインの整備やEclipse等の開発環境の整備などのタスクを実施して、標準化とクオリティの担保を推進しています。

さて、Java 8においては、Java 7において実装見送りとなったStream APIやラムダ式といった大きな機能追加がありました。とはいえ、これらの機能を使ったとして、性能的に大丈夫なのかとか、どういったコーディングスタイルが良いのか？ など、エンタープライズ領域において、どう利用していくのかを決める必要がありました。

世の中のJavaのコーディング規約

インターネット上で公開されているJavaのコーディング規約で、主要なものを挙げてみました。

作成したのが2000年代で古いものが多く、Stream APIやラムダ式についての記載があるものは見当たらず、参考になりませんでした。

Future Enterprise Coding Standards

世の中になければ自分たちで作るというのは弊社の行動理念でもあるので、社内で伝統的に受け継がれ、細かな改善をしてきたコーディング規約をベースに技術チーム有志にてJava 8対応版を作りました。
(ちなみにモチーフがなぜハチなのかは、わかりますよね？ )

そして、「Future Enterprise Coding Standards」と名付けて、せっかくなので、Githubに公開しました！！

• Future Enterprise Coding Standards for Java
  • https://future-architect.github.io/coding-standards/

Java 8対応以外にも、性能の考慮も含めたコーディング規約になってます。
その一部をここで紹介します。

Java 8に対応

例えばStream APIですが、皆さんだったら、下記3パターンのうち、どれを標準にしますか？

パターン1

List<Character> alphabetLower = list
    .stream()
    .filter(Character::isAlphabetic)
    .map(Character::toLowerCase)
    .collect(Collectors.toList());

パターン２

List<Character> alphabetLower = list.stream()
                                    .filter(Character::isAlphabetic)
                                    .map(Character::toLowerCase)
                                    .collect(Collectors.toList());

パターン3

List<Character> alphabetLower = list.stream()
    .filter(Character::isAlphabetic)
    .map(Character::toLowerCase)
    .collect(Collectors.toList());

パターン1だと1行目が気持ち悪いと感じる人がいそうですね。
パターン2は一見美しく見えるのですが、Eclipse等でフォーマッタの設定が難しかったりします。
そんなわけで、私達が設定したコーディング規約では可読性とフォーマッタとの相性も踏まえて、パターン3にしてます。

性能の考慮

次は性能についてです。下記2つのパターン、どちらも等価のリストを返しますが、
どちらのパターンで実装したらよいでしょうか？

1. 拡張for文パターン

List<String> list = //数値文字列のList
List<String> resultList = new ArrayList<>();
for (String string : list) {
    if (string.endsWith("0")) {
        resultList.add(string);
    }
}
return resultList;

2. StreamAPIパターン

List<String> list = //数値文字列のList
List<String> resultList = list.stream()
    .filter(s -> s.endsWith("0"))
    .collect(Collectors.toList());
return resultList;

Java 8を使っているし、せっかくだからStream APIで実装しようと思っても、
実は性能がシビアに求められる処理だったりしませんか？

計測結果

このように従来通りの拡張for文のほうが速かったりするので、性能も踏まえて実装をどうすべきかを判断する必要があります。こういった性能への考慮事項もコーディング規約に組み込んであります。

コーディング規約の今後について

今後はWebアプリケーション編など実践的な内容なども盛り込んでいきたいなと思ってます。
またSQLやJavaScriptなど、他の言語についても公開していきたいと思っています。

さらに、CheckStyle定義やフォーマッタ、スニペットなども用意していきたいなとチーム内でも話をしてますが、なかなか手が回らず。。。

これからも改善したいと思っていますので、プルリクお待ちしています！！

https://github.com/future-architect/coding-standards

参考：イメージキャラクターについて

特徴

• 目と触覚の配色はDukeをリスペクト
• 針ではなく葉っぱが生えてる（「Javaハチ」 -> 「Java鉢」 -> 鉢植えには葉っぱが生えてる）

イメージキャラクターは、前回記事のアイコンセット作者の木村さん作成です。