5層エラーハンドリング：AIが書いたコードの「握りつぶし」問題

1
# AIが書きがちなコード
2
def fetch_data(url):
3
    try:
4
        response = requests.get(url)
5
        return response.json()
6
    except Exception:
7
        return None  # 何が起きても None を返す

1
def get_resources():
2
    try:
3
        return api_client.fetch_resources()
4
    except Exception:
5
        return []  # エラーでも空リストを返す

1
Level 1: ロギング（全てのイベントを記録）
2
   ↓
3
Level 2: 例外処理（想定されるエラーのみキャッチ）
4
   ↓
5
Level 3: リトライロジック（一時的な障害に対応）
6
   ↓
7
Level 4: 通知システム（人間に知らせる）
8
   ↓
9
Level 5: グレースフルシャットダウン（安全に停止する）

1
from myModules.commonLogger import setupLogger
2
logger = setupLogger(__name__)
3

4
def process_task(task):
5
    logger.info(f"[TASK] Processing: {task.name} priority={task.priority}")
6
    # ... 処理 ...
7
    logger.info(f"[TASK] Complete: {task.name} result={result}")

1
# ❌ AIが書きがちな広すぎるキャッチ
2
try:
3
    result = api_call()
4
except Exception:
5
    result = default
6

7
# ✅ 想定するエラーのみキャッチ
8
try:
9
    result = api_call()
10
except ConnectionError:
11
    logger.warning("API接続エラー。リトライします")
12
    result = retry_api_call()
13
except AuthenticationError:
14
    logger.error("認証エラー。APIキーを確認してください")
15
    raise  # 認証エラーは上位に伝播（握りつぶさない）

1
import time
2

3
def api_call_with_retry(func, max_retries=3, backoff_seconds=2):
4
    for attempt in range(max_retries):
5
        try:
6
            return func()
7
        except ConnectionError:
8
            if attempt == max_retries - 1:
9
                raise  # 最終試行でも失敗なら例外を伝播
10
            wait = backoff_seconds * (2 ** attempt)  # 指数バックオフ
11
            logger.warning(f"リトライ {attempt+1}/{max_retries}（{wait}秒後）")
12
            time.sleep(wait)

1
def notify_error(message, severity="warning"):
2
    if severity == "critical":
3
        slack.send(channel="#alerts", text=f"🚨 CRITICAL: {message}")
4
    elif severity == "warning":
5
        slack.send(channel="#monitoring", text=f"⚠️ WARNING: {message}")
6
    # INFO レベルはSlack通知しない（ログのみ）

1
import sys
2

3
EXIT_SUCCESS = 0
4
EXIT_PARTIAL_FAILURE = 1  # 一部失敗（処理は継続可能）
5
EXIT_ERROR = 2            # 致命的エラー（処理不可能）
6

7
def run():
8
    try:
9
        phase_input()
10
        phase_process()
11
        phase_output()
12
        sys.exit(EXIT_SUCCESS)
13
    except CriticalError as e:
14
        logger.error(f"致命的エラー: {e}")
15
        notify_error(str(e), severity="critical")
1 collapsed line
16
        sys.exit(EXIT_ERROR)

1
## エラーハンドリングルール
2

3
1. except Exception: pass は禁止
4
   - 具体的な例外クラスをキャッチすること
5
   - キャッチしたエラーは必ずログに記録すること
6

7
2. 戻り値での握りつぶし禁止
8
   - エラー時に None や [] を返す場合は、
9
     ログにWARNING以上で記録すること
10
   - 認証エラーは絶対に握りつぶさない（raise すること）
11

12
3. リトライ対象の限定
13
   - ConnectionError, TimeoutError のみリトライ
14
   - AuthenticationError, ValueError はリトライしない