En la aplicación Symfony, necesito usar la función cambiar/caso, pero los parámetros de solicitud me lo impiden, y uso una condición fea como:
$roles = ["ROLE_USER"]; if ($request->get('isAdmin')) { array_push($roles, "ROLE_ADMIN"); } if ($request->get('isFreemium')) { array_push($roles, "ROLE_FREEMIUM"); } if ($request->get('isPremium')) { array_push($roles, "ROLE_PREMIUM"); } if ($request->get('isExternal')) { array_push($roles, "ROLE_EXTERNAL"); } if ($request->get('isVip')) { array_push($roles, "ROLE_VIP"); }
¿Puedes ayudarme a simplificarlo?
Puedes intentar lo siguiente.
$definedRoles = new ArrayObject([ 'Admin', 'Freemium', 'Premium', 'External', 'Vip', ]); $roles = new ArrayObject([ 'ROLE_USER' ]); foreach ($definedRoles as $role) { if ($request->get('is' . $role)) { $roles->append(strtoupper('role_' . $role)); } }
¿Por qué estoy usando la clase ArrayObject
? Porque ya funciona como un rendimiento. A diferencia de una matriz, el objeto ocupa memoria solo para la entrada actual cuando se itera. Por lo tanto, su bucle ocupa poca memoria y no carga todas las entradas en la memoria.
Coloque todos sus roles válidos en la primera matriz $definedRoles
. Todos los roles se repetirán y se verificará una condición if, si está en el objeto $request
. Si es así, el rol se agregará a la matriz $roles
.