0

我正在编写一个登录页面，当无效用户尝试登录时，我重定向到登录操作，错误参数等于 1。

private String username;
private String password;
private int error;
@Override
public String execute()
{
    //validate user input
    if (username == null || password == null || username.isEmpty() || password.isEmpty())
    {
        error = 2;
        return LOGIN;   
    }

    LoginModel loginModel = new LoginModel(username, password);

    HttpBuilder<LoginModel, User> builder = new HttpBuilder<LoginModel, User>(User.class);
    builder.setPath("service/user/authenticate");
    builder.setModel(loginModel);

    IHttpRequest<LoginModel, User> request = builder.buildHttpPost();
    User user = request.execute(URL.BASE_LOCAL);

    //redirects to login page
    if (user == null)
    {
        error = 1;
        return LOGIN;
    }
    else
    {   
        return SUCCESS;
    }
}

   //Getters/Setters

如果无效用户尝试登录，它会重定向到 localhost:8080/app/login.action?error=1。我试图通过使用 if 标记访问错误参数来向用户显示错误消息，但它不起作用，消息未显示。

<s:if test="error == 1">
<center>
    <h4 style="color:red">Username or Password is invalid!!</h4>
</center>

我究竟做错了什么？

4

2 回答 2

2

就我而言，你做错了什么是完全忽略了框架。

粗略地说，IMO 这应该看起来更像这样：

public class LoginAction extends ActionSupport {

    private String username;
    private String password;

    @Override
    public String validate() {
        if (isBlank(username) || isBlank(password)) {
            addActionError("Username or Password is invalid");
        }

        User user = loginUser(username, password);
        if (user == null) {
            addActionError("Invalid login");
        }
    }

    public User loginUser(String username, String password) {
        LoginModel loginModel = new LoginModel(username, password);

        HttpBuilder<LoginModel, User> builder = new HttpBuilder<LoginModel, User>(User.class);
        builder.setPath("service/user/authenticate");
        builder.setModel(loginModel);

        IHttpRequest<LoginModel, User> request = builder.buildHttpPost();
        return request.execute(URL.BASE_LOCAL);
    }

}

您将获得"input"包含表单的结果，并使用您想要的任何样式显示存在的任何操作错误。如果根据登录错误的类型更改样式至关重要，那么您将不得不再玩一些游戏，但这似乎过分了。

不相关，但我会将该loginUser代码完全移出操作并进入实用程序/服务类，但至少将它包装在一个单独的方法中，您可以更轻松地模拟它。它当然不属于该execute方法。

于 2013-07-22T17:41:33.893 回答
0

您需要提供字段 'error' 的getter和setter才能从值堆栈中访问它。

public int getError() 
{
   return error;
}

public void setError(int error) 
{
   this.error = error;
}

并尝试在 OGNL 中访问它：

<s:if test="%{#error==1}"></s:if>

或者使用 JSTL：

<c:if test="${error==1}"></c:if>
于 2013-07-22T17:27:13.480 回答