Как можно оптимизировать такой код?

/**
     * @Rest\Post("/users/{userId}/group/{groupId}")
     */
    public function addUserToGroupAction($userId, $groupId)
    {
        $userManager = $this->get('fos_user.user_manager');
        $groupManager = $this->get('group_manager');
 
        $user = $userManager->findUserBy(['id' => $userId]);
 
        if (is_null($user)) {
            return $this->handleView($this->view('User not found', Response::HTTP_NOT_FOUND));
        }
 
        $group = $groupManager->getById($groupId);
 
        if (is_null($group)) {
            return $this->handleView($this->view('Group not found', Response::HTTP_NOT_FOUND));
        }
 
        try {
 
            $user->addGroup($group);
            $userManager->updateUser($user);
 
            return $this->handleView($this->view('User assign to group successfull', Response::HTTP_OK));
 
        } catch (\Exception $e) {
            return $this->handleView($this->view($e->getMessage()));
        }
    }
 
    /**
     * @Rest\Delete("/users/{userId}/group/{groupId}")
     */
    public function deleteUserFromGroupAction($userId, $groupId)
    {
        $userManager = $this->get('fos_user.user_manager');
        $groupManager = $this->get('group_manager');
 
        $user = $userManager->findUserBy(['id' => $userId]);
 
        if (is_null($user)) {
            return $this->handleView($this->view('User not found', Response::HTTP_NOT_FOUND));
        }
 
        $group = $groupManager->getById($groupId);
 
        if (is_null($group)) {
            return $this->handleView($this->view('Group not found', Response::HTTP_NOT_FOUND));
        }
 
        try {
 
            $user->removeGroup($group);
            $userManager->updateUser($user);
 
            return $this->handleView($this->view('User remove from group successfull', Response::HTTP_OK));
 
        } catch (\Exception $e) {
            return $this->handleView($this->view($e->getMessage()));
        }
    }

{"error":{"code":500,"message":"Internal Server Error","exception":[{"message":"An exception occurred while executing 'INSERT INTO users (username, username_canonical, email, email_canonical, enabled, salt, password, last_login, confirmation_token, password_requested_at, roles) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)' with params [null, null, null, null, 1, null, null, null, null, null, \"a:1:{i:0;s:0:\\\"\\\";}\"]:\n\nSQLSTATE[23000]: Integrity constraint violation: 1048 Column 'username' cannot be null","class":"Doctrine\\DBAL\\Exception\\NotNullConstraintViolationException","trace":[{"namespace":"","short_class":"","class":"","type":"","function":"","file":"\/home\/dev\/Desktop\/Projects\/nfq_task\/vendor\/doctrine\/dbal\/lib\/Doctrine\/DBAL\/Driver\/AbstractMySQLDriver.php","line":112,"args":[]},{"namespace":"Doctrine\\DBAL\\Driver","short_class":"AbstractMySQLDriver","class":"Doctrine\\DBAL\\Driver\\AbstractMySQLDriver","type":"->","function":"convertException","file":"\/home\/dev\/Desktop\/Projects\/nfq_task\/vendor\/doctrine\/dbal\/lib\/Doctrine\/DBAL\/DBALException.php","line":128,"args":[["string","An exception occurred while executing 'INSERT INTO users (username, username_canonical, email, email_canonical, enabled, salt, password, last_login, confirmation_token, password_requested_at, roles) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)' with params [null, null, null, null, 1, null, null, null, null, null, \"a:1:{i:0;s:0:\\\"\\\";}\"]:\n\nSQLSTATE[23000]: Integrity constraint violation: 1048 Column 'username' cannot be null"],["object","Doctrine\\DBAL\\Driver\\PDOException"]]},

$user = $um->createUser();
        $user
            ->setUsername($request->get('username'))
            ->setEmail($request->get('email'))
            ->setPlainPassword($request->get('password'))
            ->setRoles([$request->get('role')])
            ->setEnabled(true);