AddBirth 메소드가 NULL을 반환하는 이유는 무엇입니까?
-
19-09-2019 - |
문제
이 방법이 있지만 런타임에는 NullPointerException이 발생하는 이유는 무엇입니까?
내 방법 :
public static boolean isAddBirth(String name, String family, String fatherName, String mName, String dOfBirth, String pOfBirth) {
ResultSet rst;
boolean bool = false;
Statement stmt;
try {
stmt = conn.createStatement();
rst = stmt.executeQuery("SELECT * FROM birthtable");
while (rst.next()) {
if (rst.getString(2).equals(name) && rst.getString(3).equals(family) && rst.getString(4).equals(fatherName) && rst.getString(5).equals(mName) && rst.getString(6).equals(dOfBirth) && rst.getString(7).equals(pOfBirth)) {
bool = false;
} else {
bool = true;
}
}
} catch (SQLException ex) {
Logger.getLogger(Manager.class.getName()).log(Level.SEVERE, null, ex);
}
return bool;
}
스택 트레이스 :
java.lang.NullPointerException
at database.Manager.isAddBirth(Manager.java:164)
at AdminGUI.AddNewBornInformation.submit(AddNewBornInformation.java:356)
at AdminGUI.AddNewBornInformation.setButtonActionPerformed(AddNewBornInformation.java:283)
at AdminGUI.AddNewBornInformation.access$800(AddNewBornInformation.java:28)
at AdminGUI.AddNewBornInformation$9.actionPerformed(AddNewBornInformation.java:140)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:6038)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5803)
at java.awt.Container.processEvent(Container.java:2058)
at java.awt.Component.dispatchEventImpl(Component.java:4410)
at java.awt.Container.dispatchEventImpl(Container.java:2116)
at java.awt.Component.dispatchEvent(Component.java:4240)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4322)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3986)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3916)
at java.awt.Container.dispatchEventImpl(Container.java:2102)
at java.awt.Window.dispatchEventImpl(Window.java:2429)
at java.awt.Component.dispatchEvent(Component.java:4240)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:273)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:183)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:173)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:168)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:160)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:121)
또한 이것들은 내 수업에 있습니다.
Logger logger = Logger.getLogger(this.getClass().getName());
private static Connection conn = DBManager.getConnection();
해결책
당신이 줄 번호 164를 제공하지 않았기 때문에 다음과 같이 추측 할 것입니다.
if (rst.getString(2).equals(name) && rst.getString(3).equals(family) && rst.getString(4).equals(fatherName) && rst.getString(5).equals(mName) && rst.getString(6).equals(dOfBirth) && rst.getString(7).equals(pOfBirth))
먼저 그 라인은 내가 울고 싶게 만듭니다.
고치합시다 :
String a;
String b;
String c;
String d;
String e;
String f;
a = rst.getString(2);
b = rst.getString(3);
c = rst.getString(4);
d = rst.getString(5);
e = rst.getString(6);
f = rst.getString(7);
if (!(a.equals(name))
{
bool = false;
}
if(!(b.equals(family))
{
bool = false;
}
if(!(c.equals(fatherName))
{
bool = false;
}
if(!(d.equals(mName))
{
bool = false;
}
if(!(e.equals(dOfBirth))
{
bool = false;
}
if(!(f.equals(pOfBirth))
{
bool = false;
}
그것은 적어도 당신에게 널 포인터가있는 줄을 보여줄 것입니다 (내 추측이 맞다고 가정).
또한, AE는 끔찍한 이름입니다 ... 당신은 내가했던 것보다 더 나은 이름을 선택해야합니다.
여기서 실제 해결책은 의도 된대로 객체 지향 프로그래밍을 사용하는 것입니다 ... 사람 수업을 만들어 보자.
public class Person
{
private final String firstName;
private final String lastName;
private final String middleName; // guessing that is what mName is...
private final String fathersName;
private final String dateOfBirth;
private final String placeOfBirth; // guessing that is what pOfBirth is...
public Person(final String firstName,
final String lastName,
final String middleName,
final String fathersName,
final String dateOfBirth,
final String placeOfBirth)
{
if(firstName == null)
{
throw new IllegalArgumentException("firstName cannot be null");
}
if(lastName == null)
{
throw new IllegalArgumentException("lastName cannot be null");
}
... etc for all of the other arguments ...
// I would never do the this.fristName thing.. .I would name the parameter different than the instance vairable...
this.firstName = firstName;
this.lastName = lastName;
... etc for all of the other arguments ...
}
public boolean equals(final Object o)
{
final Person person;
if(!(o instanceof Person))
{
return (false);
}
other = (Person)o;
// the code you I put above + your code for checking if they are equal
}
public int hashCode()
{
// this is probably good enough
return (firstName.hashCode() + lastName.hashCode());
}
}
그런 다음 방법에서 다음과 같은 코드가 있습니다.
rst = stmt.executeQuery("SELECT * FROM birthtable");
while (rst.next())
{
final Person person;
// I would use temp variables rather than passing in the result of getString directly...
person = new Person(rst.getString(2),
rst.getString(3),
rst.getString(4),
rst.getString(5),
rst.getString(6),
rst.getString(7));
// otherPerson would be passed into the method instead of the String you are passing now
bool = person.equals(otherPerson);
... etc ...
}
다른 팁
우선, 그것이 작동하더라도 그 코드는 당신이 원하는 것을 수행하지 않을 것입니다. 근처에도 안.
둘째, 우리가 예외가 어떤 줄이 있는지 알면 도움이 될 것입니다. 그러나 그 대신에, 우리는 그것을 다음 중 하나로 좁힐 수 있습니다.
conn
NULL입니다.conn.createStatement();
NULL을 반환합니다.stmt.executeQuery()
NULL을 반환합니다.- 중 하나
rst.getString()
s는 null을 반환합니다.
나는 당신이 거기에서 그것을 알아낼 수 있다고 확신합니다.
원인은 "GetString"방법이 NULL을 반환하고 있기 때문입니다.
이 방법의 API 문서는 NULL을 반환 할 수 있음을 나타냅니다.
String GetString (int columnIndex) [...
반환 : 열 값; 값이 SQL NULL 인 경우 리턴 된 값은 없는
위의 API DOC 참조는 다음과 같습니다.결과 세트
귀하의 프로그램이 NullPointerExceptions에 덜 취약하기 위해 추정하다 입력 변수가 무효가 아님 (또는 보장하다 방법의 시작 부분에서) 및 비교를 반전시킵니다.
if (name.equals(rst.getString(1)) && ...
문제가없는 다른 대안은 Commons-Lang 라이브러리에서 객체 equals.equals (obj1, obj2)를 사용하는 것입니다. 두 객체가 모두 null이거나 obj1.equals 인 경우에만 true를 반환합니다. OBJ2), 다른 모든 가능성은 거짓을 반환합니다. 클래스 객관성 및 문자열은 널 안전한 많은 방법을 가지고 있습니다.