Tell me more ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite
1

I have a problem with my codes, dunno how to fix it, because I've been doing it over and over again... and same error occurred can anyone tell me what to do?

Here is my codes :

For my connection path :

public void Connection(String path){
    try{
            Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
            String url = "jdbc:odbc:*****";
            String user = "*****";
            String pass = "*****";
            connection =  DriverManager.getConnection(url, user, pass);
} catch(Exception e) {}}

My update codes :

try{
            String update = ("UPDATE dbo.Patients set pLName=?,pFName=? where pIDNo" +txtPatientID.getText()+"");
            ps = connection.prepareStatement(update);
            ps.setString(1, txtpFName.getText());
            ps.setString(2, txtpLName.getText());
            ps.execute();
            }
            catch(Exception e)
            {
            e.printStackTrace();
            System.exit(1);
            }

And here is the error that I've got :

java.lang.NullPointerException
    at javaapplication25.EditPatient.btnUpdateActionPerformed(EditPatient.java:407)
    at javaapplication25.EditPatient.access$100(EditPatient.java:14)
    at javaapplication25.EditPatient$2.actionPerformed(EditPatient.java:187)
    at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2018)
    at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2341)
    at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
    at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
    at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
    at java.awt.Component.processMouseEvent(Component.java:6505)
    at javax.swing.JComponent.processMouseEvent(JComponent.java:3321)
    at java.awt.Component.processEvent(Component.java:6270)
    at java.awt.Container.processEvent(Container.java:2229)
    at java.awt.Component.dispatchEventImpl(Component.java:4861)
    at java.awt.Container.dispatchEventImpl(Container.java:2287)
    at java.awt.Component.dispatchEvent(Component.java:4687)
    at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4832)
    at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4492)
    at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4422)
    at java.awt.Container.dispatchEventImpl(Container.java:2273)
    at java.awt.Window.dispatchEventImpl(Window.java:2713)
    at java.awt.Component.dispatchEvent(Component.java:4687)
    at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:707)
    at java.awt.EventQueue.access$000(EventQueue.java:101)
    at java.awt.EventQueue$3.run(EventQueue.java:666)
    at java.awt.EventQueue$3.run(EventQueue.java:664)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:87)
    at java.awt.EventQueue$4.run(EventQueue.java:680)
    at java.awt.EventQueue$4.run(EventQueue.java:678)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
    at java.awt.EventQueue.dispatchEvent(EventQueue.java:677)
    at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:211)
    at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:128)
    at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:117)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:113)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:105)
    at java.awt.EventDispatchThread.run(EventDispatchThread.java:90)
Java Result: 1
share|improve this question
Just a side node: why don't you use a param for your where condition value like for your other input values? Always remember Exploits Of A Mom – Filburt Apr 6 at 7:40
4  
the stacktrace suggest looking at EditPatient.java:407 that is line 407 of EditPatient.java. What code is on that line? – James OB Apr 6 at 7:41
are you sure the connection was successfully established? The caught exceptions in your Connection method are not logged – david Apr 6 at 7:54
sir James this is the codes on that line : ps.setString(2, txtpLName.getText()); – Crystal Maiden Apr 6 at 8:08
sir david I already have my INSERT statement almost the same format as this, and I'm sure that it is successfully established the connection, I don't know now what to do... – Crystal Maiden Apr 6 at 8:09

closed as too localized by Filburt, Sean Owen, Jayamohan, jeb, Audrius Meškauskas Apr 6 at 14:16

This question is unlikely to help any future visitors; it is only relevant to a small geographic area, a specific moment in time, or an extraordinarily narrow situation that is not generally applicable to the worldwide audience of the internet. For help making this question more broadly applicable, see the FAQ.

1 Answer

up vote 1 down vote

If your exception occurs on this line

ps.setString(2, txtpLName.getText());

It seems that txtplName is null, as your code already called a method of ps.

But you should really invest into your exception handling, otherwise you will always run into such "follow-up" errors. Especially swallowing the exception in your Connection method may bite you, as you won't see that the connection was not established. Printing the stack trace helps for debugging, but not for your application. At least return true or false and handle it accordingly (false means you cannot establish a connection and thus you don't need to continue anyway).

share|improve this answer

Not the answer you're looking for? Browse other questions tagged or ask your own question.