Skip to main content
Título orgânico. Remoção de "o que você acham?", essa não é o tipo de pergunta que o site promociona, tem até um motivo de fechamento pra isso: "principalmente baseada em opiniões"
Fonte Link
brasofilo
  • 6,7mil
  • 5
  • 30
  • 50

Estou abusando muito do Strategy Pattern? em Java?

Em uma aplicação para autenticação de usuários via Radius achei que seria interessante a utilização do Design Patter Strategy com Enum.

Então o código ficou desse jeito:

public enum TipoAutenticacao {

LIVRE("Acesso Livre"){
    @Override
    public String autentica(LoginService service, String user, String mac) throws NoResultException, AccessException{
        Login login = service.findByUser(user);
        return login.autentica(user, login.getPass());
    }
},
MAC("Filtro de MAC"){
    @Override
    public String autentica(LoginService service, String user, String mac) throws NoResultException, AccessException{
        Login login = service.findByUser(user);
        return login.autentica(user, login.getPass(), mac);
    }
};

private String descricao;


TipoAutenticacao(String descricao){
    this.descricao = descricao;
}

public String getDescricao(){
    return this.descricao;
}

public abstract String autentica(LoginService service , String user, String mac) throws NoResultException, AccessException;
}

Método autentica no Login

    public String autentica(String user, String pass, String mac) throws AccessException {
    if(!this.mac.equals(mac))
        throw new AccessException(TipoLoginResposta.MAC_INVALID);
    
    return autentica(user, pass);
}

public String autentica(String user, String pass) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}

TipoLoginResposta

public enum TipoLoginResposta {

USER_PASS_INVALID("Login/Senha Inválidos"),
MAC_INVALID("MAC Inválido"),
PENDENCIA_FINANCEIRA("Pendência Financeira"),
BLOQUEADO("Bloqueado");

private String descricao;

TipoLoginResposta(String descricao){
    this.descricao = descricao;
}

public String getDescricao(){
    return this.descricao;
}
}

A ideia de utilizar o AccessException passando no construtor TipoLoginResposta é para que seja especificado mais tarde no banco de dados o motivo pelo qual o usuário não conseguiu se conectar.

Utilizando essa implementação eu estou amarrando o código de alguma maneira? Gostaria de saber a opinião de vocês.

Estou abusando muito do Strategy Pattern? Java

Em uma aplicação para autenticação de usuários via Radius achei que seria interessante a utilização do Design Patter Strategy com Enum.

Então o código ficou desse jeito:

public enum TipoAutenticacao {

LIVRE("Acesso Livre"){
    @Override
    public String autentica(LoginService service, String user, String mac) throws NoResultException, AccessException{
        Login login = service.findByUser(user);
        return login.autentica(user, login.getPass());
    }
},
MAC("Filtro de MAC"){
    @Override
    public String autentica(LoginService service, String user, String mac) throws NoResultException, AccessException{
        Login login = service.findByUser(user);
        return login.autentica(user, login.getPass(), mac);
    }
};

private String descricao;


TipoAutenticacao(String descricao){
    this.descricao = descricao;
}

public String getDescricao(){
    return this.descricao;
}

public abstract String autentica(LoginService service , String user, String mac) throws NoResultException, AccessException;
}

Método autentica no Login

    public String autentica(String user, String pass, String mac) throws AccessException {
    if(!this.mac.equals(mac))
        throw new AccessException(TipoLoginResposta.MAC_INVALID);
    
    return autentica(user, pass);
}

public String autentica(String user, String pass) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}

TipoLoginResposta

public enum TipoLoginResposta {

USER_PASS_INVALID("Login/Senha Inválidos"),
MAC_INVALID("MAC Inválido"),
PENDENCIA_FINANCEIRA("Pendência Financeira"),
BLOQUEADO("Bloqueado");

private String descricao;

TipoLoginResposta(String descricao){
    this.descricao = descricao;
}

public String getDescricao(){
    return this.descricao;
}
}

A ideia de utilizar o AccessException passando no construtor TipoLoginResposta é para que seja especificado mais tarde no banco de dados o motivo pelo qual o usuário não conseguiu se conectar.

Utilizando essa implementação eu estou amarrando o código de alguma maneira? Gostaria de saber a opinião de vocês.

Estou abusando muito do Strategy Pattern em Java?

Em uma aplicação para autenticação de usuários via Radius achei que seria interessante a utilização do Design Patter Strategy com Enum.

Então o código ficou desse jeito:

public enum TipoAutenticacao {

LIVRE("Acesso Livre"){
    @Override
    public String autentica(LoginService service, String user, String mac) throws NoResultException, AccessException{
        Login login = service.findByUser(user);
        return login.autentica(user, login.getPass());
    }
},
MAC("Filtro de MAC"){
    @Override
    public String autentica(LoginService service, String user, String mac) throws NoResultException, AccessException{
        Login login = service.findByUser(user);
        return login.autentica(user, login.getPass(), mac);
    }
};

private String descricao;


TipoAutenticacao(String descricao){
    this.descricao = descricao;
}

public String getDescricao(){
    return this.descricao;
}

public abstract String autentica(LoginService service , String user, String mac) throws NoResultException, AccessException;
}

Método autentica no Login

    public String autentica(String user, String pass, String mac) throws AccessException {
    if(!this.mac.equals(mac))
        throw new AccessException(TipoLoginResposta.MAC_INVALID);
    
    return autentica(user, pass);
}

public String autentica(String user, String pass) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}

TipoLoginResposta

public enum TipoLoginResposta {

USER_PASS_INVALID("Login/Senha Inválidos"),
MAC_INVALID("MAC Inválido"),
PENDENCIA_FINANCEIRA("Pendência Financeira"),
BLOQUEADO("Bloqueado");

private String descricao;

TipoLoginResposta(String descricao){
    this.descricao = descricao;
}

public String getDescricao(){
    return this.descricao;
}
}

A ideia de utilizar o AccessException passando no construtor TipoLoginResposta é para que seja especificado mais tarde no banco de dados o motivo pelo qual o usuário não conseguiu se conectar.

Utilizando essa implementação eu estou amarrando o código de alguma maneira?

removeu 307 caracteres do conteúdo
Fonte Link
SoabTI
  • 440
  • 1
  • 4
  • 12
    public String autentica(String user, String pass, String mac) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(!this.mac.equals(mac))
        throw new AccessException(TipoLoginResposta.MAC_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
   return if(!this.getUsuario().isPendenciaFinanceiraautentica())
        throw newuser, AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRApass);
    return this.pass;
}

public String autentica(String user, String pass) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}
public String autentica(String user, String pass, String mac) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(!this.mac.equals(mac))
        throw new AccessException(TipoLoginResposta.MAC_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}

public String autentica(String user, String pass) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}
    public String autentica(String user, String pass, String mac) throws AccessException {
    if(!this.mac.equals(mac))
        throw new AccessException(TipoLoginResposta.MAC_INVALID);
    
    return autentica(user, pass);
}

public String autentica(String user, String pass) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}
Fonte Link
SoabTI
  • 440
  • 1
  • 4
  • 12

Estou abusando muito do Strategy Pattern? Java

Em uma aplicação para autenticação de usuários via Radius achei que seria interessante a utilização do Design Patter Strategy com Enum.

Então o código ficou desse jeito:

public enum TipoAutenticacao {

LIVRE("Acesso Livre"){
    @Override
    public String autentica(LoginService service, String user, String mac) throws NoResultException, AccessException{
        Login login = service.findByUser(user);
        return login.autentica(user, login.getPass());
    }
},
MAC("Filtro de MAC"){
    @Override
    public String autentica(LoginService service, String user, String mac) throws NoResultException, AccessException{
        Login login = service.findByUser(user);
        return login.autentica(user, login.getPass(), mac);
    }
};

private String descricao;


TipoAutenticacao(String descricao){
    this.descricao = descricao;
}

public String getDescricao(){
    return this.descricao;
}

public abstract String autentica(LoginService service , String user, String mac) throws NoResultException, AccessException;
}

Método autentica no Login

public String autentica(String user, String pass, String mac) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(!this.mac.equals(mac))
        throw new AccessException(TipoLoginResposta.MAC_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}

public String autentica(String user, String pass) throws AccessException {
    if(!this.user.equals(user) || !this.pass.equals(pass))
        throw new AccessException(TipoLoginResposta.USER_PASS_INVALID);
    if(bloqueado)
        throw new AccessException(TipoLoginResposta.BLOQUEADO);
    if(!this.getUsuario().isPendenciaFinanceira())
        throw new AccessException(TipoLoginResposta.PENDENCIA_FINANCEIRA);
    return this.pass;
}

TipoLoginResposta

public enum TipoLoginResposta {

USER_PASS_INVALID("Login/Senha Inválidos"),
MAC_INVALID("MAC Inválido"),
PENDENCIA_FINANCEIRA("Pendência Financeira"),
BLOQUEADO("Bloqueado");

private String descricao;

TipoLoginResposta(String descricao){
    this.descricao = descricao;
}

public String getDescricao(){
    return this.descricao;
}
}

A ideia de utilizar o AccessException passando no construtor TipoLoginResposta é para que seja especificado mais tarde no banco de dados o motivo pelo qual o usuário não conseguiu se conectar.

Utilizando essa implementação eu estou amarrando o código de alguma maneira? Gostaria de saber a opinião de vocês.