Có cách nào thông minh hơn để làm điều này ngoài một chuỗi dài các câu lệnh if hoặc switch không?

Tôi đang triển khai bot IRC nhận được tin nhắn và tôi đang kiểm tra tin nhắn đó để xác định chức năng nào cần gọi. Có cách nào thông minh hơn để làm điều này? Có vẻ như nó sẽ nhanh chóng thoát khỏi tầm tay sau khi tôi nhận được tới 20 lệnh.

Có lẽ có một cách tốt hơn để trừu tượng này?

 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);
        }
    }

Sử dụng một bảng công văn . Đây là bảng chứa các cặp ("phần thông báo", pointer-to-function). Bộ điều phối sau đó sẽ trông như thế này (bằng mã giả):

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

(các equalsIgnoreCase có thể được xử lý như một trường hợp đặc biệt ở đâu đó trước đây hoặc nếu bạn có nhiều thử nghiệm trong số đó, với bảng công văn thứ hai).

Tất nhiên, những gì pointer-to-functionphải trông giống như phụ thuộc vào ngôn ngữ lập trình của bạn. Đây là một ví dụ trong C hoặc C ++. Trong Java hoặc C #, bạn có thể sẽ sử dụng các biểu thức lambda cho mục đích đó hoặc bạn mô phỏng "con trỏ tới các hàm" bằng cách sử dụng mẫu lệnh. Cuốn sách trực tuyến miễn phí " Thứ tự cao hơn Perl " có một chương hoàn chỉnh về các bảng điều khiển sử dụng Perl.

Có lẽ tôi sẽ làm một cái gì đó như thế này:

public interface Command {
  boolean matches(String message);

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

Sau đó, bạn có thể có mọi lệnh thực hiện giao diện này và trả về true khi nó khớp với thông báo.

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
    }
}

Bạn đang sử dụng Java - vì vậy hãy làm cho nó đẹp ;-)

Tôi có thể sẽ làm điều này bằng cách sử dụng Chú thích:

1. Tạo chú thích phương thức tùy chỉnh

   @IRCCommand( String command, boolean perfectmatch = false )

2. Thêm chú thích cho tất cả các Phương thức có liên quan trong Lớp, vd

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

3. Trong hàm tạo của bạn, hãy sử dụng Reflection để tạo HashMap của các Phương thức từ tất cả các Phương thức được chú thích trong lớp của bạn:

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

4. Trong onMessagePhương thức của bạn , chỉ cần thực hiện một vòng lặp commandListcố gắng khớp chuỗi trên mỗi chuỗi và gọi method.invoke()nơi phù hợp.

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

Điều gì sẽ xảy ra nếu bạn xác định một giao diện, giả sử IChatBehaviourcó một phương thức được gọi là Executelấy trong một messagevà một cmdđối tượng:

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

Trong mã của bạn, sau đó bạn triển khai giao diện này và xác định các hành vi bạn muốn:

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

Và như vậy cho phần còn lại.

Trong lớp chính của bạn, sau đó bạn có một danh sách các hành vi ( List<IChatBehaviour>) mà bot IRC của bạn thực hiện. Sau đó, bạn có thể thay thế các iftuyên bố của mình bằng một cái gì đó như thế này:

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

Ở trên nên giảm số lượng mã bạn có. Cách tiếp cận trên cũng sẽ cho phép bạn cung cấp các hành vi bổ sung cho lớp bot của mình mà không cần sửa đổi chính lớp bot (theoStrategy Design Pattern ).

Nếu bạn chỉ muốn một hành vi kích hoạt bất cứ lúc nào, bạn có thể thay đổi chữ ký của executephương thức để mang lại true(hành vi đã kích hoạt) hoặc false(hành vi không kích hoạt) và thay thế vòng lặp trên bằng một thứ như thế này:

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

Ở trên sẽ tẻ nhạt hơn khi thực hiện và khởi tạo vì bạn cần tạo và vượt qua tất cả các lớp bổ sung, tuy nhiên, nó sẽ làm cho bot của bạn dễ dàng mở rộng và có thể sửa đổi vì tất cả các lớp hành vi của bạn sẽ được đóng gói và hy vọng độc lập với nhau.

"Thông minh" có thể (ít nhất) ba điều:

Hiệu suất cao hơn

Đề xuất Bảng công văn (và tương đương của nó) là một gợi ý tốt. Một bảng như vậy được gọi là "CADET" trong nhiều năm qua cho "Không thể thêm; thậm chí không thử." Tuy nhiên, hãy xem xét một nhận xét để hỗ trợ người duy trì người mới về cách quản lý bảng đã nói.

Bảo trì

"Làm cho nó đẹp" không phải là lời khuyên nhàn rỗi.

và, thường bị bỏ qua ...

Khả năng phục hồi

Việc sử dụng toLowerCase có một cạm bẫy ở chỗ một số văn bản trong một số ngôn ngữ phải trải qua quá trình tái cấu trúc đau đớn khi thay đổi giữa magiscule và miniscule. Thật không may, những cạm bẫy tương tự tồn tại cho toUpperCase. Chỉ cần lưu ý.

Bạn có thể có tất cả các lệnh thực hiện cùng một giao diện. Sau đó, một trình phân tích cú pháp thư có thể trả về cho bạn lệnh thích hợp mà bạn sẽ chỉ thực hiện.

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);
    }
}

Có vẻ như chỉ cần nhiều mã hơn. Có, bạn vẫn cần phân tích thông điệp để biết lệnh nào sẽ được thực thi nhưng bây giờ nó đã được xác định đúng. Nó có thể được tái sử dụng ở nơi khác. (Bạn có thể muốn tiêm MessageParser nhưng đó là một vấn đề khác. Ngoài ra, mẫu Flykg có thể là một ý tưởng hay cho các lệnh, tùy thuộc vào số lượng bạn dự kiến ​​sẽ được tạo.)

Những gì tôi sẽ làm là đây:

1. Nhóm các lệnh bạn có thành các nhóm. (bạn có ít nhất 20 cái ngay bây giờ)
2. Ở cấp độ đầu tiên, phân loại theo nhóm, do đó bạn có lệnh liên quan đến tên người dùng, lệnh bài hát, lệnh đếm, v.v.
3. Sau đó, bạn đi theo phương pháp của mỗi nhóm, lần này bạn sẽ nhận được lệnh ban đầu.

Điều này sẽ làm cho điều này dễ quản lý hơn. Lợi ích nhiều hơn khi số lượng 'khác nếu' tăng quá nhiều.

Tất nhiên, đôi khi có những 'nếu khác' sẽ không phải là một vấn đề lớn. Tôi không nghĩ 20 là xấu.