プログラマーが知るべき「良いコメント」の条件【第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
-
事例
[サイボウズ株式会社] DXに必要な「Dスキル」「Xスキル」を持った人材を育成するには? -
市場調査・トレンド
[サイボウズ株式会社] データで見る、DXが「順調に進む企業」と「つまずく企業」の違い -
製品資料
[サイボウズ株式会社] 賛否が割れがちな「Notesからの移行」 新環境への移行を納得してもらうには? -
事例
[ServiceNow Japan合同会社] 農林中金に学ぶ内製開発 処理効率を約2倍に高めAI活用も加速させた方法とは? -
事例
[ServiceNow Japan合同会社] NTTグループのデジタル変革術、17万人が利用する決裁プロセス刷新の全貌
こんなメディアも見られています
TechTargetジャパンに関連する情報をお探しであれば、こちらのメディアもお役に立てるかもしれません。
ベンダーコンテンツ PR
From Informa TechTarget
SpecialPR
アクセスランキング
-
1
【基本情報技術者試験】「デュプレックスシステム」と「デュアルシステム」の違いは?
-
2
100億円の「Linux更新」を回避 みずほ銀行が選んだ“おきて破り”のRHEL延命策
-
3
MS月例パッチが1000件突破 人手不足の情シスを襲う「月1回メンテ」の崩壊
-
4
サーバ約70台をAWSへ ヤナセが移行前にやった「通信要件の可視化」
-
5
AIインフラの理想形? 「5層のケーキ」を垂直統合するための近道とは
-
6
「VMware離れ」は本当か 3000社がVCF 9にかじを切った現実的な理由
-
7
脱VMwareの真実:データセンター大手がNutanixを選んだ「コスト以上の理由」
-
8
多品種小ロットの「手書き・配合ミス」を克服 キャニオンスパイスの食品工場DX
-
9
「Microsoft一択」で本当にいいのか 知らぬ間にライセンス費用が膨らむ真相
-
10
Oracle巨大ITプロジェクトはなぜつまずいたのか 8年で導入1割、追加で170億ドル
ホワイトペーパーランキング PR
-
1
生成AIのハルシネーションを防止 回答精度を高めるセマンティックレイヤーとは
-
2
AIエージェントで多様な日常業務を効率化するための入門ガイド
-
3
5回聞くだけじゃ足りない? トヨタ式「なぜなぜ分析」の正しい実践方法
-
4
AIが「わざわざ使うツール」になっていない? 業務で自然に使う導線にする秘訣
-
5
JR西日本ITソリューションズが「監視業務の属人化」を解消した方法とは?
-
6
「脱Excel」か「Excel快適化」か? 現場にやさしい業務改善の進め方
-
7
インシデント対応工数を約3割削減、東京ガスの事例に学ぶ監視体制刷新のコツ
-
8
マンガで解説:「ゼロトラスト」「SASE」の必要性とメリット
-
9
5分で分かる「セキュア大容量ファイル転送サービス」の機能とメリット
-
10
情報セキュリティ対策早分かりガイド:25の自社診断で弱点と解決策を理解
TechTargetジャパン SNS
インフォメーション
注目情報をチェック
TechTargetジャパンをフォロー