はじめに

業務アプリやバッチ処理のコードを読んでいると、こんな書き方を見かけることがあります。

foreach ($items as $item) {
    $response = $externalService->fetch($item->id); // ← loop内でAPI叩き
    ...
}

一見すると普通ですが、実はこれ
パフォーマンス劣化・レートリミット・障害耐性低下
を同時に招く危険なアンチパターンです。

この記事では、

なぜ loop 内で外部 API/Service を呼んではいけないのか
実際どんな問題が起きるのか
どう直すべきか
良い例 / 悪い例をコード付きで解説

をまとめます。

結論

外部API・サービス呼び出しは 必ず loop の外に出すべき です。

☑️ 1. パフォーマンスが最悪になる（ネットワーク × N 回）

APIのレスポンスが 100ms の場合、100件処理しただけで 10秒かかります。
内部処理は高速でも、ネットワークIOは常に最も遅い。

☑️ 2. レートリミットに簡単に到達する

外部APIはたいてい何らかの回数制限があります。
loop 内で呼ぶと、意図せず制限を踏み抜き、急にエラー多発 → 障害化。

☑️ 3. 外部障害の影響を全件が受けて止まる

loop 内で毎回 API を呼ぶと、

APIが一時的に落ちる
タイムアウトが発生する

だけで 全件処理が即停止 します。
外部依存処理は、障害時の影響範囲を最小化すべきです。

☑️ 4. 再実行が困難（どこまで進んだ？問題）

途中で落ちると「どこまで処理したか？」が不明になり、リトライがしにくい。

☑️ 5. 従量課金APIの場合はコストが爆増

ループ件数 × APIコスト = 請求額
開発者にとっては見えづらいが、運用コストを押し上げます。

❌ 悪い例：loop 内で API を叩く

foreach ($hotels as $hotel) {
    $planId = $externalApi->getPlanId($hotel->plan_code); // ❌ 毎回API
    $this->savePlan($hotel, $planId);
}

毎回API
障害が起きたら即終了
遅い
コスト高い

典型的なアンチパターンです。

🟢 良い例：loop 外で一括取得する

☑️ API側がまとめ取得に対応している場合（最も理想）

$planCodes = $hotels->pluck('plan_code')->all();

// 一括で API 呼び出し（まとめて取得）
$planMap = $externalApi->getPlanIds($planCodes);

foreach ($hotels as $hotel) {
    $planId = $planMap[$hotel->plan_code] ?? null;
    $this->savePlan($hotel, $planId);
}

効果

API コール回数 100 → 1 回
パフォーマンス上昇
障害影響最小化
コスト削減

「APIのあるべき姿」は bulk API を用意することです。

API側が対応していない場合：キャッシュで回避

APIがまとめ取得に対応していないケースでも、loop 内呼び出しを避ける工夫 ができます。

$cache = [];

foreach ($hotels as $hotel) {

    // すでに取得したコードはAPI呼ばない
    if (!isset($cache[$hotel->plan_code])) {
        $cache[$hotel->plan_code] = $externalApi->getPlanId($hotel->plan_code);
    }

    $planId = $cache[$hotel->plan_code];
    $this->savePlan($hotel, $planId);
}

重複コードに対する API 呼び出しが 1回に削減 されます。

🔧 どうしても loop 内で API が必要な場合の対策

① キャッシュする（最重要）

$cache = [];

foreach ($items as $item) {

    if (!isset($cache[$item->code])) {
        $cache[$item->code] = $api->fetch($item->code);
    }

    $value = $cache[$item->code];
}

② リトライ・サーキットブレーカーを設定する

障害中でも無限リトライしないようにする。

③ キューに逃がす

外部API依存処理はできるだけ非同期化。

④ まとめて投げられる API を作る（サーバ側改善）

自身でAPIを持っている場合はbulk API を用意するのが最強です。

「Service呼び出し」も同じ

外部APIだけでなく、以下も同じ原則が当てはまります。

DBクエリ（N+1問題）
別マイクロサービス
外部ライブラリでのIO処理
ファイルアクセス

loop 内の I/O はすべてアンチパターンになり得ます。

🎯 まとめ

外部APIやサービス呼び出しは、必ず loop の外に出すことを最優先で考えるべきです。

性能改善
レートリミット回避
障害耐性アップ
コスト削減

とメリットが非常に大きいです。

もし loop 内で呼ばれているコードを見つけたら、
「これ、外に出せない？」
と必ず疑って改善していきましょう。