모든 솔리드 프라이스에 준하는 클래스의 올바른 버젼

Question

Activation Function를 적용하는 클래스를 만들고 싶습니다. Functions 다른 기능이 정의되는 enum이다

public class ActivationFunction { 

    public static double function(double value, Functions functions) { 
     if(functions.equals(Functions.SIGMOID)) { 
      return sigmoid(value); 
     } 
     return 0f; 
    } 
    public static double derivativeOfFunction(double value, Functions functions) { 
     if(functions.equals(Functions.SIGMOID)) 
      return sigmoidDerivative(value); 

     return 0f; 
    } 

    private static double sigmoid(double value) { 
     return 1/(1 + Math.exp(0 - value)); 
    } 
    private static double sigmoidDerivative(double value) { 
     return (Math.exp(0 - value)/((1 + Math.exp(0 - value)) * (1 + Math.exp(0 - value)))); 
    } 

} 

을 다음과 같이 클래스이다. 지금은 sigmoid 기능 만 있지만 더 추가됩니다.

질문

나는 것은 그것 (더를 위반 한 것 할 수 있음) Open-Closed Principle, OOP의 5 개 SOLID 원칙 중 하나를 위반입니다. 앞으로 더 많은 함수를 추가 할 수 있도록이 클래스를 작성하는 올바른 방법은 무엇입니까?

도움을 주시면 감사하겠습니다.

Answer 1

enum 자체에 공통 인터페이스 구현을 배치 할 수 있습니다. 이 같은

뭔가 :

ActivationFunction 다음 쓰기 매우 간단하게 당신의

public enum Functions { 
    SIGMOID { 
    public double function(double value) { return 1/(1 + Math.exp(0 - value)); } 
    public double derivative(double value) { return ...; } 
    }, 
    OTHER_FUNCTIONS { ... } 

    public abstract double function(double value); 
    public abstract double derivative(double value); 
} 

- 그것은 더 이상 아마 도움이 아니에요.

Answer 2

enum 값을 다음과 같이 지정하면 어떨까요?

enum MyFunctions { 
    FUNC_1 { 
     @Override double compute(double value) { 
      return 1/(1 + Math.exp(0 - value)); 
     } 
    }, 

    FUNC_2 { 
     @Override double compute(double value) { 
      return (Math.exp(0 - value)/((1 + Math.exp(0 - value)) * (1 + Math.exp(0 - value)))); 
     } 
    }; 

    abstract double compute(double value); 
} 

당신은 FUNC_3처럼, 또 다른 열거 값을 정의, 더 많은 기능을 필요로 할 때마다. 하나 이상의 방법이 필요한 경우 computecompute2 다른 방법을 추가하십시오.

내가 선택한 모든 이름은 바보 같지만 중요한 점을 얻길 바랍니다. 아니면 질문의 목표를 완전히 놓쳤습니까?

public interface Function { 
    double doIt(double value); 
} 

public class Sigmoid implements Function { 
    @Override 
    public double doIt(double value) { 
     return 0; 
    } 
} 

public class ActivationFunction { 
    public static double callFunction(Function function,double value){ 
     return function.doIt(value); 
    } 
} 

당신은 새로운 기능에 대한 새로운 클래스를 생성하고 기능 인터페이스를 구현해야합니다

Answer 3

여기 내 솔루션입니다. callFunction 인수 유형은 Function이므로 새 함수를 추가 할 때이를 변경할 필요가 없으며 Open-Closed Principle에 대해 정확합니다.