1

继上一个问题之后，我发现代码中的罪魁祸首是这个对象

我在 Java 中的 Connection 类中有一个名为“conn”的对象。

在我的旧代码中，没有突出显示对象名称。正如你在下面看到的。

在此处输入图像描述

但是，在我在 Eclipse 上的较新版本的代码中，由于该对象始终为空，因此该对象一直被突出显示。

在此处输入图像描述

我的DatabaseLogic类

    package org.ari;

//class dependencies
import java.io.IOException;
import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NamingException;
import javax.sql.DataSource;

public class DatabaseLogic
{
    private static Connection conn;

    public static String openDatabase() throws IOException, SQLException,
            NamingException
    {


        // context class gives naming standards of the surrounding environment
        // of the servlet i.e. the web server ,
        // allowing the servlet to interface with the web servers resources
        Context initialContext = new InitialContext();
        Context envContext = (Context) initialContext.lookup("java:comp/env");
        // servlet looks up for a connection pool called "jdbc/POOL"
        DataSource ds = (DataSource) envContext.lookup("jdbc/POOL");
        // connection is then made/requests to connection pool

        String result = ds.toString();


        try
        {
            conn = ds.getConnection();
        }
        catch (SQLException e)
        {
            System.out.println( e.toString());          
        }


        return result;
    }

    public static void closeDatabase()
    {
        try
        {
            conn.close();
        }
        catch (SQLException e)
        {

        }
    }

    // queryId is the parameter to be used for querying for relevant records
    // - possibly change name to make it less specific e.g. recordid
    public static String getData(String queryId, int requestNumber)
            throws SQLException
    {
        String result = "";
        if (queryId != null)
        {
            try
            {

                if (conn == null)
                { 
                    //result = "We are in here";
                    result = openDatabase(); 
                }

                // prepare a statement for use in query

                Statement stmt = conn.createStatement();
                // query parameratised with queryId
                String qry = "SELECT RECORD_ID, USER_ID, OPERATION_CD, BUSCOMP_NAME, OPERATION_DT, FIELD_NAME, OLD_VAL, NEW_VAL, AUDIT_LOG, ROW_ID, BC_BASE_TBL FROM S_AUDIT_ITEM WHERE RECORD_ID='"
                        + queryId + "'";
                ResultSet results = stmt.executeQuery(qry);
                result = XMLBuilder.xmlBuilder(results, queryId,
                        requestNumber);
                // close the connection
                stmt.close();
                results.close();
            }
            catch (Exception e)
            {
                // log.error("Cannot connect to database :" + e);


            }
        }
        else
        {
            // not sure if ever reached
            result = "The query parameter  is a null value";
        }
        return result;
    }

}

这个突出显示是什么意思，我猜 Eclipse 正在以不同的方式处理这个对象，这就是为什么这个对象不再正常运行的原因。

我该如何解决这个问题，有什么想法吗？我已尝试恢复旧版本的代码，但此突出显示仍然相同。

我猜这是IDE中某处的一些设置。

谢谢

4

1 回答 1

3

因此，执行时会收到 NullPointerException

conn.createStatement();

这意味着它conn是空的。

在哪里conn初始化？就在之前，在调用openDatabase().

如何openDatabase()初始化变量？

    try
    {
        conn = ds.getConnection();
    }
    catch (SQLException e)
    {
        System.out.println( e.toString());          
    }

abobe 表示要么ds.getConnection()返回一个连接，并且conn不能为空（但它是），要么conn为空，因为ds.getConnection()抛出了一个 SQLException，但你继续好像什么也没发生，只是显示抛出的异常的 toString()。以上应改写为：

conn = ds.getConnection();

这样，您就不会忽略 SQLException。它将传播给调用者，并被识别为 SQLException，并带有明确的错误消息，指示代码中问题的来源，而不是稍后发生的晦涩的 NullPointerException，并且更难诊断。

让异常传播，或者至少将代码替换为：

try {
    conn = ds.getConnection();
}
catch (SQLException e) {
    throw new RuntimeException("Something really bad happened, and it makes no sense to continue further, so I'll throw a runtime exception wrapping the original cause", e);
}

现在您只需要获取由 抛出的 SQLException 的堆栈跟踪ds.getConnection()，并了解它的错误消息的含义。

请注意，我怀疑该异常不是由 引发的conn.createStatement()，而是由您在以下内容中也忽略的其他一些异常引发的：

catch (Exception e)
{
    // log.error("Cannot connect to database :" + e);
}

再一次，不要捕获异常。让它传播。通过抓住它，您很难自己注意到问题，并且无法诊断它。

如果您无法处理异常，请不要捕获它。

于 2012-12-24T12:36:16.070 回答