Tbpgr Blog

Employee Experience Engineer tbpgr(てぃーびー) のブログ

2012-08-26から1日間の記事一覧

不要な分岐

概要 レビュー指摘「不要な分岐」について ※Javaの話です 内容 私が実際に指摘した「不要な分岐」について。 コードを見ていただくのが手っ取り早いかと思います。こういうコードを書くJava開発経歴10年のプログラマーとかが 居たりするのが日本の開発現場の…

不適切な命名

概要 レビュー指摘「不適切な命名」について ※Javaの話です 内容 「不適切な命名」のコードは可読性が下がります。 項目 内容 不適切な理由 汎用的な名前 param,obj等広い意味を表す名前 用途を読み取れない 日本語のローマ字読み helloではなくkonnichiwa等…

Java | レビュー指摘例 | 2013/04/15更新

概要 実際にレビューで指摘した事項についてまとめる。 実務でどのような「好ましくない」コードが作成されるのか、 という情報を元にレビューの視点や規約化などに活かすことを目的とする。随時更新 Javaのレビュー指摘 変数名関連 ※頻度は指摘頻度です。 …

アート・オブ・コミュニティ | 風通しの良いコミュニケーション

パンくずリスト アート・オブ・コミュニティ 風通しの良いコミュニケーション 概要 風通しの良いコミュニケーションについて コミュニケーションの手段 コミュニケーションの分類 ・受信のためのコミュニケーション ・発信のためのコミュニケーション ・内部…

マジック分岐

概要 レビュー指摘「マジック分岐」について ※Javaの話です 内容 自分が独自に定義した数値に意味を持たせて 自分にしか分からない分岐を作成すること。 サンプルコード package review.ex3.magic_branch; public class MusicPlayer { enum MusicType { JPOP…