今天在学习servlet和jsp时,遇到一个头疼的问题
即不做任何输入时提交,代码提示空指针异常,位置在于Servlet的doPost()和doGet()方法。
查阅代码,发现已经做了处理。
if (doctor != null)
后来debug调试发现当空字符串提交时,竟然判定 doctor != null 条件为真!当时就很诧异,后来查阅资料发现当对象被实例化后,该对象!=null。但又发生了新的疑问,如下。
Doctor doctor = doctorDao.findDoctorByPym(pym);
按照道理,若find方法返回为空,则该doctor不该被实例化。再后来仔细检查过find的实现方法后,恍然大悟。
@Override
public Doctor findDoctorByPym(String pym) {
Connection conn = null;
PreparedStatement ps = null;
ResultSet rs = null;
Doctor doctor = new Doctor();
conn = DBHelp.getConn();
String sql = "select * from tb_doctor where pym=?";
try {
ps = conn.prepareStatement(sql);
ps.setString(1, pym);
rs = ps.executeQuery();
if (rs.next()) {
doctor.setId(rs.getInt("ID"));
doctor.setDname(rs.getString("dname"));
doctor.setDept(rs.getString("dept"));
doctor.setPym(rs.getString("pym"));
}
} catch (Exception e) {
e.printStackTrace();
} finally {
DBHelp.closeAll(rs, ps, conn);
}
return doctor;
}
这就是自身水平的不足,敲代码没有形成良好的风格和理论知识的匮乏的表现。这种细节问题之前是万万想不到的。
Doctor doctor = new Doctor();
在获取之前先new了一个对象,实例化了,那在后面的业务处理中,doctor当然不等于null了。我认为最好的写法应该如下:
@Override
public Doctor findDoctorByPym(String pym) {
Connection conn = null;
PreparedStatement ps = null;
ResultSet rs = null;
Doctor doctor = null;
conn = DBHelp.getConn();
String sql = "select * from tb_doctor where pym=?";
try {
ps = conn.prepareStatement(sql);
ps.setString(1, pym);
rs = ps.executeQuery();
if (rs.next()) {
doctor = new Doctor();
doctor.setId(rs.getInt("ID"));
doctor.setDname(rs.getString("dname"));
doctor.setDept(rs.getString("dept"));
doctor.setPym(rs.getString("pym"));
}
} catch (Exception e) {
e.printStackTrace();
} finally {
DBHelp.closeAll(rs, ps, conn);
}
return doctor;
}
将对象的实例化也放入赋值的判断中,这样一来若没有从数据库中查到数据,则该对象也不会被实例化,再后面的是否等于null的判断中也不会出现奇怪的错误了。
在java的路上还有很长的路要走。