コードでドキュメント化されていない匿名関数を過度に使用するのを避けるべき理由

ルール
～しすぎないで やりすぎないで 文書化されていない 匿名 関数を
大規模な 大規模な 関数 なし ドキュメントがない 
は 難しい 理解 理解 難しく、 再利用が

対応言語： 45以上

はじめに

コールバックやイベントハンドラーとして渡される匿名関数は、実装の詳細の裏にその目的を隠します。20行の矢印関数は .map() または .filter() 読者は何が変換されるのかを理解するために、ロジック全体を解析する必要があります。説明的な名前を持つ名前付き関数は、意図を即座に示し、複雑なロジックは、実装に深く入り込む前に関数名を読むことで理解できます。

コード例

❌ 非準拠：

app.get('/users', async (req, res) => {
    const users = await db.users.find({});
    const processed = users.filter(u => {
        const hasActiveSubscription = u.subscriptions?.some(s => 
            s.status === 'active' && new Date(s.expiresAt) > new Date()
        );
        const isVerified = u.emailVerified && u.phoneVerified;
        return hasActiveSubscription && isVerified && !u.deleted;
    }).map(u => ({
        id: u.id,
        name: `${u.firstName} ${u.lastName}`,
        email: u.email,
        memberSince: new Date(u.created).getFullYear(),
        tier: u.subscriptions[0]?.tier || 'free'
    })).sort((a, b) => a.name.localeCompare(b.name));
    res.json(processed);
});

誤っている理由： filter関数には、複雑なビジネスロジック（サブスクリプション検証、検証チェック）が匿名関数内に埋め込まれています。このロジックは再利用できず、独立してテストすることも、すべての行を読まずに理解することもできません。フィルタリングロジックが失敗した場合、スタックトレースには匿名関数が表示されます。

✅ 準拠済み:

function hasActiveSubscription(user) {
    return user.subscriptions?.some(subscription => 
        subscription.status === 'active' && 
        new Date(subscription.expiresAt) > new Date()
    );
}

function isVerifiedUser(user) {
    return user.emailVerified && user.phoneVerified && !user.deleted;
}

function isEligibleUser(user) {
    return hasActiveSubscription(user) && isVerifiedUser(user);
}

function formatUserResponse(user) {
    return {
        id: user.id,
        name: `${user.firstName} ${user.lastName}`,
        email: user.email,
        memberSince: new Date(user.created).getFullYear(),
        tier: user.subscriptions[0]?.tier || 'free'
    };
}

function sortByName(a, b) {
    return a.name.localeCompare(b.name);
}

app.get('/users', async (req, res) => {
    const users = await db.users.find({});
    const processed = users
        .filter(isEligibleUser)
        .map(formatUserResponse)
        .sort(sortByName);
    res.json(processed);
});

これが重要である理由： 複雑なビジネスロジックは、テスト可能な関数として抽出されます。 hasActiveSubscription() そして isVerifiedUser() 単体テストが可能で再利用できます。スタックトレースには関数名が表示されるため、デバッグが高速化されます。エンドポイントのロジックはクリーンで自己文書化されています。

まとめ

2〜3行を超えるロジック、または再利用される可能性のあるロジックには、名前付き関数を使用してください。関数名が実装よりも長くなるような些細な操作には、匿名関数を予約してください。記述的な関数名は、インラインドキュメントとして機能します。