0

我创建了一个基本的日志记录类,目前它是一个二维数组,最多存储 100 个事件的日志信息(然后开始覆盖信息,所以我只保留最后 100 个事件)。

在某个级别的日志事件中,此数组数据被转储到 AsyncTask 线程上的数据库中。

目前该数组是在类级别定义的,因此 AsyncTask 线程正在访问 GUI 线程正在访问的同一个数组(这给了我 NullPointerExceptions,应该如此)。

我需要一种方法来复制数组的值,因为它是在 AsyncTask 线程启动的那一刻,以便这个后台线程可以读取所有数据并将其正确发送到数据库 - 这就是我的重点卡在。

// Defined in the class
private static String[][] logHistory = new String[100][6];


// Class to log errors to a database
private static class asyncLog extends AsyncTask<Void,Void,Void>
{
    // This is the background thread
    @Override
    protected Void doInBackground(Void... params)
    {
        Thread.currentThread().setName("LogToDB.asyncLog.doInBackground");              

        // Make a copy of the array in its current state
        final String[][] logTemp = new String[100][6];
        for (int i = 0; i < 100; i++) 
        {
              for (int j = 0; j < 6; j++) 
              {
                logTemp[i][j]=logHistory[i][j];
              }
        }

        // ...snip
    }
}

显然final修饰符在这里是错误的,因为我仍然遇到异常

我想知道:

  • 这是做我想做的事情的正确方法吗?
  • 如果没有,有什么更好的方法?
  • 如何在后台线程中复制字符串数组,以便无论 GUI 线程对原始数组进行什么更改,副本始终具有相同的值,直到线程结束并被销毁

我见过很多处理一维数组的例子,但这些大多是 int 数据类型(例如这个,据我了解字符串,它们总是引用:/

编辑根据要求,请参阅堆栈跟踪:

08-23 09:17:35.570: E/AndroidRuntime(8030): FATAL EXCEPTION: LogToDB.asyncLog.doInBackground
08-23 09:17:35.570: E/AndroidRuntime(8030): java.lang.RuntimeException: An error occured while executing doInBackground()
08-23 09:17:35.570: E/AndroidRuntime(8030):     at android.os.AsyncTask$3.done(AsyncTask.java:299)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:352)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at java.util.concurrent.FutureTask.setException(FutureTask.java:219)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at java.util.concurrent.FutureTask.run(FutureTask.java:239)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:230)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1080)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:573)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at java.lang.Thread.run(Thread.java:856)
08-23 09:17:35.570: E/AndroidRuntime(8030): Caused by: java.lang.NullPointerException
08-23 09:17:35.570: E/AndroidRuntime(8030):     at coty.production.downtimeaquisition.LogToDB.Insert(LogToDB.java:393)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at coty.production.downtimeaquisition.LogToDB.access$10(LogToDB.java:387)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at coty.production.downtimeaquisition.LogToDB$asyncLog.doInBackground(LogToDB.java:558)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at coty.production.downtimeaquisition.LogToDB$asyncLog.doInBackground(LogToDB.java:1)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at android.os.AsyncTask$2.call(AsyncTask.java:287)
08-23 09:17:35.570: E/AndroidRuntime(8030):     at java.util.concurrent.FutureTask.run(FutureTask.java:234)
08-23 09:17:35.570: E/AndroidRuntime(8030):     ... 4 more

LogToDB.java:558 是对 Insert 方法的调用

LogToDB.java:387 是插入方法

LogToDB.java:393 是突出显示的行:

Statement stmt=null;

try 
{
    stmt=dbConn.createStatement(); // This line
    stmt.executeUpdate(sql);
}  
4

1 回答 1

1

我将假设您要做的是拥有一个“幻像日志”事件,只有在发生某些不希望的情况时才会被转储,这样您就不会弄乱主日志,但可以看到发生错误时的详细信息。如果是这样的话,我会这样做:

  1. 定义一个LogMessage包含有关您的消息的必要信息的类,通常是 aString和一个日志级别(应该是 an enum),可能带有一个用于 a 的字段Throwable。有关此接口通常外观的示例,请参见 log4j 和 slf4j。
  2. 定义一个固定长度的缓冲区类;不幸的是,虽然 1.6 并发 API 具有支持固定长度队列的接口,但我无法找到一个好的、高效的实现。最好的选择很大程度上取决于读取和转储的频率,但我会先做一些事情,比如分配一个固定大小的数组,使用 anAtomicInteger作为数组索引。标准(无限大小)实现基于论文“Simple, Fast, and Practical Non-Blocking and Blocking Concurrent Queue Algorithms”。对该数组的引用需要volatileAtomicReference.
  3. 当跟踪日志级别的“漏斗”API 看到一条具有适当重要性的日志消息时,它会自动换出整个日志结构并启动一个作业以转储它。如果您正在登录到一个会被多个日志线程混淆的设施,您甚至可以将整个日志缓冲区提交到BlockingQueue一个单个长时间运行的转储线程读取的地方。
于 2013-08-23T08:32:13.430 回答