すべてのカテゴリ

ChatGPT、Claude Code、Cursorでコードは作れます。でも「動いた」と「公開して大丈夫」は、まったく別です。画面が開いた。ボタンも押せた。だから本番へ出す。ここを一気に進むと、利用者が増えた瞬間に問題が出ます。私も生成AIを使って実装やコードレビューを進めますが、最後に見るのはコードのきれいさだけではありません。データ、権限、同時利用、再実行、ログ、画面操作、更新後の確認まで見ます。生成されたコードが悪いという話ではないんですよね。確認せず公開する流れが危ない、という話です。1人の動作確認では見えない、本番の条件ローカル環境で1人が使うと正常でも、サーバーで複数人が使うと状況は変わります。実際、結果画面を開いている間、同じAPIへ何度もアクセスしているように見えるケースがありました。1回の処理は軽くても、複数人が同時に使えば呼び出しは積み上がります。ほかにも、ブラウザ操作のE2Eテストでは、対象が見えているのにクリックがタイムアウトすることがあります。画面上の重なり、読み込み中の状態、対象要素の揺れ。コードだけ眺めても分からない問題です。はい。「自分のPCでは動きました」は、公開許可証ではありません。公開前に残す7つのチェック1つ目は、入力値。空欄、長すぎる文字、想定外の記号、0件、最大件数を試します。正常なサンプル1件だけで通すと、現場のデータに負けます。2つ目は、権限。ログインしていない人が画面やAPIへ直接アクセスできないか。一般ユーザーが管理機能を実行できないか。画面上でボタンを隠すだけでは足りません。3つ目は、機密情報。APIキー、パスワード、接続先

最近はChatGPTなどの生成AIに、「Pythonで〇〇をするプログラムを作って」とお願いすると、数秒でコードを作ってもらえるようになりました。プログラミング初心者の方でも、以前より簡単にアプリや自動化ツールを作れる便利な時代になったと思います。一方で、実際に使おうとすると、次のような問題もよく起こります。・コードを実行してもエラーになる・一応動いたけれど、正しい処理なのか分からない・処理に時間がかかりすぎる・途中から、どこを修正すればよいのか分からなくなる・セキュリティ上の問題がないか不安・機能を追加したら、それまで動いていた部分が動かなくなったAIが作成したコードは、とても便利です。しかし、表示されたコードが必ずしも、そのまま安全に使えるとは限りません。今回は、現役SEである私が、AI生成のPythonコードを確認するときに見ているポイントをご紹介します。1．まずは「何をしたいコードなのか」を整理するコードを確認する前に大切なのは、目的をはっきりさせることです。例えば、・Excelのデータを自動で集計したい・Webサイトから情報を取得したい・CSVファイルをデータベースに登録したい・簡単なWebアプリを作りたいといった目的があります。AIに何度も追加の指示を出していると、最初の目的とは違うコードになってしまうことがあります。そのため、コードの修正を始める前に、「このプログラムで最終的に何を実現したいのか」を整理することが重要です。2．エラーが出ている場所だけを見ないエラーメッセージが表示されると、その行だけを直そうとしてしまいがちです。しかし、実際には別の場所に原因がある

コードレビューというと、「細かい指摘をされる場」「ダメ出しされる時間」というイメージを持つ人も多いかもしれません。ですが実際の現場では、レビューはバグ探しやマウントの場ではなく、チーム全体で品質を上げるための作業です。では、エンジニアはコードレビューの時に、具体的にどんな点を見ているのでしょうか。1. 正しく動くか（仕様を満たしているか）一番大前提になるのがここです。要件通りの挙動になっているか仕様の勘違いや実装漏れがないかエッジケース（想定外の入力）に弱くないかどんなに綺麗なコードでも、仕様を満たしていなければ意味がありません。レビューでは「このコードは正しいか？」という視点がまず最初に入ります。2. 可読性（他人が読めるか）レビューでとても重視されるのが可読性です。変数名・関数名が役割を表しているか処理の流れが追いやすいか「なぜこの処理をしているか」が読み取れるかコードは「自分のため」ではなく「未来の誰か（他人や将来の自分）のため」に書くものです。一瞬で意図が伝わらないコードは、レビューで指摘されやすくなります。3. 設計として無理がないか次に見られるのが、設計の妥当性です。責務が適切に分離されているか1つの関数やコンポーネントが肥大化していないか将来の変更に弱くない構造になっていないか「今は動くけど、あとで地獄になるコード」になっていないか、という視点で見られていることが多いです。4. 同じことを何度も書いていないかレビューでは「重複」もよく見られます。似た処理を複数箇所に書いていないか共通化できる部分が放置されていないか重複が多いと、修正時にバグが生まれやすくなります。

「ChatGPTにコードレビューを頼めば、人間のエンジニアは要らない？」最近、そんな声をよく聞きます。確かに、ChatGPTはLaravelやPHPのコードを解析し、改善案を提示するのが得意です。しかし、ChatGPTだけに頼ると見落としが発生することもあります。今回は、ChatGPTを使ったLaravelコードレビューのコツと注意点を現役エンジニアの視点で解説します。1. ChatGPTをコードレビューに活用するメリット・可読性の低いコードを改善案付きでリファクタリングしてくれる・ベストプラクティスや一般的なセキュリティ対策（XSS、CSRFなど）を教えてくれる・簡単なサンプルコードやテストコードの生成が得意2. ChatGPTでレビューする時のコツ・コードの一部ではなく、できるだけ全体のコンテキストを渡す・「このコードのセキュリティ面の改善点は？」など、具体的な質問をする・修正案をもらったら、すぐ本番に反映せず必ずテスト環境で動作確認する3. ChatGPTの限界・システム全体の設計や依存関係までは理解できないことが多い・複雑なバグ（DB設計やLaravel特有の動き）に対しては不完全な回答になることがある・「このコードは安全？」と聞いても、実行環境によるエラーは見抜けない4. AI＋人間レビューのハイブリッドが最強ChatGPTはヒントをくれる優秀なアシスタントですが、最終的な安全性や保守性を保証するのは人間のエンジニアのレビューです。特にLaravelはバージョン差異やパッケージの依存関係が複雑なため、プロの目が必要なことも多いです。まとめChatGPTはコードレビュー

コードレビューとは？コードレビューとは、他の開発者が書いたコードを確認し、品質やバグ、改善の余地がある箇所を指摘するプロセスです。これはソフトウェア開発の一部として広く行われており、チーム全体のスキル向上やプロジェクトの品質を向上させるための重要なステップです。コードレビューの重要性1.バグの早期発見コードを書く際には、人間のミスが避けられません。コードレビューによって、他の開発者が自分の書いたコードをチェックし、潜在的なバグを早い段階で見つけることができます。バグが早期に発見されれば、その修正コストも低く済むため、プロジェクト全体の効率が向上します。2.コードの品質向上コードレビューを行うことで、コードが標準的なコーディング規約に沿っているか、パフォーマンスが最適化されているか、冗長な部分がないかなどをチェックできます。これにより、長期的なプロジェクトのメンテナンス性が向上し、将来的なバグ発生のリスクも低減します。3.知識の共有とスキル向上コードレビューは単にバグを見つけるだけではなく、チーム内での知識共有の場にもなります。他の開発者が書いたコードを読むことで、新しい技術やベストプラクティスを学ぶことができ、自分のスキルアップにも繋がります。また、レビューを受けることで、自分の書いたコードの改善点を知ることができるため、フィードバックを元に成長する機会にもなります。4.チームの一体感を育むコードレビューを定期的に行うことで、チーム全体のコードに対する理解が深まります。全員がプロジェクトのコードベースに関わることで、責任感が生まれ、チームとしての一体感も育まれます。実践的なフィ

こんにちは、桜井隆二です。フリーランスエンジニアとして複数のプロジェクトに関わっていると、日常的に「コードレビュー」をする機会があります。他の人が書いたコードを確認し、改善点を指摘したり、良い点を共有したりする作業です。一見すると技術的なやりとりに思えるこの作業ですが、実は「人の優しさ」が垣間見える瞬間がたくさんあるんです。丁寧なコメントに込められた配慮先日、あるジュニアエンジニアの方が書いたコードをレビューしました。ロジックは正しいものの、もう少し効率的な書き方がある箇所がありました。指摘しようとPRコメントを書き始めたとき、ふと「この人はどんな気持ちでこのコードを書いたんだろう」と考えました。きっと、調べながら、試行錯誤しながら、一生懸命書いたはずです。だから僕は、こう書きました。「このロジック、しっかり動いていて素晴らしいです。もし今後パフォーマンスを意識するなら、こういう書き方もありますよ」技術的に正しいだけでなく、相手の努力を尊重する。それがコードレビューにおける「優しさ」だと思っています。 受け取る側の優しさもある逆に、自分が書いたコードにレビューをもらうこともあります。「ここ、こう書いた方が読みやすいかもしれません」そんなコメントをもらったとき、素直に「ありがとうございます、修正します」と返せるかどうか。それもまた、優しさだと思うんです。指摘を攻撃と受け取らず、成長のチャンスとして受け入れる。そういう姿勢がチーム全体の雰囲気を良くします。コードの向こう側には、人がいる結局のところ、コードレビューは「人と人とのコミュニケーション」なんですよね。技術と真摯に向き合うこ