Quantcast
Channel: Forum Pasja Informatyki - Najnowsze pytania bez odpowiedzi
Viewing all articles
Browse latest Browse all 21942

Spring - czy używanie serwisu w innym serwisie to prawdidłowy sposób?

$
0
0

Posiadam przykładowe dwa serwisy

private AuthorizationService authorizationService;
private SecurityService securityService;

których używam do procesu logowania(tworzenie ciasteczek itd.). Kontroler, który za to odpowiada to

@GetMapping("/signInSuccessfully")
    public ModelAndView signInSuccessfully(HttpServletRequest request) {
        if(authorizationService.isLoginProcess(request)) {
            authorizationService.login();
        }

        return new ModelAndView("redirect:/");
    }

który wykorzystuje jeden z powyższych serwisów.

Natomiast w serwisie 

AuthorizationService

niektóre metody korzystają z innego serwisu

@Service
public class AuthorizationServiceImpl implements AuthorizationService {

    private SecurityService securityService;

    @Autowired
    public AuthorizationServiceImpl(SecurityService service) {
        this.securityService = service;
    }

    @Override
    public boolean isLoginProcess(HttpServletRequest request) {
        return true;
    }

    @Override
    public boolean login() {
        String username = securityService.getUsername();

        System.out.println(username);

        return false;
    }
}

i tu pojawia się ten mój problem. Czy jest to prawidłowy sposób na używanie serwisu, czy może nie powinienem zawiązywać innych serwisów w serwisie, a przekazywać tylko bezpośrednio z kontrolera parametry do metody, czyli np. zmiana kontrolera na

@GetMapping("/signInSuccessfully")
    public ModelAndView signInSuccessfully(HttpServletRequest request) {
        if(authorizationService.isLoginProcess(request)) {
            authorizationService.login(securityService.getUsername());
        }

        return new ModelAndView("redirect:/");
    }

i

@Service
public class AuthorizationServiceImpl implements AuthorizationService {

    @Override
    public boolean isLoginProcess(HttpServletRequest request) {
        return true;
    }

    @Override
    public boolean login(String username) {
        System.out.println(username);

        return false;
    }
}

Wtedy w serwisie nie ma żadnego innego serwisu i jest zachowana zasada SOLID. Jak uważacie?


Viewing all articles
Browse latest Browse all 21942