プログラマーが知るべき「良いコメント」の条件【第2回】
Pythonのソースコードで考える「こんなコメントは無駄なだけで無意味」
「コメント」は、ソースコードを読むだけでは分かりづらい情報を補足するのに役立つ。ただし書き方によっては、コメントはほとんど有益な情報を生み出さなくなってしまう。それはどのようなコメントなのか。
プログラミングにおいて良いコメントをソースコード内に残すことは、ソースコードの記述や修正の効率を向上させるために重要だ。良いコメントを書こうとするのであれば、まずは「悪いコメント」とはどのようなコメントなのかを理解するとよい。まずは「本来の役割を担えないコメント」とは何かを考えよう。
こんなコメントは「無駄なだけで無意味」
併せて読みたいお薦め記事
連載:プログラマーが知るべき「良いコメント」の条件
ソースコードの書き方
以下はプログラミング言語「Python」で実装した、Webアプリケーションのログイン機能の例だ。Webアプリケーションを実装するためのフレームワーク(特定の設計思想を具現化するプログラム部品やドキュメントの集合体)「Flask」を利用している。
@app.route('/login', methods=['POST'])
def login():
info = json.loads(request.data)
username = info.get('username')
password = info.get('password')
user = User.objects(name=username, password=password).first()
if user:
login_user(user)
return jsonify(user.to_json())
else:
return jsonify({"status": 401,
"reason": "Username or Password Error"})
上述のソースコードを簡単に説明すると、以下のようになる。
- 1行目
- Webアプリケーション内のログインURL(「/login」)でPOSTメソッド(サーバへのデータ送信要求)を実行した際に、2行目以降で定義する関数を呼び出すことを示す。POSTメソッドで受け取ったJSON形式のデータを変数「request.data」に格納する。
- 2行目
- 関数の名前を「login」と定義する。
- 3行目
- 受け取ったデータ(request.data)を変数「info」に代入する。
- 4行目
- info中にあるユーザー名(username)のデータを変数「username」に代入する。
- 5行目
- info中にあるパスワード(password)のデータを変数「password」に代入する。
- 6行目
- データベースから、usernameとpasswordが一致する最初のレコード(データの組)を参照し、該当するレコード(ユーザーデータ)を変数「user」に代入する。
- 8行目
- ユーザーが存在するかどうか(userにデータが代入されているかどうか)を確認する。
- 9行目
- (ユーザーが存在する場合)Flaskの拡張機能群「Flask-Login」の関数「login_user」でログイン処理をする。
- 10行目
- (ユーザーが存在する場合)userが保持するデータをJSON形式にしてから、HTTP通信用のデータに加工し、呼び出し元に渡す。
- 12、13行目
- (ユーザーが存在しない場合)JSON形式のエラーメッセージを、呼び出し元に渡す。
このソースコードの場合、ソースコードの処理の流れをそのままコメントとして書き写すことは悪手だ。ソースコードを読んだ人が関数やメソッド(データに対する処理)の意味を理解できるならば、それと同じ情報を伝えるコメントは意味がない。ソースコード内に存在する関数やメソッドの説明は、プログラミング言語の公式ドキュメントを参照してもらう方がより適切だ。
分かりやすくソースコードを説明するコメントがあったとしても、間違っていたり、古くなっていたりする場合がある。ソースコードに関わった人が同じようなコメントをどんどん書き足していくと、ソースコードが乱れて読みにくくなり、混乱や矛盾を招く恐れがある。
上述のソースコードにコメントを追加する場合、以下のようにするとよい。「#」で始まる行がコメントだ。
# 既存ユーザーであればログインを実行し、ユーザーが存在しなければ401エラーを返す
@app.route('/login', methods=['POST'])
def login():
info = json.loads(request.data)
username = info.get('username')
password = info.get('password')
# データベースから、入力されたユーザー名とパスワードに一致するユーザー情報を取得する
user = User.objects(name=username, password=password).first()
if user:
login_user(user)
return jsonify(user.to_json())
else:
return jsonify({"status": 401,
"reason": "Username or Password Error"})
次回は、良いコメントとはどのようなものかを考える。
TechTarget発 エンジニア虎の巻
米国TechTargetの豊富な記事の中から、開発のノウハウや技術知識など、ITエンジニアの問題解決に役立つ情報を厳選してお届けします。
Copyright © ITmedia, Inc. All Rights Reserved.
TechTarget発 エンジニア虎の巻
米国TechTargetの豊富な記事の中から、開発のノウハウや技術知識など、ITエンジニアの問題解決に役立つ情報を厳選してお届けします。
この記事の著者
関連記事
新着ホワイトペーパー PR
-
事例
[フォーティネットジャパン合同会社] 複雑化しがちな「SASE運用」 IT・医療・金融などの事例に見る導入成功の秘訣 -
製品資料
[フォーティネットジャパン合同会社] 「SD-WAN」導入で失敗しないための5つの重要事項 -
技術文書・技術解説
[フォーティネットジャパン合同会社] VPNの代替候補「ZTNA」を理解できるようになる“3大要素”とは? -
事例
[フォーティネットジャパン合同会社] すぐ分かる「SD-WAN」選定のポイント 成功企業はどんな成果を出した? -
製品資料
[SCSK株式会社] 「Dropbox」管理・運用の困りごと解決ガイド
こんなメディアも見られています
TechTargetジャパンに関連する情報をお探しであれば、こちらのメディアもお役に立てるかもしれません。
ベンダーコンテンツ PR
From Informa TechTarget
SpecialPR
アクセスランキング
-
1
半数がブルーカラー転向を視野に? ベテランエンジニアを追い込む“54歳の壁”
-
2
PHP脆弱性「PHP2Shell」で2260万件に影響 情シスの緊急チェックリスト
-
3
みずほ証券が「液冷サーバ」を採用 1日3.6億件の計算を支えるための要件
-
4
「Microsoft 365」が乗っ取られる MFAを破る“本物の画面”の恐怖
-
5
「Windows」の不要なファイルを安全に消して容量を空ける方法
-
6
Microsoft月例パッチ1400件超で過去最多 「限界点」を迎えたパッチ管理
-
7
損保ジャパンはなぜ「COBOL」を捨てなかったのか? 脱メインフレームの真相
-
8
RAGで「データをAIに送る」運用が限界を迎えている理由
-
9
SAP導入後も残る“Excel依存” 京急電鉄は「kintone」でどう解消した?
-
10
なぜ「Gemini 4 Argon」は出遅れたのか? Googleが狙う“逆転のシナリオ”
ホワイトペーパーランキング PR
-
1
AI駆動攻撃にどう対抗する? 超速の攻撃からエンドポイントを保護する方法
-
2
2026年のサイバーセキュリティには何が必要か? 最新データに学ぶ傾向と対策
-
3
経験豊富な即戦力人材に直接アプローチできる、「攻めの採用」への転換術
-
4
「IT史に名を残すシステム障害」8選
-
5
最速72分でデータが流出? 最新の脅威トレンドや侵入手法から分かった防衛策
-
6
AI駆動開発のスピードと安全性を両立 「品質・セキュリティ管理」の実践ガイド
-
7
不審メールの経路や見せ方に変化? 2026年夏の3事例から見えた動向と対処方法
-
8
HDDを使わない「SSDオンリー」が無謀なのはなぜ?
-
9
「標的型攻撃メール」事例・サンプル集
-
10
Microsoft 365のメールセキュリティ、3つの“死角”と追加で検討すべき対策とは
TechTargetジャパン SNS
インフォメーション
注目情報をチェック
TechTargetジャパンをフォロー