重构 Java Servlet - 寻找一个好的命名方案

Question

我正在重构一个现有的 (Ajax) Java Web 应用程序，该应用程序目前直接在 servlet 中保存许多逻辑。 （servlet 甚至使用一些令人讨厌的技巧互相调用......）

重构本身很痛苦，但非常简单（现在重构所有内容以使用依赖注入）。

我的问题是，我真的无法想出完美的命名方案：

重构的 Servlet 现在只包含最少量的逻辑（仅处理 ServletRequest 和 ServletResponse、发送 HTTP 错误代码等）。 ），然后他们将我当前所说的称为处理器（好/坏名字？！？），它可以被多个 Servlet、测试类重用……处理器执行以下操作所需的逻辑：动作，类似于如果我们使用 EJB，那么会话 Bean 就可以了。

我从来没有想过，“Session Bean”会是一个好名字。而且我也不太热衷于定义一个单一的“外观”。我主要是在寻找一个比“Processor”更好的名称，或者可能是关于如何构建 Servlet 处理代码的一些想法。

示例

这是一个简化的示例（真正的应用程序使用 GWT 和 Gilead，...但这并不那么重要）：

public class UserRegistrationServlet extends HttpServlet {

@Inject
UserRegistrationProcessor userRegistrationProcessor;

@Override
protected void doPost(final HttpServletRequest req, 
        final HttpServletResponse resp)
        throws ServletException, IOException {

    RegistrationRequest registrationRequest = parseRegistrationRequest(req);

    RegistrationResult registrationResult = 
                userRegistrationProcessor.process(registrationRequest);

    pw.print(toJson(registrationResult));

    ...
}
}

正如示例所示，我更喜欢拥有专门的处理器，它们仅负责一项任务（或者可能是一些非常重要的任务） ：强相关的任务） - 与处理许多不同操作的大类相反。

Answer 1

这是我基于之前讨论的示例：

public class UserRegistrationServlet extends HttpServlet {

    @Inject
    private UserRequestExtractor userRequestExtractor;

    @Inject
    private UserRegistrationService userRegistrationService;

    @Override
    protected void doPost(final HttpServletRequest request, final HttpServletResponse response) throws ServletException, IOException {
        PrintWriter pw = response.getWriter();
        pw.print(createResponse(userRequestExtractor.extract(request)));
    }

    protected String createResponse(User newUser) {
        if (newUser == null) {
            return "{'message' : 'the user parameters are not valid'}";
        }
        else {
            UserRegistrationResult result = userRegistrationService.register(newUser);
            switch (result) {
                case SUCCESS:
                    return "{'message' : 'hurray'}";
                case EMAIL_IN_USE:
                    return String.format("{'message' : 'the email address %s is already in use'}", newUser.email);
                case USERNAME_IN_USE:
                    return String.format("{'message' : 'the user name %s is already in use'}", newUser.username);
                default:
                    return "{'message' : 'an error occurred'}";
            }
        }
    }
}

public class User {
    private String username;
    private String email;

    public boolean isValid() {
        return username != null && email != null;
    }
}

/**
 * Extractor which instantiates an object of type T from a request
 * 
 * @param <T>
 */
public interface RequestExtractor<T> {
    public T extract(HttpServletRequest request);
}

public class UserRequestExtractor implements RequestExtractor<User> {
    public User extract(HttpServletRequest request) {
        User user = new User();
        user.username = request.getParameter("username");
        user.email = request.getParameter("email");
        // validation could also be a responsibility of the RequestExtractor
        if (!user.isValid()) {
            return null;
        }
        return user;
    }
}

public interface UserRegistrationService {
    public UserRegistrationResult register(User user);
}

// implementation of UserRegistrationService omitted

public enum UserRegistrationResult {
    SUCCESS, EMAIL_IN_USE, USERNAME_IN_USE;
}

/**
 * Unit test
 */
public class UserRegistrationServletTest {

    @Test
    public void test() {
        UserRegistrationServlet cut = new UserRegistrationServlet();
        User user = new User();
        user.username = null;
        user.email = "[email protected]";

        String response = cut.createResponse(user);

        Assert.assertEquals("{'message' : 'the user parameters are not valid'}", response);
    }
}

希望这会有所帮助。

Answer 2

为什么不按原样称呼它们：Servlet？

如：

com.mycompany.app.admin.user.SaveServlet.java
com.mycompany.app.admin.user.ListServlet.java