[m.....t]

#php #codereview

Jak bardzo zle wyglada ta metoda?

public function changeForgottenPassword($email, $token, $newPassword, Closure $onError = null)

    {

        try {

            $this->authRepository->changeForgottenPassword($email, $token, $newPassword);

        } catch (ApiException $e) {

            if ($onError) {

                $onError($e->getMessages());

            }

return $e->getMessages();

        }

    }

[krulgimby]

@matiit: Bardzo

[m.....t]

@krulgimby: rozwin prosze ;D

[krulgimby]

@matiit: no mięty podoba mi się, ale ja się na tym nie znam.

[m.....t]

@krulgimby: lol :D

[MacDada]

@matiit:

1. Po co zwracasz treść błędu jeśli obsługujesz go przez

[m.....t]

@MacDada: O to chodzilo.. dzieki.

Ja sie tylko zastanawiam czy to zmieniac, stara metoda.

[paranoiddd]

@matiit: wszyscy tak mówią :D

[m.....t]

@paranoiddd: ;)

[stacktrace]

@matit to ja dodam od siebie, że nie podoba mi się nazwa auth repository bo nie wiadomo co to repozytorium przechowuje. Nie podoba mi się też metoda changeForgottenPassword w repozytorium, repozytorium służy raczej do pobierania. Zwracanie treści błędu lub przekazywanie też uważam za bezsensowne, jeśli już to lepiej kod błędu. Poza tym jeśli to jest jakaś warstwa usługowa to ona powinna prawie wszystko obsłużyć, więc taki callback wydaje się zbędny.