プログラマーが知るべき「良いコメント」の条件【第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
-
製品資料
[株式会社MatrixFlow] 「物流リソース最適化」ガイド:人員・配車・傭車を出庫依頼の確定前に決めきる -
製品資料
[株式会社キーエンス] なぜRPA導入は頓挫する? シナリオ作成の壁を乗り越える解決策とは -
製品資料
[株式会社セールスフォース・ジャパン] 「CRMは設計と無関係」は本当か? PLMとの融合で実現する高速開発 -
事例
[日本ヒューレット・パッカード合同会社] AIエージェントの時代にどう備える? 「新たな働き手」を支える3要素とは -
製品資料
[日本ヒューレット・パッカード合同会社] “横並びの自動化”から脱却、AI活用で生産性と競争力を高める秘訣
こんなメディアも見られています
TechTargetジャパンに関連する情報をお探しであれば、こちらのメディアもお役に立てるかもしれません。
ベンダーコンテンツ PR
From Informa TechTarget
SpecialPR
アクセスランキング
-
1
なぜ「全社配布Copilot」は使われないのか? 失敗に学ぶAI定着
-
2
法務と開発者で「言葉が通じない」問題 トヨタやソニーが語るOSS管理の真実
-
3
なぜ「Gemini 4 Argon」は出遅れたのか? Googleが狙う“逆転のシナリオ”
-
4
損保ジャパンはなぜ「COBOL」を捨てなかったのか? 脱メインフレームの真相
-
5
情シスの約8割が転職や退職を意識 調査で分かった“辞めたくなる最大の理由”
-
6
ChatGPTは“検索しまくり”でGeminiは“淡泊”? データが明かすAIの裏側
-
7
情報漏えいはなぜ繰り返されるのか 今すぐ見直すべき「境界」
-
8
「Wi-Fi 7」経由でWindowsが乗っ取られる? 最高権限奪取の恐怖
-
9
「結局使わなくなる」Microsoft 365 Copilotを半年で定着 キリンの3施策
-
10
「中堅・中小企業のネットワーク・セキュリティ運用実態」に関するアンケート
ホワイトペーパーランキング PR
-
1
不審メールの経路や見せ方に変化? 2026年夏の3事例から見えた動向と対処方法
-
2
家庭用Wi-Fiルーターの業務利用は危険? 避けるべき理由と具体的な対策
-
3
Microsoft 365を安全に運用 うっかりミスやサイバー攻撃に備えるデータ保護術
-
4
財務部門がAIを最大限に活用する方法 無駄のない戦略的リーダーシップへの道
-
5
LLMが兵器化? 元FBI高官が鳴らす警鐘とセキュリティツール統合のポイント
-
6
「オンプレミス回帰」せざるを得ない“合理的な理由”
-
7
なぜRPA導入は頓挫する? シナリオ作成の壁を乗り越える解決策とは
-
8
生成AIを開発に導入しても効果が見えない? 実証実験で分かった成果と課題
-
9
経産省DX指針から読み解く、受発注業務デジタル化ロードマップ
-
10
HDDを使わない「SSDオンリー」が無謀なのはなぜ?
TechTargetジャパン SNS
インフォメーション
注目情報をチェック
TechTargetジャパンをフォロー