Refactor mã java

Question

rồi đoán câu hỏi này trông rất giống:

What is the best way to replace or substitute if..else if..else trees in programs?

xem xét câu hỏi này ĐÓNG CỬA!

---

Tôi muốn refactor code mà trông giống như sau:

String input; // input from client socket. 
if (input.equals(x)) { 
    doX(); 
} else if (input.equals(y)) { 
    doY(); 
} else { 
    unknown_command(); 
} 

Nó là mã mà kiểm tra đầu vào từ ổ cắm để thực hiện một số hành động, nhưng tôi không thích if else xây dựng vì mỗi thời gian một lệnh mới được thêm vào máy chủ (mã) một cái mới nếu cần phải thêm vào đó là xấu. Ngoài ra khi xóa lệnh, if else phải được sửa đổi.

Answer 1

Thu thập các lệnh đó trong một Map<String, Command> trong đó Command là một interface với phương thức execute().

Map<String, Command> commands = new HashMap<String, Command>(); 
// Fill it with concrete Command implementations with `x`, `y` and so on as keys. 

// Then do: 
Command command = commands.get(input); 
if (command != null) { 
    command.execute(); 
} else { 
    // unknown command. 
} 

Để có được một bước xa hơn, bạn có thể xem xét để điền vào bản đồ tự động bằng cách quét cho các lớp thực hiện một giao diện cụ thể (Command trong trường hợp này) hoặc một chú thích cụ thể trong classpath. Google Reflections có thể giúp ích rất nhiều cho việc này.

Cập nhật (từ nhận xét) Bạn cũng có thể cân nhắc việc kết hợp answer of Instantsoup với câu trả lời của tôi. Trong phương thức buildExecutor(), trước tiên hãy lấy lệnh từ Map và nếu lệnh không tồn tại trong Map, hãy thử tải lớp được liên kết và đặt nó trong Map. Sắp xếp tải chậm. Điều này hiệu quả hơn quét toàn bộ classpath như trong câu trả lời của tôi và tạo ra nó mọi lúc như trong câu trả lời của Instantsoup.

Answer 2

Một cách có thể để có một giao diện ICommand đó là hợp đồng chung cho một lệnh, ví dụ:

public interface ICommand { 
    /** @param context The command's execution context */ 
    public void execute(final Object context); 
    public String getKeyword(); 
} 

Và sau đó bạn có thể sử dụng cơ chế SPI Java tự động khám phá hiện thực khác nhau của bạn và đăng ký chúng trong a Map<String,ICommand> và sau đó thực hiện knownCommandsMap.get(input).execute(ctx) hoặc một cái gì đó giống nhau.

Điều này thực tế cho phép bạn tách dịch vụ của mình khỏi việc triển khai lệnh, thực hiện hiệu quả các plugin đó.

Đăng ký lớp triển khai với SPI được thực hiện bằng cách thêm tệp có tên là tên đủ điều kiện của lớp ICommand của bạn (vì vậy nếu nó nằm trong gói giả, tệp sẽ là META-INF/dummy.ICommand trong đường dẫn lớp học của bạn), sau đó bạn ' sẽ tải và đăng ký chúng dưới dạng:

final ServiceLoader<ICommand> spi = ServiceLoader.load(ICommand.class); 
for(final ICommand commandImpl : spi) 
    knownCommandsMap.put(commandImpl.getKeyword(), commandImpl); 

Answer 3

Làm thế nào về giao diện, nhà máy và phản ánh một chút? Bạn vẫn sẽ cần phải xử lý các ngoại lệ về đầu vào xấu, nhưng bạn sẽ luôn luôn cần phải làm điều này. Với phương pháp này, bạn chỉ cần thêm một thực thi mới của Executor cho một đầu vào mới.

public class ExecutorFactory 
{ 
    public static Executor buildExecutor(String input) throws Exception 
    { 
     Class<Executor> forName = (Class<Executor>) Class.forName(input); 
     return (Executor) executorClass.newInstance(); 
    } 
} 

public interface Executor 
{ 
    public void execute(); 
} 


public class InputA implements Executor 
{ 
    public void execute() 
    { 
     // do A stuff 
    } 
} 

public class InputB implements Executor 
{ 
    public void execute() 
    { 
     // do B stuff 
    } 
} 

mã ví dụ của bạn sau đó trở thành

String input; 
ExecutorFactory.buildExecutor(input).execute(); 

Answer 4

Xây dựng guốc lệnh trên một lớp enum có thể giảm bớt một số mã soạn sẵn. Giả sử rằng x trong input.equals(x) là "XX" và y trong input.equals(y) là "YY"

enum Commands { 
    XX { 
    public void execute() { doX(); }   
    }, 
    YY { 
    public void execute() { doY(); }   
    }; 

    public abstract void execute(); 
} 

String input = ...; // Get it from somewhere 

try { 
    Commands.valueOf(input).execute(); 
} 
catch(IllegalArgumentException e) { 
    unknown_command(); 
} 

Answer 5

Bạn nói rằng bạn đang xử lý đầu vào từ một ổ cắm. Bao nhiêu đầu vào? Nó phức tạp thế nào? Nó có cấu trúc như thế nào?

Tùy thuộc vào câu trả lời cho các câu hỏi đó, bạn có thể viết tốt hơn ngữ pháp và để trình tạo trình phân tích cú pháp (ví dụ: ANTLR) tạo mã xử lý đầu vào.