本文介绍如何优化 wordpress ajax 处理逻辑，避免重复代码，利用 foreach 遍历用户角色配置数组，统一获取各启用角色的用户显示名称，显著提升代码可维护性与简洁性。
在 WordPress AJAX 开发中，常见需求是根据后台插件设置（如 show_users 选项）动态筛选并返回特定用户角色的数据。原始实现中，开发者为每个角色（如 administrator、editor、author 等）单独编写几乎相同的代码块，导致大量冗余，难以维护。
问题根源在于：自定义函数 save_time() 试图复用逻辑，但未正确传递和返回关键变量（如 $result 和 $count），且依赖外部作用域的 $user_roles —— 而 PHP 函数默认无法直接修改或访问外部变量，除非显式使用 global 或传参/返回值机制。
✅ 推荐解法：直接遍历配置数组，消除函数封装必要性
既然 get_option('show_users') 已返回一个以角色名为键、布尔值为值的关联数组（例如 ['administrator' => true, 'editor' => false, 'author' => true]），最简洁、高效且符合 WordPress 最佳实践的方式是直接遍历该数组：
$user_roles = get_option('show_users');
$result = []; // 显式初始化结果数组
$count = 0;
foreach ($user_roles as $role => $enabled) {
if ($enabled) { // 仅处理启用的角色
$users = get_users([
'role' => $role,
'fields' => ['ID', 'display_name'] // 可选：限制字段提升性能
]);
foreach ($users as $user) {
$result[] = $user->display_name; // 使用 [] 自动追加，无需手动计数
}
}
}
// 最终输出 JSON 响应（AJAX 回调中）
wp_send_json($result);
? 关键优化点说明：
- 移除手动计数：使用 $result[] = ... 替代 $result[$count] + $count++，更安全、更符合 PHP 惯例；
- 按需加载：if ($enabled) 确保只查询启用的角色，避免无效 get_users() 调用；
- 性能增强：添加 'fields' => ['ID', 'display_name'] 可减少数据库负载（尤其用户量大时）；
- 无状态设计：不依赖全局变量或闭包，逻辑清晰、易于测试与复用。
⚠️ 若仍需封装为函数（例如多处调用），应显式传入参数并返回结果：
function get_enabled_user_display_names($roles_config) {
$result = [];
foreach ($roles_config as $role => $enabled) {
if ($enabled) {
$users = get_users(['role' => $role, 'fields' => ['display_name']]);
foreach ($users as $user) {
$result[] = $user->display_name;
}
}
}
return $result;
}
// 调用方式：
$user_roles = get_option('show_users');
$result = get_enabled_user_display_names($user_roles);
wp_send_json($result);
总结：在 WordPress AJAX 后端逻辑中，优先利用原生数组结构进行迭代，比强行抽象为“通用函数”更直观、健壮且高效。保持代码扁平、意图明确，是保障可维护性的核心原则。
