if 문이나 스위치의 긴 체인 외에도이를 수행하는보다 지능적인 방법이 있습니까?

메시지를 수신하는 IRC 봇을 구현하고 있으며 호출 할 함수를 결정하기 위해 해당 메시지를 확인하고 있습니다. 이것을하는 더 영리한 방법이 있습니까? 내가 20 명령을 좋아하면 빨리 손을 get 것 같습니다.

아마도 이것을 추상화하는 더 좋은 방법이 있습니까?

 public void onMessage(String channel, String sender, String login, String hostname, String message){

        if (message.equalsIgnoreCase(".np")){
//            TODO: Use Last.fm API to find the now playing
        } else if (message.toLowerCase().startsWith(".register")) {
                cmd.registerLastNick(channel, sender, message);
        } else if (message.toLowerCase().startsWith("give us a countdown")) {
                cmd.countdown(channel, message);
        } else if (message.toLowerCase().startsWith("remember am routine")) {
                cmd.updateAmRoutine(channel, message, sender);
        }
    }

디스패치 테이블을 사용하십시오 . 쌍을 포함하는 테이블입니다 ( "메시지 부분", pointer-to-function). 그러면 디스패처가 의사 코드로 다음과 같이 표시됩니다.

for each (row in dispatchTable)
{
    if(message.toLowerCase().startsWith(row.messagePart))
    {
         row.theFunction(message);
         break;
    }
}

(이것은 equalsIgnoreCase이전의 어딘가에서 특별한 경우로 처리 할 수 ​​있거나 두 번째 디스패치 테이블을 사용하여 이러한 테스트가 많은 경우 처리 할 수 ​​있습니다).

물론, pointer-to-function모양은 프로그래밍 언어에 따라 다릅니다. 다음 은 C 또는 C ++의 예입니다. Java 또는 C #에서는 해당 목적으로 람다 식을 사용하거나 명령 패턴을 사용하여 "포인터 대 함수"를 시뮬레이션합니다. 무료 온라인 서적 " Higher Order Perl "에는 Perl을 사용한 디스패치 테이블에 대한 완전한 장이 있습니다.

아마 다음과 같이 할 것입니다 :

public interface Command {
  boolean matches(String message);

  void execute(String channel, String sender, String login,
               String hostname, String message);
}

그런 다음 모든 명령이이 인터페이스를 구현하도록하고 메시지와 일치하면 true를 리턴 할 수 있습니다.

List<Command> activeCommands = new ArrayList<>();
activeCommands.add(new LastFMCommand());
activeCommands.add(new RegisterLastNickCommand());
// etc.

for (Command command : activeCommands) {
    if (command.matches(message)) {
        command.execute(channel, sender, login, hostname, message);
        break; // handle the first matching command only
    }
}

Java를 사용하고 있으므로 아름답게 만드십시오. ;-)

아마도 주석을 사용 하여이 작업을 수행 할 것입니다.

1. 사용자 정의 메소드 어노테이션 작성

   @IRCCommand( String command, boolean perfectmatch = false )

2. 클래스의 모든 관련 메소드에 주석 추가

   @IRCCommand( command = ".np", perfectmatch = true )
   doNP( ... )

3. 생성자에서 Reflections를 사용하여 클래스의 모든 주석이 달린 메소드에서 메소드의 HashMap을 만듭니다.

   ...
   for (Method m : getDeclaredMethods()) {
   if ( isAnnotationPresent... ) {
       commandList.put(m.getAnnotation(...), m);
       ...

4. 당신에 onMessage방법, 그냥 루프 할 commandList각각의 문자열과 일치하는 것을 시도하고 호출 method.invoke()이 맞는 곳을.

   for ( @IRCCommand a : commanMap.keyList() ) {
       if ( cmd.equalsIgnoreCase( a.command )
            || ( cmd.startsWith( a.command ) && !a.perfectMatch ) {
           commandMap.get( a ).invoke( this, cmd );

인터페이스를 정의하면 a 와 객체 를 취하는 IChatBehaviour하나의 메소드가 있습니다 .Executemessagecmd

public Interface IChatBehaviour
{
    public void execute(String message, CMD cmd);
}

그런 다음 코드에서이 인터페이스를 구현하고 원하는 동작을 정의하십시오.

public class RegisterLastNick implements IChatBehaviour
{
    public void execute(String message, CMD cmd)
    {
        if (message.toLowerCase().startsWith(".register"))
        {
            cmd.registerLastNick(channel, sender, message);
        }
    }
}

나머지도 마찬가지입니다.

메인 클래스에는 IRC 봇이 구현하는 동작 목록 ( List<IChatBehaviour>)이 있습니다. 그런 다음 if진술을 다음과 같이 바꿀 수 있습니다 .

for(IChatBehaviour behaviour : this.behaviours)
{
    behaviour.execute(message, cmd);
}

위의 코드는 줄어 듭니다. 위의 접근 방식을 사용하면 봇 클래스 자체를 수정하지 않고도 봇 클래스에 추가 동작을 제공 할 수 있습니다 Strategy Design Pattern.

한 번에 하나의 동작 만 실행하려면 execute메소드 의 서명을 변경하여 true(동작이 발생 했음) 또는 false( 동작이 발생 하지 않음) 위의 루프를 다음과 같이 바꾸십시오.

for(IChatBehaviour behaviour : this.behaviours)
{
    if(behaviour.execute(message, cmd))
    { 
         break;
    }
}

위의 내용은 모든 추가 클래스를 생성하고 전달해야하기 때문에 구현하고 초기화하는 것이 더 지루하지만, 모든 행동 클래스가 캡슐화되고 희망적 으로 서로 독립적 이기 때문에 봇을 쉽게 확장하고 수정할 수 있어야합니다 .

"지능형"은 (적어도) 세 가지가 될 수 있습니다.

더 높은 성능

디스패치 테이블 (및 해당 항목) 제안은 좋은 것입니다. 이러한 표는 과거에 "추가 할 수없고 시도조차하지 않음"으로 "CADET"이라고 불 렸습니다. 그러나 초보자 유지 관리자가 해당 테이블을 관리하는 방법에 도움이되는 의견을 고려하십시오.

유지 보수성

"아름답게 만드십시오"는 유휴 훈계가 아닙니다.

그리고 종종 간과 ...

복원력

toLowerCase를 사용하면 일부 언어의 일부 텍스트가 자기와 작은 부분 사이를 바꿀 때 고통스러운 구조 조정을 거쳐야한다는 단점이 있습니다. 불행히도, toUpperCase에도 동일한 함정이 존재합니다. 그냥 알아 둬

모든 명령이 동일한 인터페이스를 구현하도록 할 수 있습니다. 그런 다음 메시지 파서는 사용자에게만 적합한 명령을 반환 할 수 있습니다.

public interface Command {
    public void execute(String channel, String message, String sender) throws Exception;
}

public class MessageParser {
    public Command parseCommandFromMessage(String message) {
        // TODO Put your if/switch or something more clever here
        // e.g. return new CountdownCommand();
    }
}

public class Whatever {
    public void onMessage(String channel, String sender, String login, String hostname, String message) {
        Command c = new MessageParser().parseCommandFromMessage(message);
        c.execute(channel, message, sender);
    }
}

더 많은 코드처럼 보입니다. 예, 실행할 명령을 알기 위해 메시지를 구문 분석해야하지만 이제는 올바르게 정의 된 지점에 있습니다. 다른 곳에서 재사용 할 수 있습니다. MessageParser를 주입하고 싶을 수도 있지만 또 다른 문제입니다. 또한 Flyweight 패턴 은 생성 할 것으로 예상되는 수에 따라 명령에 적합합니다.

내가 할 일은 이것입니다 :

1. 가지고있는 명령을 그룹으로 그룹화하십시오. (지금은 20 이상입니다)
2. 첫 번째 수준에서는 그룹별로 분류되므로 사용자 이름 관련 명령, 노래 명령, 카운트 명령 등이 있습니다.
3. 그런 다음 각 그룹의 방법으로 이동하여 이번에는 원래 명령을 얻습니다.

이렇게하면 관리가 더 쉬워집니다. 'else if'의 수가 너무 많아지면 더 많은 이점이 있습니다.

물론, 때때로 '다른 경우'를 갖는 것은 큰 문제가되지 않습니다. 나는 20이 그렇게 나쁘지 않다고 생각합니다.