Go初学者へのコードレビューでよくあったコメント20選

2024/11/14 235 users

はじめに こんにちは、ソーシャルベッティング事業本部 海外ベッティング事業部の山崎です。 本記事では、Effective GoやGoogle のスタイルガイド、Code Review Commentsといった公式資料、Future Architectの記事などを参考に、Go を初めて触る開発者を対象にした汎用的なレビューコメントの 20 選を紹介します。 大き... 続きを読む

コトに向かうコードレビュー | Marginalia

2024/11/07 10 users Marginalia narumi コト ひと 親和性

他人や自分など「ヒト」ではなく「コト」に向かうコードレビューについて考えてみる。 参考:「コトに向かう」について => DeNA南場智子さんの講演「ことに向かう力」がいい話だった｜narumi （本来の「コトに向かう」話とそれほど親和性のある話ではない気はしつつも、響きがいいので借用しています） このコードは良く... 続きを読む

ChatGPTがコーディング支援を強化。コードエディタ機能を統合、コードレビューやデバッグ、他言語への移植ツールなどを含む新機能「Canvas」が登場

2024/10/07 18 users OpenAI chatgpt canvas デバッグ 統合

ChatGPTがコーディング支援を強化。コードエディタ機能を統合、コードレビューやデバッグ、他言語への移植ツールなどを含む新機能「Canvas」が登場 OpenAIはChatGPTの新機能「Canvas」を発表しました。 ChatGPTはさまざまな指示や依頼にAIが対応する汎用目的で使えるサービスですが、新機能「Canvas」は特に文章の作成と... 続きを読む

コードレビュー観点表を作った話

2024/08/07 315 users コーディングガイドライン GitHub ガイドライン 事項

はじめに 今回は、コードレビュー観点表を作った話について少し書かせていただきます。 社内ではGitHubを用いてコードレビューを行っていて、バックエンドの開発においては、コーディングガイドラインも策定しています。 しかし開発において、ガイドラインに書かれている事項が全てではないため、コードレビューを行う際... 続きを読む

大吉祥寺.pmに参加し、登壇し、DJをやった #kichijojipm - 私が歌川です

2024/07/15 11 users 知見 帰還 吉祥寺 kichijojipm 観点

大吉祥寺.pmに参加しました。この記事の公開をもって吉祥寺からの帰還といたします。 登壇した speakerdeck.com タイトルは「君たちはどう生きるか」をモジりました。 コードレビューの目的は「コードの品質を保つこと」「知見を共有すること」である、という観点を踏まえ、自分がどのようにしてコードレビューに向き合... 続きを読む

TypeScriptの型と値とバリデーション

2024/06/11 144 users TypeScript バリデーション バリデーシ 何度 影響

TypeScript は本質的に自分に型が付与されていると思っているだけの JavaScript です。 いくら型を付与しようが、それが実行時に影響を与えることはありません。 コードレビューをしているとここを誤解している人が本当に多いです。何度も解説しているのですが、なかなか浸透しないので、TypeScript におけるバリデーシ... 続きを読む

「誰にもできる」より良いコードを書くためのテクニック - Qiita

2024/04/29 17 users Qiita リーダブルコード テクニック Webエンジニア

はじめに こんにちは、愛知でWEBエンジニアをしているkokuboと申します。 ここ最近の業務でコードレビューをすることが多くなり、人によって読みやすさの違いがあることや過去に自分で書いたコードが読みにくかったことから、良いコードとは何かを調べて見ました。 今回は「リーダブルコード」を元に良いコードとは何か... 続きを読む

sed awk make 互換性 つらい うんざりザリガニ - Lambdaカクテル

2024/04/19 109 users ザリガニ Lambdaカクテル sed awk 互換性

仕事でsedのコードをパパッと書いて使ったら、コードレビューで動かんと言われてしまった。そういえば、同僚はMacで、おれはLinuxだったな。 まぁsedならよくある話なので、しょうがないと思ってawkで同じことをした。awkは比較的もうちょっと互換性がある気がしているからだ(どっこいどっこいな気もする)。 しかしまだ... 続きを読む

Reactを使ってプロダクト開発している開発者だけでなく、マネージャにも読んでほしい「Fluent React」 - ROUTE06 Tech Blog

2024/03/25 250 users React マネージャ フレームワーク ライブラリ プロダクト

チームでReactを使って開発していると、コードレビューをする際に、「この書き方はしない方がいいが、それを説明するには800文字くらい必要。図も描きたい。でもそれらを準備する時間はない。」ということが度々ありました。 また、フレームワークやライブラリの技術選定をする際、マネージャに「どうして技術選定が必要... 続きを読む

単なる品質チェックではなく、技術的な思想も共有・議論する　ソニックガーデンの“妥協しないコードレビュー”

2024/02/29 20 users ソニックガーデン 思想 田中 田中氏 共有

単なる品質チェックではなく、技術的な思想も共有・議論する ソニックガーデンの“妥協しないコードレビュー” 妥協できないソニックガーデンのコードレビュー 田中氏の自己紹介 田中義人氏：こんばんは。ソニックガーデンの田中です。ここでは「妥協できないソニックガーデンのコードレビュー」というお題で発表していき... 続きを読む

良いソフトウェアとコードレビュー / Good software and code review

2024/01/30 274 users 良いソフトウェア

Scala + Caliban で作るGraphQL バックエンド / Making GraphQL Backend with Scala + Caliban 続きを読む

めんどくさい作業を改善できるようになるには - Konifar's ZATSU

2023/12/21 215 users Konifar's ZATSU めんどくさい作業 lint

めんどくさい作業にぶち当たった時、一気に改善してしまう人がいる。ガッと自動化したり仕組みそのものを変えたりしてしまうのだ。「めんどくさい」と心の中で思ったなら、その時スデに行動は終わっているのである。 たとえばコードレビューで都度同じ指摘をしだしたらLintとCIを整備したり、期限のリマインドを何度もし... 続きを読む

チームで1年間コードレビューを最優先に実施したら開発生産性に良い影響を与えてくれたかも

2023/12/11 97 users 最優先 チーム 開発生産性 良い影響 soda

＼スニダンを開発しているSODA inc.の Advent Calendar 10日目の記事です!!!／ こんにちは！！！！SODA開発部の矢野です！！！ はじめに 私のチームでは一年前からコードレビューを最優先に実施するという取り組みをしています。この取り組みを継続した結果開発生産性にも良い影響を与えてくれたかもしれないので記事に... 続きを読む

コードレビューの思想や心構え - Qiita

2023/12/03 30 users Qiita ASANO chatgpt 心構え 入力プロンプト

株式会社ブレインパッドでデータサイエンティストをしているasanoです。 この記事はBrainPad Advent Calender 2023 1日目の記事シリーズ2です。 ※シリーズ1は@fuyu_quantさんの入力プロンプトを復元する技術 #ChatGPTです！ 今日はコードレビューの思想や心構えについて書きます。 はじめに コードレビューをより生産的... 続きを読む

その例外、いつキャッチするの？

2023/11/03 89 users 例外 Java tl;dr イッ 若手

はじめに 最近、若手のコードレビューをしていて例外の使い方を教える機会があったので、ブログの方にもまとめたいと思います。今回はバッチ編。オンラインだとまた少し違う観点があると思います。また、言語はJavaを前提していますが考え方は例外機構をもつ言語ならあまり変わりません。 TL;DR 例外は原則キャッチしな... 続きを読む

長くなりがちだったコードレビューを改善した話 - 弁護士ドットコム株式会社 Creators’ blog

2023/09/08 152 users Creators’ blog 拡充 タスク 施策 エンジニア

弁護士ドットコム クラウドサイン事業本部でエンジニアをしている山田です。 主にフロントエンドを担当しています。 普段の業務でフロントエンド開発のコードレビューをすることが多く、今回は長い時間がかかりがちだったコードレビューを以下の施策で改善した話をします。 タスクへの認識合わせを拡充 タスクを小さく分... 続きを読む

クソコードを見る度に怖くなる

2023/07/04 19 users クソコード マジ コード レビュー 完成

業務で結構な量のコードレビューを毎日してるんだけど 最近マジでクソコードが多い 適当に書き殴ったコードなんじゃなくて とにかく思い付いたところからコーディングして 実際に動作させたら思い通りに行かないから継ぎ接ぎで修正して 最終的に機能を満たしたから完成、PR作成、レビューよろしく、みたいなのが本当に多... 続きを読む

エンジニアが開発しやすい環境作りをする

2023/04/19 192 users エンジニア 環境作り 直近 ディレクトリ設計 渋谷

はじめに 自分は渋谷のWeb系開発会社にて執行役員兼エンジニアをやっています。(新卒入社3年目) 直近では6~8名程のエンジニアがいるプロジェクトで、ディレクトリ設計やissue作成、コードレビュー、スケジュール管理、PMへのUI/UX及び機能提案などを行なっています。 その中で自分が「エンジニアチームにとって開発しや... 続きを読む

リーダブルコードの要点整理と活用例まとめ

2023/04/02 141 users リーダブルコード 要点 要点整理 実務 把握

はじめに 最近コードレビューの機会が増えてきたので、「リーダブルコード」を読み直しました。 リーダブルコードを読んでいく中で要点を整理し、実務の現場でコードを書いたりレビューをする際にどのように活用していくべきかを自分なりにまとめてみました。 この記事を読むことで、リーダブルコードの要点の把握と実際... 続きを読む

コードレビューにラベルを付けるだけでチームの心理的安全性を高めた話

2023/03/06 708 users ラベル チーム 齟齬 安全性 必須

ラベルを付けることにより、このような利点があります。 すべてのコメントに対して修正が必須でないので、対等な関係性となる するべきアクションが明確に伝わるため、認識の齟齬が起こらない コメントに重み付けがされるので、優先順位が明確になる 後でやるべきタスクを、タスク化し忘れない コメントの敷居が下がるこ... 続きを読む

コードレビュー時のコメントの意図を明確にする

2023/02/15 29 users 意図 疑念 レビュイー コードレビュー時 コメント

コードレビューあるあるなんですけど、レビューする時に「これってなんでこうしてるんですか？」みたいなピュアなコメントを書いてしまうと、その意図がレビュイーに伝わらなくて、「詰められてる」「コードに疑念を持たれている」「修正依頼をされている」「純粋に質問されている」という解釈のブレを生んでしまって物... 続きを読む

コードレビューの「純粋に質問ですが」は「勝手に指摘だと受けとってよくわからん修正するんだろ？質問なんだよ！修正すんなよ！」という意思表示な説

2023/02/14 188 users 指摘 修正 意思表示 殺伐 質問

やきにくのすえなみ @a_suenami コードレビューの「純粋に質問ですが」は表現を柔らかさ目的でなく、「お前は質問だって言わないと勝手に指摘だと受けとってよくわからん修正するんだろ？質問なんだよ！！修正すんなよ！」という意思表示で、むしろ元より殺伐となってる可能性すらあります。 2023-02-13 16:43:45 続きを読む

VSCodeにChatGPTの拡張機能を入れてコードレビューやバグを発見してもらう - Qiita

2023/01/15 394 users Qiita vscode OpenAI chatgpt パク

ChatGPTとは？ OpenAIが開発するGPT-3という言語モデルをベースとしたチャットアプリです。 こちらの質問に対して、AIが色々な質問に答えてくれて、一般的な内容だけではなく、コードレビューやバグなども発見してくれるめっちゃ凄いやつです。 細かい内容は以下の記事がとても参考となります。 VSCodeと連携して、ブラ... 続きを読む

コードレビューの生産性を上げるためのTips | Offers Tech Blog

2023/01/10 154 users Tips Offers Tech Blog 生産性 藤井 文化

はじめに こんにちは。 プロダクト開発人材の副業転職プラットフォーム Offers を運営する株式会社 overflow のエンジニアの藤井です。 エンジニアであれば誰しも日頃からコードレビューをしたり、されたりしていることと思います。 健全な開発組織を育む意味でもレビューの文化を根付かせることはとても大切ですが、小... 続きを読む

コードレビューをAIに手伝ってもらい楽をしてみる - hiroppy's site

2022/12/15 16 users hiroppy's site Issue REVIEW 依頼

この機能の登場により、PR でのレビューのオーバヘッドを少なくすることが期待されます。この PR では何を変更したのかを説明したり、更には review の依頼を投げることもできます。 また、Issue でも AI にどうしたらよいか？を聞くこともできるそうです。詳しくは公式の動画を見てください。 How many times have you ... 続きを読む