出会いは突然に いつものように眠い目を擦りながらコードレビューをしていると以下のようなコードに遭遇しました。 (※部分的に書き換えてあるので、実際には動作していないコードです) def fetch_companies(setting_id) setting = Setting.find_by(id: setti…

動的型付け言語(Ruby)でも関数の戻り値の型は可能な限り統一した方が良い

リモートワークのファシリってつらい 現在、弊社ではコロナ対策として、完全リモートワークに移行しており、よほどのことがない限りは在宅にて勤務をしております。開発業務がメインなのですが、スクラムマスターという立場上、ファシリテーターとして主体的…

ファシリテーターをまかせたら普通にみんなやってくれた

APIの仕様書が送られてきた 外部連携させて頂く企業様から、新規に追加されるAPIの仕様書を頂きました。ユーザーが登録しているカテゴリの総数と、その内訳を取得することが出来るAPIだそうで、新たに追加されたとのことです。しかしながら、蓋を開けてみる…

【API設計】jsonを返す時はできる限りフィールドのkeyを統一してあげよう

プロジェクトの肥大化 自分が参加しているプロジェクトが早いもので、スタートから2年が経ちました。 ありがたいことにお客様の数は増え続けており、今でも多くのお客様に使って頂いております。初期の頃と比べると、かなり機能がリッチになりました。 元々…

開発者が複雑だと思う機能はユーザーには使いこなせない

コードはどんどん汚くなっていく プロジェクトのスタート時にどれだけ入念に設計をしてコードを書き始めたとしても、プロジェクトの年数が経つにつれて、コードの状態は悪くなってきます。 新機能追加 既存仕様の変更 メンバーの入れ替わり リリース日が変更…

ボーイスカウト・ルールが保守開発に役立った話

コードレビューはじめました 昨年の10月から勤務先企業で開発チームのリーダーになりました。元々はコードをガリガリ書くような業務をしていたのですが、ポジションが変わったため、業務の内容も変化しました。大きく変化した点としてコードをレビューしても…

Rubyのコードレビューでよく指摘すること

とりあえず書き込まれるTODO ある日のことです。新機能開発のためにコードを眺めていると該当箇所に対応するテストコードが全くないことに気づきました。 「TDDでやってるのになぁ」と思っていると、テストコードのファイルだけが作られていて、中にTODOコメ…

TODOコメントってあんまり意味がないんじゃないか説