Servlet.service () для сервлета jsp выдал исключение java.lang.NullPointerException при вызове EJB-компонента - PullRequest
Новая
       17

Servlet.service () для сервлета jsp выдал исключение java.lang.NullPointerException при вызове EJB-компонента

2 голосов
/ 02 декабря 2011

При вызове EJB-компонента на моей простой странице jsp у меня возникает такая ошибка: 

 WARNING: StandardWrapperValve[jsp]: PWC1406: Servlet.service() for servlet jsp threw 
 exception java.lang.NullPointerException
at appjpa.ejb.UsersFacade.findAllStudents(UsersFacade.java:65)
at org.apache.jsp.Students_jsp._jspService(Students_jsp.java from :75)
at org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:111)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:847)
at org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:403)
at org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:492)
at org.apache.jasper.servlet.JspServlet.service(JspServlet.java:378)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:847)
at org.apache.catalina.core.StandardWrapper.service(StandardWrapper.java:1539)
at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:281)
at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175)
at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:655)
at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:595)
at com.sun.enterprise.web.WebPipeline.invoke(WebPipeline.java:98)
at com.sun.enterprise.web.PESessionLockingStandardPipeline.invoke(PESessionLockingStandardPipeline.java:91)
at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:162)
at org.apache.catalina.connector.CoyoteAdapter.doService(CoyoteAdapter.java:330)
at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:231)
at com.sun.enterprise.v3.services.impl.ContainerMapper.service(ContainerMapper.java:174)
at com.sun.grizzly.http.ProcessorTask.invokeAdapter(ProcessorTask.java:828)
at com.sun.grizzly.http.ProcessorTask.doProcess(ProcessorTask.java:725)
at com.sun.grizzly.http.ProcessorTask.process(ProcessorTask.java:1019)
at com.sun.grizzly.http.DefaultProtocolFilter.execute(DefaultProtocolFilter.java:225)
at com.sun.grizzly.DefaultProtocolChain.executeProtocolFilter(DefaultProtocolChain.java:137)
at com.sun.grizzly.DefaultProtocolChain.execute(DefaultProtocolChain.java:104)
at com.sun.grizzly.DefaultProtocolChain.execute(DefaultProtocolChain.java:90)
at com.sun.grizzly.http.HttpProtocolChain.execute(HttpProtocolChain.java:79)
at com.sun.grizzly.ProtocolChainContextTask.doCall(ProtocolChainContextTask.java:54)
at com.sun.grizzly.SelectionKeyContextTask.call(SelectionKeyContextTask.java:59)
at com.sun.grizzly.ContextTask.run(ContextTask.java:71)
at com.sun.grizzly.util.AbstractThreadPool$Worker.doWork(AbstractThreadPool.java:532)
at com.sun.grizzly.util.AbstractThreadPool$Worker.run(AbstractThreadPool.java:513)
at java.lang.Thread.run(Thread.java:662)

Код для моей страницы jsp такой: 

<%@page import="appjpa.entities.Users"%>
<%@page import="java.util.ArrayList"%>
<%@page import="socnet2.Student"%> <!-- class Student emulates real student -->
<%@page contentType="text/html" pageEncoding="UTF-8"%>
 <!DOCTYPE html>
<html>
<head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
    <title>JSP Page</title>
</head>
<body>
    <jsp:useBean class="appjpa.ejb.UsersFacade" id="dao" scope="application"></jsp:useBean>
    <p><b>List of all students:</b></p>
    <form action="/AppJPA/Edit" method="POST">
       <jsp:scriptlet>
       int counter = 0;  if (dao != null){
       for (Users s : dao.findAllStudents()) {
        </jsp:scriptlet>
        <p> <jsp:expression> s.getName() + " " + s.getSurname()</jsp:expression></p>
        <input type="submit" name="Edit_<%=counter%>" value="Edit" />
        <input type="submit" name="Delete_<%=counter%>" value ="Delete"/> 
         <jsp:scriptlet>
                counter++;
            }}
        </jsp:scriptlet>
        <input type="submit" name="Add_new_student" value ="Add new student"/>
    </form>
</body>

Класс сущности, созданный мастером NetBeans 7.0.1 для пользователей, выглядит следующим образом: 

 @Entity
@Table(name = "USERS")
@NamedQueries({
@NamedQuery(name = "Users.findAll", query = "SELECT u FROM Users u"),
@NamedQuery(name = "Users.findById", query = "SELECT u FROM Users u WHERE u.id = :id"),
@NamedQuery(name = "Users.findByName", query = "SELECT u FROM Users u WHERE u.name =      :name"),
@NamedQuery(name = "Users.findBySurname", query = "SELECT u FROM Users u WHERE u.surname = :surname"),
@NamedQuery(name = "Users.findByLogin", query = "SELECT u FROM Users u WHERE u.login = :login"),
@NamedQuery(name = "Users.findByPassword", query = "SELECT u FROM Users u WHERE u.password = :password"),
@NamedQuery(name = "Users.findByRole", query = "SELECT u FROM Users u WHERE u.role = :role")})
public class Users implements Serializable {
private static final long serialVersionUID = 1L;
@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@Basic(optional = false)
// @NotNull
@Column(name = "ID")
private Integer id;
@Size(max = 40)
@Column(name = "NAME")
private String name;
@Size(max = 40)
@Column(name = "SURNAME")
private String surname;
@Size(max = 40)
@Column(name = "LOGIN")
private String login;
@Size(max = 40)
@Column(name = "PASSWORD")
private String password;
@Size(max = 40)
@Column(name = "ROLE")
private String role;
@ManyToMany(mappedBy = "usersList")
private List<SubjectGroup> subjectGroupList;
@OneToMany(cascade = CascadeType.ALL, mappedBy = "idCreator")
private List<SubjectGroup> subjectGroupList1;
@OneToMany(cascade = CascadeType.ALL, mappedBy = "idSender")
private List<Messages> messagesList;

public Users() {
}

public Users(Integer id) {
    this.id = id;
}

public Integer getId() {
    return id;
}

public void setId(Integer id) {
    this.id = id;
}

public String getName() {
    return name;
}

public void setName(String name) {
    this.name = name;
}

public String getSurname() {
    return surname;
}

public void setSurname(String surname) {
    this.surname = surname;
}

public String getLogin() {
    return login;
}

public void setLogin(String login) {
    this.login = login;
}

public String getPassword() {
    return password;
}

public void setPassword(String password) {
    this.password = password;
}

public String getRole() {
    return role;
}

public void setRole(String role) {
    this.role = role;
}

public List<SubjectGroup> getSubjectGroupList() {
    return subjectGroupList;
}

public void setSubjectGroupList(List<SubjectGroup> subjectGroupList) {
    this.subjectGroupList = subjectGroupList;
}

public List<SubjectGroup> getSubjectGroupList1() {
    return subjectGroupList1;
}

public void setSubjectGroupList1(List<SubjectGroup> subjectGroupList1) {
    this.subjectGroupList1 = subjectGroupList1;
}

public List<Messages> getMessagesList() {
    return messagesList;
}

public void setMessagesList(List<Messages> messagesList) {
    this.messagesList = messagesList;
}

@Override
public int hashCode() {
    int hash = 0;
    hash += (id != null ? id.hashCode() : 0);
    return hash;
}

@Override
public boolean equals(Object object) {
    // TODO: Warning - this method won't work in the case the id fields are not set
    if (!(object instanceof Users)) {
        return false;
    }
    Users other = (Users) object;
    if ((this.id == null && other.id != null) || (this.id != null && !this.id.equals(other.id))) {
        return false;
    }
    return true;
}

@Override
public String toString() {
    return "appjpa.entities.Users[ id=" + id + " ]";
}

}

И, наконец, класс UsersFacade таков: 

@Stateless
public class UsersFacade extends AbstractFacade<Users> implements UsersFacadeLocal {
@PersistenceContext(unitName = "AppJPAPU")
private EntityManager em;

protected EntityManager getEntityManager() {
    return em;
}

public UsersFacade() {
    super(Users.class);
}

@Override
public Users checkInBase(String login, String password) {
    Users u = (Users)(em.createNamedQuery("Users.findByLogin").setParameter("login",      login).getSingleResult());

    return u;
}

@Override
public void removeUserFromSubjectGroup(Users user, SubjectGroup group) {
    Users u = em.find(Users.class, user.getId());
    SubjectGroup g = em.find(SubjectGroup.class, group.getId());
    if (u != null && g != null) {
        user.getSubjectGroupList().remove((SubjectGroup)group);
        group.getUsersList().remove(user);
        em.merge(user);
        em.merge(group);
    }
}

@Override
public void addUserToSubjectGroup(Users user, SubjectGroup group) {
    Users u = em.find(Users.class, user.getId());
    SubjectGroup g = em.find(SubjectGroup.class, group.getId());
    if (u != null && g != null) {
        user.getSubjectGroupList().add((SubjectGroup)group);
        group.getUsersList().add(user);
        em.merge(user);
        em.merge(group);
    }
}

@Override
public List<Users> findAllStudents() {
    String role = new String("student");
     List<Users> students = (List<Users>)(em.createNamedQuery("SELECT u FROM Users u WHERE u.role = :role").setParameter("role", role).getResultList());
     if (students == null){
         System.out.println("Students are null!!!");
     }

    return students;
}

@Override
public boolean deleteStudent(Users user) {
    Users u = em.find(Users.class, user.getId());
    em.remove(user);
    return true;
}

@Override
public boolean updateStudent(Users studentToEdit, String newName, String newSurname, String newLogin, String newPassword) {
    Users u = em.find(Users.class, studentToEdit.getId());
    u.setName(newName);
    u.setSurname(newSurname);
    u.setLogin(newLogin);
    u.setPassword(newPassword);
     em.merge(studentToEdit);
    return true;
}

@Override
public void insertStudenttoDB(Users student, String newName, String newSurname, String newLogin, String newPassword) {
   em.persist(student);
   Users u = em.find(Users.class, student.getId());
    if (u != null) {
    student.setName(newName);
    student.setSurname(newSurname);
    student.setLogin(newLogin);
    student.setPassword(newPassword);  
    em.merge(student);
    }
}     
}

Я новичок в EJB и, возможно, яЯ не использую их правильно.Я пытался вызвать другие методы из UsersFaced на других страницах, но получил ту же ошибку.Пожалуйста, объясните, что я делаю не так ...

Ответы [ 2 ]

1 голос
/ 02 декабря 2011

@PersistenceContext не будет внедрено, если вы вручную создадите экземпляр класса EJB как простой Java-бин в JSP с помощью <jsp:useBean> вместо получения экземпляра управляемого контейнера с помощью @EJB. Я не знаю, как и где вы научились писать JSP таким образом, но это, несомненно, основано на сильно устаревшей книге / учебнике. Вам необходимо внести радикальные изменения в ваш текущий подход модель-представление-без-контроллера. Вам нужно добавить контроллер. Выбросьте все эти <jsp:xxx> теги и создайте сервлет, который будет действовать как настоящий контроллер.

1007 * Е.Г. * 

@WebServlet("/students")
public class StudentsServlet extends HttpServlet {

    @EJB
    private UsersFacade usersFacade;

    @Override
    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        request.setAttribute("users", usersFacade.findAllStudents());
        request.getRequestDispatcher("/WEB-INF/students.jsp").forward(request, response);
    }

}

Обратите внимание, что я переместил ваш JSP в папку /WEB-INF, чтобы невозможно было преднамеренно или случайно вызвать его без вызова сервлета.

Теперь измените свой JSP следующим образом: 

<%@page pageEncoding="UTF-8" %>
<%@taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %>
<!DOCTYPE html>
<html lang="en">
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
        <title>JSP Page</title>
    </head>
    <body>
        <h4>List of all students:</h4>
        <form action="${pageContext.request.contextPath}/edit" method="post">
            <c:forEach items="${users}" var="user" varStatus="loop">
                <p><c:out value="${user.name} ${user.surname}" /></p>
                <input type="submit" name="Edit_${loop.index}" value="Edit" />
                <input type="submit" name="Delete_${loop.index}" value ="Delete"/> 
            </c:forEach>
            <input type="submit" name="Add_new_student" value ="Add new student"/>
        </form>
    </body>
</html>

Откройте его по URL сервлета http://localhost:8080/AppJPA/students, чтобы его doGet() вернулось к нужному заданию и чтобы JSTL <c:forEach> перебрал users соответственно.

Кстати, у меня есть сомнения относительно того, как вы назвали кнопки. Идентификация выбранного пользователя будет очень неуклюжей. Но это тема для другого вопроса.

Смотри также:

0 голосов
/ 02 декабря 2011

Если это ваша целая пачка именованных запросов, то я думаю, что этот код: 

 List<Users> students = (List<Users>)(em.createNamedQuery(
         "SELECT u FROM Users u WHERE u.role = :role").setParameter("role", role)
         .getResultList());

испорчен. Вы пытаетесь создать запрос с именем , имя которого равно SELECT u FROM Users u WHERE u.role = :role.

Вы указали именованные запросы, определенные в аннотации Users, и один из них вы пытаетесь использовать: 

@NamedQuery(name = "Users.findByRole", 
            query = "SELECT u FROM Users u WHERE u.role = :role")})

Поэтому вы должны использовать его имя - Users.findByRole. Более того, вам не нужно приводить к List<Users>, так как есть соответствующий параметр метода для определения типа результата. Это должно быть скорее: 

List<Users> students = em.createNamedQuery("Users.findByRole", Users.class)
                         .setParameter("role", role).getResultList();

В вашем коде есть несколько плохих практик, таких как:

  • используя множественное число для ваших сущностей; это должно быть скорее User, чем Users,
  • с использованием скриптлетов; и использование <jsp:scriptlet> не делает его лучше.
...