コードレビュー

フロントエンドエンジニアをしている瀧です。 フロントエンドとは、ユーザーが直接目にする画面や動きの部分のことで、ユーザーインタフェースや、ユーザー体験を開発しています。 この記事では私のチームで行っているコードレビューについて紹介します。 瀧 章倫（タキ アキノリ） シナジーマーケティング2005年入社の古参のメンバーです。 現在はSynergy!のフロントエンドの開発に携わっています。 はじめに コードレビューとは開発工程の１つで、自分が書いたコードを見てもらったり他人が書いたコードを見たりする仕事のことです。 レビューと聞くと印象的には評価・評論・批評といった感じですよね。粗探しをされてい…

2021年11月に株式会社クラウドワークスへ入社した小西です。エンジニアとしてクラウドテックというハイクラスなエンジニア・デザイナーを中心とした人材と企業をマッチングするエージェントサービスを開発しています。 クラウドワークスに入社するまでの経緯や入社後に始めたペアプロのことなどをお伝えしていこうと思います。 目次 目次 入社経緯 入社してどうだったか ペアプロの提案 ペアプロの良いところ 技術スキルや業務知識などのノウハウが共有できる コードレビューの前にアドバイスが受けられる チームの醸成になる ペアプロで変えていく組織 クラウドワークスに興味のある方へ 入社経緯 入社までのエンジニア歴は…

どんな開発現場でもコードレビューは重要です。今の時代だとGithubのプルリクエストなどをベースとしたコードレビューは、開発の現場ではもはや必須ともいえる制度です。 ただ私はスタートアップでの立ち上げ期を経て、新規開発立ち上げ期においてはコードレビューの役割を変えたほうがいいということを感じました。この記事ではコードレビューの役割について考えてみます。 平常時におけるコードレビューの役割 まずは平常時におけるコードレビューの役割を見ていきましょう。 1. ダブルチェックになるのでバグが発見できる可能性がある バグがリリースされる可能性を減らす役割です。 2. 技術的負債になりうるコードを発見で…

ソフトウェアエンジニアが直接的に生み出すものはプログラムのコードそのものです。 しかし個人的な経験として成果物をコードそのものではなくプルリクエスト全体だと捉えることでぐっと仕事の質が良くなったので、その話について書いてみます。 ※ 注：以降”プルリクエスト”は長いので”PR”と略します。 ================== 「PRを成果物として捉える」というのはどういう意味かというと、「PRをなるべく丁寧に作ることを心がけ、他の人やあるいは未来の自分が見たときに背景や意図がわかるようにする」ことを指します。 実際PRのキレイさというトピックは実際世の中的にも重要だと認識されています。『プル…

サイト開発における属人的なリスクを減らし、品質確保や生産性向上を図るために、dodaサイト開発チームへの「開発プロセス標準」導入が進められています。内製エンジニアの有志メンバーから始まったこの取り組みは、体制づくり、現場浸透のための取り組みへと、歩みを止めずに前進し続けてきました。 組織やサービスに合う仕組みを作るために、そして作って終わりではなく現場に浸透させるために、どのような思いが込められているのか－－取り組みを主導する坂井と、ガイドライン策定に携わった古沢に、取り組みの裏側と思いを聞きました。 有志の取り組みから、開発ルールやプロダクトの技術方針を扱う新体制へ 皆の手で長期的に運用して…

コードレビューのコメントに記述されることがある「nit」ですが、これは「nitpicking（つまらぬあら探し）」の略語です。 コメントに「nit」が付与されている場合、「細かいところを指摘するようだけど ... あなたの判断で、これについては必ずしもコードを修正する必要はないですよ」という意味になります。 コードレビューの不要な混乱を避けるためには、定義を正しくしておくことは大切だと思います。日本人には謙虚な心がありますが、控えめな気持ちを表現するために「nit」を書き込むのは止めたほうが良いでしょう。 コードレビューにかかる開発者それぞれの負担を、可能な限り小さくすることは大切で、そのため…

なぜこの記事を書くか 実装するとき コードレビューするとき コードレビューしてもらうとき なぜこの記事を書くか 検索すれば、実装時の注意、コードレビューの時の注意点は書いてあると思うので、これは完全に自分用のメモ 特に失敗してしまったこと自分の言葉で書いて、次に同じことが起きないようにする。 粒度大きめ＞粒度小さめ の順番で書いていく 実装するとき 仕様がある場合は仕様を把握する 仕様で漏れがないか、わからない点がある場合、確認事項をリストアップし早めに確認しておく 仕様がない場合は、出来上がりのイメージを確認したり、仕様を作っていく 既存のコードを使う場合、既存コードが何を担保しているか、把…

ソフトウェア開発をしていると、他の人のコードや設計のレビューをすることが頻繁にある。これまでコードレビューや設計レビューというと、他の人の判断にケチをつけるようで心理的負担があった。最近は考え方が変わって、レビューに対する心理的負担は減っている。 少し前に『NOISE 上: 組織はなぜ判断を誤るのか?』という本を読んだ。この本では人間の判断には、専門家であってもばらつきがあるということが言われていた。それがノイズと本の中で言われているものだ。アメリカの裁判で、判事によって量刑のばらつきがあること、誕生日だったり、気温が暑かったり、そう言ったものでも判断の傾向が変わることなどが書かれていた。 ソ…

更新日：2022/8/8 エンジニアを目指していますが、sesとして働くのはイヤだなぁなので自社開発に行きたいのだけど、自社開発企業へ就職できるプログラミングスクールはあるの？ 今回は、上記の疑問に答える記事になります。 この記事を書いている僕は、 ✅現在エンジニアで、約6年以上勤務 ✅自社開発もやってましたが、ses経験はかなり豊富です ✅現場でも仕事しながら、プログラミング経験ない人を教えています ✅通学型プログラミングスクールで学習してました 上記の経験があります。 また、当ブログではプログラミングスクールの体験談や、プログラミング（スクールの紹介も含む）に関する記事を90記事ほど書いて…

去年の8月1日から今の会社に中途採用されたのでちょうど1年くらいたった。 3ヶ月の研修を経て今の配属先に着いて、3月くらいから本格的に案件をやるようになったからプログラマー歴で言ったら6ヶ月と言う感じなのか。 言語はJavaとJavaのフレームワークの Struts2、SQL、たまにHTML(jsp)、javascriptを書いてる。正直よく分かってなくてノリでやってる感が強い。というか同僚の人にだいぶ助けられてるなぁ。 案件でやる事は主にエンハンスなので、既存のコードをコピペしてエラー対応の部分とかを手直ししている。 やってて辛いのはコードレビューで指摘がキツいこと。理詰めでめちゃくちゃに言…

プロダクト開発部の小堀です。 コドモンには0.5投資という学習制度があり、業務時間を使って勉強会を実施したりセミナーに参加したりできます。 そこで私は、有志のエンジニア10名弱といっしょに『良いコード／悪いコードで学ぶ設計入門―保守しやすい 成長し続けるコードの書き方』(以下『良いコード』)の輪読会を実施しています。 gihyo.jp この輪読会がとても楽しくやれているので、本記事では輪読会の内容や工夫した点を紹介します！ 背景 輪読会の進め方 現在 まとめ 背景 以前から「良いコード」をテーマに、開発者たちと話せる場をつくりたいと思っていました。 というのは「良いコード」を書くためには、誰か…

こんにちは。estieの青木啓剛(Keigo)です。社内いくつかのチームのエンジニアリングマネージャーをやりつつ、コーポレートITなども担当しています。先日同僚のエンジニアリングマネージャーから「1日密着したい！」といわれる機会があったのでブログにしてみることにしました。 どんな仕事も同じだと思うのですが、特に規模の小さい会社では肩書が同じでもやっている仕事はぜんぜん違う、みたいなことが往々にしてあると思います。estieのエンジニアリングマネージャーはこんな1日を送っていますよ、という雰囲気が伝われば幸いです。 estieのエンジニアリングマネージャー 私たちestieではプロダクト開発の最…

先日、「リーダブルなテストコードについて考えよう～VeriServe Test Automation Talk No.3～」というイベントで登壇してきました。 veriserve-event.connpass.com 今回は発表内容に対する補足と、発表に対していただいた質問に回答します。気になるところだけでも読んでもらえればと思います。 目次 目次 発表内容 発表に対する補足 【補足1】都道府県のテストについて 【補足2】Parameterized Testsへの利用について いただいた質問の回答 【質問1】リーダブルなテストコードの勉強方法はありますか？ 【質問2】テストコードのメンテナンス…

簡単に結論 開発効率を下げる1番の要因は手戻り 実装に入る前に実装方針を検討することで手戻りを少なくできる 記事の背景 dev.to 先日こんな記事を読んで、自分も気をつけていることをまとめようと思ったのがきっかけです。 困っていたこと DeNAに新卒として入社したときに研修の講師から「すぐにコードを書くな」とよく言われました。 どんなクラス設計にするのか 要件に見落としているポイントはないか を深く検討した上でコードを書き始めないとダメだという意味でした。 実際に僕も新卒入りたての頃はとにかく手を動かし始めるというアンチパターンを踏んでしまっていました。 最も開発効率を下げる要因とは何か 開…

こんにちは。エンジニアの加藤です。LIFULL HOME'Sの注文住宅領域を支えるエンジニアチームのマネジメントを担当しています。 弊社では昨年度よりKPIマネジメントを導入し、さまざまな組織課題の解決や目標達成に向け活用しています。 私がマネジメントするエンジニアチームでも同様に開発生産性向上を目標とし、KPIマネジメントの手法を用いて実現への取り組みを実施してきました。 そのような中、今回はこの半期においての私のチームでの取り組みについて紹介したいと思います。 KPIマネジメントの概要や今までの取り組みについては以前こちらのブログでも投稿いたしましたので、よければ併せてご覧ください。 ww…

はじめに STORES 予約 でモバイルアプリの開発を行なっているnekoと申します。 私は2022年の3月にheyに入社し、まもなく5ヶ月が経過しました。 STORES 予約 のモバイルアプリチームは、私が入社した時点ではエンジニアが1人しらおらず、私が入社して2人になりました。 さらにその時点で、翌月にはもう1人の入社が決まっており、当面はその3人で開発を進める体制に移行するというタイミングでした。 その中で、入社当初から今まで、メインの開発タスクと並行し、プロジェクト構成の変更や、CI/CD周りのフロー構築など、複数人でのチーム開発に移行する準備を進めてきました。 今回はその中で主だった…

タコス、amaneko、くる の3人でチーム「タコタコ」で参加。 1年前 3人でisucon予選出たいなと言いつつ誰も予選登録できず終了。参加が難しい。 今年 6月11日 今年も、3人でisucon予選出たいなと言いつつ誰も予選登録できず終了......かと思われたがタコス君が登録できてた。えらい。僕は寝てました。何もしないまま参加登録完了！ 持つべきものは早起きできるエンジニアだね。うんうん。 6月16日 タコス太郎が招待urlとかを貼ってくれる。チームメンバーとして認証するための設定とかがあるらしい。 僕はLINEを無視していた。 6月18日 無視してたら怒られた。タコス君からクビ宣言をく…

連載目次記事 今回の記事はあまり望ましい実装方法ではありません。 望ましい実装方法では データ バインディング を使用します、 今回望ましくない方法で書くのは、この部分が本連載の主題ではないからです。データ バインディング について説明するとそれだけで一つの連載記事になってしまうため、今回は使わない事としました。 もし皆さんが仕事でコードを書く場合は、データ バインディング を使っていない場合間違いなくコードレビューでリジェクトをもらうことになるので、注意してください。 ■ 前提記事 画面の UI は XAML という XML で書きました。 rksoftware.hatenablog.com…

veriserve-event.connpass.com こちらのイベントに参加してきたので、会の様子と感想を書いていこうと思います。 会の概要 会の様子 リーダブルテストコード〜伊藤淳一さん〜 コンテキストとセマンティクスを意識して リーダブルなE2Eテストコードを書こう〜末村拓也さん〜 テストコードにはテストの意図を込めよう〜ブロッコリーさん〜 質疑応答&パネルトーク パラメタライズテストの可読性を上げたい すでに書いたテストを捨てる基準は？ 定数はプロダクションコードとテストコードで揃える？(同じ定数を参照する？) リーダブルなテストコードをどのように浸透させるか？(良いコードの共通理解…